cannot change owner for recipe - picker shows 'undefined'

Bug #778129 reported by Robert Collins
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Henning Eggers

Bug Description

visit https://code.launchpad.net/~launchpad/+recipe/lpreview-body-daily, click on the edit icon beside the owner, the picker comes up with 'undefined' rather than a drop-down list of teams I could change it to, or a vocab.

Related branches

Revision history for this message
Deryck Hodge (deryck) wrote :

This works for me so something about you is special. Do you get a js console error that might point to the issue?

Revision history for this message
Robert Collins (lifeless) wrote :

Nope. Chromium 64-bit on natty FWIW.

Revision history for this message
William Grant (wgrant) wrote :

Works for me, Chromium 11 on natty amd64.

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 778129] Re: cannot change owner for recipe - picker shows 'undefined'

yay I'm so special.

'13.0.759.0 (84563) Ubuntu 11.04'

Revision history for this message
Ian Booth (wallyworld) wrote :

Works for me too using FF 4.01 and Chrome (not Chromium) 12.0.742.30 beta

Could you please load the page with Firebug enabled and post the content of the Javascript console. That will show the error stopping the full Javascript from being loaded and hence preventing the picker from being wired up correctly.

Revision history for this message
Robert Collins (lifeless) wrote :

I didn't think chromium had firebug? Is there an equivalent I can do in chromium?

Revision history for this message
William Grant (wgrant) wrote :

Menu->Tools->Developer Tools. I suspect this is breakage in your Chromium daily build.

Changed in launchpad:
status: Triaged → Incomplete
Revision history for this message
Robert Collins (lifeless) wrote :

I've updated, same thing.
13.0.768.0 (Developer Build 85577 Linux) Ubuntu 11.04

Revision history for this message
Robert Collins (lifeless) wrote :

loading lazr.activator
launchpad.js:2Got search event:[]
launchpad.js:2Got cancel event.
launchpad.js:2Got search event:[]
launchpad.js:2Got cancel event.

Changed in launchpad:
status: Incomplete → Triaged
Revision history for this message
William Grant (wgrant) wrote :

That's still a daily build, and nobody else has reported this.

Changed in launchpad:
status: Triaged → Incomplete
Revision history for this message
Robert Collins (lifeless) wrote :

It is, but you asked for data and I've provided it, thus this isn't incomplete.

Changed in launchpad:
status: Incomplete → Triaged
Revision history for this message
Robert Collins (lifeless) wrote :

ff 4.0.1 does it too

Revision history for this message
William Grant (wgrant) wrote :

Works fine for me, Firefox 4.0.1 on natty amd64, that same recipe. Can you check the AJAX log for anything obvious wrong?

Revision history for this message
Robert Collins (lifeless) wrote :

[18:35:51.703] GET https://code.launchpad.net/%7Elaunchpad/+recipe/lpreview-body-daily/@@+huge-vocabulary?name=UserTeamsParticipationPlusSelf&search_text=&batch=6&start=0 [HTTP/1.1 200 Ok 568ms]

which returns
{"total_size": 123, "entries": [{"api_uri": "/~lifeless", "css": "sprite person", "description": "<my email>", "value": "lifeless", "title": "Robert Collins (lifeless)"}, {"api_uri": "/~ayatana", "css": "sprite team", "description": "<email address hidden>", "value": "ayatana", "title": "Ayatana Discussion (ayatana)"}, {"api_uri": "/~bzr", "css": "sprite team", "value": "bzr", "title": "Bazaar Developers (bzr)"}, {"api_uri": "/~bzr-gtk", "css": "sprite team", "value": "bzr-gtk", "title": "Bazaar GTK maintainers (bzr-gtk)"}, {"api_uri": "/~bzr-log-rss-devel", "css": "sprite team", "value": "bzr-log-rss-devel", "title": "Bazaar Log RSS developers (bzr-log-rss-devel)"}, {"api_uri": "/~ubuntuone-beta-testers", "css": "sprite team", "value": "ubuntuone-beta-testers", "title": "Beta Testers (ubuntuone-beta-testers)"}]}

for me

Revision history for this message
William Grant (wgrant) wrote :

The picker breaks when there are more than 20 pages to show. In your case there are 21.

Revision history for this message
Henning Eggers (henninge) wrote :

The problem is in addPickerPatcher. It configures a picker that tells the suer to "narrow down the search" if too many results are displayed but does not offer a search box because it does not consider the vocabulary to be huge.

The definition of "huge" probably needs to be revised.

Changed in launchpad:
status: Triaged → In Progress
assignee: nobody → Henning Eggers (henninge)
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Revision history for this message
Deryck Hodge (deryck) wrote :

Testing another case works fine, so I marked this qa-ok. Would love for Robert to confirm for himself, though.

tags: added: qa-ok
removed: qa-needstesting
Revision history for this message
Robert Collins (lifeless) wrote :

works for me on qastaging, though I dunno if I'm in the same group count there.

Revision history for this message
Henning Eggers (henninge) wrote :

Robert, if you get more than 20 batches, the bug can be considered fixed. It not, you need to join a few more teams or remind me who the guy with 200+ teams was. ;-)

Revision history for this message
Robert Collins (lifeless) wrote :

22 batches.

Revision history for this message
Henning Eggers (henninge) wrote :

Huzzah!

William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.