From 6f22e2821126fa6eaf1c7b6845abf8cc6b4ddc93 Mon Sep 17 00:00:00 2001
From: Web Access <rohan.katkar@webaccessglobal.com>
Date: Wed, 10 Jun 2015 17:04:49 +0530
Subject: [PATCH] Warning fixes

---
 CRM/Upgrade/Incremental/php/FourFive.php | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/CRM/Upgrade/Incremental/php/FourFive.php b/CRM/Upgrade/Incremental/php/FourFive.php
index ed8c99ead4..9d8892cb48 100755
--- a/CRM/Upgrade/Incremental/php/FourFive.php
+++ b/CRM/Upgrade/Incremental/php/FourFive.php
@@ -332,7 +332,7 @@ DROP KEY `{$dao->CONSTRAINT_NAME}`";
    *
    * @return bool TRUE for success
    */
-  static function updateSavedSearch(CRM_Queue_TaskContext $ctx) {
+  public static function updateSavedSearch(CRM_Queue_TaskContext $ctx) {
     $sql = "SELECT id, form_values FROM civicrm_saved_search";
     $dao = CRM_Core_DAO::executeQuery($sql);
     while ($dao->fetch()) {
@@ -342,22 +342,22 @@ DROP KEY `{$dao->CONSTRAINT_NAME}`";
         if (preg_match('/^custom_/', $field) && is_array($data_value) && !array_key_exists("${field}_operator", $formValues)) {
           // Now check for CiviCRM_OP_OR as either key or value in the data_value array.
           // This is the conclusive evidence of an old-style data format.
-          if(array_key_exists('CiviCRM_OP_OR', $data_value) || FALSE !== array_search('CiviCRM_OP_OR', $data_value)) {
+          if (array_key_exists('CiviCRM_OP_OR', $data_value) || FALSE !== array_search('CiviCRM_OP_OR', $data_value)) {
             // We have old style data. Mark this record to be updated.
             $update = TRUE;
             $op = 'and';
-            if(!preg_match('/^custom_([0-9]+)/', $field, $matches)) {
+            if (!preg_match('/^custom_([0-9]+)/', $field, $matches)) {
               // fatal error?
               continue;
             }
-            $fieldID= $matches[1];
+            $fieldID = $matches[1];
             if (array_key_exists('CiviCRM_OP_OR', $data_value)) {
               // This indicates data structure identified by jamie in the form:
               // value1 => 1, value2 => , value3 => 1.
               $data_value = array_keys($data_value, 1);
 
               // If CiviCRM_OP_OR - change OP from default to OR
-              if($data_value['CiviCRM_OP_OR'] == 1) {
+              if ($data_value['CiviCRM_OP_OR'] == 1) {
                 $op = 'or';
               }
               unset($data_value['CiviCRM_OP_OR']);
-- 
2.25.1