Undefined index

Door Max M op 17-09-2017 10:50

2.892 views

Wie wil me hier mee even helpen ?
Sinds ik over ben gegaan naar php versie 5.6 staat mijn hele errorlog vol met de volgende error:
Undefined index: size in /home/domein/domains/domein.nl/public_html/image.php on line 45
Undefined index: crop in /home/domein/domains/domein.nl/public_html/image.php on line 45

ik kome er zelf niet uit, hier onder het stukje uit het script line 45

alvast bedankt, Max

Code (php)

/*
* Caching of images
*/
$cacheFile = "id{$_GET['mediaID']}-".md5("{$_GET['type']}-{$_GET['mediaID']}-{$_GET['folderID']}-{$_GET['size']}-{$_GET['crop']}").'.jpg'; // Name of cached file (line 45)

//$_SESSION['testing'][$_GET['mediaID']] = $cacheFile; // Testing

$cachePathFile = BASE_PATH."/assets/cache/{$cacheFile}";

if(file_exists($cachePathFile))
{
//if(!$_SESSION['debugMode'] and $config['cacheImages'] and !$offSiteRequest) // Check for debug mode
?>

Ariën

20-09-2017 09:43

$_SESSION['debugMode'] bestaat hier niet. Ook hier zou ik een isset() op toepassen.

Max M

20-09-2017 10:10

Denk dat ik toch nog iets fout heb staan nu ?


<?php
/*
    * Caching of images
    */

    $mediaID = isset($_GET['mediaID']) ? $_GET['mediaID'] : '';
    $type = isset($_GET['type']) ? $_GET['type'] : '';
    $folderID = isset($_GET['folderID']) ? $_GET['folderID'] : '';
    $size = isset($_GET['size']) ? $_GET['size'] : '';
    $crop = isset($_GET['crop']) ? $_GET['crop'] : '';
    
    $cacheFile = "id{'$mediaID'}-".md5("{'$type'}-{'$mediaID'}-{'$folderID'}-{'$size'}-{'$crop'}").'.jpg'; // Name of cached file
        
    //$_SESSION['testing'][$_GET['mediaID']] = $cacheFile; // Testing
    
    $cachePathFile = BASE_PATH."/assets/cache/{$cacheFile}";
    
    if(file_exists($cachePathFile))

    $debugMode = isset($_SESSION ['debugMode']) ? $_SESSION ['debugMode'}] : '';
    $cacheImages = isset($_SESSION ['cacheImages']) ? $_SESSION ['cacheImages'}] : '';
    
    {    
        if(!$_SESSION['$debugMode'] and $config['$cacheImages'] and !$offSiteRequest) // Check for debug mode (line 61)
        {
            $cacheTime = gmdate("U")-$config['cacheImagesTime'];
            $fileTime = filemtime($cachePathFile);
            
            if($cacheTime < $fileTime)
            {    
                header("Content-type: image/jpeg");
                //header('Last-Modified: ' . gmdate('D, d M Y H:i:s', filemtime($thumbnail)) . ' GMT');
                
                //ob_clean();
                //flush();

                readfile($cachePathFile);
                exit;
            }
            else // Cleanup old cached file
                @unlink($cachePathFile);
        }
     
?>

Ariën

20-09-2017 10:24 gewijzigd op 20-09-2017 10:25

$_SESSION['$debugMode']
eh...?

Je behandelt $debugMode hier als een string, en bovendien is die variabele al een kopie van de waarde van $_SESSION['debugMode']. Die hoort niet in zijn index in ieder geval. ;-)

Gewoon $debugMode is al voldoende.

$config['$cacheImages'] lijkt mij ook niet echt correct om deze reden.

- SanThe -

20-09-2017 11:42

Regel 18 een if() en op regel 23 de openings { van de if() met daartussen 2 regels...???

Max M

20-09-2017 12:07

Ja Sante sorry, maar ik probeer ook maar wat en soms lukt het :-) Maar vind het wel leuk om te doen, dus ga me er wel in verder in verdiepen.

Je mag me wel laten zien hoe het dan wel moet, daar leer je toch ook weer van.

Ariën

20-09-2017 12:10 gewijzigd op 20-09-2017 12:11

Na een if, waarin een voorwaarde wordt gesteld volgt, normaal gesproken, altijd een {
die afgesloten wordt met } op het moment dat de actie uitgevoerd is.

Misschien frist dit je geheugen even op:
https://www.w3schools.com/php/php_if_else.asp

Reageren

Inloggen om te reageren