Ticket #6786 (closed defect: fixed)

Opened 3 years ago

Last modified 2 years ago

usage of strftime() causes a "Genshi UnicodeDecodeError"

Reported by: Jens.Langner@light-speed.de Assigned to: rjollos
Priority: normal Component: TicketChangePlugin
Severity: critical Keywords:
Cc: Trac Release: 0.11

Description

Similar to ticket #4820 this plugins suffers from incorrectly using strftime() which in turn causes a "Genshi UnicodeDecodeError?" to appear when a ticket is changed and the Locale is not english.

The following patch fixes the issues: 

===================================================================
--- ticketchange/web_ui.py	(revision 7751)
+++ ticketchange/web_ui.py	(working copy)
@@ -168,11 +168,11 @@
         row = cursor.fetchone()
         if not row:
             raise ResourceNotFound("Unable to update comment on Ticket #%d at time '%s' ('%s') - existing change not found.\n" \
-                                   % (id, time, strftime('%A, %d %b %Y %H:%M:%S', localtime(time))))
+                                   % (id, time, strftime('%x %X', localtime(time))))
         old_author, old_comment = (row[0], row[1])
         cursor.execute("UPDATE ticket_change SET newvalue=%s WHERE ticket = %s AND time = %s AND field = 'comment'", (comment, id, time))
         db.commit()
         self.env.log.info("Ticket #%d comment of '%s' by '%s' has been updated by '%s':\nold value: '%s'\n\nnew value: '%s'\n" \
-                        % (id, strftime('%A, %d %b %Y %H:%M:%S', localtime(time)), old_author, author, old_comment.replace('\r', ''), comment.replace('\r','')))
+                        % (id, strftime('%x %X', localtime(time)), old_author, author, old_comment.replace('\r', ''), comment.replace('\r','')))

Attachments

Change History

04/16/10 23:24:51 changed by anonymous

Some up of this bug in 0.11 

UnicodeDecodeError: 'ascii' codec can't decode byte 0xf1 in position 22: ordinal not in range(128)

04/16/10 23:27:07 changed by anonymous

Up to front! 

File "C:\Python25\lib\site-packages\trac\web\main.py", line 450, in _dispatch_request
  dispatcher.dispatch(req)
File "C:\Python25\lib\site-packages\trac\web\main.py", line 206, in dispatch
  resp = chosen_handler.process_request(req)
File "build\bdist.win32\egg\ticketchange\web_ui.py", line 89, in process_requestFile "build\bdist.win32\egg\ticketchange\web_ui.py", line 176, in _update_ticket_comment

System Information: 

User Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; ru; rv:1.9.2.3) Gecko/20100401 Firefox/3.6.3
Trac: 	0.11.7
Python: 	2.5.4 (r254:67916, Dec 23 2008, 15:10:54) [MSC v.1310 32 bit (Intel)]
setuptools: 	0.6c11
SQLite: 	3.3.4
pysqlite: 	2.3.2
Genshi: 	0.5.1
mod_python: 	3.3.1
Pygments: 	1.3.1
Subversion: 	1.6.6 (r40053)
RPC: 	1.0.6
jQuery:	1.2.6

12/10/10 08:38:29 changed by rjollos

  • status changed from new to assigned.
  • owner changed from SergeiLuchko to rjollos.

This can be fixed by proper use of trac.util.datefmt.

12/10/10 08:42:28 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

Fixed in [9655] and [9656]. The post-commit hook failed to reference this ticket.

Add/Change #6786 (usage of strftime() causes a "Genshi UnicodeDecodeError")




Change Properties
Action