Closed (fixed)
Project:
Advanced CSS/JS Aggregation
Version:
7.x-2.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
19 Jun 2020 at 14:01 UTC
Updated:
22 Sep 2020 at 20:19 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
thallesComment #3
kevin morse commentedSo a few lines above, the code checks
if (empty($info_on_files[$cid]['content_hash']))Since this line isn't throwing any error, I am assuming we could use this same test to see if
$info_on_files[$cid]['content_hash']is null before doing the test on line 872 that is throwing the Noticeif ($info_on_files[$cid]['content_hash'] !== $file_contents_hash)Since something that is null is not going to equal the hash of a file. We can probably change the test to be if empty() or the original test.
Thoughts?
Comment #4
rishabhthakur commentedComment #5
kevin morse commentedHere is my attempt at a patch.
This also fixes another PHP 7.4 issue that I was having.
Comment #6
kevin morse commentedI think my patch was not in utf8 so that's why it couldn't apply?
Comment #7
rishabhthakur commentedhi @kavin Morse
Please let me know if need then i will look on it
Comment #8
rishabhthakur commentedModify @kevin patch with 7.x-2.x-dev branch code and add some coding standard format.
Comment #9
rishabhthakur commentedComment #10
kevin morse commentedHi Rishabh,
Thanks for fixing my patch. I have queued up an additional test using PHP 7.4 and MySQL 5.7 since this patch is meant to address a PHP 7.4 issue.
Comment #11
kevin morse commentedI have been using this patch in production for a few weeks now with no issue.
Would be great if someone else could also test it.
Comment #12
joaopauloscho commentedThe patch #9 works for me.
Comment #14
thallesComment #15
thallesThanks @all!