Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #2006 (closed defect: fixed)

Opened 8 years ago

Last modified 8 years ago

redirect no longer works with a list of url components

Reported by: PeterRussell Owned by: faide
Priority: normal Milestone: 1.0.x bugfix
Component: TurboGears Version: 1.0.7
Severity: normal Keywords:
Cc:

Description

Sorry for snowing you under with bug reports!

Since [5145] passing a list of url components to turbogears.redirect no longer works. The attached patch fixes this.

Attachments

redirect.patch Download (1.5 KB) - added by PeterRussell 8 years ago.
Patch for turbogears 1.1 HEAD
redirect.2.patch Download (1.5 KB) - added by PeterRussell 8 years ago.
Patch against 1.0 HEAD

Change History

Changed 8 years ago by PeterRussell

Patch for turbogears 1.1 HEAD

Changed 8 years ago by PeterRussell

Patch against 1.0 HEAD

comment:1 Changed 8 years ago by chrisz

  • Status changed from new to closed
  • Resolution set to fixed

Though this was not a documented feature of redirect(), this should work since url() allows a list as well. Unfortunately, there was not unit test for this, so thanks for the patch. I have added tests for tuples and both relative and absolute paths (the problem appeared only with relative paths). Applied in r5511.

Note: See TracTickets for help on using tickets.