1. This site uses cookies. By continuing to use this site, you are agreeing to our use of cookies. Learn More.

Fixed set_magic_quotes_runtime problem with php5.3

Discussion in 'Resolved Bug Reports' started by ragtek, Dec 3, 2010.

  1. ragtek

    ragtek Guest

    I can't run xenforo, because set_magic_quotes_runtime is deprecated


    ;)
     
  2. ragtek

    ragtek Guest

    Am i the only, who have this problem, or is nobody using 5.3?:D
     
  3. Brogan

    Brogan XenForo Moderator Staff Member

    I notice the error is on your local install.
    I use XAMPP and I don't have that problem.
     
  4. ragtek

    ragtek Guest

  5. Kier

    Kier XenForo Developer Staff Member

    Am I to understand that your server has magic quotes runtime enabled??

    FWIW, we are running 5.3 here and have always done so.
     
  6. ragtek

    ragtek Guest

    my php.ini
    Code:
    magic_quotes_gpc = Off
    magic_quotes_runtime = Off
    error_reporting = E_ALL | E_STRICT
    display_errors = On
    
    It's because of enabled display_errors (tested it)
    On a developer enviroment IMHO it has to be enabled, and it's fact that the function is deprecated.


    I don't want to sound rude, it's my bad english;)
     
  7. Kier

    Kier XenForo Developer Staff Member

    Can you open library/XenForo/Application.php and add an @ to the function call on line 154 like this:
    PHP:
    @set_magic_quotes_runtime(false);
    Then let me know if it resolves the problem?
     
  8. ragtek

    ragtek Guest

    Yes, it's working, but why not change it to ini_set ?
    Hidding the error because you're using a deprecated function is also no nice coding style.
    Any reason you don't want to use it?:D

    Just to be clear, both methods are working:
    PHP:
    @set_magic_quotes_runtime(false);
    ini_set('magic_quotes_runtime'0);
     
  9. Mike

    Mike XenForo Developer Staff Member

    ini_set can have problems in certain situations - I've actually seen hosts disable it, bizarrely.

    It's an edge case, but ini_set will fail when the value is set via php_admin_value.
     
  10. DiscoRevilo

    DiscoRevilo Member

    On a clean install of XAMPP (installed today)
    (I am glad to see I am not the only person to have noticed this though and yes, I did read above and add the @ to suppress error reporting :))
     
  11. Floris

    Floris Guest

    I agree :) While @ hides it, it doesn't really 'resolve' it.
    that said, i understand why kier mentioned it :D
     

Share This Page