status filtering only works with a package not with a url

Bug #139679 reported by Brian Murray
2
Affects Status Importance Assigned to Milestone
Bug Helper
Fix Released
Medium
Markus Korn

Bug Description

Using python-launchpad-bugs revision 53 and bughelper revision 193 - I noticed that status filtering with bugnumbers only works if you specify a package name not a url. For example:

./bugnumbers --status "New" -l https://bugs.launchpad.net/ubuntu/+source/inkscape/+bugs
115589 (Confirmed, Low) - inkscape pyxml missing python-xml
123336 (New, Undecided) - inkscape crashed with SIGSEGV
* and a bunch more

./bugnumbers --status "New" -p inkscape
123336 (New, Undecided) - inkscape crashed with SIGSEGV
139672 (New, Undecided) - inkscape does not adapt the window to screen size (e.g. portrait)

This becomes more problematic when using a url for bugs assigned to a team or an individual.

Revision history for this message
Markus Korn (thekorn) wrote :

Thanks Brian for reporting thsi, I'm going to work on this.

Changed in bughelper:
assignee: nobody → thekorn
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Markus Korn (thekorn) wrote :

It is easy to fix, but there is one question open for me:
A querie like
./bugnumbers --status "New" -l https://bugs.edge.launchpad.net/bughelper/+bugs?field.status%3Alist=Incomplete
is possible. Which result do you expect in this case:
 * --status="New,Incomplete"
 * --status option overwrites the url option: --status="New"
 * url option overwrites the --status option: --status="Incomplete"

Markus

Revision history for this message
Brian Murray (brian-murray) wrote :

I think the thing that would lead to the least amount of confusion would be to have query exit indicating that there are "conflicting" options in the filtering. Barring that I would have --status overwrite the url option as --status is optional.

Revision history for this message
Markus Korn (thekorn) wrote :

This two patches fix this bug. The patch against bughelper.main adds a --ignore_conflicts option to bughelper and bugnumbers (in this case the commandline option would overwrite the url option)

Example usage:

markus@thekorn:~/devel/bughelper/main$ ./bugnumbers --status "New" -l https://bugs.launchpad.net/ubuntu/+source/inkscape/+bugs
146352 (New, Undecided) - [Gutsy] Use Tango styled icon for Inkscape
140529 (New, Undecided) - inkscape shows wrong stroke weight of a "plain" svg
123336 (New, Undecided) - inkscape crashed with SIGSEGV
139672 (New, Undecided) - inkscape does not adapt the window to screen size (e.g. portrait)
145145 (New, Undecided) - ill2svg.pl uses Image::Magick instead of Graphics::Magick
149034 (New, Undecided) - Broken illustrator importer
markus@thekorn:~/devel/bughelper/main$ ./bugnumbers --status "Invalid" -l "https://bugs.launchpad.net/ubuntu/+source/inkscape/+bugs?field.status%3Alist=New"
ValueError: conflict filter options (field.status:list)
markus@thekorn:~/devel/bughelper/main$ ./bugnumbers --status "Invalid" -l "https://bugs.launchpad.net/ubuntu/+source/inkscape/+bugs?field.status%3Alist=New" --ignore_conflicts
17411 (Invalid, Medium) - crash upon start
36502 (Invalid, Medium) - [due to xgl]Inkscpae crashes
116877 (Invalid, Medium) - inkscape crashed tracing lo-res bitmap
10768 (Invalid, High) - inkscape: FTBFS on mips(el)
9219 (Invalid, Medium) - inkscape: exports empty ps/eps
107670 (Invalid, Medium) - [apport] inkscape crashed with SIGSEGV in ()
56983 (Invalid, Undecided) - Inkscape v0.44
63770 (Invalid, Wishlist) - Inkscape cannot render Chinese text in bold or italic
14496 (Invalid, Medium) - Sync with upstream release of inkscape 0.41?
[...]
markus@thekorn:~/devel/bughelper/main$

Revision history for this message
Markus Korn (thekorn) wrote :

patch against py-lp-bugs.main r59: Adding ignore_conflicts argument to get_modified_url() and get_url_opt() in buglist_filter.py

Revision history for this message
Markus Korn (thekorn) wrote :

patch against bughelper.main r196: adding --ignore_conflicts cmd-line option to bughelper/bugnumbers, modify given url if necessary.

Changed in bughelper:
status: Confirmed → Fix Committed
Revision history for this message
Markus Korn (thekorn) wrote :

python-launchpad-bugs (0.2.20) gutsy; urgency=low

  * buglist_filter.py, connector.py: adding --ignore_conflicts cmd-line option
    to bughelper/bugnumbers, modify given url if necessary (LP: #139679)

 -- Markus Korn <email address hidden> Mon, 08 Oct 2007 11:39:49 +0200

bughelper (0.2.9) gutsy; urgency=low

  [ Markus Korn ]
  * bugHelper/commandLine.py, bughelper, bugnumbers: dding ignore_conflicts
    argument to get_modified_url() and get_url_opt() in buglist_filter.py
    (LP: #139679)

  [ Daniel Holbach ]
  * debian/control: bumped Depends to python-launchpad-bugs (>= 0.2.20).

 -- Markus Korn <email address hidden> Mon, 08 Oct 2007 11:41:41 +0200

Changed in bughelper:
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.