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 #1966 (closed defect: fixed)

Opened 11 years ago

Last modified 11 years ago

r5204 breaks "view.render()/view.stdvars()" when TurboGears server is not started

Reported by: Chris Arndt Owned by: chrisz
Priority: normal Milestone: 1.1
Component: TurboGears Version: 1.0.4.4
Severity: normal Keywords:
Cc:

Description

r5204 sets the default value of turbogears.startup.webpath to None and thus breaks turbogears.view.stdvars() when the TurboGears server is not started.

The attached patch fixes this for the 1.1 branch but I wonder why webpath can't be set to the empty string by default as it used to be.

Attachments

view-webpath.diff Download (1.6 KB) - added by Chris Arndt 11 years ago.

Change History

Changed 11 years ago by Chris Arndt

comment:1 Changed 11 years ago by Chris Arndt

  • Summary changed from r5204 break using of "view.render()" without starting the TurboGears server to r5204 breaks "view.render()/view.stdvars()" when TurboGears server is not started

comment:2 Changed 11 years ago by chrisz

I had changed it to "None" to indictate "not yet set up." But you're probably right, it should be better declared as "" so that it can be used without calling startTurboGears, where it is set according to the config.

comment:3 Changed 11 years ago by Chris Arndt

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

Ok, I changed this back in r5267 in the 1.1 branch so that startup.webpath defaults to ''. I also applied the attached patch in slightly different form to make view.base.stdvars more robust and to clean it up.

Note: See TracTickets for help on using tickets.