foreach ($emailAdd as $key => $value) {
$email = trim($value);
$testParams['emails'][] = $email;
- $emails .= $emails ? ",'$email'" : "'$email'";
+ $emails .= ($emails ? ',' : '') . "'" . CRM_Core_DAO::escapeString($email) . "'";
if (!CRM_Utils_Rule::email($email)) {
- CRM_Core_Session::setStatus(ts('Please enter a valid email addresses.'), ts('Test not sent.'), 'error');
+ CRM_Core_Session::setStatus(ts('Please enter a valid email address.'), ts('Test not sent.'), 'error');
$error = TRUE;
}
}
}
/**
- * Is the current user permitted to add a user
- * @return bool
+ * @inheritDoc
*/
- function checkPermissionAddUser() {
- if (CRM_Core_Permission::check('administer users')) {
- return TRUE;
- }
+ public function checkPermissionAddUser() {
+ return CRM_Core_Permission::check('administer users');
}
-
/**
- * Log error to CMS
+ * @inheritDoc
*/
- function logger($message) {
+ public function logger($message) {
if (CRM_Core_Config::singleton()->userFrameworkLogging) {
- watchdog('civicrm', $message, NULL, WATCHDOG_DEBUG);
+ watchdog('civicrm', '%message', array('%message' => $message), NULL, WATCHDOG_DEBUG);
}
}