[dokuwiki] New PR: Fixed inconsistent handling of falsy values on fperm setting

  • From: "movatica" <wiki@xxxxxxxxxxxx>
  • To: dokuwiki@xxxxxxxxxxxxx
  • Date: Tue, 18 Feb 2020 19:34:45 +0100 (CET)

Hi,

movatica opened a new pull request at 
https://github.com/splitbrain/dokuwiki/pull/2985:

The $conf['fperm'] value was checked in multiple files using different methods.
This can cause permission trouble with restricted environments, i.e. when chmod 
is forbidden and file permissions are non-default.
Now, all checks use implicit cast to boolean which leads to consistent 
behaviour.

Also, a misleading variable was renamed in context to better understand one of 
the checks.

Please help us to review this pull request, so new contributors get feedback in 
a timely manner.

547ba380-527d-11ea-8a92-5ba95d539737


-- 
DokuWiki mailing list - more info at
http://www.dokuwiki.org/mailinglist

Other related posts:

  • » [dokuwiki] New PR: Fixed inconsistent handling of falsy values on fperm setting - movatica