I have been updating my site using "drush up" and getting an error on the 8002 update.

Error: Call to a member function getEffect() on null in file_browser_update_8002() (line 130 of /var/www/sitename/modules/contrib/file_browser/file_browser.install) #0

Not sure if this is a bug or if it's a problem in my site config. Any idea's on how I can get passed this error?

Thanks.
Rick.

CommentFileSizeAuthor
#2 file-browser-update-improvements.patch3.81 KBsamuel.mortenson

Comments

Rick Bergmann created an issue. See original summary.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new file-browser-update-improvements.patch3.81 KB

Hi @rick-bergmann, it looks like the update hooks I wrote weren't properly checking if the Image Style existed before attempting an update. I've already committed this to HEAD, but if you apply this patch and re-run the updates you should not see the notice.

Status: Needs review » Needs work

The last submitted patch, 2: file-browser-update-improvements.patch, failed testing.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
Status: Needs work » Needs review
rick bergmann’s picture

rick bergmann commented

Thanks. I can confirm applying the patch worked.

albertski’s picture

albertski commented
Status: Needs review » Reviewed & tested by the community

Patch applied for me and the code looks good.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Awesome! This is already committed so I'll close the issue out.

Status: Fixed » Closed (fixed)

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