diff options
| -rw-r--r-- | build/psalm-baseline.xml | 6 | ||||
| -rw-r--r-- | lib/private/Files/View.php | 6 |
2 files changed, 5 insertions, 7 deletions
diff --git a/build/psalm-baseline.xml b/build/psalm-baseline.xml index e02710054ea..ea4b4f7ba62 100644 --- a/build/psalm-baseline.xml +++ b/build/psalm-baseline.xml @@ -2209,12 +2209,10 @@ <InternalMethod occurrences="1"> <code>getParams</code> </InternalMethod> - <InvalidReturnStatement occurrences="2"> - <code>$insertQb->execute()</code> + <InvalidReturnStatement occurrences="1"> <code>$this->adapter->lastInsertId($seqName)</code> </InvalidReturnStatement> - <InvalidReturnType occurrences="2"> - <code>int</code> + <InvalidReturnType occurrences="1"> <code>string</code> </InvalidReturnType> <InvalidScalarArgument occurrences="1"> diff --git a/lib/private/Files/View.php b/lib/private/Files/View.php index 0abc870dc6f..16a9381768b 100644 --- a/lib/private/Files/View.php +++ b/lib/private/Files/View.php @@ -1191,13 +1191,13 @@ class View { throw $e; } - if ($result && in_array('delete', $hooks)) { + if ($result !== false && in_array('delete', $hooks)) { $this->removeUpdate($storage, $internalPath); } - if ($result && in_array('write', $hooks, true) && $operation !== 'fopen' && $operation !== 'touch') { + if ($result !== false && in_array('write', $hooks, true) && $operation !== 'fopen' && $operation !== 'touch') { $this->writeUpdate($storage, $internalPath); } - if ($result && in_array('touch', $hooks)) { + if ($result !== false && in_array('touch', $hooks)) { $this->writeUpdate($storage, $internalPath, $extraParam); } |