Как определить откуда клик пришел

[scalpel]

Скажите как правильно реализовать такое.
Есть 2 блока

<div class="order order-1">
	<div class="input-group input-group-order">
		<span class="input-group-addon input-group-addon-order"><span class="otkyda">1</span></span>
		<a href="javascript:" class="btn-order-map" id="order-1"><i class="fa fa-map-marker"></i></a>
		<input type="text" name="otkyda" class="form-control form-control-order error" placeholder="Выберите адрес" id="autocomplete1">
	 </div>
</div>
<div class="order order-2">
	<div class="input-group input-group-order">
		<span class="input-group-addon input-group-addon-order"><span class="kyda">2</span></span>
		<a href="javascript:" class="btn-order-map" id="order-2"><i class="fa fa-map-marker"></i></a>
		<input type="text" name="kyda" class="form-control form-control-order error" placeholder="Выберите адрес" id="autocomplete2">
	 </div>
</div>

Надо определить с какой ссылки пришел клик и в зависимость от клика добавить информацию в соответствующий input.

Пробовал таким образом но не работает.

document.body.onclick = function (e) {
        e = e || event;
        target = e.target || e.srcElement;
        if (target.tagName == "a" && target.id == "order-1") {
            return alert("good");
        } else {
            return alert("fail");
        }
    }

[Alex11223]

Как именно не работает-то?
Смотрите есть ли ошибки в консоли, смотрите что происходит в отладчике (или console.log(target))

И объявляйте переменные с var иначе они глобальные.

var target = ............;

[scalpel]

Функция работает только выводит "fail" в любых случаях даже при нажатие на ссылку и нужным ид.

[Alex11223]

Смотрите что происходит в отладчике (поставьте точку остановки в функции) или console.log(target)

[Serge_Bliznykov]

Сообщение от scalpel

даже при нажатие на ссылку и нужным ид.

во-первых, а как Вы попадаете по ссылке с нужным id, если в вашем примере ссылка пустая?
js1.png


p.s. ну и проверки по ID более чем достаточно.
Если у Вас ID в DOM не уникально - значит, у Вас неверный HTML код,
его нужно исправить так, чтобы ID однозначно идентифицировало элемент,
иначе это не ID, а фигня какая-то!

[come-on]

Если у Вас ID в DOM не уникально - значит, у Вас неверный HTML код

неправильно

[Alex11223]

Сообщение от come-on

фу, как неправильно

Не верите спецификации?
https://www.w3.org/TR/html5/dom.html#the-id-attribute

The value must be unique amongst all the IDs in the element's home subtree and must contain at least one character. The value must not contain any space characters.

Ну и да, некоторые вещи будут работать и с не уникальными, но зачем?

Сообщение от Serge_Bliznykov

если в вашем примере ссылка пустая?

там классом типа "fa fa-map-marker" должны вставляться иконки (видимо автор использует Bootstrap и иконки из шрифта font awesome)

[come-on]

The value must be unique amongst all the IDs in the element's home subtree and must contain at least one character.

где тут про ВСЕ говорят? чтож вы сюда цитируете и неверно комментируете?

[Alex11223]

Все что?

Сообщение от come-on

а какие не будут?

document.getElementById и jQuery $(''#...") вернет только один, а не все. Что может не являться ожидаемым поведением (например хотел второй элемент, а получил первый) и непонятно зачем так запутывать код. Код же для людей, а люди ожидают, что ID уникальный.

[Serge_Bliznykov]

come-on,может быть, расскажете о том, о чём намекаете.

Сообщение от come-on

фу, как неправильно

про то, что ID должно быть уникально..
Расскажите, как должно быть правильно.

Сообщение от Alex11223

там классом типа "fa fa-map-marker" должны вставляться иконки

да, точно, это я проглядел.
Но, даже так, разве в этом случае клик получает не сам <i class="fa fa-map-marker"></i> ?
если да, тогда в этом случае нужно проверять id родительского узла.
ну, так, например:

	if (target.parentNode.id == "order-1") {
            return alert("good");
....