littlebizzy / slickstack

Lightning-fast WordPress on Nginx
https://slickstack.io
GNU General Public License v3.0
643 stars 113 forks source link

Fatal error due to leftover "tmp" folder in the mu-plugins directory #115

Closed corzel closed 2 years ago

corzel commented 3 years ago

Hi! I'm getting this error and notices in settings tabs. https://domain.com/wp-admin/admin.php?page=slickstack&tab=blacklist (&tab=cron, &tab=sftp, &tab=caching, etc.)

Fatal error: Cannot declare class LittleBizzy\Autoloader\Autoloader, because the name is already in use in /var/www/html/wp-content/mu-plugins/tmp/autoloader.php on line 34 

The cause of the fatal error is the last line in autoloader.php

// Autostart
new Autoloader;

If it's removed settings works again.

jessuppi commented 3 years ago

Hmm... I'm not sure why there is a /tmp folder inside your mu-plugins directory? It seems autoloader.php exists twice and that is causing a fatal error, it should only exist once (and definitely not in the /tmp/ child folder).

jessuppi commented 2 years ago

A follow up here... this was caused by a bug in ss-install-wordpress-mu-plugins that seemed to affect sites using a custom list of MU plugins in their ss-config file... in the past few months, I have done a lot of cleanup on that script.

Probably still some improvement to be had with how custom MU plugins are installed, but for now it should be working better and more importantly, the script will force delete the tmp folder every time it runs now.