Edgewall Software
Modify

Opened 16 years ago

Closed 15 years ago

#7469 closed defect (fixed)

Refactoring gone bad, patch to correct

Reported by: Pedro Algarvio, aka, s0undt3ch Owned by: Jeroen Ruigrok van der Werven
Priority: highest Milestone: 0.12
Component: general Version: 0.12dev
Severity: blocker Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

The correct refactoring would have been:

  • trac/web/main.py

     
    276276                translation.get_available_locales()]
    277277
    278278            preferred = req.session.get('language', req.languages)
    279             if isinstance(preferred, list):
     279            if not isinstance(preferred, list):
    280280                preferred = [preferred]
    281281            return Locale.negotiate(preferred, available, sep='-')

Attachments (0)

Change History (4)

comment:1 by Pedro Algarvio, aka, s0undt3ch, 16 years ago

Owner: set to Jeroen Ruigrok van der Werven

comment:2 by Jeroen Ruigrok van der Werven, 16 years ago

Resolution: fixed
Status: newclosed

Fixed in r7412. Thanks.

comment:3 by anonymous, 15 years ago

Resolution: fixed
Status: closedreopened

comment:4 by anonymous, 15 years ago

Resolution: fixed
Status: reopenedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Jeroen Ruigrok van der Werven.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Jeroen Ruigrok van der Werven to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.