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

[Dries]

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:   Dries
 Status:       patch

All other configuration pages (eg. locale module, menu module, etc.) all
use form_item().  I'm likely to drop that bit, and have yet to see the
feasability of a 'confirmation API' (it sounds difficult).
(Ber: it is not clear from your review whether you tested the patch or
not.)


Dries



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

February 25, 2005 - 19: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.


------------------------------------------------------------------------

February 28, 2005 - 15:05 : Bèr Kessels

+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.