Love Fuel?    Donate

FuelPHP Forums

Ask your question about FuelPHP in the appropriate forum, or help others by answering their questions.
SimpleAuth improvement
  • Fuel RussiaFuel Russia
    Please add activation by email option to SimpleAuth. This is the best spam blocker.
  • Fuel RussiaFuel Russia
    Another improvements:
    1) Change last_login type from varchar(25) to int(10) in DB table
    2) Add fields:
    registered int(10)
    registration_ip varchar(39)
  • huglesterhuglester
    I'm not sure if Jelmer will agree on all this. That's why it is called SimpleAuth. I think the best would be just to clone the SimpleAuth, rename it to smth different, and just to add those more options. Good luck!
  • Ben CorlettBen Corlett
    +1 for email activation. is such a simple process :)
  • Fuel RussiaFuel Russia
    Jaroslav Petrusevic wrote on 02/03/11 7:39 am:
    I'm not sure if Jelmer will agree on all this. That's why it is called SimpleAuth.
    I think the best would be just to clone the SimpleAuth, rename it to smth different, and just to add those more options.

    This options is very simple. PS: I will make SuperAuth with avatars, twitter/fb integration & etc stuff;)
  • Jelmer SchreuderJelmer Schreuder
    At this point forgotten password still has to be written so there's work to be done on it in general, but I do agree that some kind of activation is basic enough to be part of the simpleauth driver. On the field suggestions for the DB: I agree with modification to last_login & adding registered. "registration_ip" seems kinda useless to me, that's a one-off field that has no use afterwards. A last_ip might be usefull.
  • Fuel RussiaFuel Russia
    Also I think you should add field screen_name, because there is method in SimpleAuth: get_user_screen_name() (now screen_name=username). And on create_user method we can set screen_name to username if screen_name is empty.
    Field screen_name useful, because user can use short login jsmith, and full name John Smith.
  • Jelmer SchreuderJelmer Schreuder
    This is one of those parts where SimpleAuth should remain simple. Adding screennames adds a whole layer of complexity (may screen_name/username match, conflict, must be unique) and to much implementation is involved. I added support for screen_name just for the reason you gave, but to use it with your own Auth driver you'll have to extend SimpleAuth. SimpleAuth must remain simple and contain as little specific implementation as possible to allow people to extend it without having to rewrite huge chunks of it. It can be a basis for one's own or be an example for one.
Add a Comment

Howdy, Stranger!

It looks like you're new here. If you want to get involved, click one of these buttons!

Sign In Apply for Membership

Categories

In this Discussion