- Issue created by @joegraduate
- Status changed to Needs review
12 months ago 12:56am 9 September 2023 - last update
12 months ago 54 pass - πΊπΈUnited States joegraduate Arizona, USA
The attached patch changes the code that disables the form element for the
use_collection
setting in theSettingsForm
to only disable the form element if the setting is already enabled and multiple containers already exist.It occurred to me that another way this could potentially be addressed would be to add validation to the
SettingsForm
instead of dynamically disabling the form element but this patch should fix the problem of not being able to enable the setting if it isn't already enabled and multiple container already exist. -
japerry β
committed 09e68d28 on 2.0.x authored by
joegraduate β
Issue #3386220 by joegraduate: Cannot modify non-default containers or...
-
japerry β
committed 09e68d28 on 2.0.x authored by
joegraduate β
- Status changed to Fixed
7 months ago 4:57pm 9 February 2024 - πΊπΈUnited States japerry KVUO
At first I was confused why you'd want to do this, but yes you're right -- in case somehow you ended up with this box unchecked, you should be able to check it even if other containers are there.
Fixed.
Automatically closed - issue fixed for 2 weeks with no activity.