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 10 years ago

Last modified 10 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 10 years ago.
Patch for turbogears 1.1 HEAD
redirect.2.patch Download (1.5 KB) - added by PeterRussell 10 years ago.
Patch against 1.0 HEAD

Change History

Changed 10 years ago by PeterRussell

Patch for turbogears 1.1 HEAD

Changed 10 years ago by PeterRussell

Patch against 1.0 HEAD

comment:1 Changed 10 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.