Modify

Opened 4 years ago

Last modified 4 years ago

#7174 new enhancement

[PATCH] Properly register all configuration options so IniAdmin can edit them

Reported by: matthijs Owned by: obs
Priority: normal Component: BlackMagicTicketTweaksPlugin
Severity: normal Keywords: patch
Cc: Trac Release: 0.11

Description

The plugin currently only registers a few options through the Option class. Others are requested using self.config.get directly, since these have dynamic names, based on the value of the tweaks configuration value.

The attached patch fixes this by registering all options, even those with dynamic names. The motivation to do this, is so that the IniAdminPlugin can then find the options and allows you to change them through the web interface.

Furthermore, I've changed the default values of most options to '' instead of None, since the Option class didn't seem to work with None. However, looking at #7124 suggests that it didn't work before either, so this patch probably fixes #7124 as well.

I've also documented all registered options, including the existing ones.

Attachments (1)

bmtt-configuration.diff (15.4 KB) - added by matthijs 4 years ago.
Updated patch against 0.11 version, r8017

Download all attachments as: .zip

Change History (3)

comment:1 Changed 4 years ago by matthijs

Ah, one more thing. I've tested the entire thing, except for the stuff in IPermissionPolicy, since check_permission didn't seem to get called at all for me (but also not with a clean trunk, so this is probably another bug...).

Changed 4 years ago by matthijs

Updated patch against 0.11 version, r8017

comment:2 Changed 4 years ago by matthijs

I've updated the patch, fixing an occurence of '' where [] was required.

Add Comment

Modify Ticket

Action
as new .
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.