Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
If using the patches in #1356276: Allow profiles to define a base/parent profile the libraries module is not aware of parent profiles which may have libraries directories to be searched.
Proposed resolution
Rely on adjustment to drupal_system_listing_profile from #1356276: Allow profiles to define a base/parent profile to add paths to the directories searched so all relevant profiles are considered.
Remaining tasks
None known, this issue is to make a patch available if needed.
User interface changes
Non
API changes
libraries_get_libraries function may now search more directories than previously done.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | inherited-profiles-1783598-5.patch | 1.92 KB | dagomar |
| #4 | libraries-profile_inheritance-1783598-4.patch | 635 bytes | pmelab |
| #2 | libraries-profile_inheritance-1783598-2.patch | 680 bytes | tekante |
| #1 | libraries-profile_inheritance-1356276-1.patch | 680 bytes | tekante |
Comments
Comment #1
tekante CreditAttribution: tekante commentedPatch is attached. Will mark this as needs review just so the maintainer can take a quick look but I suspect that since this is a patch dependent on a patch this should likely be closed as a won't fix or as designed (pending real support of inherited profiles in core).
Comment #2
tekante CreditAttribution: tekante commentedCorrection of patch naming and status change. Patch is identical but improper name on previous version would make it hard to track back to this issue from a make file.
Comment #3
tstoecklerPatch looks good, but I'm setting to postponed until the core issue gets in.
Comment #4
pmelab CreditAttribution: pmelab commentedRecreated the patch for libraries version 2.1
Comment #5
dagomar CreditAttribution: dagomar commentedThe patch in #4 goes with patch #25 here: #1356276-25: Allow profiles to define a base/parent profile. However, after that the function drupal_system_listing_profile was removed from the patches. So, #4 will now result in a WSOD (call to undefined function).
To make matters more complicated, the referred issue was split up to allow D7 and D8 development to live independently from one another. So, I have created a new patch that takes into account the latest available patch (surprise, also by me) in the newly created issue here: #2067229-41: Allow install profiles to declare base profiles for Drupal 7
The patch basically lifts the drupal_system_listing_profile from the old patch and adds it to the libraries module as libraries_system_listing_profile.
Comment #6
tstoecklerMarking duplicate of #2081579: Base profile support