[dokuwiki] New PR: Fix handling of unknown array settings

  • From: "splitbrain" <wiki@xxxxxxxxxxxx>
  • To: dokuwiki@xxxxxxxxxxxxx
  • Date: Wed, 26 Oct 2022 16:49:31 +0200 (CEST)

Hi,

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

When a plugin used an array setting and was then installed, the old 
configuration can remain in local.php. It is handled by the SettingUndefined 
class. However this class used the base out() mechanism which didn't know about 
arrays. The result was a fatal error resulting in a completely empty config 
file.

This moves the error handling from the SettingArray class to the base class 
which fixes the original problem.

It also adds a check if the Writer is about to write a completely empty config 
file and refuses to do so, throwing an Exception.

Probably related to #3777

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

64af4ed0-553d-11ed-99db-b45aa613e4fb


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

Other related posts:

  • » [dokuwiki] New PR: Fix handling of unknown array settings - splitbrain