Changeset 3883 in subversion

Timestamp:

Aug 9, 2010 2:35:10 AM (3 years ago)

Author:

alec

Message:

• Rename hooks: contact_save -> contact_update, identity_save -> identity_update, delete_identity -> identity_delete
• Print warning to the log about deprecated hook names usage

Location:

trunk/roundcubemail/program

Files:

• include/rcube_plugin_api.php (modified) (3 diffs)
• steps/addressbook/save.inc (modified) (1 diff)
• steps/settings/delete_identity.inc (modified) (1 diff)
• steps/settings/save_identity.inc (modified) (1 diff)

trunk/roundcubemail/program/include/rcube_plugin_api.php

@@ -59 +59 @@
     'get_address_book'  => 'addressbook_get',
     'create_contact'    => 'contact_create',
-    'save_contact'      => 'contact_save',
+    'save_contact'      => 'contact_update',
+    'contact_save'      => 'contact_update',
     'delete_contact'    => 'contact_delete',
     'manage_folders'    => 'folders_list',
@@ -68 +69 @@
     'list_identities'   => 'identities_list',
     'create_identity'   => 'identity_create',
-    'save_identity'     => 'identity_save',
+    'delete_identity'   => 'identity_delete',
+    'save_identity'     => 'identity_update',
+    'identity_save'     => 'identity_update',
   );
 
@@ -194 +197 @@
     if (is_callable($callback)) {
       if (isset($this->deprecated_hooks[$hook])) {
-        /* Uncoment after 0.4-stable release
         raise_error(array('code' => 522, 'type' => 'php',
           'file' => __FILE__, 'line' => __LINE__,
           'message' => "Deprecated hook name. ".$hook.' -> '.$this->deprecated_hooks[$hook]), true, false);
-        */
         $hook = $this->deprecated_hooks[$hook];
       }

trunk/roundcubemail/program/steps/addressbook/save.inc

@@ -55 +55 @@
 if (!empty($cid))
 {
-  $plugin = $RCMAIL->plugins->exec_hook('contact_save', array('id' => $cid, 'record' => $a_record, 'source' => get_input_value('_source', RCUBE_INPUT_GPC)));
+  $plugin = $RCMAIL->plugins->exec_hook('contact_update',
+    array('id' => $cid, 'record' => $a_record, 'source' => get_input_value('_source', RCUBE_INPUT_GPC)));
   $a_record = $plugin['record'];
   

trunk/roundcubemail/program/steps/settings/delete_identity.inc

@@ -31 +31 @@
 if ($iid && preg_match('/^[0-9]+(,[0-9]+)*$/', $iid))
 {
-  $plugin = $RCMAIL->plugins->exec_hook('delete_identity', array('id' => $iid));
+  $plugin = $RCMAIL->plugins->exec_hook('identity_delete', array('id' => $iid));
   
   if (!$plugin['abort'] && $USER->delete_identity($iid)) {

trunk/roundcubemail/program/steps/settings/save_identity.inc

@@ -62 +62 @@
 {
   $iid = get_input_value('_iid', RCUBE_INPUT_POST);
-  $plugin = $RCMAIL->plugins->exec_hook('identity_save', array('id' => $iid, 'record' => $save_data));
+  $plugin = $RCMAIL->plugins->exec_hook('identity_update', array('id' => $iid, 'record' => $save_data));
   $save_data = $plugin['record'];