I'm looking for a part-time remote job.

Hire me


I'm the author of:

Mastering Redmine is a comprehensive guide with tips, tricks and best practices, and an easy-to-learn structure.

Check the book's project or

Buy the book

Social pages of the book:

By buying this book you also donate to Redmine (see this page).


Follow me:

Bug #1966

use registration settings

Added by bynn ies about 12 years ago. Updated about 12 years ago.

Status:
Closed
Priority:
Major
Assignee:
Target version:
Start date:
10 Feb 2012
Due date:
16 Apr 2012
% Done:

100%

Redmine version:
External issue:

Description

This plugin automatically accepts anyone who registers using openid even if the administration setting is set to something else like manual registration. The following fies this bug.

replace:
register_automatically(user) do
onthefly_creation_failed(user)
end

with:
case Setting.self_registration
when '1’
register_by_email_activation(user) do
onthefly_creation_failed(user)
end
when '3’
register_automatically(user) do
onthefly_creation_failed(user)
end
else
register_manually_by_administrator(user) do
onthefly_creation_failed(user)
end
end

Associated revisions

Revision 3 (diff)
Added by Andriy Lesyuk about 12 years ago

Added support for manual activation (fixes #1966)

History

#1 Updated by Andriy Lesyuk about 12 years ago

  • Status changed from New to Open
  • Assignee set to Andriy Lesyuk

Hi! Thanks for report!

I thought about this earlier... Let me explain why I did the way I did:

E-Mail activation is needed to verify email... But should it be verified if user uses a trusted OpenID source? I believe it’s redundant. What do you think?

Regarding manual actiovation... Yes, it can have a sense...

#2 Updated by bynn ies about 12 years ago

Hey,

I agree, include the automatic and manual activation and leave out the email activation.

#3 Updated by Andriy Lesyuk about 12 years ago

  • Status changed from Open to Under Verification
  • Target version set to 0.0.2
  • % Done changed from 0 to 100

I believe I have fixed it! Please test if possible (take code from SVN)...
Thanks for contribution!

#4 Updated by Andriy Lesyuk about 12 years ago

  • Due date set to 16 Apr 2012
  • Status changed from Under Verification to Closed

Also available in: Atom PDF

Terms of use | Privacy policy