- Issue created by @aklump
- πΊπΈUnited States caesius
This issue doesn't occur on a fresh install of Drupal without contributed or custom code. The "steps to reproduce" need to be more specific, especially if this only happens when using a contributed module or writing custom code.
- πΊπΈUnited States aklump
Copy that. I'm working on a patch. It's probably that some custom code created #attributes as an object, not an array. But I haven't tracked it that far yet. I'll see if I can nail it down better.
- @aklump opened merge request.
- πΊπΈUnited States aklump
I've created a merge request that is working in my installation. Feel free to decline it if you think it's edge case. This may be a helpful reference: https://www.drupal.org/node/1727592 β .
BTW, great work on this module. Several of my clients have asked for this functionality and I was able to install it for them so they had what they wanted. Nice job!
Aaron
- Status changed to Needs review
over 1 year ago 7:11pm 15 August 2023 - Status changed to Postponed: needs info
over 1 year ago 7:17pm 15 August 2023 - πΊπΈUnited States caesius
This looks like a simple and harmless patch, but I'll still need specific reproduction steps before merging since I need to see the issue as described. If you find custom code that is causing this, make sure you update the IS with some example code for a generalized use case -- it should be something that can be dropped into a custom module with minimal changes e.g. updating placeholders.
(also, I did nothing to develop this module -- I just came on after the original developers abandoned it...)
- πΊπΈUnited States aklump
I think that should do it. I've tested on a vanilla Drupal with only that custom module code that I zipped and uploaded. It will set up the scenario that exposes the issue.
Thank you for keeping the module going!
- Status changed to RTBC
over 1 year ago 1:17am 16 August 2023 -
caesius β
committed 87f25c0c on 8.x-1.x authored by
aklump β
Issue #3381269 by aklump: save_edit may share the same #attribute...
-
caesius β
committed 87f25c0c on 8.x-1.x authored by
aklump β
- Status changed to Fixed
over 1 year ago 1:20am 16 August 2023 - πΊπΈUnited States aklump
You bet. Thank you for the quick turn-around. Take care.
Automatically closed - issue fixed for 2 weeks with no activity.