View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0000145 | v1.1 Release (Closed) | [All Projects] General | public | 2009-04-30 05:33 | 2009-09-09 22:37 | ||||
Reporter | caseydk | ||||||||
Assigned To | caseydk | ||||||||
Priority | high | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Product Version | |||||||||
Target Version | Fixed in Version | 1.1 | |||||||
Summary | 0000145: Id generation for Custom Fields | ||||||||
Description | In /classes/CustomFields.class.php around lines 76-80, the Custom Fields class itself calculates the next available primary key for the table using a "max(value_id)" structure. While this should always work, it's kind of silly to do... the underlying value_id field is an auto_increment so the database should handle this. | ||||||||
Additional Information | If this was consistent behavior across all classes, I wouldn't note this as a problem but as is, this method of id calculation is only used in this single class. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files |
|
![]() |
|
caseydk (administrator) 2009-06-13 20:08 Last edited: 2009-06-13 20:08 |
Resolved in r440; |
caseydk (administrator) 2009-09-09 22:37 |
Closed for release. |
![]() |
|||
Date Modified | Username | Field | Change |
---|---|---|---|
2009-04-30 05:33 | caseydk | New Issue | |
2009-06-07 21:07 | caseydk | Project | Pending Requests => v1.1 Release (Closed) |
2009-06-11 06:34 | caseydk | Status | new => assigned |
2009-06-11 06:34 | caseydk | Assigned To | => caseydk |
2009-06-13 20:08 | caseydk | Status | assigned => resolved |
2009-06-13 20:08 | caseydk | Resolution | open => fixed |
2009-06-13 20:08 | caseydk | Note Added: 0000301 | |
2009-06-13 20:08 | caseydk | Note Edited: 0000301 | |
2009-09-09 22:37 | caseydk | Status | resolved => closed |
2009-09-09 22:37 | caseydk | Note Added: 0000430 | |
2009-09-09 22:37 | caseydk | Fixed in Version | => 1.1 |