PHPhulp.nl PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
M

Undefined index

Door Max M op 17-09-2017 10:50
2.894 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
?>
- SanThe -
17-09-2017 11:07
$_GET['size'] en $_GET['crop'] bestaan blijkbaar niet.
M
Max M
17-09-2017 11:45
Ik heb het er even tussen uit gehaald, maar dan komen de foto's een stuk kleiner terug als ik ze uit de cash haal.
Het werkt dus wel, en er kwamen gelijk weer een aantal andere foutmeldingen.
P
Pipo Clown
17-09-2017 11:50
Grote kans dat, wanneer je naar PHP7 gaat, dat er nog veel meer problemen bij komen.

Advies: Maak alles meteen PHP7-proof.
M
Max M
17-09-2017 15:18
Ja dat wil ik ook doen, maar vind het wel makkelijk om het in kleine stapjes te doen, het script werk wel op php 7.1
Maar ik hoop dat iemand met een oplossing voor deze foutmelding komt.
O
Obelix Idefix
17-09-2017 16:26
Max M op 17/09/2017 10:50:42

Sinds ik over ben gegaan naar php versie 5.6

Max M op 17/09/2017 15:18:07

het script werk wel op php 7.1

Ben je overgestapt op 5.6 of op 7.1?


Max M op 17/09/2017 15:18:07

Maar ik hoop dat iemand met een oplossing voor deze foutmelding komt.

Is al gegeven:
- SanThe - op 17/09/2017 11:07:46

$_GET['size'] en $_GET['crop'] bestaan blijkbaar niet.

Je zult dus moeten controleren of / zorgen dat deze bestaan.
M
Max M
17-09-2017 20:15
Nou heb het nu op php 7.0 werkend de fout zit er nog wel in maar ook nog een andere:
PHP Deprecated: Methods with the same name as their class will not be constructors in a future version of PHP

En dit is dat stukje:

# FORMAT DATE FROM LANG CALENDAR FILE
class kdate
{
var $distime = 0;
var $time_zone;
var $date_sep;
var $date_format;
var $date_display;
var $clock_format;
var $daylight_savings;
var $adjust_date = 1;

function kdate($dateSettings='')
{
Ariën
17-09-2017 21:06
Tegenwoordig (al sinds PHP 5) moet je __construct() gebruiken, in plaats van een functie die dezelfde naam als de class heeft.

Verder raad ik ook aan om je te verdiepen in 'visibility'.

Eigenlijk is de hele tutorial wel aardig om te lezen.
M
Max M
18-09-2017 08:56
Bedankt Ariën s ook opgelost, nu deze nog: Undefined variable: page

dit is die regel:
if( !in_array($page, $_SESSION["admin_user"]["permissions"]) && $page != "welcome" && $page != "toolslinks" && $page != "permissions_error" && $page != "" && $page != "options" )

En hoe krijg je de code netjes in een vensterje op dit forum ?
Ariën
18-09-2017 09:24 gewijzigd op 18-09-2017 09:27
Dan moet je de code-tags gebruiken. Zie ook de 'Veel gestelde vragen'.

En over je probleem: waar komt $page vandaan?
M
Max M
20-09-2017 09:31 gewijzigd op 20-09-2017 09:36
Bijna alles opgelost, alleen deze kom ik niet uit line 61

PHP Notice: Undefined index: debugMode in /home/.../domains/domein.nl/public_html/image.php on line 61, 


/*
	* 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))
	{	
		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);
		}

Reageren

Inloggen om te reageren