1. Software Freedom Conservancy
  2. Kallithea
  3. kallithea
  4. Issues

Issues

Issue #95 resolved

Error 500 on mail notification creation when "Approved" translation is unicode

Jérémy Longo
created an issue

traced it to model/notification.py,

Fix :

replace

l = [str(x) for x in [kwargs.get('status_change'), kwargs.get('closing_pr') and _('Closing')] if x]

with

l = [unicode(x) for x in [kwargs.get('status_change'), kwargs.get('closing_pr') and _('Closing')] if x]

Comments (9)

  1. Mads Kiilerich

    I wonder why I put that str() there in the first place. I guess that is a leftover from previous iterations.

    Does it also work for you if you use 'x for x in ...' instead?

  2. Log in to comment