Page MenuHomePhabricator

VisualEditor: [Regression pre-wmf14] Each category is getting added twice in Betalabs, throws TypeError: categoryItem is undefined
Closed, ResolvedPublic

Description

Screenshot

Steps to reproduce:

1.Open a page with VE
2.Go to Page Settings option>Categories
3.Try to add categories

Observed Result:
After typing into a category name, it getting added twice and trying again throws error:

TypeError: categoryItem is undefined

See the screenshot attached


Version: unspecified
Severity: major

Attached:

Screen_Shot_2014-07-23_at_3.45.56_PM.png (464×737 px, 61 KB)

Details

Reference
bz68484

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:28 AM
bzimport set Reference to bz68484.

Seems to be working again. Probably a deployment gremlin. Again. :-(

Reproduced it today following these steps

1.Open a page with VE
2.Go to Page Settings option>Categories
3.Type a category name in the text field
4.When the drop down box appears, close the dialog
5.Again, reopen the Page Settings option>Categories
6.Now select a category from the drop down box

Observed Result:

It gets added twice,if you cancel the dialog box again it will throw the error:
TypeError: categoryItem is undefined

If you reopen the Page Settings option>Categories, you will get stuck inside it.

Change 151088 had a related patch set uploaded by Alex Monk:
Tear down meta dialog pages when cancelling

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

The patch is not even merged yet, assuming the bug was closed by mistake.

Change 151088 merged by jenkins-bot:
Tear down meta dialog pages when cancelling

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

This is still happening on Betalabs

Can now successfully reproduce on deployment-prep, but not my local wiki...

Also, I am getting a different error this time while closing the dialog second time:
Uncaught TypeError: Cannot read property 'disconnect' of undefined

And I can't reproduce it on deployment-prep any more. Rummana?

Hmm, I can't either. Marking it as fixed.

Verified the fix in production