Menu

#435 (ok 3.1.2) php5.2+ warning does not work

closed-accepted
PHP errors (22)
1
2009-01-19
2008-12-31
No

there are hundreds of pages containing
"Parse error: syntax error, unexpected T_STRING, expecting T_OLD_FUNCTION or T_FUNCTION or T_VAR or '}'
in .... /libraries/Error_Handler.class.php on line 25" to be found in google, see also
https://sourceforge.net/forum/forum.php?thread_id=2692135&forum_id=72909 and
https://sourceforge.net/forum/forum.php?thread_id=2620874&forum_id=72909 etc.

pma should avoid revealing the path and give a meaningful explanation instead;

my (simple) suggestion:

* @version $Id: Error_Handler.class.php 10738 2007-10-08 16:02:58Z cybot_tm $
*/

if (version_compare(PHP_VERSION, '5.2.0', '<')) {
die ('PHP 5.2+ is required! ');
}

All the best for 2009,
Jürgen

Discussion

  • Marc Delisle

    Marc Delisle - 2009-01-01
    • assigned_to: nobody --> lem9
     
  • Marc Delisle

    Marc Delisle - 2009-01-01
    • summary: pma3 needs php5.2+ warning --> php5.2+ warning does not work
     
  • Marc Delisle

    Marc Delisle - 2009-01-01

    There is already such warning in common.inc.php which worked in the past but with additions made later, the first PHP 5 problem occurs higher in the code.

     
  • Marc Delisle

    Marc Delisle - 2009-01-01

    Your patch merged but in common.inc.php, thanks and best wishes for 2009!

     
  • Marc Delisle

    Marc Delisle - 2009-01-01
    • milestone: --> For_immediate_release
    • priority: 5 --> 1
    • summary: php5.2+ warning does not work --> (ok 3.1.2) php5.2+ warning does not work
    • status: open --> open-accepted
     
  • Jürgen Wind

    Jürgen Wind - 2009-01-02

    >There is already such warning in common.inc.php which worked in the past
    yeah, I saw that *after* I hit the send button ;)

     
  • Marc Delisle

    Marc Delisle - 2009-01-19
    • status: open-accepted --> closed-accepted
     
Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.