Hi Bob, today I tried your SiteCheck with MODX 3 and it works as far as I can say. Thanks!
Herewith I would like to report about it and some small issues.
I fixed a sql collation on my new MODX 3 installations: modx_actions_fields, had the sql collation utf8mb3_general_ci instead of utf8mb4_general_ci
Q1: This seems to be the case with all MODX 3 installations and maybe it is desired?
Checking Transport Packages shows these errors: Checking Package: resizer-1.0.1-pl -> Source: resizer-1.0.1-pl.transport.zip Package has no transport file Checking Package: tinymcerte-2.0.9-pl -> Invalid package_name
Q2: I guess it appears because the resizer is a beta, but I do not understand what is invalid about the Tiny RTE package?
open_basedir in Error Log generated by SiteCheck: (ERROR @ /var/www/MyDomain.tld/httpdocs/modx/core/components/sitecheck/model/sitecheck/pathcheck.class.php : 659) PHP warning: file_exists(): open_basedir restriction in effect. File(/var/www/MyDomain.tld/httpdocs/modx/core/components/sitecheck/model/sitecheck/pathcheck.class.php/config.core.php) is not within the allowed path(s): (/var/www/MyDomain.tld/:/tmp/)
Q3: What do I need to add in php.ini at the line open_basedir? This doesn’t work: open_basedir = /var/www/MyDomain.tld/:/tmp/:/var/lib/php/sessions/
If I leave it blank, the error does not appear. Is that important? It seems to work even with this error.
Two more messages in Error Log generated by SiteCheck: (ERROR @ /var/www/MyDomain.tld/httpdocs/modx/config.core.php : 7) PHP warning: Constant MODX_CORE_PATH already defined (ERROR @ /var/www/MyDomain.tld/httpdocs/modx/config.core.php : 8) PHP warning: Constant MODX_CONFIG_KEY already defined
on point 1:
Also in a new MODX 3.0.0-pl installation (now 3.0.2-pl) that I installed locally for testing, the modx_actions_fields had a mb3 reference instead of mb4. Weird, my collation default is utf8mb4_general_ci. No matter why, but your SiteCheck has found my fault
btw: Locally I use MariaDB and on the live site MySQL.
on point 2:
No, the resizer-1.0.1-pl doesn’t exist, because it is a new installation and only the resizer-1.0.2-beta works with MODX 3. I checked the TinyMCE Rich Text Editor from MODX EXTRA (loaded and installed via manager), everything looks fine to me.
OK, it doesn’t seem important to me, because everything was installed properly and is working
on point 3:
I also think that this is not a fault of your tool. It has something to do with the settings of open_basedir.
on point 4:
I suspect that this message appears from SiteCheck? I get this on my live site and on my local test site.
On point 4, I think you’re correct. Each of the config.core.php are “included” and checked. There’s an @ sign on the include to suppress that error. I don’t see the error, and don’t know why you do, but the error is bound to occur, since I need to include all three of the files to check them. I could substitute my own error handler, but it seems like overkill.