| commit | 1eb39399c1e00540d94c71b0ff2cf1779edec388 | |
| author | fiwswe <53953985+fiwswe@users.noreply.github.com> | |
| Wed, 22 Mar 2023 19:43:52 +0000 (22 20:43 +0100) | ||
| committer | GitHub <noreply@github.com> | |
| Wed, 22 Mar 2023 19:43:52 +0000 (22 20:43 +0100) | ||
| tree | b15e2e237470040250107c5b1c7c9ab10e1bc9aa | treesnapshot (tar.gz zip) |
| parent | 9c6974793ee2feadeac51bf08bf45b6d07275e8e | commitdiff |
Correction to fix of corner case
Fixes https://github.com/dokuwiki/dokuwiki/pull/3928/commits/e7794be61135b07c6227c367ebf735ba94a2b200
IMHO paying the test on the same variable (`$size`) as in the original test in https://github.com/dokuwiki/dokuwiki/blob/775003a7dfc8824e655f040724abfb8255c6da03/lib/plugins/logviewer/admin.php#L143 is better/more clear than basing it on something caused as a result of that test, i.e. `$toread` having a specific value.
Fixes https://github.com/dokuwiki/dokuwiki/pull/3928/commits/e7794be61135b07c6227c367ebf735ba94a2b200
IMHO paying the test on the same variable (`$size`) as in the original test in https://github.com/dokuwiki/dokuwiki/blob/775003a7dfc8824e655f040724abfb8255c6da03/lib/plugins/logviewer/admin.php#L143 is better/more clear than basing it on something caused as a result of that test, i.e. `$toread` having a specific value.
| lib/plugins/logviewer/admin.php | diffblobblamehistory |