[PHP]Bug in PHP? Ik zou het haast gaan denken...

[NaliXL]

Hallo!

Ik heb hier een stukje PHP-script, wat telkens een simpele parse-error produceert. Ik heb dit script steeds versimpeld tot ik niet veel verder kon, maar kon geen fouten ontdekken. Ook mijn 2 collega's konden niets vinden. Om het nog erger te maken blijft de error ook nog bestaan als ik de fout-producerende lijn commentariseer, met /* en */ . De file op een andere plaats zetten helpt ook niet. Verder staat dit script in een simpele door Dreamweaver gegenereerde basis-HTML tussen de body-tags. De allereerste regel bevat het stukje script <?php session_start(); ?> om de sessie te starten. Mijn excuses dat de posting zo lang is, maar ik wilde met mijn commentaar zo goed mogelijk laten zien waarom ik denk dat dit niet mijn fout is. Hier is in ieder geval de code :

code:

<?php
    // Get the current path, and put it in the currentpath directory
    $CurrentPath = getcwd();
    
    // Replace windows-slashed by unix-slashes 
    $CurrentPath = str_replace("\", "/", $CurrentPath);
    
    // Separate all directory elements in an array  
    $CurrentPath = explode("/", $CurrentPath); 
    
    // Assign the number of elements of $CurrentPath to $Temp
    $Temp = count($CurrentPath);
     
    // Substract 1 from $Temp
    $Temp = $Temp - 1;
     
    // Assign the last value of Currentpath (count($CurrentPath) - 1) to $Pietje
    $Pietje = $CurrentPath[$Temp]; 
    
    // Check if the last element of CurrentPath ($Pietje) is an empty string. This
    // is where the parse-error is given every time...              
    if ($Pietje == ""){
        $PathBack = 2; // If the last element is empty, take the one before last element
    } else {
        $PathBack = 1; // Else just take the last element
    }
    
    // Use the number obtained above to take the right element of Currentpath, and put
    // that element in currentpath. I don't need the rest of the array anyway.
    $CurrentPath = $CurrentPath[count($CurrentPath) - $PathBack];
    
    // Register currentpath as a session-variable. Session_start has been done at the very beginning
    // of the file, in the form of <?php session_start(); ?>
    session_register("CurrentPath");
    
    // Print a piece of javascript that will transport us to another page.
    printf("\n<SCRIPT Language=\"JavaScript\">\n");
    printf("parent.location = '../index.php';\n");
    printf("</SCRIPT>\n");
?>

Heel hartelijk bedankt voor je hulp!!

[wiho]

Vervang

code:

$CurrentPath = str_replace("\", "/", $CurrentPath);

door

code:

$CurrentPath = str_replace("\\", "/", $CurrentPath);

[_DeWie_]

Welke error krijg je precies ??

[NaliXL]

Hmm, die slashes had ik inderdaad over het hoofd gezien. Ik kreeg eerst een parse-error, maar nu dat met die slashes is aangepast, krijg ik geen error meer. Er gaat echter nog steeds wat fout, want alles onder de regel

code:

session_register("CurrentPath");

word gewoon in de HTML gezet, in plaats van door PHP ge-parsed. Ten eerste zou die fout met die dubbele slashes op een veel eerdere regel een fout hebben moeten geven. Ten tweede als dat de oorzaak van mijn probleem zou zijn, dan denk ik dat het nu had moeten werken. In ieder geval denk ik nog steeds dat het hiet om een PHP-bug gaat.

[Glabbeek]

Verander de commentaarregel erboven

code:

// of the file, in the form of <?php session_start(); ?>

eens zodat er geen <?php en ?> meer in voorkomen. Als het daarop misloopt is het zeker een grote bug. Het is een wilde gok, maar het enige wat ik nu kan bedenken.

[LuCarD]

PHP:

<?      // Get the current path, and put it in the currentpath directory      $CurrentPath = getcwd();            // Replace windows-slashed by unix-slashes       $CurrentPath = str_replace("\", "/", $CurrentPath);            // Separate all directory elements in an array        $CurrentPath = explode("/", $CurrentPath);             // Assign the number of elements of $CurrentPath to $Temp      $Temp = count($CurrentPath);             // Substract 1 from $Temp      $Temp = $Temp - 1;             // Assign the last value of Currentpath (count($CurrentPath) - 1) to $Pietje      $Pietje = $CurrentPath[$Temp];             // Check if the last element of CurrentPath ($Pietje) is an empty string. This      // is where the parse-error is given every time...                              if ($Pietje == ""){?>

Bekijk het nu nog eens... En ben nog steeds van mening dat het een php bug is?????

de session_register fout komt door ?> in de commentaar...

?> is end php... ok in de commentaar regels!

[drm]

NaliXL:
Mijn excuses dat de posting zo lang is, maar ik wilde met mijn commentaar zo goed mogelijk laten zien waarom ik denk dat dit niet mijn fout is

Man, als iedereen 's zo duidelijk was

Glabbeek:
Als het daarop misloopt is het zeker een grote bug.

Daar kan je volgens mij heel lang over discussieren, want daar ben ik het dus niet mee eens

[NaliXL]

Ik dacht dat alles wat in commentaar staat genegeerd zou moeten worden door een compiler?

[Fatamorgana]

Op vrijdag 24 mei 2002 10:40 schreef NaliXL het volgende:
Ik dacht dat alles wat in commentaar staat genegeerd zou moeten worden door een compiler?

Daar heb je gelijk in, dat ZOU die ook moeten doen. Selectief commentaar heb je weinig aan, je moet er gewoon alles in kunnen zetten.

[NaliXL]

Hmm, in ieder geval bedankt voor het meedenken voor een oplossing, het script doet nu in ieder geval waar het voor bedoeld is. Dat een sluittag in commentaar word meegenomen vind ik eigenlijk wel een bug, of op z'n minst een zwak punt van PHP, maar goed, er valt mee te leven. Weten we dat ook weer...

[LuCarD]

Op vrijdag 24 mei 2002 10:48 schreef NaliXL het volgende:
Hmm, in ieder geval bedankt voor het meedenken voor een oplossing, het script doet nu in ieder geval waar het voor bedoeld is. Dat een sluittag in commentaar word meegenomen vind ik eigenlijk wel een bug, of op z'n minst een zwak punt van PHP, maar goed, er valt mee te leven. Weten we dat ook weer...

Meld hem aan....!

http://bugs.php.net
open-source is groot geworden door bijdrage van iedereen....

[Glabbeek]

Op vrijdag 24 mei 2002 10:39 schreef drm het volgende:
Daar kan je volgens mij heel lang over discussieren, want daar ben ik het dus niet mee eens

Nu ik er nog eens over nadenk, is het inderdaad geen bug. Anders zou je ook nooit iets als

code:

Selected item: <?php echo $selected; //show selected ?> <BR>

kunnen gebruiken, al gaat mijn voorkeur dan uit naar

code:

Selected item: <?php echo $selected; /* show selected */ ?> <BR>

[SJR]

't Is `normal behavior' inderdaad: http://bugs.php.net/bug.php?id=4879

zie ook: http://www.php.net/manual/language.basic-syntax.comments.php3

[djc]

It's a bug, not a feature!

Deze is al verscheidene malen voorbij gekomen op de mailinglists van PHP en dergelijke, maar het is zo bedoeld. De PHP delimiters hebben zeg maar eerste precedence, zelfs boven commentaar.

[drm]

Nou, als je dat soort regels gaat commentaren, dan ben je volgens mij een beetje overdreven bezig.

plus dat php4 die prachtige asp ("prachtig" en "asp" in 1 zin ) notatie heeft overgenomen. Is wel handig (moet je uiteraard wel ff configureren in je php.ini, maar daar komt men vast wel uit)

code:

Nutteloze var: <?=$nutteloos // Nutteloos commentaar?>

Maar dat was het punt natuurlijk niet
^</offtopic>

[Glabbeek]

Op vrijdag 24 mei 2002 11:23 schreef drm het volgende:
[...]
Maar dat was het punt natuurlijk niet
^</offtopic>

Parseerror: </offtopic> without opening tag

[drm]

Glabbeek:
Parseerror: </offtopic> without opening tag

Update notes
in this version the following bug was updated:

</offtopic> previously generated a parse error when started with "Door drm". This bug is fixed. When "Door drm" is encountered, it is automatically replaced by a start-tag for <offtopic> by the precompiler