Forums - Thread Details
  • State Not Answered
  • Replies 17 replies
  • Subscribers 319 subscribers
  • Views 5620 views
  • Users 0 members are here
Forum Actions
Related

How to update a field in an after_save logic hook?

Jaume Albaigès
Hello all,

I have created a custom field to hold an autoincrement value for each record in the module. This works ok.

I would like to use this number as a part of the name of the record. So, I have created a logic_hook based on the after_save event.

Unfortunately, it seems that $bean data is not available when after_save-ing.

It I test the logic_hook in before_save event, the code works ok and Name field is properly updated. However I can't use the code in before_save because it only would work when editing records not when creating them, because the logic hook would be fired before setting the autoincrement value.

Any ideas?

Here is my code, so simple. I'm on SugarCRM CE 6.5.11.
class my_class{
  public function my_function(&$bean, $event, $arguments){
    $bean->name = $bean->my_autoincrement_field_c;
  }
}
Parents
  • 0
    In a case like this I'd go straight to the database and do an update query. Either that or in your after_save call do a $bean->save(). There is some built-in protection against nested logic hooks, but it would be wise to also set a static var to ensure that your hook only runs once if you call ->save() again.
  • 0 in reply to Jason Eggers
    Jaume,

    To add to eggsurplus's notes, in your after_save hook, you just need to :

    $bean->db->query(" UPDATE {$bean->table_name} SET name='{$bean->my_autoincrement_field_c}' WHERE id='{$bean->id}' ");

    This will update the name field in the bean's table with the value from the 'my_autoincrement_field_c'.

    But be careful whether you need the name to be set on EVERY save of the record as doing this without any checks/conditions will overwrite the "name" field if any other logic or users have updated it to something else.

    ie you might want to only update the name once and not everytime and/or based on certain checks/conditions.

    eg set it to the my_autoincrement_field_c value only if the name is empty

    class my_class{
      public function my_function(&$bean, $event, $arguments){        if(empty($bean->name)){
           $bean->db->query(" UPDATE {$bean->table_name} SET name='{$bean->my_autoincrement_field_c}' WHERE id='{$bean->id}' ");      }
      }
    }
    Cheers
    Kedar.
  • 0 in reply to Kedar Gawande
    Why do an after save? That will change the name and save again which will call the after save again...
    Why not do a before_save instead? You should be able to just set the $bean->name as you have above:
    $bean->name = $bean->my_autoincrement_field_c;

    My general approach is that if you are changing fields of the bean you are saving then use a before_save, if you are changing another bean then you can use an after_save.

    HTH
    FrancescaS
  • 0 in reply to Francesca Shiekh
    FrancescaS, I agree that before_save is better but in this case, its an autoincrement field which doesnt get a value until the record is created in the DB, hence the need for an after_save

    Cheers
    Kedar
Reply Children
No Data