In order to get Auto nodetitle work with IEF (see #1719948: Integration with inline entity form) it is needed the node controller to call node_submit and hook_node_submit.

CommentFileSizeAuthor
#4 1719956-4-am.patch1.29 KBjherencia
#4 1719956-4.patch953 bytesjherencia
#2 1719956-2.patch786 bytesjherencia
#2 1719956-2-am.patch1.15 KBjherencia
#1 1719956-1-am.patch1.14 KBjherencia
#1 1719956-1.patch779 bytesjherencia

Comments

jherencia’s picture

jherencia commented
Title: Let modules alter node submition » Let other modules execute hook_node_submit
StatusFileSize
new 1719956-1.patch779 bytes
new 1719956-1-am.patch1.14 KB
jherencia’s picture

jherencia commented
StatusFileSize
new 1719956-2-am.patch1.15 KB
new 1719956-2.patch786 bytes

I've fixed a bug in the previous patch.

bojanz’s picture

bojanz commented
Status: Needs review » Needs work

Blurgh, D7 still has some crappy code.

You're passing $form_state to hook_node_submit, that's the form state of the parent form, not of the embedded node form.
That won't work for obvious reasons (autonodetitle uses $form_state['values']['title'], for example).

Perhaps just construct a fake form state that contains just values:

$fake_form_state = form_state_defaults();
$fake_from_state['values'] = drupal_array_get_nested_value($form_state['values'], $entity_form['#parents']);

and hope that works?

jherencia’s picture

jherencia commented
Status: Needs work » Needs review
StatusFileSize
new 1719956-4.patch953 bytes
new 1719956-4-am.patch1.29 KB

You are right.

bojanz’s picture

bojanz commented

Committed.

bojanz’s picture

bojanz commented
Status: Needs review » Fixed

Closing issue.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

aitala’s picture

aitala commented
Issue summary: View changes

Hi,

I think I'm still having an issue with Auto nodetitle and IEF - I'm still getting the node title field showing up, when it should be hidden, and being required.

I can complete the IEF form, but the new node comes back with no title. If I edit it, still no title.

The only things that 'works' is to add the node containing the IEF, then go back and edit the IEF node. If that makes sense.

E