Closed (fixed)
Project:
Bootstrap Layouts
Version:
8.x-5.4
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Oct 2024 at 11:36 UTC
Updated:
3 Mar 2025 at 15:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
svicer commentedComment #3
raveen_thakur51 commented@svicer, I am also getting the same error when i am enabling the module.
https://prnt.sc/dZcsHDUHtazn
I am using drupal version 11
Comment #4
sarwan_verma commentedComment #6
rajab natshahComment #8
rajab natshahRaveen, Sarwan, thanks for reporting and following up on the issue.
I created a starting point MR (basic changes).
For sure we need to do more than that.
Comment #10
smulvih2Sorry didn't mean to hide visibility of the branch, accidently clicked the wrong link. Add fix to the MR, broken reference to
ContainerAwareInterface.Comment #12
smulvih2Adding lates MR changes to patch to use in my project.
Comment #13
mlzrHello, thanks for the good work for the community!
If I deploy this patch I have a new error. If i go to the page /backend/modules/uninstall then the website reports "The website encountered an unexpected error. Try again later.".
A peek in the page /backend/reports/dblog (the log) I find this:
Error: Call to undefined method Drupal\bootstrap_layouts\Plugin\BootstrapLayouts\Ds::setContainer() in Drupal\bootstrap_layouts\Plugin\BootstrapLayouts\BootstrapLayoutsHandlerBase->__construct() (line 22 of /home/map/public_html/modules/contrib/bootstrap_layouts/src/Plugin/BootstrapLayouts/BootstrapLayoutsHandlerBase.php).
I am not a coder so I hope you can do something about it. Thanks.
Marcel
Comment #14
smulvih2Ran into the same issue as #13, although in my case it was through PageManager.php instead of Ds.php. Fixed the issue in the PR and attaching a new patch to use in my project.
Comment #15
mlzrThanks smulvih2 / Stephen!
I used the new 3481450-14.patch and now it works fine, no errors anymore.
Thank you for your time!
Marcel
Comment #16
smulvih2Based on feedback from #15, and because I'm using this in my distro to fix the reported error, setting this to RTBC.
Comment #18
rajab natshahComment #19
rajab natshahThank you :)
Merged
✅ Released Bootstrap Layouts 8.x-5.5