The Media library currently implements an install hook that changes the path of the default "media" view so that the Grid appears first at /admin/content/media. This works fine if the library is installed after the Media and Views modules, but if the library is installed as a profile dependency the "media" view does not exist when the library's install hook is invoked. This is due to a quirk in profile installs where optional config is installed _after_ all modules are installed.
Handle the case where the "media" view is created _after_ the library is installed.
Review the patch.
None.
None.
None.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | interdiff_4-5.txt | 2.14 KB | shaal |
| #5 | core-media-view-in-profile-2992056-5.patch | 1.25 KB | shaal |
| #4 | 2992056.patch | 1.21 KB | eric morand |
| #2 | 2992056-2.patch | 1.09 KB | samuel.mortenson |
Comments
Comment #2
samuel.mortensonPatch.
Comment #4
eric morand commentedThe previous patch doesn't apply anymore with 8.7.1. Here is a new one that does.
Comment #5
shaalI re-rolled this patch for 8.8.x
Comment #6
phenaproximaI think we should postpone this one on #2981105: Media Library should not modify the media view, since that one will probably dramatically change how the media view is modified.
Comment #7
phenaproximaComment #8
phenaproximaNow that #2981105: Media Library should not modify the media view is in, this should no longer be an issue. Media Library no longer modifies the Media view in any way upon install. Closing as oudated :)