feature request or how to implement: Created_by automatically filled by session variable

[Gianna Giavelli]

I'd like to have some audit in my tables of who made the record change. 

It would be nice if I could have a "created_by" like "created_at" that auto-filled upon save, using a userid value from the session

Is there an easy way to do some kind of IOC intercept and make that happen globally? If not, please consider it a feature request. 

thanks

Gia

[Igor Polevoy]

Gianna,. this will not work, as I do not want to make ActiveJDBC to be dependent on ActiveWeb.

What you can do, is create a parent controller and extend it by all other controllers. 

In the parent controller, you can create a method like this: 

public class MySuperController<T extends Model> extends AppController{

  protected T createModel(Class<T> clazz){

         User user = (User)session("user");

         Model m = clazz.newInstance();

         m.set("user_id", user.getId());

  }

}

Then, in your child controller you can:

public class BookController extends MySuperController{

  @POST

  public void save(){

     Person p = createModel(Person.class);

     /// more code 

  }

}

will this work?

[Gianna Giavelli]

I think that will work. But for the models which do not contain a created_by column I don't want to fail, how do I check to see if the model contains the field? I won't have it in EVERY table. Yes, I could only extend the supercontroller in the cases where I DO have the column in the model but then I that defeats the "don't worry about it it just works" nice-ness of the feature

thanks

Gia

[Igor Polevoy]

this is easy, just add a condition: 

protected T createModel(Class<T> clazz){

         User user = (User)session("user");

         Model m = clazz.newInstance();

         if(m.attributeNames().contains("created_by"))

             m.set("user_id", user.getId());

  }

Thanks

[Igor Polevoy]

Sorry, I meant: 

protected T createModel(Class<T> clazz){

         User user = (User)session("user");

         Model m = clazz.newInstance();

         if(m.attributeNames().contains("user_id"))

[Gianna Giavelli]

awesome! I'll integrate it right away!