last comment date searching is limited

Bug #177202 reported by Brian Murray
4
Affects Status Importance Assigned to Milestone
python-launchpad-bugs
Fix Released
Undecided
Markus Korn

Bug Description

I think it would be useful if it were also possible to search on last comment dates after a specific date or within a date range rather than just for comments before a certain date. This would probably require --lc to take an operator though. So something like:

--lc "d:<2007-11-18"
--lc "d:>2007-10-18"

Related branches

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

I'm changing the package back to python-launchpad-bugs as last comment filtering is done in launchpadbugs/buglist_filter.py . Once it was implemented in p-l-b then a bughelper task would be appropriate.

Revision history for this message
Daniel Holbach (dholbach) wrote :

Ok, sorry. My mistake.

Revision history for this message
Markus Korn (thekorn) wrote :
Changed in python-launchpad-bugs:
status: New → Fix Committed
Revision history for this message
Brian Murray (brian-murray) wrote :

That works great! One other thing that occurred to me would be "d:=2007-01-08" to see all the bugs I commented on today.

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

yes, you are right this would be nice, working on it.

a temporary workaround could be something like:
$ ./bugnumbers -p python-launchpad-bugs -U --lc="d:>2007-12-28&d:<2007-12-29" --format url

Markus

Changed in python-launchpad-bugs:
assignee: nobody → thekorn
status: Fix Committed → In Progress
Revision history for this message
Markus Korn (thekorn) wrote :

Fixed in bughelper.text.dev r223 and py-lp-bugs.text.de r113:

$ ./bugnumbers -p python-launchpad-bugs -U --lc="d:=2007-12-28" --format url
/home/devel/bughelper/text.dev/launchpadbugs/text_bug.py:22: FutureWarning: text_bug API not stable yet
  warnings.warn("text_bug API not stable yet", FutureWarning)
/home/devel/bughelper/text.dev/launchpadbugs/text_buglist.py:7: FutureWarning: text_buglist API not stable yet, LP returns a list of all bugs ever reported
  warnings.warn("text_buglist API not stable yet, LP returns a list of all bugs ever reported", FutureWarning)
https://launchpad.net/bugs/156404
https://launchpad.net/bugs/153846
https://launchpad.net/bugs/172882
https://launchpad.net/bugs/162814
https://launchpad.net/bugs/178940

Markus

Changed in python-launchpad-bugs:
status: In Progress → Fix Committed
Revision history for this message
Brian Murray (brian-murray) wrote :

I believe the default is now "d:<" if there no symbol is given and I'm not certain this is what people would expect. Having it error if there is not a symbol seems best. Or maybe having the default change to "=".

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

It seems the "-U" is causing the issue as:

bugnumbers -p python-launchpad-bugs -U --lc="d:2007-12-28" --format url

returns a different set than:

bugnumbers -p python-launchpad-bugs --lc="d:2007-12-28" --format url

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

My last comment is wrong and really should be expunged.

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

fix released with version 0.2.27 of py-lp-bugs in ubuntu

Changed in python-launchpad-bugs:
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.