package picker lists unpublished (invalid) packages
Bug #42298 reported by
Luis Villa
This bug affects 8 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Colin Watson | ||
Bug Description
I just filed a bug against 'kernel-image-2.6'- I picked it out a list of results for 'kernel-image-2.6', since I figured that was most likely to get the most coverage on searches- and got the following note:
"Thank you for your bug report.
Note that the package 'kernel-image-2.6' is not published in Ubuntu; the bug has therefore been targeted to the distribution only."
If the package is not in the distro, and I'm filing a bug against the distro, *don't make it an option to file a bug against it*. The results shouldn't include kernel-image-2.6; it should include whatever is actually in the distro I'm filing against.
The field validator does not match the vocabulary. The vocabulary is doing a lot of work show suggestions that will never be accepted.
Related branches
lp:~cjwatson/launchpad/fix-dsp-vocab-picker
- William Grant: Approve (code)
-
Diff: 781 lines (+216/-173)12 files modifiedlib/lp/app/javascript/picker/picker_patcher.js (+4/-2)
lib/lp/app/javascript/picker/tests/test_picker_patcher.js (+35/-2)
lib/lp/app/widgets/popup.py (+14/-1)
lib/lp/app/widgets/templates/distributionsourcepackage-picker.pt (+29/-0)
lib/lp/app/widgets/templates/form-picker-macros.pt (+1/-1)
lib/lp/app/widgets/tests/test_launchpadtarget.py (+2/-12)
lib/lp/app/widgets/tests/test_popup.py (+33/-7)
lib/lp/bugs/browser/tests/test_bugalsoaffects.py (+1/-18)
lib/lp/bugs/browser/widgets/bugtask.py (+11/-2)
lib/lp/registry/tests/test_distributionsourcepackage_vocabulary.py (+45/-92)
lib/lp/registry/vocabularies.py (+29/-35)
lib/lp/services/webapp/configure.zcml (+12/-1)
lp:~cjwatson/launchpad/bugtaskeditview-spn-dsp-vocab
- William Grant: Approve (code)
-
Diff: 173 lines (+70/-16)3 files modifiedlib/lp/bugs/browser/bugtask.py (+27/-4)
lib/lp/bugs/browser/tests/test_bugtask.py (+42/-11)
lib/lp/bugs/browser/widgets/bugtask.py (+1/-1)
lp:~cjwatson/launchpad/distribution-filebug-dsp-vocab
- William Grant: Approve (code)
-
Diff: 750 lines (+363/-75)9 files modifieddatabase/sampledata/current-dev.sql (+1/-1)
database/sampledata/current.sql (+1/-1)
lib/lp/bugs/browser/bugtarget.py (+48/-20)
lib/lp/bugs/browser/tests/test_bugtarget_filebug.py (+22/-2)
lib/lp/bugs/browser/widgets/bugtask.py (+46/-5)
lib/lp/bugs/doc/bugtask-package-widget.txt (+128/-23)
lib/lp/bugs/tests/test_doc.py (+26/-1)
lib/lp/registry/tests/test_distributionsourcepackage_vocabulary.py (+52/-3)
lib/lp/registry/vocabularies.py (+39/-19)
lp:~cjwatson/launchpad/git-repository-target-widget-dsp-vocab
- William Grant: Approve (code)
-
Diff: 113 lines (+35/-4)2 files modifiedlib/lp/code/browser/widgets/gitrepositorytarget.py (+12/-2)
lib/lp/code/browser/widgets/tests/test_gitrepositorytargetwidget.py (+23/-2)
lp:~cjwatson/launchpad/productseries-ubuntupkg-dsp-vocab
- William Grant: Approve (code)
-
Diff: 526 lines (+164/-86)7 files modifiedlib/lp/app/widgets/popup.py (+85/-1)
lib/lp/bugs/browser/bugtracker.py (+2/-2)
lib/lp/bugs/browser/widgets/bugtask.py (+11/-74)
lib/lp/registry/browser/productseries.py (+33/-4)
lib/lp/registry/browser/tests/test_packaging.py (+20/-3)
lib/lp/registry/tests/test_distributionsourcepackage_vocabulary.py (+11/-0)
lib/lp/registry/vocabularies.py (+2/-2)
lp:~cjwatson/launchpad/potemplate-dsp-vocab
- William Grant: Approve (code)
-
Diff: 583 lines (+347/-28)8 files modifiedlib/lp/app/widgets/popup.py (+8/-0)
lib/lp/app/widgets/templates/distributionsourcepackage-picker.pt (+5/-0)
lib/lp/bugs/browser/widgets/bugtask.py (+1/-2)
lib/lp/bugs/doc/bugtask-package-widget.txt (+18/-21)
lib/lp/translations/browser/potemplate.py (+51/-2)
lib/lp/translations/browser/tests/test_potemplate_views.py (+68/-3)
lib/lp/translations/browser/widgets/potemplate.py (+62/-0)
lib/lp/translations/browser/widgets/tests/test_potemplate.py (+134/-0)
lp:~cjwatson/launchpad/translation-import-queue-entry-dsp-vocab
- William Grant: Approve (code)
-
Diff: 315 lines (+192/-5)4 files modifiedlib/lp/translations/browser/tests/test_translationimportqueueentry.py (+46/-2)
lib/lp/translations/browser/translationimportqueue.py (+34/-3)
lib/lp/translations/browser/widgets/tests/test_translationimportqueue.py (+78/-0)
lib/lp/translations/browser/widgets/translationimportqueue.py (+34/-0)
Revision history for this message
| Luis Villa (luis-villa) wrote | #1 |
| Changed in malone: | |
| status: | New → Triaged |
Revision history for this message
| Diogo Matsubara (matsubara) wrote Re: Popup package search lists unpublished packages | #2 |
| description: | updated |
| summary: |
- Popup package search lists unpublished packages + package picker lists unpublished (invalid) packages |
| tags: | added: package-picker vocabulary |
| Changed in launchpad: | |
| importance: | Medium → High |
| tags: | added: disclosure |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #3 |
| Changed in launchpad: | |
| assignee: | nobody → Steve Kowalik (stevenk) |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| tags: |
added: qa-untestable removed: qa-needstesting |
| tags: |
added: target-picker removed: package-picker |
| Changed in launchpad: | |
| status: | In Progress → Fix Released |
| Changed in launchpad: | |
| assignee: | Steve Kowalik (stevenk) → nobody |
| status: | Fix Released → In Progress |
| tags: | removed: qa-untestable |
| tags: | removed: disclosure |
| Changed in launchpad: | |
| status: | In Progress → Triaged |
| Changed in launchpad: | |
| assignee: | nobody → Curtis Hovey (sinzui) |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| Changed in launchpad: | |
| status: | In Progress → Triaged |
| Changed in launchpad: | |
| assignee: | Curtis Hovey (sinzui) → nobody |
| Changed in launchpad: | |
| status: | Triaged → In Progress |
| assignee: | nobody → Colin Watson (cjwatson) |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #5 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Colin Watson (cjwatson) wrote | #6 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #8 |
| tags: |
added: qa-needstesting removed: qa-ok |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #9 |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #10 |
| tags: |
added: qa-needstesting removed: qa-ok |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #11 |
| tags: |
added: qa-needstesting removed: qa-ok |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #12 |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #13 |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #14 |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Colin Watson (cjwatson) wrote | #15 |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #16 |
| Changed in ubuntu: | |
| status: | New → Confirmed |
| no longer affects: | ubuntu |
To post a comment you must log in.
And I note when trying to fix the problem that you already have the ability to do this filtering, so fixing this should be less difficult than I first assumed (though potentially still difficult.)