- Issue created by @ajits
- Status changed to Needs review
over 1 year ago 1:48pm 15 March 2023 - 🇮🇳India ajits India
Handled an error thrown
Error: Call to a member function getPath() on null in components_library_info_alter() (line 18 of /var/www/html/docroot/modules/contrib/components/components.module).
- Status changed to Needs work
over 1 year ago 3:37pm 23 March 2023 - 🇨🇭Switzerland stefanos.petrakis@gmail.com Biel, Switzerland
It looks good.
Maybe we can be more defensive regarding the$extension_path
variable?
As in, if still an empty string, do not enter the main loop.if (!empty($extension_path) && isset($info['components']) && isset($info['components']['autodetect_libraries']) && isset($info['components']['namespaces'])) {
- 🇸🇮Slovenia alecsmrekar
Updating the patch to read from the module/theme list service, instead of reading from cache directly. I also incorporated feedback from Stefanos.
- Status changed to Needs review
about 1 year ago 1:14pm 26 October 2023 - last update
about 1 year ago 61 pass - 🇨🇦Canada RobLoach Earth
Thanks for all the work here, everyone. How does this approach differ from https://www.drupal.org/project/components/issues/2707849 ✨ Allow components to define asset libraries Needs work ?