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 #369 (closed enhancement: fixed)

Opened 13 years ago

Last modified 12 years ago

[PATCH] Extend fastdata with related join (many-to-many) functionality

Reported by: ksenia Owned by: anonymous
Priority: normal Milestone: 0.9
Component: TurboGears Version:
Severity: normal Keywords:
Cc:

Description

DataController now accept (SQL)RelatedJoin? name in field_list. The join is mapped to CheckBoxList?. Other joins still need to be implemented.

Attachments

relatedjoins.patch Download (6.3 KB) - added by ksenia 13 years ago.
relatedjoins
relatedjoins_modified.patch Download (12.0 KB) - added by ksenia 13 years ago.

Change History

Changed 13 years ago by ksenia

relatedjoins

comment:1 Changed 13 years ago by anonymous

  • Milestone set to 0.9

comment:2 Changed 13 years ago by kevin

Overall, this is a good patch. But, I don't like putting SQLObject in widgets.base. I'd recommend a new widget (CheckboxList? subclass) in fastdata itself for the purpose at hand.

comment:3 Changed 13 years ago by kevin

  • Summary changed from [PATCH] Extend fastdata with related join (many-to-many) functionality to Extend fastdata with related join (many-to-many) functionality

Changed 13 years ago by ksenia

comment:4 Changed 13 years ago by ksenia

  • Summary changed from Extend fastdata with related join (many-to-many) functionality to [PATCH] Extend fastdata with related join (many-to-many) functionality

I created new file datawidgets.py in the fastdata directory, for SQLObject widgets. DataGrid is also moved to this file, so if anyone is using

from turbogears.fastdata.datacontroller import DataGrid

it must be changed to

from turbogears.fastdata.datawidgets import DataGrid

This patch also includes #354, hope it's ok...

comment:5 Changed 13 years ago by kevin

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

committed in [562]! Thanks for the updated patch

Note: See TracTickets for help on using tickets.