Login

12-21-2009, 07:19 PM

[eluser]Eric Abruzzese[/eluser]
Hello,

I'm relatively new to CodeIgniter, so I'm still getting used to "the CodeIgniter way", so bear with me.

I have an application called Tiquette, and issue-tracking system.

I am trying to minimize code duplication as much as possible, so I created models for all of my database tables.

As an example, I have a model for the users who can log into the system. Naturally, they have fields like: username, password, full name, email address, phone number, notes, "is-logged-in", last login, permissions, etc.

Code:
class Tiquette_user extends Model {

    /*

     * Active Directory credentials

     */

    private $ad_imported_from_active_directory;

    private $ad_active_directory_username;

    private $ad_active_directory_domain;

    /*

     * The unique ID generated by MySQL

     */

    private $db_user_id;

    /*

     * Information about the user

     */

    private $tu_full_name;

    private $tu_email_address;

    private $tu_phone_number;

    private $tu_notes;

    /*

     * Login information (password is only set if user is not part of 

     * Active Directory)

     */

    private $tu_username;

    private $tu_password;

    /*

     * Session and login information.

     */

    private $tu_is_logged_in;

    private $tu_last_login;

    private $tu_permission_set;

        /* From here it's just getters and setters... */

}

If I'm not mistaken, I should be able to do the following, according to the documentation on the Active Record class:

Code:
$CI->db->insert('rts_users',$tiquette_user_object);

Obviously, I'm doing something wrong, because when I try that, I get this error:

"You must use the "set" method to update an entry."

It may be important to note that some of the fields in the model are never set (should I set them to null?)

Thank you for your assistance,

Eric Abruzzese

12-21-2009, 07:46 PM

[eluser]Cro_Crx[/eluser]
That error you're getting would occur if the $tiquette_user_object variable is null or not set. It's complaining that there's nothing to insert.

The error is sort of worded funny, I think it's worded like that because, you could do this instead

Code:
$CI->db->set($tiquette_user_object);

$CI->db->insert('rts_users');

And nothing is actually being set at the moment so it assumes you've missed out the set method.

Hope this helps.

12-21-2009, 09:09 PM

[eluser]Eric Abruzzese[/eluser]
Thanks for the quick reply. I have temporarily replaced the insert() with a print_r() to display the contents of my $tiquette_user_object.

It gives me:

Code:

Etc...

Obviously that doesn't match my database table. Is there a built-in way to grab only the first few fields? Am I doing something wrong?

Thanks for all the help!

Eric Abruzzese

12-21-2009, 09:15 PM

[eluser]Cro_Crx[/eluser]
Looks like $tiquette_user_object is being assigned to the current object ($this).

You're not doing $tiquette_user_object = $this; anywhere are you ?

12-21-2009, 09:19 PM

[eluser]Eric Abruzzese[/eluser]
I see what you mean. Here's my function I'm calling it from in the controller:

Code:

function add_user()

    {

        $this->load->model('tiquette_user');

        $this->tiquette_user->set_username('admin');

        $this->tiquette_user->set_password('my_password1');

        $this->tiquette_user->set_full_name('Eric C. Abruzzese');

        $this->tiquette_user->set_email_address('[email protected]');

        $this->tiquette_user->set_phone_number('(XXX) XXX-XXXX');

        $this->tiquette_user_manager->add_user($this->tiquette_user);

    }

EDIT: I should probably clarify that add_user simply takes it's parameter and passes it to insert() (or, as is the case right now, print_r()).

The last line in that function I don't think is correct. Ideas?

Thanks!

12-21-2009, 11:14 PM

[eluser]Eric Abruzzese[/eluser]
I'm dumb.

Being a Java programmer by day, I set all my variables in my model to private... hence, no visibility... hence they cannot be accessed by insert().

Thanks though!