EGroupware /
egroupware
| 1 | <?php |
||
| 2 | /** |
||
| 3 | * EGroupware - Mail hooks |
||
| 4 | * |
||
| 5 | * @link http://www.egroupware.org |
||
| 6 | * @package api |
||
| 7 | * @subpackage amil |
||
| 8 | * @author Klaus Leithoff <leithoff-AT-stylite.de> |
||
| 9 | * @author Ralf Becker <[email protected]> |
||
| 10 | * @copyright (c) 2008-16 by leithoff-At-stylite.de |
||
| 11 | * @license http://opensource.org/licenses/gpl-license.php GPL - GNU General Public License |
||
| 12 | * @version $Id$ |
||
| 13 | */ |
||
| 14 | |||
| 15 | namespace EGroupware\Api\Mail; |
||
| 16 | |||
| 17 | /** |
||
| 18 | * diverse static Mail hooks |
||
| 19 | */ |
||
| 20 | class Hooks |
||
| 21 | { |
||
| 22 | /** |
||
| 23 | * Password changed hook --> unset cached objects, as password might be used for email connection |
||
| 24 | * |
||
| 25 | * @param array $hook_data |
||
| 26 | */ |
||
| 27 | public static function changepassword($hook_data) |
||
| 28 | { |
||
| 29 | if (!empty($hook_data['old_passwd'])) |
||
| 30 | { |
||
| 31 | Credentials::changepassword($hook_data); |
||
| 32 | } |
||
| 33 | } |
||
| 34 | |||
| 35 | /** |
||
| 36 | * Hook called before an account get deleted |
||
| 37 | * |
||
| 38 | * @param array $data |
||
| 39 | * @param int $data['account_id'] numerical id |
||
| 40 | * @param string $data['account_lid'] account-name |
||
| 41 | * @param int $data['new_owner'] account-id of new owner, or false if data should get deleted |
||
| 42 | */ |
||
| 43 | static function deleteaccount(array $data) |
||
| 44 | { |
||
| 45 | self::run_plugin_hooks('deleteAccount', $data); |
||
| 46 | |||
| 47 | // as mail accounts contain credentials, we do NOT assign them to user users |
||
| 48 | Account::delete(0, $data['account_id']); |
||
| 49 | } |
||
| 50 | |||
| 51 | /** |
||
| 52 | * Hook called before a group get deleted |
||
| 53 | * |
||
| 54 | * @param array $data |
||
| 55 | * @param int $data['account_id'] numerical id |
||
| 56 | * @param string $data['account_name'] account-name |
||
| 57 | */ |
||
| 58 | static function deletegroup(array $data) |
||
| 59 | { |
||
| 60 | Account::delete(0, $data['account_id']); |
||
| 61 | } |
||
| 62 | |||
| 63 | /** |
||
| 64 | * Hook called when an account get added or edited |
||
| 65 | * |
||
| 66 | * @param array $data |
||
| 67 | * @param int $data['account_id'] numerical id |
||
| 68 | * @param string $data['account_lid'] account-name |
||
| 69 | * @param string $data['account_email'] email |
||
| 70 | */ |
||
| 71 | static function addaccount(array $data) |
||
| 72 | { |
||
| 73 | $method = $data['location'] == 'addaccount' ? 'addAccount' : 'updateAccount'; |
||
| 74 | self::run_plugin_hooks($method, $data); |
||
| 75 | } |
||
| 76 | |||
| 77 | /** |
||
| 78 | * Run hook on plugins of all mail-accounts of given account_id |
||
| 79 | * |
||
| 80 | * @param string $method plugin method to run |
||
| 81 | * @param array $data hook-data incl. value for key account_id |
||
| 82 | */ |
||
| 83 | protected static function run_plugin_hooks($method, array $data) |
||
| 84 | { |
||
| 85 | foreach(Account::search((int)$data['account_id'], 'params') as $params) |
||
| 86 | { |
||
| 87 | if (!Account::is_multiple($params)) continue; // no need to waste time on personal accounts |
||
| 88 | |||
| 89 | try { |
||
| 90 | $account = new Account($params); |
||
| 91 | if ($account->acc_smtp_type != __NAMESPACE__.'\\Smtp' && ($smtp = $account->smtpServer(true)) && |
||
|
0 ignored issues
–
show
|
|||
| 92 | is_a($smtp, __NAMESPACE__.'\\Smtp') && get_class($smtp) != __NAMESPACE__.'\\Smtp') |
||
| 93 | { |
||
| 94 | $smtp->$method($data); |
||
| 95 | } |
||
| 96 | if ($account->acc_imap_type != __NAMESPACE__.'\\Imap' && $account->acc_imap_admin_username && |
||
| 97 | $account->acc_imap_admin_password && ($imap = $account->imapServer(true)) && |
||
| 98 | is_a($imap, __NAMESPACE__.'\\Imap') && get_class($imap) != __NAMESPACE__.'\\Imap') |
||
| 99 | { |
||
| 100 | $imap->$method($data); |
||
| 101 | } |
||
| 102 | } |
||
| 103 | catch(\Exception $e) { |
||
| 104 | _egw_log_exception($e); |
||
| 105 | // ignore exception, without stalling other hooks |
||
| 106 | } |
||
| 107 | } |
||
| 108 | } |
||
| 109 | } |
||
| 110 |
This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.
If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.