jquery:SyntaxError: unterminated string literal

Question

when ever i am giving a space between a word & i click to call a function, then i am getting the below error:

SyntaxError: unterminated string literal
[Break On This Error]   

$(this).SelectProjectBox(363,"ssss

mypage# (line 1, col 29)

Browser generated html:

<span ssasas")="" onclick="$(this).SelectProjectBox(363,"ssss" href="#">ssss ssasas</span>

My actual code

  echo '<span href="#"  onClick=$(this).SelectProjectBox(' . $cat->project_id . ',"'.$cat->project_name.'")>' . $cat->project_name . '</span>';

what is the cause Please suggestion, how can i modify my php code for this?

@Sundara Try echo '<span href="#" onClick="$(this).SelectProjectBox(\''.$cat->project_id .'\',\''.$cat->project_name.'\')">'.$cat->project_name.'</span>'; — Tushar Gupta - curioustushar
– Tushar Gupta - curioustushar, Commented Apr 29, 2014 at 6:52
@TusharGupta Yes, your comment worked for me.. Can anyone describe your solution. — Dan
– Dan, Commented Apr 29, 2014 at 6:55
StackOverflow should make essential to write comment if -ve vote is done! — Dan
– Dan, Commented Apr 29, 2014 at 17:24

Rakesh Sharma · Accepted Answer · 2014-04-29 06:54:48Z

3

you missing quotes on onclick event try

echo '<span href="#"  onClick="$(this).SelectProjectBox(' . $cat->project_id . ','.$cat->project_name.')">' . $cat->project_name . '</span>';

or try

<span href="#"  onClick="$(this).SelectProjectBox(<?php echo $cat->project_id ;?>,<?php echo $cat->project_name;?>)"><?php echo $cat->project_name;?></span>

edited Apr 29, 2014 at 6:54

answered Apr 29, 2014 at 6:47

Rakesh Sharma

13.7k5 gold badges41 silver badges44 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Gumbo Over a year ago

Quotes appearing in an unquoted value are not allowed.

Dan Over a year ago

error: Uncaught SyntaxError: Unexpected token ILLEGAL Html: <span href="#" onclick="$(this).SelectProjectBox("363","ssss" ssasas")="">ssss ssasas</span>

GuyT · Accepted Answer · 2014-04-29 06:53:56Z

1

If I may give you some advice: split up your code a bit more. It is easier for people to understand:

$content  = "<span href='#' ";  
$content .= "onClick=\"$(this).SelectProjectBox($cat->project_id,'$cat->project_name')\"";
$content .= ">";
$content .= $cat->project_name . "</span>";

echo $content;

answered Apr 29, 2014 at 6:53

GuyT

4,4562 gold badges20 silver badges34 bronze badges

3 Comments

Dan Over a year ago

if i do so then, my code will be long. already i have 832 lines in my code

GuyT Over a year ago

But it will be much better for maintenance. And if other programmers have to work with this code it will take them ages to understand. If your file is really getting to big then you could split it up in seperate files.

GuyT Over a year ago

And as you see, when you made a mistake in your own code it took a long time(and even a SO question) to correct it. Just consider my code. Rather use 4000 lines of readable code then 800 of unreadable code.

Tushar Gupta - curioustushar · Accepted Answer · 2014-04-29 06:56:54Z

1

Escape single quote(') using \.

\' will echo '

To specify a literal single quote, escape it with a backslash (\). To specify a literal backslash, double it (\\). All other instances of backslash will be treated as a literal backslash: this means that the other escape sequences you might be used to, such as \r or \n, will be output literally as specified rather than having any special meaning.

Code

echo '<span href="#"  onClick="$(this).SelectProjectBox(\''.$cat->project_id .'\',\''.$cat->project_name.'\')">'.$cat->project_name.'</span>';

Read String

answered Apr 29, 2014 at 6:56

Tushar Gupta - curioustushar

57.2k24 gold badges106 silver badges110 bronze badges

Comments

Jenz · Accepted Answer · 2014-04-29 06:58:38Z

0

Instead of

echo '<span href="#"  onClick=$(this).SelectProjectBox(' . $cat->project_id . ',"'.$cat->project_name.'")>' . $cat->project_name . '</span>';

try with

echo '<span href="#"  onClick="$(this).SelectProjectBox(\''.$cat->project_id .'\',\''.$cat->project_name.'\')">'.$cat->project_name.'</span>';

Here, we are echoing the html part and we are using single quotes for enclosing the entire html string. So whenever we want single quotes in the string , here we want it for functions, we have to escape it using backslash as /' which will display a single quote. Otherwise the string will get stopped at that point.

edited Apr 29, 2014 at 6:58

answered Apr 29, 2014 at 6:46

Jenz

8,3718 gold badges50 silver badges83 bronze badges

3 Comments

Dan Over a year ago

SyntaxError: unterminated regular expression literal [Break On This Error] 	  $(this).SelectProjectBox('359','ddddddddd')>ddddddddd</span><img alt=

analytics.js (line 16, col 54)

Dan Over a year ago

now worked. Can you please describe your answer, so that next time i can solve this type of issue by myself

Jenz Over a year ago

@Sundra..Please refer this link php.net/manual/en/language.types.string.php This will help you.

Collectives™ on Stack Overflow

jquery:SyntaxError: unterminated string literal

4 Answers 4

2 Comments

3 Comments

Comments

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

3 Comments

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related