Inline entity forms are not saving with 'save edit' button

This works for me, I suggest RTBC then..

This seems to be working for me as well. Thanks @Niyas

However, this doesn't look like it should be a final solution for this problem since it creates a dependency on Inline entity forms module.

Attached patch resolving the dependency on Inline Entity Forms.

@arnaldop,
Your patch is wrong, you patched "bootstrap_site_alert" module ! :)

@Niyas Thanks for this patch, I'll review today or this week-end.

I upload a new patch : I check if inline entity form is enable

@q__nt_n, I'm so sorry! I do not know what happened!

Quick question, though. According to this page, system_get_info requires a type and the module name is optional.
I haven't tested this patch, but to be consistent with the documentation, should we change the line to read as follows?

if (system_get_info('module', 'inline_entity_form'))

Thanks!

@arnaldop
Yeah, sure ! I failed my patch :) I'll reupload in a few hours.
Thanks for your attention !

I re-upload patch.

I really need your feedback on this. I dont know if it's a good pratice to fix dependance between 2 modules like that.
Thanks !

Add new patch to fix directly call inline_entity_form callback function, instead let this save_edit button to inherit all these callback existed for default submit button.

Patch from #12is working as desired and is looking good to me!

Patch #12 looks good to me and fixed the issue perfectly in my use case.

Patch #2 is working fine with me.

Thanks for Patch #12

Patch #2 works for me but Patch #12 doesn't work as it doesn't unpublish the node when clicking on the save and edit button.

Patch #12 let me finally save the inline_entity_form edits/adds.

I was able to swap between Draft and Published with #12 @rviner. But I think the workflow has had some tweaks and we're using other modules like content_lock with save_edit and inline_entity_form.

It's true what rviner says, that the unpublish feature of save_edit is not working with patch #12. That is because the submit callbacks need to be provided in some order to work properly, especially the callback save_edit_form_submit_presave needs to be called before the ::save callback callback. Patch #12 attaches the two save_edit submit callbacks just at the end of the existing callbacks, thus to late for save_edit_form_submit_presave.

Providing a patch that should fix this. Maybe one could handle the two foreachs a bit more elegant, but should do it like this.

BTW: If you are using save_edit with Drupal 9 and want to use the unpublish feature you will need this patch first: https://www.drupal.org/project/save_edit/issues/3239440

I confirm that #19 fixes the issue for me.

Thanks to everyone who has been working on this issue! I've also implemented patch in #19 and it's working for me.

One caveat: I'm not using the unpublish functionality, so I can't speak to that working or not.

I happened to be experiencing this issue too and the patch fixed it.

Committed to dev.