TypeError: object of type 'NoneType' has no len()

[Ryan J Ollos]

Navigating directly to /newticket?preview=1 results in the following error:

2015-06-15 22:05:07,962 Trac[main] ERROR: Internal Server Error:
Traceback (most recent call last):
  File "/srv/trac-hacks.org/pve/lib/python2.6/site-packages/Trac-1.0.6post2-py2.6.egg/trac/web/main.py", line 534, in _dispatch_request
    dispatcher.dispatch(req)
  File "/srv/trac-hacks.org/pve/lib/python2.6/site-packages/Trac-1.0.6post2-py2.6.egg/trac/web/main.py", line 227, in dispatch
    resp = chosen_handler.process_request(req)
  File "/srv/trac-hacks.org/pve/lib/python2.6/site-packages/Trac-1.0.6post2-py2.6.egg/trac/ticket/web_ui.py", line 183, in process_request
    return self._process_newticket_request(req)
  File "/srv/trac-hacks.org/pve/lib/python2.6/site-packages/Trac-1.0.6post2-py2.6.egg/trac/ticket/web_ui.py", line 466, in _process_newticket_request
    valid = self._validate_ticket(req, ticket)
  File "/srv/trac-hacks.org/pve/lib/python2.6/site-packages/Trac-1.0.6post2-py2.6.egg/trac/ticket/web_ui.py", line 1282, in _validate_ticket
    if len(ticket['summary']) > self.max_summary_size:
TypeError: object of type 'NoneType' has no len()

This is not of much concern, it's just an error I found in the logs. The fix appears to be simple though:

trac/ticket/web_ui.py

@@ -1279 +1279 @@
             valid = False
 
         # Validate summary length
-        if len(ticket['summary']) > self.max_summary_size:
+        if len(ticket['summary'] or '') > self.max_summary_size:
             add_warning(req, _("Ticket summary is too long (must be less "
                                "than %(num)s characters)",
                                num=self.max_summary_size))

[Ryan J Ollos]

Committed to 1.0-stable in [14112], merged to trunk in [14113].