If we implement hook_content_extra_fields(), we can allow users of CCK to be able to re-order the CAPTCHA field on node forms. Handy!
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 658366-mollom-cck-field-D6.patch | 934 bytes | dave reid |
| #1 | 658366-mollom-cck-field-D6.patch | 819 bytes | dave reid |
| #1 | screenshot_011.png | 42.23 KB | dave reid |
Comments
Comment #1
dave reidPatch attached for review with a screenshot of how it looks on the CCK 'Manage Fields' interface.
Comment #2
dave reidRevised patch with a much lower weight so it should sink to the bottom on the CCK UI.
Comment #3
dries commentedThis looks great. Before we commit this, let's add some extra documentation to specify _where_ the hook comes from and _why_ it is implemented. Let's also specify why we set the weight to 99999. Otherwise RTBC.
Comment #4
sunActually, this is a common hook for D6 and known to developers, and we inject the element to the end of the form in mollom_form_alter(), so I think this patch is perfectly fine.
Needs to be forward-ported. Not sure how it's called in D7 though.
Comment #5
dave reidWe do basically the same thing with hook_field_extra_fields() in D7, but there is a critical bug since re-ordering fields in the interface doesn't actually do anything: #672794: Re-ordering fields does not take affect in forms.
Comment #6
dries commentedI added some extra documentation and committed the patch to DRUPAL-6--1. The patch still needs to go into CVS HEAD for Mollom 7, so I've update the issue accordingly. Thanks Dave.
Comment #7
dave reidCommitted the same thing to the D7 branch adjusted 'CCK' to 'Field UI'.