Fatal Error Crash

  • Resolved cvladan

    (@cvladan)


    5 months, 1 week ago

    I’ve also posted an issue on GitHub

    Please consider adding a check to see if writing to wp-config.php is possible, because on some hosts, such as this one ( raidboxes.io ), the wp-config.php file is locked, and I think this is a good practice. It’s fine to modify it, but the code should be resilient to the fact that it can’t be modified, so if it can’t write to wp-config.php, it should simply report this instead of causing the site to crash.

    PHP Warning:  file_put_contents(/.../wp-config.php): Failed to open stream: Permission denied in /.../wp-content/plugins/admin-site-enhancements/classes/class-wp-config-transformer.php on line 418

    PHP Fatal error:  Uncaught Exception: Failed to update the config file. in /.../wp-content/plugins/admin-site-enhancements/classes/class-wp-config-transformer.php:421

    Once again – thank you for the excellent plugin.

Viewing 14 replies - 1 through 14 (of 14 total)
  • Plugin Author Bowo

    (@qriouslad)

    @cvladan thanks for reporting this. Will see what I can do.

    p.s. I wonder why they are locking wp-config.php? What is the rationale? Who can edit them then?

    I can confirm the same issue. We lock the wp-config.php file and only us can do changes to it. Many hostings would lock the file for security reasons. I don’t even have to activate the “Disable the plugin and theme editor.” and site still crashes. Downgrading to 7.4.4 for now

    https://video.ultimatemarketing.group/conversations/5a9d1efa-82bb-555a-a5a2-63cc46acdaa4

    Same Here, but happens on one of my WP sites!!!!

    • This reply was modified 5 months, 1 week ago by DJABHipHop.
    • This reply was modified 5 months, 1 week ago by DJABHipHop.

    Same file different bug, but happens on one of my WP sites!!!!

    Plugin Author Bowo

    (@qriouslad)

    @everyone I’ve implemented a mechanism to check if wp-config.php is writeable and whether the DISALLOW_FILE_EDIT constant is already defined there or not. The resulting logic is smarter in the sense that it will try to modify wp-config.php if possible, and if not, will use PHP define() method instead. Please test when you see it.

    Thanks again for reporting the issue.

    Thread Starter cvladan

    (@cvladan)

    Thanks ??

    Still a problem. And our client is really upset that the site is crashing. WP 6.6,1 & ASE version 7.4.5…had to disable ASE because of the instability. To our knowledge this is the only site of 62 that its happening on. But maybe there are others we’re not aware of.

    PHP Fatal error: Uncaught Exception: Failed to update the config file. in /nas/content/live/centerpoints/wp-content/plugins/admin-site-enhancements/classes/class-wp-config-transformer.php:421#012Stack trace:#012#0 /nas/content/live/centerpoints/wp-content/plugins/admin-site-enhancements/classes/class-wp-config-transformer.php(324): ASENHA\Classes\WP_Config_Transformer->save(‘<?php\n# Databas…’)#012#1 /nas/content/live/centerpoints/wp-content/plugins/admin-site-enhancements/classes/class-disable-smaller-components.php(201): ASENHA\Classes\WP_Config_Transformer->update(‘constant’, ‘DISALLOW_FILE_E…’, ‘false’, Array)#012#2 /nas/content/live/centerpoints/wp-includes/class-wp-hook.php(324): ASENHA\Classes\Disable_Smaller_Components->enable_plugin_theme_editor(”)#012#3 /nas/content/live/centerpoints/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters(NULL, Array)#012#4 /nas/content/live/centerpoints/wp-includes/plugin.php(517): WP_Hook->do_action(Array)#012#5 /nas/content/live/centerpoints/wp-settings.php(555): do_action(‘plugins_loaded’)#012#6 /nas/content/live/centerpoints/wp-config.php(120): require_once(‘/nas/content/li…’)#012#7 /nas/content/live/centerpoints/wp-load.php(50): require_once(‘/nas/content/li…’)#012#8 /nas/content/live/centerpoints/wp-blog-header.php(13): require_once(‘/nas/content/li…’)#012#9 /nas/content/live/centerpoints/index.php(17): require(‘/nas/content/li…’)#012#10 {main}#012 thrown in /nas/content/live/centerpoints/wp-content/plugins/admin-site-enhancements/classes/class-wp-config-transformer.php on line 421Thu, Oct 3, 2024, 09:48 AM MDT

    PHP Warning: file_put_contents(/nas/content/live/centerpoints/wp-config.php): Failed to open stream: Permission denied in /nas/content/live/centerpoints/wp-content/plugins/admin-site-enhancements/classes/class-wp-config-transformer.php on line 418

    Thread Starter cvladan

    (@cvladan)

    @imaginarytrout … I don’t think we have update with these changes.
    At least, I didn’t get it. It’s an easy fix so it will work, I’m sure.

    Plugin Author Bowo

    (@qriouslad)

    @imaginarytrout please downgrade to v7.4.4 on the problematic site. The fix will be released in v7.4.6, scheduled for Monday.

    @qriouslad Same file different bug, but happens on one of my WP sites!!!!
    Undefined variable: anchor in?wp-content/plugins/admin-site-enhancements/classes/class-wp-config-transformer.php?on line?242

    Plugin Author Bowo

    (@qriouslad)

    @pressthemes1 thanks for reporting. On that site where you’re seeing the error, can you check whether the wp-config.php file contains one of the following lines?

    • /* That’s all, stop editing! Happy publishing. */
    • /* That’s all, stop editing! Happy blogging. */

    Either comes by default in wp-config, and is required for the wp-config-transformer.php script to work properly.

    DJABHipHop

    (@pressthemes1)

    @qriouslad? yes i have it,

    Plugin Author Bowo

    (@qriouslad)

    @pressthemes1 thanks for checking. That’s strange. Anyway, I made a small change that adds a 3rd option for lines to check in wp-config.php beside the two listed in my previous reply. When v7.4.7 is out, please test again on that one site of yours.

    Plugin Author Bowo

    (@qriouslad)

    @everyone, I have not heard further reports of a similar issue. So, considering this resolved for now at v7.5.2. Please report back if you still experience errors / issues.

Viewing 14 replies - 1 through 14 (of 14 total)
  • You must be logged in to reply to this topic.