giterlizzi / dokuwiki-plugin-datatables

Add DataTables support to DokuWiki
GNU General Public License v2.0
10 stars 11 forks source link

Empty excluded pages configuration property behavior #14

Closed tienod closed 8 years ago

tienod commented 8 years ago

If the excludedPages setting is empty i would assume that no page is excluded.

But the line 63 in action.php interprets it as if every page is excluded in this case: if ((bool) preg_match('/'.$this->getConf('excludedPages').'/', $ID)) {

Adding a check to this condition to check if excludedPages is not empty would clarify an empty excludedPages setting.

cheers

tienod commented 8 years ago

Great, Thanks!