When testing the sections module with the latest 4.7 beta sections can't be created. Function section_settings_execute needs to be renamed to section_settings_submit. Patch attached.

CommentFileSizeAuthor
#4 sections.module_5.patch893 bytesyched
sections.module_3.patch927 bytesdreed47

Comments

Bèr Kessels’s picture

Bèr Kessels commented

Could you please explain why tis change is needed? I am not following the development in HEAD and fAPI closely, so I think I might have missed something :)

eaton’s picture

eaton commented
Status: Active » Reviewed & tested by the community

In a nutshell, the 'execute' hook for form handling was changed to 'submit' for clarity -- thus, the sections module's 'execute' handle never gets called and no sections are ever created.

+1 simple patch, fixes the problem.

yched’s picture

yched commented

Sections.module is just completely useless with HEAD for nearly two months now ?
This patch is quite trivial - i'm surprised it has not been commited yet.
Has this module been abandonned ?

yched’s picture

yched commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new sections.module_5.patch893 bytes

Me and my big mouth...

OK, the patch HAS been committed indeed. This issue just has'nt been updated.

Anyway the problem remains : you currently can't add or edit sections sections with HEAD.
The problem now lies in the code of section_settings_submit. $edit doesn't contain the text of the submit button : it's in $_POST['op']
This new patch corrects this

Bèr Kessels’s picture

Bèr Kessels commented
Status: Needs review » Fixed

Thanks for the patch. It is now comitted.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)