70559 - Duplicate entries are not getting checked

Status: VERIFIED DUPLICATE of bug 67918

(Core :: DOM: Editor, defect)

Description

[shrirang khanzode]

this bug spawned after 70558 was patched (entries do not show up in list)

Steps:
1 Click on the image icon in Composition Toolbar in composer
2 Add any image
3 Double click on the image to open Image Properties dialog
4 Click on 'Advanced Edit'
5 Select the third tab 'Javascript Events'
6 Add a event handler,say onClick()' and enter a value for it
7 Again, in the Name field add 'onClick()' and a value for it and it actually 
  gets added to the list

Expected: Addition of duplicate events should not be possible.

patch coming ...

Comment 1

[shrirang khanzode]

reassign :cmanske, correcting summary

Comment 2

[shrirang khanzode]

Attachment: patch

Comment 3

[Charles Manske]

Hmmm. This problem doesn't exist with patch to fix bug 67918.
My version of EdAEJSEAttributes.js has:
if( !name)
  return;
near where your patch does a similar test, but that code isn't new (isn't part
of patch to 67918), which seems strange!
Test the fix for 67918 and see if this problem occurs. In my tree, if you try
to type the same "name" as an existing attribute, it disables the "Add" button,
so you can't add a duplicate.


*** This bug has been marked as a duplicate of 67918 ***

Comment 4

[sujay]

verified in 3/1 build.