Ticket #1219 (closed defect: wontfix)

Opened 3 years ago

Last modified 1 year ago

Appwizard creates KeyField as string instead tuple

Reported by: sunday Assigned to: somebody
Priority: major Milestone: 0.9.2
Component: app Version: 0.9.1
Keywords: Cc:

Description

Appwizard should create multiple field PKs as tuples and not string

Change History

06/22/09 13:46:18 changed by paul

I think this is a 'no-fix' as:

1) dBizobj.KeyField? *is* a string (for multiple field pk, separate with comma).

2) There isn't any great way to detect the pk field list automatically, so you need to edit the generated code anyway. No big deal to add the other PK fields at that time.

AppWizard? is meant to get you started.

On the other hand, if you have a fix sitting around that you want to propose...

07/05/09 15:10:14 changed by sunday

1) dBizobj.KeyField?? *is* a string (for multiple field pk, separate with comma).

I think I have seen pieces of code treating it otherwise, so it was not clear what it really should be. If it is defined that way ok, although I think a tuple would be more "pythonic"

2) There isn't any great way to detect the pk field list automatically, so you need to edit the generated code anyway. No big deal to add the other PK fields at that time.

Ok then...

02/03/11 10:22:33 changed by paul

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

Closing as it was an agreed non-issue.