Opened 10 years ago

Closed 8 years ago

#9696 closed defect (patchwelcome)

Widget won't run on IE7 with "object does not support this property or method error" while it runs correctly at other browser

Reported by: gchan Owned by: Neil Roberts
Priority: low Milestone: future
Component: DojoX DTL Version: 1.3.2
Keywords: Cc:
Blocked By: Blocking:

Description

An issue that only occurs on IE7 where IE7 would throw an "object does not support this property or method error"

The error occurs in "dojoRenderIssue.widgets.pagingalbumsb". It occurs after startIndicator() around dojox.data.JsonRestStore?.fetch() is called.

Attached are the HTML test file, dojo.js file, the layer file (both compressed and uncompressed)

Attachments (7)

dojo.js (79.7 KB) - added by gchan 10 years ago.
indexPageNormal.js (129.2 KB) - added by gchan 10 years ago.
pagingalbums.html (929 bytes) - added by gchan 10 years ago.
dojoRender.profile.js (329 bytes) - added by gchan 10 years ago.
The profile that create the build
pagingalbumsb.js (5.2 KB) - added by gchan 10 years ago.
The source code of the widget
pagingalbums.2.html (929 bytes) - added by gchan 10 years ago.
template file for the widget
indexBugNormal.html (860 bytes) - added by gchan 10 years ago.
HTML file that drive the widget

Download all attachments as: .zip

Change History (17)

Changed 10 years ago by gchan

Attachment: dojo.js added

Changed 10 years ago by gchan

Attachment: indexPageNormal.js added

Changed 10 years ago by gchan

Attachment: pagingalbums.html added

comment:1 Changed 10 years ago by gchan

got problem upload uncompressed layer. Upload raw source code and its html instead

comment:2 Changed 10 years ago by bill

Component: GeneralDojoX Data
Owner: changed from anonymous to Jared Jurkiewicz

You don't need to upload dojo since we all have copies of dojo, but you do need to find some way to upload uncompressed versions of your test files, and also a test case much shorter than 129KB.

Is this a JsonRestStore error?

comment:3 Changed 10 years ago by Jared Jurkiewicz

Owner: changed from Jared Jurkiewicz to kriszyp

Changed 10 years ago by gchan

Attachment: dojoRender.profile.js added

The profile that create the build

Changed 10 years ago by gchan

Attachment: pagingalbumsb.js added

The source code of the widget

Changed 10 years ago by gchan

Attachment: pagingalbums.2.html added

template file for the widget

Changed 10 years ago by gchan

Attachment: indexBugNormal.html added

HTML file that drive the widget

comment:4 Changed 10 years ago by gchan

Is this a JsonRestStore? error?

No, the error is actually "object does not support this property or method error". It only occurs on IE7. Other browser can actually run the widget correct.

For the test case, I uploaded 4 files 1) build profile file dojoRender.profile.js 2) widget source pagingalbums.js 3) widget template pagingalbums.html 4) HTML that drive the widget indexBugNormal.html

folder structure for the widget dojoRender/dojoRenderIssue/widgets/pagingalbums.js dojoRender/dojoRenderIssue/widgets/template/pagingalbums.html

Since there is an upload limit. The above file should be enough to build and create the test case.

comment:5 Changed 10 years ago by bill

I hope this test doesn't really require doing a build, you can likely reproduce the problem with just a dojo.require() of the things you need.

I tried running the test but it's failing because the data isn't there: Error: Unable to load /api2/album/recent?page=1 status:404, same problems on FF or IE.

(Also, it's unclear what component this bug is in, could be a DTL problem etc.)

comment:6 Changed 10 years ago by gchan

I got permission to put the code on our server on trac. The offending code can be at http://gerald.digisphereinc.com/indexBugNormal.html

I just double check with FF and IE7. IE7 got an error but FF does not.

comment:7 Changed 10 years ago by bill

Component: DojoX DataDojoX DTL

OK, I'm seeing an error about this.rendered being undefined, which is a problem in DTL.

comment:8 Changed 10 years ago by bill

Owner: changed from kriszyp to Neil Roberts

comment:9 Changed 10 years ago by dante

Milestone: tbdfuture

comment:10 Changed 8 years ago by ben hockey

Priority: highlow
Resolution: patchwelcome
Status: newclosed

dtl is unsupported. any further patches are welcome.

Note: See TracTickets for help on using tickets.