Priority: Normal
Current State: Closed
Released In: n/a
Target Release: n/a
Applies To: Engine
Component: Configure
Branches:
I was using
DatabasePlugin, and I entered an invalid value for {Plugins}{!DatabasePlugin}{Databases} (I missed a comma). Configure script did not help me to discover the misconfiguration ... I created a small perl script to discover the error.
The screen that displays the variable changes and asks for the password to continue could be improved:
The script could evaluate the new values for perl data types, and give an error message if the new setting does not compile.
Foswiki did not break ... I could not find the new value in LocalSite.cfg. So it seems that this check is already being done somewhere. It should also be done a little earlier to help admins to enter valid values.
--
KipLubliner - 29 Dec 2011
Thanks for reporting this issue.
The current (trunk) version of configure validates PERL data items, so if this Extension is using configure - I don't have DatabasePlugin installed, but assume it is since you opened the item against configure - you shouldn't experience this problem in the future.
Configure now parses PERL items to make sure they conform to the restricted subset of PERL that is acceptable, and also checks for errors when it
eval s the item. Errors are reported in real-time - usually with a pointer to the line in error. (Always with a pointer to the line where the problem is detected - not always the same thing.)
I'm closing this item as it is fixed in trunk, and
Item12180 owns the check-ins/release process; if you still have your reproducer and can make it fail, feel free to re-open it.
I'm also documenting the checkins specific to this issue here - although they depend on a number of other checkins in the configure overhaul and can not be applied to previous versions of Foswiki.
--
TimotheLitt - 11 Dec 2012
If a task has checkins, and is a bug that was reported against a prior release, it should probably go into Waiting For Release status. The only task that should be closed with checkins are ones that fix bugs new to the code that were never released. This way the specific summary gets added to the release notes. This seems like one worth calling out in the release notes.
--
GeorgeClark - 11 Dec 2012