Fixed cache backends bug #11322

[ifsnow]

It's a PR to save number 0 in cache. (issue #11322)

[sergeyklay]

Can you please provide a small test?

[ifsnow]

@sergeyklay

I think it's just a issue related to checking if variable is null.

if !content {
    // if content is null
} else {
    // if content is not null
}

In this case, if content parameter is 0 or "0", It works as if content is null.
It's not the operation that user wanted. (user wanted to save "0")

apc.zep and memory.zep already use content === null condition to check the parameter.

[sergeyklay]

In any case, you must show that your PR has solved the issue. Now it is impossible to check without accepting your PR. A little test would have shown - issue is solved or not without accepting the PR to the main codebase

[ifsnow]

@sergeyklay
I added some codes in testDataFileCache function of CacheTest.php.
It would failed at this test in 2.0.x branch.

[sergeyklay]

Can you please rebase onto 2.0.x

[ifsnow]

@sergeyklay I have rebased onto 2.0.x

[sergeyklay]

Fine. How about:

$cache->save('test-zero-data', 0); // int, not string. as described in issue #11322

// or

$cache->save('test-zero-data', false);

can you please also provide these test cases

[sergeyklay]

Put your changes at new line

[ifsnow]

@sergeyklay added another test cases & changed wrong CHANGELOG.md

[sergeyklay]

Thanks