Problem/Motivation

The file_delete_ui_install() hook doesn't work well for profile installations because there's a possibility that the views.view.files config might not exist at that point.

Steps to reproduce

Add the module to a profile installation will create an invalid views.view.files config.

Proposed resolution

Check that the config exists before attempting to update it.

Remaining tasks

Provide issue fork/patch.

User interface changes

N/A

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
#3 3318919-file_delete_ui-support-profile-installs-3.diff612 bytescodebymikey

Issue fork file_delete_ui-3318919

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

codebymikey created an issue. See original summary.

codebymikey opened merge request !1

codebymikey’s picture

codebymikey commented
Status: Active » Needs review
StatusFileSize
new 3318919-file_delete_ui-support-profile-installs-3.diff612 bytes

douggreen’s picture

douggreen commented
Status: Needs review » Fixed

Thanks! I committed a slight tweak from this so that it still catches the case when $config does not exist at all (I'm not sure that it can ever happen, but I didn't want to lose that check). If you confirm this works, I'll tag a new release.

codebymikey’s picture

codebymikey commented
Status: Fixed » Reviewed & tested by the community

All looks good to me.

codebymikey’s picture

codebymikey commented
Status: Reviewed & tested by the community » Fixed

This issue has been fixed and tagged.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.