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.
I was seeing the following php warnings in the logs:
Warning: readdir() expects parameter 1 to be resource, boolean given in advagg_install_get_first_advagg_file() (line 1225 of advagg.install)
Since opendir() can return a boolean, $handle shouldn't be passed directly to readdir() without checking if it's false first.
Comment | File | Size | Author |
---|---|---|---|
#1 | advagg-2445153-1-opendir-check.patch | 736 bytes | mikeytown2 |
Comments
Comment #1
mikeytown2 CreditAttribution: mikeytown2 commentedThanks for the report. Easy fix. Patch has been committed.
Comment #3
hankpalan.com CreditAttribution: hankpalan.com commentedI feel like a real idiot here. I've disabled, uninstalled, and removed the module from my server. But I'm still getting the hundreds of errors from the old code. Even if I install/update the module, its still seeing the the old error.
I've cleared every cache I know possible. I'm stuck here.
Comment #4
mikeytown2 CreditAttribution: mikeytown2 commentedDon't know why you'd still be getting those errors after the module has been uninstalled. But I can tell you that the code in question with that line number (1278) means you're not using the latest dev. Latest dev would throw that error on line 1282 if the issue still existed.
APC/opcode cache?
Comment #5
hankpalan.com CreditAttribution: hankpalan.com commentedIt was some caching outside of my knowledge. It finally cleared out. I just left it alone for the night. Its all up and running now. Thank you!