Simple code random error

[Mudsaf]

This messes whole page with no erros.

foreach ($file_allowed_extensions as $i => $file_allowed_extensions_value) {if ($fileupload == 0  || !isset($fileupload)) {if ($file_extension == $file_allowed_extensions_value) {$fileupload = 1;} else {$fileupload = 0;}}} foreach ($thumbnail_allowed_extensions as $i => $thumbnail_allowed_extensions_value) {if ($thumbnail_extension == $thumbnail_allowed_extensions_value) {if ($thumbnailupload == 0 || !isset($thumbnailupload)) {$thumbnailupload = 1;} else {$thumbnailupload = 0;}}}

This doesnt

foreach ($file_allowed_extensions as $i => $file_allowed_extensions_value) {if ($fileupload == 0  || !isset($fileupload)) {if ($file_extension == $file_allowed_extensions_value) {$fileupload = 1;} else {$fileupload = 0;}}}foreach ($thumbnail_allowed_extensions as $i => $thumbnail_allowed_extensions_value) {if ($thumbnail_extension == $thumbnail_allowed_extensions_value) {//if ($thumbnailupload == 0 || !isset($thumbnailupload)) {$thumbnailupload = 1;} else {$thumbnailupload = 0;}//}}

Seems like had problems with $thumbnailupload, changed it to $thumbupload and works.

Edited February 13, 2013 by Mudsaf

[callumacrae]

So the problem is fixed now?A couple notes:First, you should indent your code. I can't read it. Something like this:

foreach($file_allowed_extensions as $i => $file_allowed_extensions_value) {	if ($fileupload == 0 || !isset($fileupload)) {		if ($file_extension == $file_allowed_extensions_value) {			$fileupload = 1;		} else {			$fileupload = 0;		}	}}foreach($thumbnail_allowed_extensions as $i => $thumbnail_allowed_extensions_value) {	if ($thumbnail_extension == $thumbnail_allowed_extensions_value) {		if ($thumbnailupload == 0 || !isset($thumbnailupload)) {			$thumbnailupload = 1;		} else {			$thumbnailupload = 0;		}	}}

Second, you can use ternary to shorten your code quite a bit. The following code does the same as above:

foreach($file_allowed_extensions as $i => $file_allowed_extensions_value) {	if ($fileupload == 0 || !isset($fileupload)) {		$fileupload = ($file_extension == $file_allowed_extensions_value) ? 1 : 0;	}}foreach($thumbnail_allowed_extensions as $i => $thumbnail_allowed_extensions_value) {	if ($thumbnail_extension == $thumbnail_allowed_extensions_value) {		$thumbnailupload = ($thumbnailupload == 0 || !isset($thumbnailupload)) ? 1 : 0;	}}

[Mudsaf]

So the problem is fixed now? A couple notes: First, you should indent your code. I can't read it. Something like this:

foreach($file_allowed_extensions as $i => $file_allowed_extensions_value) {	if ($fileupload == 0 || !isset($fileupload)) {		if ($file_extension == $file_allowed_extensions_value) {			$fileupload = 1;		} else {			$fileupload = 0;		}	}} foreach($thumbnail_allowed_extensions as $i => $thumbnail_allowed_extensions_value) {	if ($thumbnail_extension == $thumbnail_allowed_extensions_value) {		if ($thumbnailupload == 0 || !isset($thumbnailupload)) {			$thumbnailupload = 1;		} else {			$thumbnailupload = 0;		}	}}

Second, you can use ternary to shorten your code quite a bit. The following code does the same as above:

foreach($file_allowed_extensions as $i => $file_allowed_extensions_value) {	if ($fileupload == 0 || !isset($fileupload)) {		$fileupload = ($file_extension == $file_allowed_extensions_value) ? 1 : 0;	}} foreach($thumbnail_allowed_extensions as $i => $thumbnail_allowed_extensions_value) {	if ($thumbnail_extension == $thumbnail_allowed_extensions_value) {		$thumbnailupload = ($thumbnailupload == 0 || !isset($thumbnailupload)) ? 1 : 0;	}}

Yea i agree with identifying the code, but if i would ternary shorten my code i could not understand it as i do now.