hey Dan,

Found a small bug.

in the library (row 250) should be

because if I create an custom roll, I’m also a valide user.

Thanks wolfear for this good suggestions, just two things to say.

The first :

wolfear - 22 February 2007 06:35 PM

————————————————————-
In template_admin -> add.php

Incorrect nesting of elements
Change all:

<p><label

to

<div><label

Change Lines 19, 40, 49,  58, 67, 77, 92
sample:

<span><?=(isset($this->validation) ? $this->validation->{$field.'_error'} : '')?></span></p>

should be:

<?=(isset($this->validation) ? $this->validation->{$field.'_error'} : '')?></div>

this nesting is correct for me

I just found a bad id=”“, we could replace

with

The last thing :

——————————————————————————-
In template_admin -> list.php
Duplicate “submit” -> I just removed the top form.

I think that this second “add user” button is convenient,
we could just rename the second id=“submit2”

Also, in this file, I found a small typo :

grahack-
There is not really a “functional” problem, but it is a problem if we want valid code.
I changed those because this:

Produces a div, inside of a span, inside of a paragraph, when combined with the label.
You end up with paragraph block-> inline span-> division block.

One cannot “legally” have a block element contained inside of an inline element if you want to be valid (CSS level 2 specs).

The proper nesting would be division block-> paragraph block-> inline span.

By replacing the the paragraph element with a div, we keep the block level hierarchy correct since it is permissible to nest divs.

————————
True, the duplicate submits are convenient, especially if you have a large number of entries.
The current code just produces a duplicate ID error.  Your idea to rename would work fine also. I just took the easy way out.
I should have stated that there were other options. My bad.

jaapio - 23 February 2007 06:58 PM

hey Dan,

Found a small bug.

if ($_username != false AND $_role != false AND $_role=='user')

in the library (row 250) should be

if ($_username != false AND $_role != false)

because if I create an custom roll, I’m also a valide user.

Hey Jaapio this is not a bug.
User is one of the 3 default roles in FreakAuth_light along with admin and superadmin.
The method you refer to isValidUser() returns TRUE if the logged-in person has a role of user

There are other 3 methods to check for different roles:
- isAdmin()
- isSuperAdmin()
- belongsToGroup()

All this methods are described in detail in the DOCUMENTATION

The user_temp table in DB has the purpose of storing registered but not yet activated people that subscribed to the website. Once they activate their account they get moved from the user_temp table to the user table.

To Jaapio and all the gang of FreakAuth_light users:
if you are not sure about some code, please ask WHY some code has been written before reporting it as a bug.
One thing is to report possible customisations, another is to report bugs.

Danfreak: Bloody fantastic system you have made here. I love it.
To the rest of the community: Great work guys. It is good to see so much support for this application, and the CI framework in general.

Thanks Steve. This fix really helped me.

stevepaperjam - 19 February 2007 09:10 AM

Thanks to Dan for getting this updated version out so quickly.

I was going to post this in a separate thread, but it shouldn’t take long: any PHP4 users had a go with a new version yet?

I had a problem when opening the view user (or admins) page, which the log file tracked down to line 1139 in libraries/Freakauth_light.php

$db_fields =$this->CI->userprofile->getTableFields();

In discussion with Dan last night, I thought it might be my set-up (PHP4.4.0, Apache 2.something).

I’ve just had a bit more of a fiddle with it, and moving the loading of the model (just above that line) to the constructor seems to fix the issue.

This change also still works in PHP5.0.4 as well. (I’m using MAMP to flip between PHP versions, by the way…)

I had two postgres related issues with using FreakAuth and have come up with a solution for both. Hope these help any postgres users out there. These fixes are probably only relevant for people using v7.x of postgres.

Bug 1
Not sure if this is caused by postgres or by my verison of php 4, but when trying to add a new user it would constantly try to add the user with an id = 0 seeing as that is what the ‘add user’ form uses to denote a new user.
My fix was to remove the id=0 from the array just before it gets passed to the insert function. That way the SERIAL id field will automatically get a new id for this user record.

Bug 2
I discovered that for a postgres database 7.x the insert_id() function doesn’t work as it is being used in the adding of profiles when a user is being created.  The only way for 7.x to tell what the previous id of an insert is, is to check the sequence responsible for the SERIAL id value. The CI Postgres driver already allows for this, but a slight modification needs to be made to freakauth to take advantage of this 7.x way of doing things. To do this, I had to change the code in the application/controllers/admin/auth.php file as follows:

Also, if anyone wants it, I wrote a converted PostgreSQL script for installing the base tables for FreakAuth.

Please let me know if there is a way of fixing the above bugs better than I have done.

Cheers,

Gerard

there’s something I’m trying to do with the menus that i didn’t get it yet in CI
how can i make links only available for a group use?

i know that i need to do a if condition to see if the user is logged in and there’s info at the session
if he is logged in show links if not show other links…i’m doing this and its not working :(

Let’s say you want to display a menu item only to members of users (belonging to group users) if they are logged in.

EXAMPLE (views/FreakAuth_light/menu.php)

To use this example you need:
- a myprofile.php controller
- to set this in your config/routes.php

Check the method belongsToGroup($group=null, $only=null) if you want to display something to members of your custom groups beside users.

that’s what i’m doing and its not showing anything, gonna recode it again

and for the login, register, etc links?if a user logs in we want that those links go away how can i do that?

i was thinking to use the sessions, where session->userdata(logged_in) if this was true it would hide those links,
but that way u gave its easier thanx

Edit: isn’t guests a group to?so that would hide links from guests too,or I’m wrong?

wemago - 26 February 2007 08:31 AM

isn’t guests a group to?so that would hide links from guests too,or I’m wrong?

In a certain way it is a group.

All people not logged in belong to the group guest.

Because this group works in a kinda “negative” fashion, it doesn’t need to be defined in your application/config/freakauth_light.php

I’m looking for an auth library that can do the following things:

- has an API, nothing else! So no styles, no views, nothing. Works only through a library. Most libraries here are written as if the end user can’t code a single line…
- has ACL, or at least some roles
- is independent of the session library (native, db, ...)
- can be scaled if necessary, so it should be possible to disable ACL etc for simpler auth.

- Freakauth seems to get close, but is there a way to strip it ??

Anybody?
Thanks !

I have stripped FAL for use in my own application successfully - and it works quite well.

Iksander - 26 February 2007 05:33 PM

I have stripped FAL for use in my own application successfully - and it works quite well.

You mind sharing it with the rest of us?

Sure - if you use a similar view embedding pattern that Rick gave an example of - you should be able to simple change the ‘container’ that the auth.php file calls for _login, register, etc… to your own sites container view and load the individual content page which is completely seperate from the body of FAL’s container views - and then simply give the content view its own pseudo classes or id’s.

The only thing is - it takes some time to sift through the code and insert your own stuff…

If you want my code examples, I can give them -  I just figured giving a basic description would be quicker for you than not, is it isn’t the case - tell me so and I will upload my own examples…