Modify

Opened 3 years ago

Closed 2 years ago

#9270 closed defect (fixed)

[Patch] "Local variable referenced before assignment" error (r10732)

Reported by: rjollos Owned by: ChrisNelson
Priority: high Component: TracJsGanttPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

Following some change to a ticket, the Gantt chart stopped displaying. This occurs with r10732 of the plugin. We now see this:


I hope to send you a patch shortly.

Attachments (2)

LocalVariableStartReferrenedBeforeAssignment.png (8.0 KB) - added by rjollos 3 years ago.
th9270-r10732-0.11.7.patch (718 bytes) - added by rjollos 3 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 3 years ago by rjollos

Here is the traceback.

2011-10-15 15:16:29,216 Trac[formatter] ERROR: Macro TracJSGanttChart(milestone=M1 - I11, lwidth=600, colorBy=owner, expandClosedTickets=0, startDate=0, endDate=0, res=0) failed:
Traceback (most recent call last):
  File "/usr/lib/python2.6/site-packages/Trac-0.11.7-py2.6.egg/trac/wiki/formatter.py", line 484, in _macro_formatter
    return macro.process(args, in_paragraph=True)
  File "/usr/lib/python2.6/site-packages/Trac-0.11.7-py2.6.egg/trac/wiki/formatter.py", line 180, in process
    text = self.processor(text)
  File "/usr/lib/python2.6/site-packages/Trac-0.11.7-py2.6.egg/trac/wiki/formatter.py", line 167, in _macro_processor
    text)
  File "/usr/lib/python2.6/site-packages/Trac_jsGantt-0.9_r10732-py2.6.egg/tracjsgantt/tracjsgantt.py", line 1108, in expand_macro
    tasks = self._add_tasks(options)
  File "/usr/lib/python2.6/site-packages/Trac_jsGantt-0.9_r10732-py2.6.egg/tracjsgantt/tracjsgantt.py", line 1070, in _add_tasks
    self._schedule_tasks(options)
  File "/usr/lib/python2.6/site-packages/Trac_jsGantt-0.9_r10732-py2.6.egg/tracjsgantt/tracjsgantt.py", line 735, in _schedule_tasks
    _schedule_task_alap(t)
  File "/usr/lib/python2.6/site-packages/Trac_jsGantt-0.9_r10732-py2.6.egg/tracjsgantt/tracjsgantt.py", line 628, in _schedule_task_alap
    finish += timedelta(days=7-start.weekday())
UnboundLocalError: local variable 'start' referenced before assignment

Changed 3 years ago by rjollos

comment:2 Changed 3 years ago by rjollos

  • Summary changed from "Local variable referenced before assignment" error (r10732) to [Patch] "Local variable referenced before assignment" error (r10732)

Here is the patch.

comment:3 follow-up: Changed 3 years ago by rjollos

I've created a fork of this project at BitBucket so that I have a place to push my changes. I haven't used Git before, so please let me know if you have any suggestions on how I can do this better. Will it be easier for you to pull in this changes if I point you to the commit hash that contains the patch?

Here is the URL for the changeset that contains the patch for this ticket: https://bitbucket.org/rjollos/tracjsganttplugin/changeset/612501ab2531

comment:4 Changed 3 years ago by ChrisNelson

  • Status changed from new to assigned

comment:5 in reply to: ↑ 3 ; follow-up: Changed 3 years ago by ChrisNelson

Replying to rjollos:

I've created a fork of this project at BitBucket so that I have a place to push my changes.

I hope you mean clone, let's not diverge!

I haven't used Git before, so please let me know if you have any suggestions on how I can do this better.

Git hurts your head for a couple of months then you wonder how you ever lived without it.

Will it be easier for you to pull in this changes if I point you to the commit hash that contains the patch?

I dunno. Downloading a patch from TH is easy enough for me but I should be able to add BitBucket as an additional remote to my repo and cherry-pick your changes from there so that might be easier.

Here is the URL for the changeset that contains the patch for this ticket: https://bitbucket.org/rjollos/tracjsganttplugin/changeset/612501ab2531

I'll see if I can figure out how to use that.

comment:6 in reply to: ↑ 5 Changed 3 years ago by rjollos

Replying to ChrisNelson:

I hope you mean clone, let's not diverge!

Well, yes I suppose I do (now that I stop and think about the actual meanings of those words ;). I'm anticipating pushing up changes to the BitBucket repository and associating each with a ticket so that you can pull in the ones that you want included in your plugin. So we could diverge perhaps slightly if you don't want to include a particular change, but any customization of your plugin would just be for my own use (e.g. #8947) in order to meet the demands my team has, and I don't intend to "advertize" my clone on trac-hacks or elsewhere. For one, although I'm happy to fix bugs and try to help you support other users, I don't want to support users on any forks of the plugin. And in general, I think that everyone doing there own thing and posting it all over trac-hacks just leads to us having a very disorganized community and is not something I want to promote.

comment:7 Changed 2 years ago by ChrisNelson

[10818] should have linked here.

comment:8 Changed 2 years ago by rjollos

  • Resolution set to fixed
  • Status changed from assigned to closed

Add Comment

Modify Ticket

Action
as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from ChrisNelson. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.