Opened 8 years ago
Last modified 4 years ago
#17983 new defect
ItemFileReadStore.js (IE8)
| Reported by: | aleung | Owned by: | aleung |
|---|---|---|---|
| Priority: | undecided | Milestone: | 1.14 |
| Component: | Data | Version: | 1.9.3 |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: |
Description
Failed on line 564 (uncompressed file), IE8 array count is off by 1 because of the prototype attribute. A quick fix is to add check 'if (this._arrayOfTopLevelItems[i] === undefined) break;' within the for loop line 557 to 565.
Attachments (4)
Change History (11)
comment:1 Changed 8 years ago by
| Owner: | set to aleung |
|---|---|
| Status: | new → pending |
comment:2 Changed 8 years ago by
| Status: | pending → new |
|---|
This seems to be cause by an IE8 bug. If the storage is explicitly define as array, then the code will work. If leave it up to IE8, then array length is off by one. IE9 or above does not have the issue.
Does the assumption of array index is always a number correct? I will work on the example.
Changed 8 years ago by
| Attachment: | dojo_bug1_screen1.png added |
|---|
Changed 8 years ago by
| Attachment: | dojo_bug1_screen2.png added |
|---|
comment:3 Changed 8 years ago by
I attached two screen captures
The first one shows variable test created as an array. Variable test1 created from code similar to dojo example. The variable data is from my code created by program. Notice the type different for test1.items "[ [object Object],[object Object]]" and data.items "[object(Array)]". The second screen captures show unrolling data.items. Items enumerated to 106 and length show 108.
Changed 8 years ago by
| Attachment: | itemFileReadStore.html added |
|---|
comment:5 Changed 8 years ago by
| Status: | pending → new |
|---|
Attachment (itemFileReadStore.html) added by ticket reporter.
comment:6 Changed 6 years ago by
| Milestone: | tbd → 1.12 |
|---|
comment:7 Changed 4 years ago by
| Milestone: | 1.13 → 1.14 |
|---|
Can you attach a test case?