Ticket #592 (new Feature Request)

Opened 8 years ago

Last modified 8 years ago

Additional Fields in User registration -- Local authentication

Reported by: ravindra_b@… Owned by:
Priority: low Milestone: Not yet assigned to a Milestone
Component: Auth server, Feature request Version:
Keywords: additional fields in user registration Cc: ravindra_b@…

Description

Hi,

While exploring the wireless authentication software, I have come to your site and excited to see the developments. I am working at The Institute of Mathematical Sciences, Chennai, India (www.imsc.res.in) and we have short term/ long term visitors.

It may be better idea to add additional fields (like Full name, Institute/Organinization?, Collaborator, Collaborator's email address, etc.,) so that the application may find wider community to implement.

I hope, I am not asking too much.

Thanking you for your wornderful work. Sincerely, B. Raveendra Reddy, The Institute of Mathematical Sciences, Chennai, India.

Attachments

wifidog-auth-imsc.tar.gz Download (42.3 KB) - added by ravindra_b@… 8 years ago.
Modified files at IMSc for reference
wifidog-auth-imsc-svndiff.tar.gz Download (2.7 KB) - added by ravindra_b@… 8 years ago.
svn diff output of changed files.
wifidog-imsc.png Download (134.1 KB) - added by ravindra_b@… 8 years ago.
screen shot -- in PNG file.
wifidog-imsc.2.png Download (107.8 KB) - added by ravindra_b@… 8 years ago.
screen shot with modified fields -- in PNG file.
wifidog-imsc-new.png Download (123.2 KB) - added by ravindra_b@… 8 years ago.
screen shot after default grace time with signup fields -- in PNG file.

Change History

  Changed 8 years ago by polborges

Sorry i'm a newbie and my english is bad , i thing this link dev.wifidog.org/wiki/doc/developer/UserProfiles can help you

I have the same problem too.

Changed 8 years ago by ravindra_b@…

Modified files at IMSc for reference

  Changed 8 years ago by ravindra_b@…

I have attempted to modify the wifidog/signup.php, wifidog/classes/User.php, wifidog/classes/Network.php, wifidog/templates/site/signup.tpl, sql/wifidog-postgres-schema.sql.

I have attached the modified files for comments.

  Changed 8 years ago by ravindra_b@…

Hi,

The signup script is giving the following erro.

Unable to authenticate newly created user. Please report this bug. Error was: Login successfull

Where the login is successful and it is showing the registered user has been loged in and the activation mail is being sent to the respective user.

  Changed 8 years ago by benoitg

  • This really needs to be implemented as part of the profile system (making fields optionally mandatory). Otherwise, there is no way we can accept such patches, as every group will need different fields.
  • Please send the result of the command "svn diff" instead of complete modified files. Otherwise, it's nearly impossible to evaluate a a glance what you changed.

  Changed 8 years ago by ravindra_b@…

Thank you very much for your reply. I do agree that some of these fields such as collaborator, collaborator's email, etc., are specific the implementing site.

However, I think FULL NAME, AFFILIATION/ADDRESS should be mandatory during the registration and should not be omitted during the registartion.

Here is the detailed change log.


wifidog-auth/sql/wifidog-postgres-schema.sql

Added the following fields to users table.

fullname text, affiliation text, collaborator text, colemail text,

wifidog-atuh/wifidog/classes/User.php

Modified this function and included above fields to create statement

wifidog-auth/wifidog/install.php

Modified -- admin user creation accordingly.

wifidog-auth/wifidog/classes/Network.php

Changed the following statement to pass additional fields.

$user = User
createUser(get_guid(), $username, $this, , ,,,,);

wifidog-auth/wifidog/templates/sites/signup.tpl

Modified the registration form to have additional fields.


I am uploading the svn diff out for your checking. I may be a new bee in implementing this sort of application and my approach may be wrong. Please advise.

Changed 8 years ago by ravindra_b@…

svn diff output of changed files.

Changed 8 years ago by ravindra_b@…

screen shot -- in PNG file.

  Changed 8 years ago by benoitg

Is disagree, several groups want to preserve user's annonymity. All mandatory fields must be definable by the groups.

Changed 8 years ago by ravindra_b@…

screen shot with modified fields -- in PNG file.

follow-up: ↓ 8   Changed 8 years ago by ravindra_b@…

Hi,

Thank you very much for your reply. I need to have those additional fields during registration. Could you please advise me on the error message? Where else I need to modify the code? This is a site specific modification so that it will help me implementing at our organization. My guess is some thing to do with number of fields in the form. Please check the screen shot advise me.

Regards, ravi

in reply to: ↑ 7   Changed 8 years ago by benoitg

Run svn update, most likely this error message is completely unrelated.

  Changed 8 years ago by ravindra_b@…

Hi,

Thank you very much for fixing the error. Now the default login is successful and it is giving the 20 minute browse time along with the signup fields. I have attached the screen shot. Please have a look.

Changed 8 years ago by ravindra_b@…

screen shot after default grace time with signup fields -- in PNG file.

  Changed 8 years ago by ravindra_b@…

Hi,

I think the default setup is to display the registration page after successful registration. Is this intentional?

Is n't it better to avoid the registration screen after successful registration and login?

Thanks and regards, ravi

Note: See TracTickets for help on using tickets.