onError problem loading an image

[Jürgen Heyn]

Good afternoon,

I am handing a string as a parameter pImage = "Images/Available.jpg"
If this image is NOT available I would like to load Images/NoImage.jpg.
Unfortunately the following code does NOT work.
obj.write('<p><img src=' + pImage + '
onError="self.src="Images/NoImage.jpg"" border="0"></p>');
Where is my mistake ?
I appreciate any hint.
Best regards
Juergen Heyn, Wilhelmshaven, Germany

 

[Ivo]

Good afternoon,

I am handing a string as a parameter pImage = "Images/Available.jpg"
If this image is NOT available I would like to load Images/NoImage.jpg.
Unfortunately the following code does NOT work.
obj.write('<p><img src=' + pImage + '
onError="self.src="Images/NoImage.jpg"" border="0"></p>');

Try onError="this.src
^^^^^

 

[Ivo]

Try onError="this.src
^^^^^

That was too quick. You also have a double-quoted string inside a
double-quoted string, which the interpreter won't understand if you don't
escape the innerquotes:
onError="this.src=\"Images/NoImage.jpg\";"
or
onError="this.src='Images/NoImage.jpg';"

untested,
Ivo

 

[Richard Cornford]

That was too quick. You also have a double-quoted string inside a
double-quoted string, which the interpreter won't understand if you
don't escape the innerquotes:
onError="this.src=\"Images/NoImage.jpg\";"
or
onError="this.src='Images/NoImage.jpg';"

Take another step back any you will observe that the double quoted
string inside the double quoted string is itself inside a single quoted
string of HTML. Escaping the quotes within the onerror handler will work
for the javascript interpreter but will not be enough for the HTML
parser if the inner set of quotes remain double.

obj.write('<p><img src="'+pImage +
'" onError="this.src=\'Images/NoImage.jpg\';" border="0"></p>');

Richard.

 

[Jürgen Heyn]

Gentlemen,
thank you very much for your response.
Richard's suggestion works perfectly.
Have a nice rest of this sunday.
Best regards
Juergen Heyn, Wilhelmshaven, Germany