Merge lp:~zeitgeist/zeitgeist/timerange_and_event_value_fix into lp:zeitgeist/0.1
Proposed by
Seif Lotfy
Status: | Merged |
---|---|
Merged at revision: | 1547 |
Proposed branch: | lp:~zeitgeist/zeitgeist/timerange_and_event_value_fix |
Merge into: | lp:zeitgeist/0.1 |
Diff against target: |
47 lines (+12/-4) 1 file modified
zeitgeist/datamodel.py (+12/-4) |
To merge this branch: | bzr merge lp:~zeitgeist/zeitgeist/timerange_and_event_value_fix |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Siegfried Gevatter | Pending | ||
Review via email: mp+32574@code.launchpad.net |
Description of the change
I fixed the TimeRange.always to be consistent: so instead of (-maxint, maxint) it has become (0, maxint)
Also I check for values of the Event constructor new_for_values to detect faulty arguments and raise an exception
To post a comment you must log in.
Seif, with your changes TimeRange. always( ).is_always( ) will return False. I wouldn't call that a bugfix. :P