Support » Plugin: All In One WP Security & Firewall » Compatibility fix for WPML causes 404 error

Compatibility fix for WPML causes 404 error

  • bdevitaly

    (@bdevitaly)


    1 year, 1 month ago

    Hello,
    I would like to report a fix to be made for the compatibility with WPML. Some years ago the compatibility fix below was inserted into the file /wp-content/plugin/all-in-one-wp-security-and-firewall/classes/wp-security-process-renamed-login-page.php, lines 185-191. 

    /*
 * Compatibility fix for WPML plugin
 */
if (function_exists('icl_object_id') && strpos($home_url_with_slug,$login_slug)){
    $home_url_with_slug = home_url($login_slug);
    function qtranxf_init_language() {}
}

    Well now it causes issues, pephaps because it was deprecated and removed from version 3.3, as you can see here: https://wpml.org/documentation/support/creating-multilingual-wordpress-themes/language-dependent-ids/

    In particular, it sends 404 error if the login url has been edited in the brute force section.
    To solve this issue, the code has to be removed, as you could see in the solution posted in this WPML forum page: https://wpml.org/it/forums/topic/incompatibilita-con-all-in-one-wp-security-firewall-errore-404/
    I hope it helps and I hope that fix will be implemented soon in a future update.
    Regards

Viewing 7 replies - 1 through 7 (of 7 total)
  • Plugin Contributor mbrsolution

    (@mbrsolution)

    1 year, 1 month ago

    Thank you kindly for reporting this. I have submitted a message to the developers to investigate further your findings.

    Kind regards

    Plugin Author wpsolutions

    (@wpsolutions)

    1 year, 1 month ago

    Hi @bdevitaly
    What is the wpml plugin folder name and main plugin filename?

    Since wpml is a paid plugin I am not able to check for myself, but the reason I am asking is because if I wanted to use the get_plugin_data function I need to know the value one should use for the “$plugin_file” parameter.

    jillebehm

    (@jillebehm)

    12 months ago

    @bdevitaly : is this about the problem that if you have configured another login page that it will always give a 404 when you try to go to the ‘new’ login page?

    @wpsolutions @mbrsolution : I justed talked to WPML support and it looks indeed like the fix that bdevitaly states fixes the problem. So removing lines 185-191 from /wp-content/plugin/all-in-one-wp-security-and-firewall/classes/wp-security-process-renamed-login-page.php

    @wpsolutions to answer your question:
    – plugin folder name: sitepress-multilingual-cms
    – main plugin filename: sitepress.php

    bdevitaly

    (@bdevitaly)

    12 months ago

    @jillebehm Yes, I confirm it! And I’d like to add that I just updated the “All In One WP Security & Firewall” plugin, which brought back the lines 185-191 in /wp-content/plugin/all-in-one-wp-security-and-firewall/classes/wp-security-process-renamed-login-page.php and indeed the new login page stopped working again. At which I have again solved by commenting on those lines.
    @wpsolutions Sorry, I missed the question you asked last month, I’m glad @jillebehm has replied. In the next release of the plugin, will this issue be fixed?

    Plugin Author wpsolutions

    (@wpsolutions)

    12 months ago

    Hi @bdevitaly

    In the next release of the plugin, will this issue be fixed?

    This fix will be in the release following the new one which just came out today.

    bdevitaly

    (@bdevitaly)

    12 months ago

    That’s awesome!

    jillebehm

    (@jillebehm)

    12 months ago

    @wpsolutions : great, thanks! 😀

Viewing 7 replies - 1 through 7 (of 7 total)
  • The topic ‘Compatibility fix for WPML causes 404 error’ is closed to new replies.

Views