Form Generation Library

[eluser]2think[/eluser]
[quote author="connors" date="1261943675"]First of all, I just wanna say thanks for this plugin - really eases the form job.

I'm using 0.22 prerelease and I've ran into a few issues - Don't know if it's just me or it really is bugs.
Anyway.. First one is about the model function.

I'm not sure why, but the form isn't calling my model when I'm using form->valid after declaring it:

Code:

CI::library('form')->open('register/user')
/* Loads of fields here */
                       ->submit()
                       ->reset()
                       ->validate()
                       ->model('user','check_register');

if(CI::library('form')->valid)
{
   // Success stuff here
}

No matter where i put the ->model function, it's not called and is instead going to the if ->valid section. Bug or error in my code?

When i comment out the if ->valid part, the model is called - well, tries to - and I'm getting an error that it can't find the 'user' model.
I belive that CI models needs to by suffixed with _model, so I'm guessing this is a bug in the form lib?
Change line 2389 (the model function) from

Code:

$this->model = $model;

to

Code:

$this->model = $model.'_model';

I'm setting an error in my model for debug purposes, and when my model was finaly called it replaced my class with " error" istead of appending it.
It's fixable by modifying line 2110 from

Code:

if ($this->$el->type != 'checkbox' && $this->$el->type != 'radio') $this->$el->atts['class'] = (isset($this->$el->atts['class'])) ? ' '.$this->config['error_class'] : $this->config['error_class'];

to

Code:

if ($this->$el->type != 'checkbox' && $this->$el->type != 'radio') $this->$el->atts['class'] = (isset($this->$el->atts['class'])) ? $this->$el->atts['class'].' '.$this->config['error_class'] : $this->config['error_class'];

Once again, I dont know if you've already fixed them, just wanted to point them out [/quote]

Connors,
Have you tried the syntax used in the Getting Started section of the user guide? http://frankmichel.de/formgenlib/user_gu...arted.html I understand the CI::library syntax but haven't seen it used before in Codeigniter source code.

You also do NOT need to append _model to Codeigniter classes, this you may have seen only because it is a preference of a particular programmer.