Page MenuHomePhabricator

VisualEditor: [Regression] Don't let the user insert the same category multiple times
Closed, ResolvedPublic

Description

screenshot showing multiple Categories with same title

Environment- production, test2

1>Select Categories. Add a category for ex- “Japan”.
2>The cursor moves to the next input field. Type in “Japan” again. Click somewhere else in the modal, to move the focus.
3>Click back in the input field. While the cursor is in the field,hit enter key.
4>The category with the same name Japan gets added.See screenshot.

In beta, at step#2, after typing in the title, just click twice in the input field(to close the drop-down) and hit enter key. The category with the same title gets added.


Version: unspecified
Severity: normal

Attached:

s.png (373×1 px, 49 KB)

Details

Reference
bz72906

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:59 AM
bzimport set Reference to bz72906.

Confirmed in betalabs, test2, and production.

gerritadmin wrote:

Change 171033 had a related patch set uploaded by Alex Monk:
Kill MWCategoryWidget#onLookupEnter

https://gerrit.wikimedia.org/r/171033

gerritadmin wrote:

Change 171033 merged by jenkins-bot:
Kill MWCategoryWidget#onLookupEnter

https://gerrit.wikimedia.org/r/171033

The fix is verified in betalabs and test2.