From 7da29bae48cfff82160e8ea2b480103c7a932389 Mon Sep 17 00:00:00 2001 From: eileen Date: Fri, 17 Jan 2020 14:27:32 +1300 Subject: [PATCH] [NFC] code cleanup Declare Exceptions, use strict, use single quotes --- CRM/Member/Form/Task.php | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/CRM/Member/Form/Task.php b/CRM/Member/Form/Task.php index 72c9e23374..04a02b8f24 100644 --- a/CRM/Member/Form/Task.php +++ b/CRM/Member/Form/Task.php @@ -13,8 +13,6 @@ * * @package CRM * @copyright CiviCRM LLC https://civicrm.org/licensing - * $Id$ - * */ /** @@ -36,6 +34,7 @@ class CRM_Member_Form_Task extends CRM_Core_Form_Task { * @param * * @return void + * @throws \CRM_Core_Exception */ public function preProcess() { self::preProcessCommon($this); @@ -43,6 +42,8 @@ class CRM_Member_Form_Task extends CRM_Core_Form_Task { /** * @param CRM_Core_Form $form + * + * @throws \CRM_Core_Exception */ public static function preProcessCommon(&$form) { $form->_memberIds = []; @@ -57,7 +58,7 @@ class CRM_Member_Form_Task extends CRM_Core_Form_Task { $form->assign('taskName', $tasks[$form->_task]); $ids = []; - if ($values['radio_ts'] == 'ts_sel') { + if ($values['radio_ts'] === 'ts_sel') { foreach ($values as $name => $value) { if (substr($name, 0, CRM_Core_Form::CB_PREFIX_LEN) == CRM_Core_Form::CB_PREFIX) { $ids[] = substr($name, CRM_Core_Form::CB_PREFIX_LEN); @@ -99,7 +100,7 @@ class CRM_Member_Form_Task extends CRM_Core_Form_Task { } $searchFormName = strtolower($form->get('searchFormName')); - if ($searchFormName == 'search') { + if ($searchFormName === 'search') { $session->replaceUserContext(CRM_Utils_System::url('civicrm/member/search', $urlParams)); } else { -- 2.25.1