bugyaGTD - Yet Another Getting Things Done. - Bugs: bug #14686, yagtd: crash when ordering tasks...

 
 
Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

bug #14686: yagtd: crash when ordering tasks containing one which had been replaced

Submitted by:  MiKaël NAVARRO <klnavarro>
Submitted on:  Fri 06 Nov 2009 08:25:20 PM UTC  
 
Category: NoneSeverity: 4 - Important
Priority: 7 - HighStatus: Fixed
Privacy: PublicAssigned to: MiKaël NAVARRO <klnavarro>
Open/Closed: Closed

Fri 06 Nov 2009 08:26:24 PM UTC, comment #1:

Debian Bug report= http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=545192

MiKaël NAVARRO <klnavarro>
Project AdministratorIn charge of this item.
Fri 06 Nov 2009 08:25:20 PM UTC, original submission:

Hello Mikael,

Nicolas sent us this yagtd bug report to the Debian BTS. This
appears to affect latest yagtd. Could you have a look? Nicolas
has also included a proposed patch.

Max

---------- Forwarded message ----------
From: Nicolas Maître <valombre@mandar1ne.homelinux.org>
To: Debian Bug Tracking System <submit@bugs.debian.org>
Date: Sat, 05 Sep 2009 17:52:28 +0200
Subject: Bug #545192: yagtd: crash when ordering tasks containing one which had been replaced
Package: yagtd
Version: 0.2.8-1
Severity: normal
Tags: patch

Hi,

There's a issue when I do something like that (reproducible everytime) :

GTD> add test1 @home
Task #1 appended
Task #1 added
GTD> order
@Home
1:(2.049390) test1 @home U:1 I:3 T:1H S:2009-09-05
GTD> replace 1 test replace @home
Task #1 replaced
GTD> order
Traceback (most recent call last):
File "/usr/bin/yagtd", line 1106, in <module>
main(options, args[0])
File "/usr/bin/yagtd", line 1073, in main
gtd_cmd.cmdloop()
File "/usr/lib/python2.5/cmd.py", line 142, in cmdloop
stop = self.onecmd(line)
File "/usr/lib/python2.5/cmd.py", line 219, in onecmd
return func(arg)
File "/usr/bin/yagtd", line 868, in do_order
tasks = [ t for t in tasks if t['start'] <= today ]
TypeError: can't compare datetime.datetime to NoneType

So, I've made a patch which makes it work by setting 'start' entry in the replace function. See the attachment.

Thanks :)

-- System Information:
Debian Release: squeeze/sid
APT prefers unstable
APT policy: (990, 'unstable'), (500, 'testing'), (2, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.30-1-amd64 (SMP w/2 CPU cores)
Locale: LANG=fr_BE.UTF-8, LC_CTYPE=fr_BE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages yagtd depends on:
ii python 2.5.4-2 An interactive high-level object-o
ii python-support 1.0.3 automated rebuilding support for P

yagtd recommends no packages.

yagtd suggests no packages.

-- no debconf information

MiKaël NAVARRO <klnavarro>
Project AdministratorIn charge of this item.

 

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by klnavarro (Follow http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=545192)
  • -unavailable- added by klnavarro (Follow http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=545192)
  • -unavailable- added by klnavarro (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 06 Nov 2009 09:12:52 PM UTCklnavarroStatusNone=>Fixed
      Open/ClosedOpen=>Closed
    Fri 06 Nov 2009 08:25:21 PM UTCklnavarroAttached File-=>Added addStartTimeAfterReplace.patch, #7225
      Carbon-Copy-=>Added -unavailable-
      Carbon-Copy-=>Added -unavailable-
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup