fix: Model::save() may call unneeded countAllResults()

[kenjis]

Description
When useAutoIncrement is disabled, countAllResults() is called, even if $data does not have id value.
You'll see the query like the following:

SELECT COUNT(*) AS `numrows` FROM `news` WHERE `id` IS NULL

<?php

function f1() {
  return false && false ? true : 'a';
}
function f2() {
  return false && (false ? true : 'a');
}

var_dump(f1()); // "a"
var_dump(f2()); // false

Checklist:

• Securely signed commits
• Component(s) with PHPDoc blocks, only if necessary or adds value
• Unit testing, with >80% coverage
• User guide updated
• Conforms to style guide

[kenjis]

@paulbalandan
This bug was introduced in v4.1.4:
https://github.com/codeigniter4/CodeIgniter4/pull/4840/files#diff-1a690370bdbdc5997068f9ff210bacef314eeb65d1df73af206783ce055d8d1dL613

Is this a human error or php-cs-fixer error?

[paulbalandan]

I am not sure what happened, but I think it was an error. 😂

[kenjis]

Don't worry. Nobody's perfect.

I thought if php-cs-fixer causes this error, we have to check the rule.
But now I run php-cs-fixer, ( ) are not removed.

I don't like long conditions. It is difficult to read and do step debugging.
I refactored.

[paulbalandan]

The new code looks much better and understandable. 👍