[3.7.57]: GC throws error when "removing empty temp folders"

krizzzly commented 2 years ago

What happened?

Description

$ ./craft gc
Delete all trashed items? (yes|no) [no]:yes
Running garbage collection ...
    > purging unsaved drafts that have gone stale ... done
    > deleting stale user sessions ... done
    > deleting stale feature announcements ... done
    > deleting trashed rows in the `{{%elements}}` table ... done
    > deleting trashed rows in the `{{%categorygroups}}` table ... done
    > deleting trashed rows in the `{{%entrytypes}}` table ... done
    > deleting trashed rows in the `{{%fieldgroups}}` table ... done
    > deleting trashed rows in the `{{%sections}}` table ... done
    > deleting trashed rows in the `{{%taggroups}}` table ... done
    > deleting partial asset data in the `{{%assets}}` table ... done
    > deleting partial category data in the `{{%categories}}` table ... done
    > deleting partial entry data in the `{{%entries}}` table ... done
    > deleting partial global set data in the `{{%globalsets}}` table ... done
    > deleting partial Matrix block data in the `{{%matrixblocks}}` table ... done
    > deleting partial tag data in the `{{%tags}}` table ... done
    > deleting partial user data in the `{{%users}}` table ... done
    > deleting partial asset data in the `{{%content}}` table ... done
    > deleting partial category data in the `{{%content}}` table ... done
    > deleting partial entry data in the `{{%content}}` table ... done
    > deleting partial global set data in the `{{%content}}` table ... done
    > deleting partial tag data in the `{{%content}}` table ... done
    > deleting partial user data in the `{{%content}}` table ... done
    > deleting orphaned drafts and revisions ... done
    > deleting orphaned search indexes ... done
    > deleting trashed rows in the `{{%structures}}` table ... done
    > deleting trashed rows in the `{{%fieldlayouts}}` table ... done
    > deleting trashed rows in the `{{%sites}}` table ... done
    > deleting trashed volumes and their folders ... done
    > removing empty temp folders ... Error: Argument 1 passed to craft\base\Volume::directoryExists() must be of the type string, null given, called in /home/propatie/www/ch.propatient.beta/vendor/craftcms/cms/src/services/Gc.php on line 269

Steps to reproduce

i migrated a craft v2.9.2, pruned old revisions and tried to run garbage collector.

Craft CMS version

3.7.57

PHP version

No response

Operating system and version

7.4.30

Database type and version

No response

Image driver and version

No response

Installed plugins and versions

Compress 1.0.3 Guest Entries 2.4.0 Internal Assets 2.3.2 Kint 2.0.0 oEmbed 1.3.18 Redactor 2.10.10 Sprout Encode Email 2.0.7 Sprout Fields 3.8.5 Sprout Notes 2.2.3

brandonkelly commented 2 years ago

Thanks for reporting that. Fixed now for the next release, which should be coming soon.

brandonkelly commented 2 years ago

Craft 3.7.59 and 4.3.1 have been released with that fix.

craftcms / cms