Strict Standards: Only variables should be passed by reference in...

[dlc]

Hello,

 

I have a "working" upload script but i do get an error:

 

Strict Standards: Only variables should be passed by reference in C:\Users\UHN\Desktop\xampp-win32-1.7.4-VC6\xampp\htdocs\upload.php on line 99

 

Line 99:

$file_ext = strtolower(end(explode('.', $file_name)));

 

thanks..

 

[Muddy_Funster]

what is $file_name? an array or a flat variable?

[dlc]

I will put some more code to make ik clearer.

 

if($_SERVER['REQUEST_METHOD'] == "POST") {

$input_name = $_POST['filename']; // The given name

        $file_name = $_FILES['filen']['name']; // The file name of the uploaded file

        $file_ext = strtolower(end(explode('.', $file_name))); // Get the extension

......

 

 

[Muddy_Funster]

Out of cuiosity, what do you get if you

print_r($file_name)

[dlc]

I uploaded the file: aannp.pdf

 

And print_r($file_name); gives:

aannp.pdf

 

so that should be right.

[Muddy_Funster]

yeah, looks like it should be spot on.  How about trying this in place of it and see what you get?

$file_ext = strtolower(substr($file_name), -(strpos($file_name, '.')+1));

[dlc]

I get:

 

^{Warning: substr() expects at least 2 parameters, 1 given in C:\Users\UHN\Desktop\xampp-win32-1.7.4-VC6\xampp\htdocs\upload.php on line 100}

 

^{Warning: strtolower() expects exactly 1 parameter, 2 given in C:\Users\UHN\Desktop\xampp-win32-1.7.4-VC6\xampp\htdocs\upload.php on line 100}

 

line 100 is:

^{$file_ext = strtolower(substr($file_name), -(strpos($file_name, '.')+1));}

 

[Muddy_Funster]

my bad, malformed the code

$file_ext = strtolower(substr($file_name, -(strpos($file_name, '.')+1)));

[dlc]

Now i have a new error:

 

^{Notice: Undefined variable: extensioncheck in C:\Users\UHN\Desktop\xampp-win32-1.7.4-VC6\xampp\htdocs\upload.php on line 116}

$extensioncheck is an array holding multiple extensions including " .pdf "

 

If i use ^{print_r($file_ext);} i get:

np.pdf

 

The original name is aannp.pdf

[Muddy_Funster]

cahnge +1 to -1 and see how it goes ( I keep getting confused when working backwards through a string...and with most other things in life..... )

[dlc]

It works!

thank you for your help 

 

dlc.