Opened 4 years ago

Last modified 9 months 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 Ryan J Ollos)

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/", line 105, in process_request
    ticket.save_changes(req.authname, None, now)
  File "/usr/lib/python2.6/dist-packages/trac/ticket/", 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/", line 111, in ticket_changed
    self.add_hours_by_comment(comment,, author)
  File "/usr/local/lib/python2.6/dist-packages/TracHoursPlugin-0.6.0dev_r12205-py2.6.egg/trachours/", line 129, in add_hours_by_comment
    for match in re.finditer(self.hours_regex, comment):
  File "/usr/lib/python2.6/", 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 Ryan J Ollos

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 Ryan J Ollos

Priority: normalhigh

comment:3 Changed 4 years ago by Ryan J Ollos

Description: modified (diff)

comment:4 Changed 9 months ago by Ryan J Ollos

Owner: Ryan J Ollos deleted

Modify Ticket

Change Properties
Set your email in Preferences
as new The ticket will remain with no owner.

Add Comment

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

Note: See TracTickets for help on using tickets.