- First commit to issue fork.
- Status changed to Needs review
2 months ago 2:35pm 27 August 2024 - π¬π§United Kingdom alexpott πͺπΊπ
Reviving this issue because this code broke a commit :)
I wonder if we should deprecate this feature? My currently feeling is no. We should remove it. Since #2502785: Remove support for $form_state->setCached() for GET requests β it has been a pointless feature and we just didn't get round to completing the work. The 1 usage in contrib I can find is https://git.drupalcode.org/project/sessionless/-/blame/1.x/sessionless_f... - which is a copy and paste of core code and not something we could even issue a deprecation for. The code will not break when we merge this MR. I think all we can do here is issue a CR and open an issue for the sessionless module to remove the unnecessary code.
- Merge request !9340Resolve #2524408 "Remove formstate immutability" β (Closed) created by alexpott
- Status changed to RTBC
2 months ago 1:05pm 28 August 2024 - πΊπΈUnited States smustgrave
Change record reads straight forward. Examples are always welcome too.
Changes in MR appear pretty straight forward. All removal and didn't break anything.
Would maybe just target 11.x? And leave alone in 10.x just in case?
- π¬π§United Kingdom alexpott πͺπΊπ
@smustgrave yeah this is not for 10.x
- π¬π§United Kingdom catch
Looks good, forgot I opened this one. Committed/pushed to 11.x, thanks!
- Status changed to Fixed
2 months ago 1:18am 30 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.