#4123 (ok 4.1.0-beta2) Double config including

3.5.8
fixed
nobody
None
1
2013-12-12
2013-10-04
Alex Pianov
No

phpMyAdmin includes config file twice on prefs_manage.php, and since on my ubuntu config file contains function declaration, it fails with cannot redeclare function silently. It was very hard to find bug because phpmyadmin disables error_reporting before evaling config file, and then re-enables it.

Related

Bugs: #4123

Discussion

  • Piotr Przybylski

    Config file isn't a good place for any functions, but I will look into it. Maybe I can do it so that config.inc.php is always loaded only once.

     
  • Marc Delisle

    Marc Delisle - 2013-10-30

    Hi Piotr,
    any progress?

     
  • Piotr Przybylski

    Mostly done, I could push it this weekend to master. I got a bit
    sidetracked trying to fix some tests that fail only on Windows.

    2013/10/30 Marc Delisle lem9@users.sf.net

    Hi Piotr,
    any progress?


    Status: open
    Created: Fri Oct 04, 2013 08:57 PM UTC by Alex Pianov
    Last Updated: Sun Oct 06, 2013 09:29 AM UTC
    Owner: nobody

    phpMyAdmin includes config file twice on prefs_manage.php, and since on my
    ubuntu config file contains function declaration, it fails with cannot
    redeclare function silently. It was very hard to find bug because
    phpmyadmin disables error_reporting before evaling config file, and then
    re-enables it.


    Sent from sourceforge.net because you indicated interest in
    https://sourceforge.net/p/phpmyadmin/bugs/4123/

    To unsubscribe from further messages, please visit
    https://sourceforge.net/auth/subscriptions/

     

    Related

    Bugs: #4123

  • Piotr Przybylski

    Fixed in master

     
  • Marc Delisle

    Marc Delisle - 2013-11-03
    • summary: Double config including --> (ok 4.1.0-beta2) Double config including
    • status: open --> resolved
    • Priority: 5 --> 1
     
  • Marc Delisle

    Marc Delisle - 2013-12-12
    • Status: resolved --> fixed
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks