Error in inserting objects into array

RESOLVED INCOMPLETE

Status

()

Firefox
Untriaged
RESOLVED INCOMPLETE
a year ago
a year ago

People

(Reporter: Bob McCarthy, Unassigned, NeedInfo)

Tracking

({testcase-wanted})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(6 attachments)

13.94 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document
Details
316.70 KB, image/jpeg
Details
317.08 KB, image/jpeg
Details
316.59 KB, image/jpeg
Details
2.00 KB, text/html
Details
108.96 KB, application/x-javascript
Details
(Reporter)

Description

a year ago
Created attachment 8826201 [details]
code sample.docx

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.87 Safari/537.36

Steps to reproduce:

I have an array of objects saved  as a globally accessible (i.e window.) variable.  When I insert objects into the array of objects, Firefox propagates the latest object entry into all of the slots rather than just at the position of the last index entry.


Actual results:

I have an array of objects saved  as a globally accessible (i.e window.) variable.  When I insert objects into the array of objects, Firefox propagates the latest object entry into all of the slots rather than just at the position specified by the index entry.  This does work if only a single variable value is inserted rather than an object.


Expected results:

Firefox should have inserted the object at the correct position in the array of objects and not affected the other array entries.

Comment 1

a year ago
Could you provide a testcase on https://jsfiddle.net/ or http://codepen.io/ which are more user-fridenly to test than a .doc file.
Flags: needinfo?(rmccarthy)
Keywords: testcase-wanted
(Reporter)

Comment 2

a year ago
Created attachment 8827527 [details]
Snapshots of debugger output

Here are snapshots of the Firefox debugger output where I see the error.

To explain what is going on, I am added events to a users calendar.  The events get saved to disk and get displayed on a calendar grid with an entry for each event.  Events can be recurring where they can repeat of the same time each day, each week or each month.  The events can also be added as a custom schedule, recurring on the several days and times each week or each month.

In the first snapshot you can see where I populated the first object into the array "window.khi.calendar.events".  No problem when the first entry is added.

In the second snapshot you can see where I added the second event object in the second slot.  However in the third snapshot, you can see that the new object entry has erroneously propagated into the first array position also.

If I have two or more events, each new event will propagate into all of the previous array positions.


If you want to test the application remotely, I can give you a logon and password and push this to our server.

I've attached some sample code if you want to play with this.  I will also add this info to Bugzilla.
(Reporter)

Comment 3

a year ago
Created attachment 8827528 [details]
Debugger snapshot2

See previous comments
(Reporter)

Comment 4

a year ago
Created attachment 8827529 [details]
Debugger snapshot 3

See previous comments

Comment 5

a year ago
It doesn't really help, we need a testcase (html file) to reproduce the issue on our machines.
(Reporter)

Comment 6

a year ago
Created attachment 8828754 [details]
Samplle code

This is the sample code that I created to test for the Firefox move object bug.  I will add a second attachment with the actual code where I see the bug.  This simple test case works correctly.
(Reporter)

Comment 7

a year ago
Created attachment 8828757 [details]
This is the actual code where I see the bug.

This is the actual code. It fails at lines 1475 and 1513.  You have snapshots of the debugger output.  If you want to test the code remotely, I can send you the navigation path to the spot where it fails.

Comment 8

a year ago
How to reproduce the issue?
What does this second script do? How is it related to the 1st attachment?

Please, read carefully: could you build a self-contained testcase with JS scripts included.
Marking this as Resolved - Incomplete due to the lack of response from the reporter.
If anyone can still reproduce it on latest versions, feel free to reopen the issue and provide more information.
Status: UNCONFIRMED → RESOLVED
Last Resolved: a year ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.