Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 UTC on 18 March 2024, to get $100 off your ticket.
Currently you can only put form elements into the 'Workflow' fieldset, which is quite limiting. This patch adds in some recursive merging of form arrays to allow elements to be added anywhere.
Comment | File | Size | Author |
---|---|---|---|
#13 | node_options_2.patch | 4.29 KB | drumm |
#11 | node_setings.patch | 1.79 KB | chx |
#6 | node_options_1.patch | 5.56 KB | drumm |
#1 | node_options_0.patch | 4.34 KB | drumm |
node_options.patch | 6.42 KB | drumm | |
Comments
Comment #1
drummAnd here is a patch without extra half-finished cruft from Amsteram.
Comment #2
crunchywelch CreditAttribution: crunchywelch commentednice, I was wanting to do this today actually. +1 from me on visual review ;)
Comment #3
moshe weitzman CreditAttribution: moshe weitzman commenteddoes form_alter hook not provide a solution?
Comment #4
drummEither we should completely remove this hook and use form_alter() for everything or go with the modification to the hook to make it work completely. I beleive using the hook makes things more readable, but I'd like to hear from some others.
Comment #5
adrian CreditAttribution: adrian commented+1
we use array_merge_recursive on the node form.
Comment #6
drummHere is a new patch which removes hook_nodeapi('settings', ...) entirely.
Comment #7
chx CreditAttribution: chx commentedhmm looks nice.
Comment #8
Dries CreditAttribution: Dries commentedCommitted to HEAD. Not marking 'fixed' until this change has been documented.
Comment #9
Gerhard Killesreiter CreditAttribution: Gerhard Killesreiter commentedHow is that supposed to work? $node is undefined.
Comment #10
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedSetting to critical bug, this simply does not work.
Comment #11
chx CreditAttribution: chx commentedThis is node module patch, the other modules need similar changes.
Comment #12
drummGood point. I can make a new patch (we don't need $node anyway) on Monday.
Comment #13
drummHere is a complete fix.
Comment #14
Dries CreditAttribution: Dries commentedCommitted to HEAD. Thanks.
Comment #15
drummThe converting 4.6 -> HEAD and hook documentation are now updated.
Comment #16
(not verified) CreditAttribution: commented