Moodle

Move allowemailaddresses and denyemailaddresses settings to Email-based self-registration auth plugin

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.8.2, 1.9
  • Fix Version/s: 1.9, 2.0
  • Labels:
    None
  • Affected Branches:
    MOODLE_18_STABLE, MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE, MOODLE_20_STABLE

Description

In the admin block, the allowemailaddresses and denyemailaddresses settings are now accessible under "Server > Email".

With the new multi-authentication of Moodle 1.8, it would be more logical to see them under "User > Authentication", in the auth plugin "Email-based self-registration". (Or are the email domain restrictions used in other components?)

Issue Links

Activity

Hide
Petr Škoda (skodak) added a comment -

My +1 for this change

Show
Petr Škoda (skodak) added a comment - My +1 for this change
Hide
James Cracknell added a comment -

Or there could be a very obvious link in place!

Show
James Cracknell added a comment - Or there could be a very obvious link in place!
Hide
Martin Dougiamas added a comment -

The email domain masks could potentially be used by other parts of Moodle too, but they aren't. Are they likely to be? Probably not. Let's move them.

Show
Martin Dougiamas added a comment - The email domain masks could potentially be used by other parts of Moodle too, but they aren't. Are they likely to be? Probably not. Let's move them.
Hide
Martin Dougiamas added a comment -

--> Petr

Show
Martin Dougiamas added a comment - --> Petr
Hide
Petr Škoda (skodak) added a comment -

My +1 to move it too and store it in email plugin config. If needed we can add independent option to other plug-in that allows sign up in future.

Show
Petr Škoda (skodak) added a comment - My +1 to move it too and store it in email plugin config. If needed we can add independent option to other plug-in that allows sign up in future.
Hide
Nicolas Martignoni added a comment -

Candidate for 1.9?

Show
Nicolas Martignoni added a comment - Candidate for 1.9?
Hide
Petr Škoda (skodak) added a comment -

We should IMHO convert the auth to use admin tree settings first, but the problem is that it is does not support config plugins yet.

Show
Petr Škoda (skodak) added a comment - We should IMHO convert the auth to use admin tree settings first, but the problem is that it is does not support config plugins yet.
Hide
Petr Škoda (skodak) added a comment -

fixed, thanks!

Show
Petr Škoda (skodak) added a comment - fixed, thanks!
Hide
Nicolas Martignoni added a comment -

Verified on fresh 1.9beta3. Closing. Thanks Petr!

Show
Nicolas Martignoni added a comment - Verified on fresh 1.9beta3. Closing. Thanks Petr!

Dates

  • Created:
    Updated:
    Resolved: