Problem/Motivation

Viewing any form produces the following error:

Recoverable fatal error: Argument 2 passed to config_readonly_form_alter() must be of the type array, object given.

Proposed resolution

Update to use FormStateInterface.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#2 0001-Issue-2407593-applied-multiple-Form-API-core-changes.patch3.3 KBfgm

Comments

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Priority: Normal » Major

This hook_form_alter() is actually broken for more reasons like settings() and FormStateControllerInterface no longer existing.

fgm’s picture

fgm
Primary language French
Location Paris, France
commented
Status: Active » Needs review
StatusFileSize
new 0001-Issue-2407593-applied-multiple-Form-API-core-changes.patch3.3 KB

Seems to work now.

anavarre’s picture

anavarre
Primary language French
Location 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Applies cleanly and fixes the issue.

Anonymous’s picture

Anonymous (not verified) commented

The module is not usable without this patch. Upon applying the patch it works. +1 to the RTBC.

  • wheatpenny committed ba143b5 on 8.x-1.x authored by fgm 
    Issue #2407593 by fgm: Fixed form alter now takes FormStateInterface...
wheatpenny’s picture

wheatpenny commented
Status: Reviewed & tested by the community » Fixed

Thank you all for the work and review.

Anonymous’s picture

Anonymous (not verified) commented

Yay! Thanks for the commits. I love this module, it is a great idea and I plan to put it to good use after RC hits!

Status: Fixed » Closed (fixed)

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