I just posted a new tutorial:

=> I don’t like that “auth.php” controller!

thierryREY - 23 May 2007 02:20 PM

[quote author=“grahack”]
well, I had a big idea that doesn’t seem to be possible with ci:
a controller in a sub-sub-folder (maybe useless)
/controllers/folder_level_1/folder_level_2/ctrler.php

I confirm ,I have made many test and work on CI URI and Router class to fix a CI bug into the route with controller in subfolder.
Bug when using route with subfolder in it ????

I read this one, but I’m not speaking about config in routes.php, but only about a sub-sub-folder controller
urls like index.php/folder_level_1/folder_level_2/ctrler/fction/params
didn’t work with a controller in a folder that would be in a subfolder of controllers/

anyway, your hack is great, we just have to wait your code review !!!

Ok
I give you my last notes and sugestions about auth, FAL_front and Freakauth_light files

I will work on admin files next days….

Resume of my previous notes and fix….

File: FAL_front.php : post
delete line 234 & 235

====================================================================================

File: FAL_front.php , FAL_validation.php & Freakauth_light.php : post
Change the fal_validation->login_check() to speed Up the code by acceded the database only one time.

Change the condition in FAL_front->login() function line 150 to:

And replace the fal_validation->login_check() by:

====================================================================================

File: register.php the form template view : post

the Cancel button have no effect when ‘FAL_terms_of_service_message’ == ’’ (see my propositions)
and at line 98 replace base_url() by site_url()

====================================================================================

File: FAL_front.php : post

support for auth controller into subfolder in activation() function (only one level, CI doesn’t support more)
replace line 325 by this two lines:

====================================================================================

NEW notes and fix….

File: FAL_front.php :

Similar than the previous for activation function.
support for auth controller into subfolder in forgotten_password_reset() function (only one level, CI doesn’t support more)
replace line 430 by this two lines:

=========================================================================

File: Freakauth_light.php :(the librarie)

In many place you make wrong call to the CI->load->view() function. In fact it is in all the sending email functions…...

You do:

You have to do:

you have to delete: $this->CI->config->item(‘view’). and .EXT because CI->load->view() doesn’t need it to work.
In fact your code work because $this->CI->config->item(‘view’) doesn’t exist so it is always empty but if an user define this config for other purpose this will crash this call.

so delete this two peace of text :$this->CI->config->item(‘view’). and .EXT
at lines 1087,1111,1139 and 1162.

=================================================================

Files: activation_email.php, change_password_email.php, forgotten_password_email.php and forgotten_password_reset_email.php the email template files…

It is a little thing but as I have said for register.php (see before)
It will be better to use site_url() instead of base_url() for the site link into the emails…..

=================================================================

File: register.php the template form view.
In this view you use a global div container,<div id=“register”> </div>, around the form, but in all other view you don’t use it.

The css is also confuse with this particularity. It should be clearer if all the form view use the same scheme.
So we could delete it and modify the css for the same result into register form.

==================================================================

That’s all

@danfreak

I have look at your new tut for re routed auth method, and it seems that you have a problem with the routes.
you write:

it should be

the route with reg expression that I give work only if the activation code and reset code are only made lowercase letter…..

@thierryREY

Thanks for everything.

I corrected the tutorial and retested the code

I did a further mistake with regard to the 1st method described

here is the code for re-routing (I also updated it in the tutorial)

in application/config/freakauth_light.php

in application/config/routes.php

This works!

I have two questions:

1) how you load your views
2) how you do autoload / get config

Question 1:

Both ways look to do the same thing, just curious if you found an advantage to doing this in two lines instead of the more compact one line approach?

Question 2:
In the download, you autoload the FAL library and it’s config file.  Only after reviewing the code did I discover that you’re actually autoloading from within the FAL libray the following:

It is nice that this setup simplifies the “setup” for using FAL, however, I didn’t realize this (maybe just add comments to the instructions) that all these helpers and library were being “autoloaded” by FAL.  I was already autloading these, so I think this means I was loading these things twice?!
  Edit: it looks like CI may check if a library is loaded already before loading again, but I don’t see a check for ancillary items (plugins, helpers, etc.) which would be loaded again.

Side comment:
When you autoload the configuration, CI loads the config into the CI object, making that config available globally.
When you load the library, it also loads the config, but the config is only available when an instance of the class is created.
I was trying to compare the performance of removing the autoload of the config versus the library loading the config by default, but I didn’t know how to get past the last step shown here:

Maybe you guys already know the performance difference is negligible?  I was just thinking loading something twice would be inefficient.

Actually, when you autoload, CI creates an array called “is_loaded” (Line 105 in system/libraries/config.php).  Seems like CI should check if a config file “is_loaded” before loading it again with the library?

system/libraries/loader.php

Hi guys,

I’m a newbie on CI & FreakAuth and certainly very impressed by both of them and now planning to spice up some old apps by porting them to CI/FAL.

I installed FreakAuth and it seems to work fine, except for a small bug with the session expiration: whenever I log in as the superadmin I can perform two pageloads after which the session expires for some reason.. I’ve tried to play around with the config.php and change the expiration time, but so far no success. Any thoughts on this?

EDIT: Ok, figured it out, the autoload loaded “session” -library, which caused some unwanted effects.. :D

@danfreak

I am reading the tutorials on custom user profiles. One solution t the question of multiple profiles is adding a flag to the profile data showing whether it is extended or not. For many sites it could be sufficient to have two types of users, the regular users and the special user with the additional fields (such as Musician). To generalize this, I think it is preferable to make this field an integer to support an unlimited number of types, since some sites offer multiple user types (Director, Musician, Comedian, etc.). A mechanism is required to specify what fields belong to each type of user. Perhaps an array naming the fields.

While you are thinking about it, consider storing extended fields in their own table for each type, but that may be a bit much for now.

Anyway, will keep exploring FA.

Steve

Just wanted to let you know I encountered a problem testing FAL 1.0.4 not sure if anyone noticed this, but my existing user from the previous version was in the database. I could bring this up fine in the admin and display the form to edit the profile. On the right, the new custom profile fields defined in the FAL configuration appeared, naturally with nothing in them. I checked the database and the fields were created properly along with the profile table.

In the details (view user) to the right it says “no data in DB: please add them” but when I go to the edit profile form, enter my name and surname, it refused to populate the fields. It says “0 user edited successfully!”

I looked in the database and found there were no rows in the fa_user_profile table. Could the reason be that I did not go through the installer step, being an upgrade? Or is there some reason why it will not create the row if one is not already there (the automagic?) I suppose it could be the user entry was created under the previous system and therefore no user profile row exists.

I hope I followed the upgrade instructions correctly. They were a little confusing.

Okay, I added a row for user 1 using phpMyAdmin (the superadmin was the only one in the database left from the previous time) and it works fine. I assume this would happen for pending users as well. It could really be a problem if you had a dozens of users and had to create these rows manually for each. Perhaps this could addressed in the next release for those upgrading.

@mule
I had a similar problem at my FAL beginnings, please see this post and tell me if the ‘dirty’ hack works for you. I installed FAL on an existing CI app, and I had this problem. This problem vanished after a fresh reinstall (new CI install, new FAL install).

-sek - 24 May 2007 02:47 PM

@danfreak
I am reading the tutorials on custom user profiles. One solution t the question of multiple profiles is adding a flag to the profile data showing whether it is extended or not.

well I think that every profile type should be an ‘extended’ one. I don’t see the point of having ‘regular profiles’, webmasters could just have an ‘extended profile’ called ‘regular’

For many sites it could be sufficient to have two types of users, the regular users and the special user with the additional fields (such as Musician). To generalize this, I think it is preferable to make this field an integer to support an unlimited number of types, since some sites offer multiple user types (Director, Musician, Comedian, etc.). A mechanism is required to specify what fields belong to each type of user. Perhaps an array naming the fields.

While you are thinking about it, consider storing extended fields in their own table for each type, but that may be a bit much for now.

Good idea, I love generalisation. But I love factorisation too: what if we want similar fields in different profiles? I don’t want to have duplicate code here.
So I would just add (maybe you thought about this too) something like (in the config file)

now the tricky part: the db tables

in the actual FAL, each profile entry corresponds to a user and has a row of fields (id, field1, field2…)

What I suggest:
we could have several rows for a single user, one for each field that he has:
a row links to a field table and the row we need in it (id, user_id, field_name, id_in_this_field_table)
NOTE: id is not corresponding to the user_id as it did before

EDITED:
then we have a table for each field, containing the data: (id, data)

WAS:
then the last table contains the data: (id, data)

I’m not a db guru, tell me what you think