Same:
TypeError: str_replace(): Argument #3 ($subject) must be of type array|string, Drupal\layout_builder\Section given in str_replace() (line 32 of /var/www/html/web/modules/contrib/content_type_clone/src/Helpers/ContentTypeCloneHelper.php).
There is no patch to review here though.
- Status changed to Needs work
3 months ago 9:48pm 23 April 2025 - ๐ฎ๐ณIndia rakesh.regar Rajasthan, India
No patch attached in issue, moving to Needs work.
- ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
This should be active until there is a patch to review or work on.
@belba, this is assigned to you. Do you intend to work on it? If not, this should not be assigned to you.
- Merge request !6Issue #3411290 by venugopp, sidharth_soman: Relying on entity queries to check... โ (Open) created by Unnamed author
The issue has been resolved in the 3097623-conflict-with-layout branch.
The error with str_replace() on Drupal\layout_builder\Section objects no longer occurs.
Cloning works correctly even when Layout Builder is enabled.Kindly review the branch. Thank you!
- ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
@haritha-c Thanks for the patch. The merge request needs to target 8.x-1.x. When it is ready, please set the issue to "Needs review". Someone else can then test and review, leading to it being set "Reviewed & tested by the community".
- Merge request !7Issue #3411290 by venugopp, sidharth_soman: Relying on entity queries to check... โ (Closed) created by Unnamed author
Hi @liam morland,
Iโve updated MR !8 to target 8.x-1.x in branch 3097623-fix-clone-content-v2, fixing the str_replace() TypeError in ContentTypeCloneHelper.php.
Please review and test: https://git.drupalcode.org/project/content_type_clone/-/merge_requests/8- ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
Thanks. But the way there is no need to make a new merge request. The new code can be force-pushed onto the same branch and the target branch of the merge request can be changed.
Okay sure , I'll follow these steps in the future. Thanks for your guidance!