Remove "allow_override" property from rabbit_hole.behavior_settings.* configs

Created on 17 June 2023, about 1 year ago
Updated 20 June 2023, about 1 year ago

Problem/Motivation

"Allow these settings to be overridden for individual entities" is a checkbox on Rabbit Hole bundle settings. If it's enabled, the "Rabbit Hole" field is created on form save and the corresponding config property is updated in the config object.

The problem is that those 2 things can be unsynchronized: anyone can edit the property directly in the config, or the "Rabbit Hole" field can be removed.

The question is: can we assume that RH values can be overridden if the field exists? I think so.

If there is a need to disable overrides but keep existing values in the database, then the right way to do it is to revoke permissions.

Proposed resolution

1) Add a method that will check the RH field and return the state;
2) Keep the checkbox on the form and get its state based on the "Rabbit Hole" field presence;
3) Remove "allow_override" property from config;

πŸ“Œ Task
Status

Fixed

Version

2.0

Component

Code

Created by

πŸ‡ΊπŸ‡¦Ukraine Matroskeen πŸ‡ΊπŸ‡¦ Ukraine, Lutsk

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Production build 0.69.0 2024