Modify

Opened 4 years ago

Last modified 3 weeks ago

#10934 new defect

TracHoursByComment breaks GridModify plugin

Reported by: chris@… Owned by:
Priority: high Component: TracHoursPlugin
Severity: normal Keywords:
Cc: Trac Release:

Description (last modified by rjollos)

I just installed TracHours on my Trac that already had the GridModify plugin installed and working. After enabling all of the TracHours plugins, the GridModify plugin stopped working, ie: after editing a field in the grid, it would not save.

I turned on logging and found the following error:

Traceback (most recent call last):
  File "/usr/local/lib/python2.6/dist-packages/GridModify-0.1.6dev_r12235-py2.6.egg/gridmod/web_ui.py", line 105, in process_request
    ticket.save_changes(req.authname, None, now)
  File "/usr/lib/python2.6/dist-packages/trac/ticket/model.py", line 360, in save_changes
    listener.ticket_changed(self, comment, author, old_values)
  File "/usr/local/lib/python2.6/dist-packages/TracHoursPlugin-0.6.0dev_r12205-py2.6.egg/trachours/ticket.py", line 111, in ticket_changed
    self.add_hours_by_comment(comment, ticket.id, author)
  File "/usr/local/lib/python2.6/dist-packages/TracHoursPlugin-0.6.0dev_r12205-py2.6.egg/trachours/ticket.py", line 129, in add_hours_by_comment
    for match in re.finditer(self.hours_regex, comment):
  File "/usr/lib/python2.6/re.py", line 186, in finditer
    return _compile(pattern, flags).finditer(string)
TypeError: expected string or buffer

So I turned off the TracHoursByComment plugin, as that looked like it was expecting a comment or at least as empty string, and GridModify works OK now.

I won't be using TracHoursByComment anyway, but thought you might like to know.

Attachments (0)

Change History (4)

comment:1 Changed 4 years ago by rjollos

Your report is appreciated! I haven't worked on maintaining the plugin much lately, but I will take a look soon.

comment:2 Changed 4 years ago by rjollos

  • Priority changed from normal to high

comment:3 Changed 3 years ago by rjollos

  • Description modified (diff)

comment:4 Changed 3 weeks ago by rjollos

  • Owner rjollos deleted

Add Comment

Modify Ticket

Action
as new The ticket will remain with no owner.
Author


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

 
Note: See TracTickets for help on using tickets.