Closed Bug 83107 Opened 23 years ago Closed 23 years ago

Javascript table sort broken

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla0.9.2

People

(Reporter: casper, Assigned: jst)

References

()

Details

(Keywords: regression)

From Bugzilla Helper: User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9+) Gecko/20010523 BuildID: 2001052304 Javascript table sort demo works in IE, and in DOM 1 compliant brosers, but doesn't work on this build. I think it was working on 0.8.1. Reproducible: Always Steps to Reproduce: 1.Go to http://webfx.eae.net/dhtml/tablesort/demo.html 2.Click on the column head 3.Rows don't sort, and column title is incorrect Actual Results: Column title changed to variable name. Expected Results: Rows should sort, and column title should change to show sort order.
Also fails at http://mkruse.netexpress.net/javascript/sorttable/ IE and NS 4.77 work.
Rick, are there any errors in the JS console?
Using Mozilla nightly binary 2001052710 WinNT. Boris has the right idea: in the JS Console, you get this error every time you try to do a table sort: Error: uncaught exception: Permission denied to create wrapper for object Reassigning to Security:General. cc'ing jst, jband in case this is more of an XPConnected DOM issue than a Security one - Also happens with the Linux nightly, same date. Changing OS --> All.
Assignee: rogerl → mstoltz
Status: UNCONFIRMED → NEW
Component: Javascript Engine → Security: General
Ever confirmed: true
OS: Windows 2000 → All
QA Contact: pschwartau → ckritzer
jst: The problem here is that nsTableCellCollection does not expose nsIClassInfo. Is should, right?
Assignee: mstoltz → jst
Yup, that's the problem, thanks for tracking this down, John.
Status: NEW → ASSIGNED
Component: Security: General → DOM HTML
Keywords: regression
Hardware: PC → All
Target Milestone: --- → mozilla0.9.2
The patch in bug 85560 fixes this bug as well by adding classinfo to an ancestor class of nsTableCellCollection.
Fix checked in (reviews and so forth are in bug 85560)
Oops. Forgot to mark fixed.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Marking VERIFIED FIXED on: -MacOS91 2001-07-10-03-0.9.2 -LinRH62 2001-07-10-08-0.9.2 -Win98SE 2001-07-10-05-0.9.2
Status: RESOLVED → VERIFIED
Keywords: vtrunk
Component: DOM: HTML → DOM: Core & HTML
QA Contact: ckritzer → general
You need to log in before you can comment on or make changes to this bug.