Home page: run time error
Home page: run time error
Author
Discussion

jeremyc

Original Poster:

26,779 posts

305 months

Friday 7th May 2004
quotequote all
In www.pistonheads.com/default1.htm line 116: Unterminated String Constant.

Nacnud

2,190 posts

290 months

Monday 10th May 2004
quotequote all
Me too - it's really annoying.
Happens both on initial page load and on return to the page via 'Prev Page' after reading a feature.
Rest of PH seems to be error free.

pebbledash

795 posts

287 months

Monday 10th May 2004
quotequote all
Ditto.. ask me if I wish to debug too as i have the debugger installed.

PetrolTed

34,461 posts

324 months

Monday 10th May 2004
quotequote all
Mmm... sorry chaps. Not getting it here. Could you let me know what line number it reports (made some changes earlier to try and track it down).

jeremyc

Original Poster:

26,779 posts

305 months

Monday 10th May 2004
quotequote all
PetrolTed said:
Mmm... sorry chaps. Not getting it here. Could you let me know what line number it reports (made some changes earlier to try and track it down).
Line 117, and debugging points here:

onClick="wnd=window.open
('http://www.pistonheads.com/terms.htm', '_tc',
'width=400,height=500,scrollbars=yes,
resizable=0,toolbar=0' ); wnd.focus()" target="_self">

I'm guessing (but no expert) that its the extra inverted comma that I've emboldened that is the problem.

PetrolTed

34,461 posts

324 months

Monday 10th May 2004
quotequote all


Thanks Jeremy. For some reason by browser is refusing to display debug info currently.

jeremyc

Original Poster:

26,779 posts

305 months

Monday 10th May 2004
quotequote all
Fixed now.

Ta!

Nacnud

2,190 posts

290 months

Monday 10th May 2004
quotequote all
PetrolTed said:


Thanks Jeremy. For some reason by browser is refusing to display debug info currently.

Ditto.
Seems it was a side-effect of one of the XP upgrades.
One of my colleagues solved this by reinstalling IE.

I haven't bothered as I've got a shiny new laptop waiting to be collected next time I go to the office.