Modify

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#1162 closed defect (worksforme)

RegistrationModule can't be disabled

Reported by: iacobs@… Owned by: Matt Good
Priority: normal Component: AccountManagerPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description

It appears that IRegistrationModule doesn't check whether it is disabled or not, so it will always display the register link/provide the registration form. Patch attached to fix the issue

Attachments (1)

register_disable.diff (572 bytes) - added by iacobs@… 17 years ago.

Download all attachments as: .zip

Change History (3)

Changed 17 years ago by iacobs@…

Attachment: register_disable.diff added

comment:1 Changed 17 years ago by Matt Good

Resolution: worksforme
Status: newclosed

Well, it's just RegistrationModule, not IRegistrationModule. However, it's unnecessary for a component to test if it's enabled since it will never be called if it's been disabled. I've just tested disabling it here and it works as expected. Check your trac.ini again, or use the WebAdmin panel to disable the RegistrationModule. If you need more help check on the trac:MailingList.

comment:2 Changed 17 years ago by anonymous

The module name was a typo, sorry about that. I checked several times before checking the source. I wanted to have admin interface user management, while disallowing registration, so I disabled the module (acct_mgr.web_ui.registrationmodule = disabled in trac.ini), and the registration link still appeared in the main interface, so I'm not quite sure where this is coming from (other modules can be disabled, however, so it's not a trac issue), the easiest way was to add a check in _enable_check; my problem is solved, anyway :)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Matt Good.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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

 
Note: See TracTickets for help on using tickets.