Ticket #1586 (closed enhancement: fixed)

Opened 7 months ago

Last modified 6 months ago

[PATCH] testutil.py DbTest: attribute model.soClasses should be used if present

Reported by: ghorvath Assigned to: anonymous
Priority: normal Milestone: 1.0.4
Component: TurboGears Version: 1.0.3
Severity: normal Keywords:
Cc: gh@gregor-horvath.com

Description

I use the the SQLObject soClasses attribute in my model to define the order of class creation and hide specific classes to SQLObject. testutil.DbTest? should also use this attribute if present to create and drop the tables. Patch against TurboGears-1.0.3.2 is attached.

Attachments

testutil_soClasses.diff (477 bytes) - added by ghorvath on 10/13/07 10:54:51.
New version (the previous one did NOT work)

Change History

10/13/07 10:54:51 changed by ghorvath

  • attachment testutil_soClasses.diff added.

New version (the previous one did NOT work)

10/13/07 14:29:12 changed by ghorvath

  • cc set to gh@gregor-horvath.com.

11/25/07 12:08:50 changed by faide

Pleas use the "diff -u" command to generate your patches as the current patch does not displays correctly in trac and is not using the "good" format.

As per the patch in itself, your intention is to make sure the soClasses attribute is never used at all and we always use the dict attribute to get your classes from the model ?

11/25/07 12:11:47 changed by faide

Sorry! Your patch is in the wrong format _and_ wrong order and I did not register this at first :) you added a control on soClasses!

You have created the patch in the wrong order and it shows like you are trying to remove some code whereas in fact you want to add it.

This is a valid and interesting request.

11/25/07 12:29:52 changed by faide

  • status changed from new to closed.
  • resolution set to fixed.

Fixed in r3758