logo       

[task] Review the node table: msg#01015

php.drupal.devel

Subject: [task] Review the node table

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>
Google Custom Search

News | FAQ | advertise