Can't set component correctly

[wilmer@…]

The problem happens on this page:

http://bugs.bitlbee.org/bitlbee/ticket/26

I can't set component to IRC or BitlBee?. All other components are allowed. Possible reason: There are bugs from the IRC and BitlBee? components assigned to the 1.0 milestone, and all of them are fixed. There are also two OSCAR-related bugs in the 1.0 milestone, but one of them is still open.

This is the only connection I can find between the IRC- and BitlBee? components.

If I unset the milestone for ticket 26, I still get the same problem:

Traceback (most recent call last):
  File "/home/wilmer/trac/trac/lib/python2.4/site-packages/trac/web/standalone.py", line 303, in _do_trac_req
    dispatch_request(path_info, req, env)
  File "/home/wilmer/trac/trac/lib/python2.4/site-packages/trac/web/main.py", line 139, in dispatch_request
    dispatcher.dispatch(req)
  File "/home/wilmer/trac/trac/lib/python2.4/site-packages/trac/web/main.py", line 107, in dispatch
    resp = chosen_handler.process_request(req)
  File "/home/wilmer/trac/trac/lib/python2.4/site-packages/trac/ticket/web_ui.py", line 191, in process_request
    self._do_save(req, db, ticket)
  File "/home/wilmer/trac/trac/lib/python2.4/site-packages/trac/ticket/web_ui.py", line 349, in _do_save
    req.args.get('comment'), when=now, db=db)
  File "/home/wilmer/trac/trac/lib/python2.4/site-packages/trac/ticket/model.py", line 197, in save_changes
    self['owner'] = new_comp.owner
  File "/home/wilmer/trac/trac/lib/python2.4/site-packages/trac/ticket/model.py", line 104, in __setitem__
    self.values[name] = value.strip()
AttributeError: 'NoneType' object has no attribute 'strip'

Thanks for the great product, and I hope this will be fixed soon. :-)

[wilmer@…]

And oh yeah, I'm running trac+darcs with a bzr repository, might be a good thing to mention.

[cboos]

From the stack trace it rather looks like those component have no owner (empty string).

Ok, I can reproduce it:

1. manually assign null to the owner component2
2. create a new ticket with component set to component1
3. change component to component2 => the error

For some reason, we get a None value instead of an empty string in Python (source:trunk/trac/ticket/model.py@2539#L458). That's OK, I guess, but then we should be robust while saving the state of the ticket

model.py

@@ -129 +129 @@
             self._old[name] = self.values.get(name)
         elif self._old[name] == value: # Change of field reverted
             del self._old[name]
-        self.values[name] = value.strip()
+        self.values[name] = value and value.strip()
 
     def populate(self, values):
         """Populate the ticket with 'suitable' values from a dictionary"""

One minor issue left with that would be that once the owner of the new ticket has been cleared because of such a component change, another component change wouldn't manage to reset the owner. Therefore, we need the additional fix:

model.py

@@ -224 +226 @@
                     and not self._old.has_key('owner'):
                 try:
                     old_comp = Component(self.env, self._old['component'], db)
-                    if old_comp.owner == self.values.get('owner'):
-                        new_comp = Component(self.env, self['component'],
-                                             db)
+                    old_owner = old_comp.owner or ''
+                    current_owner = self.values.get('owner') or ''
+                    if old_owner == current_owner:
+                        new_comp = Component(self.env, self['component'], db)
                         self['owner'] = new_comp.owner
                 except TracError, e:
                     # If the old component has been removed from the database we

[cboos]

Oops, the source link above should have been:

source:trunk/trac/ticket/model.py@2539#L414

[Wilmer van der Gaast <wilmer@…>]

Oops, yeah, my conclusion was indeed wrong, turns out the problem appears with the Jabber module too. Those three are the modules without a specific owner.

Also, manually assigning someone to the bug and then setting the component seems to fix the problem. :-)

Thanks!

[cboos]

Now in 0.9-stable too (r2544)