Inconsistent when deleting a notificacion

Description

Steps to reproduce:

1. Deploy SO. Assign to the users the role "Social Office User"
2. Login as one user and send a private message to another user.
3. Login as the second user.
4. Click in "View All" in the notifications dockbar
5. Remove the notification

Notice that the notification has been removed from the notification list. Also notice that the notification still appears in the dockbar.

Activity

Show:

Cynthia Wilburn March 12, 2012 at 2:18 PM

Reopening to add 6.1.1 CE GA2. Close as Fixed.

Cynthia Wilburn February 24, 2012 at 3:25 PM

Removing 6.1.1 CE GA2 from fixed-in. We are selecting the GA2 candidates and will re-add this upon approval from product management.

Michael Saechang January 25, 2012 at 5:06 PM

Committed on:
6.1.x GIT ID: 13f7edf67e1e80765f0c9aaa4963ce9628360fd5.
6.2.x GIT ID: 25471fde7f47810a6bc6abcfe1180715e57d5c48.

Juan Fernández December 19, 2011 at 12:29 AM

Hi Brian: why did you reopened this ticket?
Thanks!

Iliyan Peychev December 15, 2011 at 4:32 AM

Here are some clarifications:

This will not throw exception in IE < 8.

If you have such array:

var testArray = [ 1, 2, 3, ];

and check array's length, like this:

alert(testArray.length);

the reported length will be different. IE will report 4, the other browsers 3. The right behaviour is not clearly specified in ECMA-262-3, but in ECMA-262-5 it is pretty clear:
"If an element is elided at the end of an array, that element does not contribute to the length of the Array."

You probably meant object literals. IE < 8 will throw syntax error if you put comma after the last element, like this:

var myObject = {

a: 1,

b: 2,

c: 3,

};

In ECMA-262-3 this is not allowed, but in ECMA-262-5 it is.

However, I am going to accept your patch in order to avoid such misunderstandings.

Fixed

Details

Assignee

Reporter

Branch Version/s

6.1.x

Backported to Branch

Committed

Git Pull Request

Affects versions

Priority

Zendesk Support

Created May 4, 2012 at 3:08 AM
Updated June 24, 2023 at 3:54 PM
Resolved May 10, 2012 at 5:37 PM
Loading...