2010-08-27 3 views
-1

Некоторое время смотрел на это и даже писал еще один вопрос и получил половину ответа, но ошибка все еще сохраняется expected ';' Персонаж 392onclick все еще дает мне проблемы

+2

Пожалуйста, не оставляйте исходный код PHP. Это бессмысленно в контексте. Всегда отправляйте полный, сгенерированный код, как показано в браузере. (Downvote not mine, хотя) –

+0

Я бы отредактировал вашу учетную запись LivePerson, не хотел бы, чтобы моя учетная запись или информация о моих клиентах была общедоступна – Mikey1980

ответ

1

Пожалуйста, следуйте советам Пекка в!

Но вы ошибаетесь в том, что вам не хватает ';, чтобы закончить одну из ваших строк.

В своем коде, у вас есть:

escape(document.location) +' imageUrl={URL} lpButtonCTTUrl = ... 

Но, я думаю, что вы хотите, чтобы закрыть эту строку и начать новое заявление. Вы должны были бы добавить '; к концу этого imageUrl строки, как это:

escape(document.location) +' imageUrl={URL}'; lpButtonCTTUrl = ... 
12

Это очень плохой стиль, чтобы поместить этот код во встроенное событие onclick именно по этой причине: для отладки становится сукой. Я бы рекомендовал создать функцию - lpButtonClick или что-то - в отдельном файле .js или в блоке <script>.

Я уверен, что когда вы это сделаете и разделите каждую команду на одну строку, причина этого ошибки ; быстро закроется.

OnClick событие Ваша ссылка была бы выглядеть следующим образом:

onClick="return lpButtonClick();" 

в качестве альтернативы, iyou может избежать установки события в разметке в целом. Если вы работаете с JQuery, давая ссылку идентификатор будет выглядеть следующим образом - поместить его в ready событие:

$("#linkID").click(function() { return lpButtonClick(); }); 
+0

Я бы также добавил, чтобы еще больше упростить ситуацию, вы могли бы сделать свой HREF и SRC значения переменных php. Это упростило бы отладку проблем с HTML/JS, делая ваш код более читаемым. –

Смежные вопросы