Maniphest T124878

Can't save edited collection
Closed, ResolvedPublic
Actions

Tags

Subscribers

TerraCodes, Jdlrobson, phuedx and 4 others

Assigned To

Jdlrobson

Authored By

Jhernandez, Jan 27 2016

Description

Uncaught TypeError: Cannot read property 'addPagesToCollection' of undefined

calls.push(this.collectionGateway.addPagesToCollection(collection.id, additions));

Fantastic...

Related Objects

Mentioned Here: T123395: Error on saving new collection

Event Timeline

Jhernandez created this task.Jan 27 2016, 11:32 AM

Jhernandez raised the priority of this task from to Needs Triage.

Jhernandez updated the task description. (Show Details)

Jhernandez added a project: Gather.

Jhernandez added a subscriber: Jhernandez.

Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald TranscriptJan 27 2016, 11:32 AM

phuedx triaged this task as Unbreak Now! priority.Jan 27 2016, 11:33 AM

phuedx added a project: Reading-Web-Sprint-64-Five Sprints at Freddy's.

phuedx set Security to None.

phuedx moved this task from Needs Analysis to To Do on the Reading-Web-Sprint-64-Five Sprints at Freddy's board.

I am unable to reproduce the problem. Can you please list the steps to do so? Thanks!

@Jhernandez: Could you add your browser and OS versions to the report, as well as steps to reproduce (as @Pranavmk98 has asked for above).

phuedx moved this task from To Do to Needs Analysis on the Reading-Web-Sprint-64-Five Sprints at Freddy's board.Jan 27 2016, 5:09 PM

Sorry, I thought it was a general problem:

OS: OSX 10.11
Chrome 47.0.2526.111 & Firefox 45

Steps:

Log in
Visit https://en.m.wikipedia.org/wiki/Special:Gather/id/617/Places_I%27ve_visited
Click Edit under description
Modal opens.
1. Click on search bar
Type Mexico for example
1. Select mexico
2. Back to the edit overlay
Click save
1. Buttton stays dimmed / disabled
Open console and see the error in the description triggered by the line in the description

Are you sure this is not fixed on beta labs?
I can edit http://en.m.wikipedia.beta.wmflabs.org/wiki/Special:Gather/id/175/A28 just fine.
Sounds a lot like T123395

@Jdlrobson You're right.

Let's keep this open to figure out what happened with this weeks deploy.

^ Given the comments above.

This was the commit that fixed it https://gerrit.wikimedia.org/r/#/c/263600/3/resources/ext.gather.page.search/CollectionPageList.js

Done the 12th, we did a release the 21st, not deployed yet.

We probably want to swat https://gerrit.wikimedia.org/r/266793 and https://gerrit.wikimedia.org/r/266792 into wmf11?

Jdlrobson moved this task from To Do to Code Review on the Reading-Web-Sprint-64-Five Sprints at Freddy's board.Jan 27 2016, 7:06 PM

Seems like a good candidate for it.

In SWAT window.
https://wikitech.wikimedia.org/w/index.php?title=Deployments&type=revision&diff=274646&oldid=274547

Confirmed fixed.

Jdlrobson moved this task from Code Review to Done on the Reading-Web-Sprint-64-Five Sprints at Freddy's board.Jan 28 2016, 12:37 AM

Whoops. I forgot to say it's fixed in wmf11 so we'll have to wait till tomorrow for it to be fixed on enwiki.

(Feel free to swat to wmf10 if you are on the European timezone and have the energy)

No need if it's going out this afternoon (GMT).

Confirmed on en.wikivoyage which was broken yesterday.

Thanks @Jdlrobson, I'm signing off on this.

Jhernandez closed this task as Resolved.Jan 28 2016, 10:22 AM

Jhernandez moved this task from Ready for Signoff to Done on the Reading-Web-Sprint-64-Five Sprints at Freddy's board.

Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptJun 2 2016, 11:10 PM