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

Opened 13 years ago

Last modified 12 years ago

Changeset 1376 broke model.py with no identity

Reported by: jorge.vargas Owned by: anonymous
Priority: normal Milestone: 1.0b1
Component: TurboGears Version: 0.9a5
Severity: minor Keywords:
Cc:

Description

a quickstarted project with no identity renders from sqlobject import *

from turbogears import identity from turbogears.database import PackageHub?

hub = PackageHub?("fooball") connection = hub

# class YourDataClass?(SQLObject): # pass

which is assuming sqlpobject was default this was a fix to #801

http://trac.turbogears.org/turbogears/browser/branches/1.0/turbogears/qstemplates/quickstart/%2Bpackage%2B/model.py_tmpl?rev=1376#L5

should read

#if $identity == "sqlobject"

I think godoy forgot $identity was a 3 value boolean variable :)

Attachments

quickstart.log Download (3.5 KB) - added by jorge.vargas 13 years ago.
sorr y I forgot to add this
quickstart.2.log Download (3.5 KB) - added by jorge.vargas 13 years ago.
sorr y I forgot to add this
model.py Download (203 bytes) - added by jorge.vargas 13 years ago.
see that the noidentity.model.py file is actually importing the identity package

Change History

comment:1 Changed 13 years ago by godoy

I've just quickstarted an application here and chose SQL Object for identity. All classes are present.

Are you sure you haven't modified anything in your code to prevent that from working? You can see here that there's a condition to inser SQL Object classes...

comment:2 Changed 13 years ago by godoy

Just to use the 1.0 branch as in your report, here there's the same condition.

You'd like to take a look at line 27--108.

Changed 13 years ago by jorge.vargas

sorr y I forgot to add this

Changed 13 years ago by jorge.vargas

sorr y I forgot to add this

Changed 13 years ago by jorge.vargas

see that the noidentity.model.py file is actually importing the identity package

comment:3 Changed 13 years ago by godoy

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

Fixed in [1519] and [1520] (1.0 and trunk respectively).

Note: See TracTickets for help on using tickets.