|
[task] Review the node table: msg#01015php.drupal.devel
Project: Drupal Version: cvs Component: node system Category: tasks Priority: critical Assigned to: killes@xxxxxxxxxxxx Reported by: killes@xxxxxxxxxxxx Updated by: killes@xxxxxxxxxxxx Status: patch queue.module is the only module that uses them. serialize() is a bit more elegant... killes@xxxxxxxxxxxx Previous comments: ------------------------------------------------------------------------ February 10, 2004 - 02:10 : killes@xxxxxxxxxxxx The node table in the current database dump contains some fields which do not stirctly belong there: - votes -> queue.module - users -> queue.module - attributes -> ? queue.module shoudl get its own table and the rest should probably be removed. I set this to 'critical' because I think it would be nice to fix this before 4.4.. ------------------------------------------------------------------------ February 21, 2004 - 19:04 : Kjartan attributes is obsolete, its what we used before taxonomy if I am not wrong. ------------------------------------------------------------------------ February 21, 2004 - 20:09 : killes@xxxxxxxxxxxx attributes was removed. The queue.module data rows still need their own table. ------------------------------------------------------------------------ February 27, 2004 - 18:59 : killes@xxxxxxxxxxxx Attachment: http://drupal.org/files/issues/queue-simplification.patch (9.76 KB) Here is the patch to give queue.module its own table. ------------------------------------------------------------------------ February 27, 2004 - 19:03 : Goba Why do you remove the field_get/set functions in this patch? -- View: http://drupal.org/node/view/5761 Edit: http://drupal.org/project/comments/add/5761 |
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| Previous by Date: | [task] Review the node table: 01015, Goba |
|---|---|
| Next by Date: | init_theme() error after upgrading to CVS: 01015, Tim Altman |
| Previous by Thread: | [task] Review the node tablei: 01015, Goba |
| Next by Thread: | [feature] task manager module: 01015, erikhopp |
| Indexes: | [Date] [Thread] [Top] [All Lists] |
| News | FAQ | advertise |