Use ... === false instead of ! ...

[WaldoMonster]

From the PHP website:

int file_put_contents ( string $filename , mixed $data [, int $flags = 0 [, resource $context ]] )
The function returns the number of bytes that were written to the file, or FALSE on failure.

This function may return Boolean FALSE, but may also return a non-Boolean value which evaluates to FALSE. Please read the section on Booleans for more information. Use the === operator for testing the return value of this function.

Code: Select all

if (!file_put_contents($dest, $data)) {
    throw new Exception('failed to create file '.$dest);

I think this and simular code shoud be changed in:

Code: Select all

if (file_put_contents($dest, $data) === false) {
    throw new Exception('failed to create file '.$dest);

Or is this to paranoia because files from 0 bytes are not possible?

[James Heinrich]

Or is this to paranoia because files from 0 bytes are not possible?

I'd have to look at all the instances where that code exists, but most likely it's fine the way it is: either it failed to access the file for writing, or something else failed and there's no data to write, either way we have a problem.

But still, it may make more sense to write it correctly.

[James Heinrich]

Actually, it turns out that that block of code has already been rewritten in the latest development version.

[WaldoMonster]

Actually, it turns out that that block of code has already been rewritten in the latest development version.

Thanks for the info.