Opened 10 years ago

Closed 10 years ago

#1551 closed defect (no action needed)

JavaScript Timeout while saving...

Reported by: guest Owned by: gogo
Priority: normal Milestone:
Component: Xinha Core Version: trunk
Severity: major Keywords: java script javascript timeout times out


Xinha V0.96.1

Plugins enabled:


I've tried with my plugin list (above) and using the example XinhaConfig?.js as it ships and had the same result.

When I click my form submit button it takes a really (REALLY) long time and Firefox gives me a warning about the script taking too long and asks if I wish to cancel. If I click continue, it takes another couple seconds and then finishes.

I just tried it with FireBug? on, but it seems to be getting stuck in different places (probably not actually getting stuck, but just taking a long time to go through the whole saving process).

One time I noticed it in XinhaCore?.js at line #5855, another time I noticed it in modules/GetHtml/Transform... at line #104 and another time at line #212.

Even if I don't type any HTML into the editor, it takes quite a while to submit, however not as long and I don't get a timeout.

If I have a fairly complex piece of HTML (with tables, images, etc) it always times out.

I've tried various HTML that I have laying around (all in the 10-15K range) and it times out on all of them.

In IE I get "Stack Overflow Line #257"

In Chrome, it seems to work better, but I think that's because Chrome has a much faster implementation of javascript...

Change History (3)

comment:1 Changed 10 years ago by guest

I'm trying to debug this with FireBug?, and I just got a different error:

"Too much recursion" in XinhaCore?.js Line #6437...

comment:2 Changed 10 years ago by guest

Apparently this was a bug in my javascript that was called during the saving of a document. For some reason it worked fine on an older version of Xinha. I changed my javascript 'save' to be an onClick event on the submit button instead of an 'onSubmit' event on the form and everything seems to work fine now.

comment:3 Changed 10 years ago by wymsy

  • Resolution set to no action needed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.