# Ticket #8663 (closed defect: fixed)

Opened 2 years ago

Reported by: Assigned to: Thierry Michel hasienda normal AccountManagerPlugin normal user registration link disable mmitar@gmail.com 0.12

### Description

There is no way to disable the register link on the login page without disabling the RegisterModule?. Having a new boolean option into the trac.ini could solve the problem.

I added this option to the AccountModule?

login_register_link = BoolOption('account-manager', 'login_register_link',
False, 'Set to True to enable register link on the login page.')


and use it like this into the LoginModule?.process_request() method :

'registration_enabled': AccountModule(self.env


## Change History

### 04/03/11 03:12:26 changed by hasienda

• keywords set to user registration link disable.

I notice a possible bug, since the link is present for me even with the RegistrationModule disabled.

Why do you think, you'll need the module without the link? I'll take a look and should be able to fix the wrong logic in the first place, without another trac.ini option.

Please discuss further on, especially in case you feel, that I misunderstood your point.

### 04/13/11 20:38:51 changed by anonymous

@hasienda: I can confirm that I'm seeing the same bug you are -- even with the RegistrationModule disabled, the "Register" link will appear in the login page, which should not be the case.

I also cannot think of a reason why the appearance of the "Register" link would not be determined by the enabling or disabling of the Registration Module.

### 04/14/11 01:28:11 changed by hasienda

Yes, without the link you need to know, how to trigger the registration page, guess from code.. Impracticable, if you ask me.

Again, why bother, if only both - the module and the link - can work together? Until now this seem a pointless discussion, at least I fail to see your point. Enlighten me, please. :-)

### 06/09/11 00:58:03 changed by hasienda

See #6821 for a hint on the background of the bug mentioned here.

### 06/16/11 22:13:55 changed by hasienda

After more feedback I've reworked the new login page concept.

You'll get a boolean option login_opt_list, so you can switch off the list inside of the central login box.

### 06/16/11 22:14:19 changed by hasienda

• status changed from new to assigned.

### 06/16/11 22:34:52 changed by hasienda

(In [10304]) AccountManagerPlugin: Fix _enable_check() in RegistrationModule refs #6821 and #8663.

The functions name has been misleading, as it has been more like a sibling of _write_check() in AccountModule.

### 06/16/11 22:46:25 changed by hasienda

(In [10306]) AccountManagerPlugin: Do the real check, and compatible to Trac 0.11 too, refs #6821 and #8663.

Plugin components can be activated implicitly, if the code is located in the plugins directory of the enironment. Method ComponentManager.is_component_enabled returns None for such a case, so it's not at all a reliable method to detect component activation.

Special thanks to David Byrne for testing and for critical hints on the issue.

### 06/28/11 21:48:58 changed by mitar

• cc set to mmitar@gmail.com.

### 07/07/11 22:11:23 changed by hasienda

• status changed from assigned to closed.
• resolution set to fixed.

(In [10393]) AccountManagerPlugin: Releasing version 0.3, pushing development to 0.4.

This new feature release finally propagates a number of solutions into an official release, after some time of testing with trunk, so explicitely closes #442, #816, #2966, #3989, #4160, #6821, #7111, #8534, #8549, #8663, #8813, #8892, #8925, #8936 and #8939.

Should have made this months ago, but felt so many pending issues were too bad for a new release. But it has been a tremendous ticket burndown since last year, so it's really worth considering an upgrade now. See fresh changelog for details.

### 07/07/11 23:10:25 changed by hasienda

(In [10395]) AccountManagerPlugin: Releasing version 0.3, pushing development to 0.4.

This new feature release finally propagates a number of solutions into an official release, after some time of testing with trunk, so explicitely closes #442, #816, #2966, #3989, #4160, #6821, #7111, #8534, #8549, #8663, #8813, #8892, #8925, #8936 and #8939.

Should have made this months ago, but felt so many pending issues were too bad for a new release. But it has been a tremendous ticket burndown since last year, so it's really worth considering an upgrade now. See fresh changelog for details.