[dokuwiki] New PR: Refactor cache.php to better conform with PSR 2

  • From: "micgro42" <wiki@xxxxxxxxxxxx>
  • To: dokuwiki@xxxxxxxxxxxxx
  • Date: Sat, 2 Feb 2019 14:23:49 +0100 (CET)

Hi,

micgro42 opened a new pull request at 
https://github.com/splitbrain/dokuwiki/pull/2689:

This is part of the PSR 2 refactoring effort. It splits up `inc/cache.php` into 
one file for every class and leaves thin wrappers behind that trigger 
`E_USER_DEPRECATED` errors.

## ToDO
There are some variables still named with a leading underscore. This used to 
indicate a private/protected scope, but at least 
`\dokuwiki\Cache\Cache::$_event` is accessed from the outside as well. How 
should we proceed here?

```
FILE: /home/michael/public_html/dokuwiki/inc/Cache/Cache.php
 16 | WARNING | Property name "$_event" should not be prefixed with an 
underscore to indicate visibility
 17 | WARNING | Property name "$_time" should not be prefixed with an 
underscore to indicate visibility
 18 | WARNING | Property name "$_nocache" should not be prefixed with an 
underscore to indicate visibility
FILE: /home/michael/public_html/dokuwiki/inc/Cache/CacheParser.php
 15 | WARNING | [ ] Property name "$_event" should not be prefixed with an 
underscore to indicate visibility
```

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

c5f28220-26ed-11e9-9722-127e970aa89a


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

Other related posts:

  • » [dokuwiki] New PR: Refactor cache.php to better conform with PSR 2 - micgro42