[drupal-devel] [task] Remove use of form_item() in taxonomy

Bèr Kessels drupal-devel at drupal.org
Mon Feb 28 13:05:53 UTC 2005


Issue status update for http://drupal.org/node/17955

 Project:      Drupal
 Version:      cvs
 Component:    taxonomy.module
 Category:     tasks
 Priority:     normal
 Assigned to:  drumm
 Reported by:  drumm
 Updated by:   Bèr Kessels
 Status:       patch

+1 from me. Untill we have a general confirmation API, we should use
something like this.
Furthermore the form_select, instead of an ugly  looped selector
genreator makes sense.


Bèr Kessels



Previous comments:
------------------------------------------------------------------------

February 25, 2005 - 18:26 : drumm

Attachment: http://drupal.org/files/issues/taxonomy.module_3.diff (4.94 KB)

form_item() isn't right in a few cases. Such as confirmation pages and
things which should be using other form functions.
There was a subtle difference between form_select() and whatever
taxonomy was doing. Thats why there is extra code here. I tested it a
bit too.





More information about the drupal-devel mailing list