From 7ff8b98a93adab68dae88bc29d87100cfcb1b86e Mon Sep 17 00:00:00 2001
From: Divya Jose <divya@divya.(none)>
Date: Wed, 4 Jun 2014 20:35:35 +0530
Subject: [PATCH] --HR-303 modification to fix notices while disabling casetype
 extension

---
 CRM/Case/XMLProcessor/Process.php | 32 +++++++++++++++----------------
 CRM/Core/BAO/UFField.php          |  2 +-
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/CRM/Case/XMLProcessor/Process.php b/CRM/Case/XMLProcessor/Process.php
index 1f57224fc4..744e746c74 100644
--- a/CRM/Case/XMLProcessor/Process.php
+++ b/CRM/Case/XMLProcessor/Process.php
@@ -314,23 +314,23 @@ class CRM_Case_XMLProcessor_Process extends CRM_Case_XMLProcessor {
    */
   function getDeclaredActivityTypes($caseTypeXML) {
     $result = array();
-
-    if ($caseTypeXML->ActivityTypes && $caseTypeXML->ActivityTypes->ActivityType) {
-      foreach ($caseTypeXML->ActivityTypes->ActivityType as $activityTypeXML) {
-        $result[] = (string) $activityTypeXML->name;
+    if (!empty($caseTypeXML)) {
+      if ($caseTypeXML->ActivityTypes && $caseTypeXML->ActivityTypes->ActivityType) {
+        foreach ($caseTypeXML->ActivityTypes->ActivityType as $activityTypeXML) {
+          $result[] = (string) $activityTypeXML->name;
+        }
       }
-    }
 
-    if ($caseTypeXML->ActivitySets && $caseTypeXML->ActivitySets->ActivitySet) {
-      foreach ($caseTypeXML->ActivitySets->ActivitySet as $activitySetXML) {
-        if ($activitySetXML->ActivityTypes && $activitySetXML->ActivityTypes->ActivityType) {
-          foreach ($activitySetXML->ActivityTypes->ActivityType as $activityTypeXML) {
-            $result[] = (string) $activityTypeXML->name;
+      if ($caseTypeXML->ActivitySets && $caseTypeXML->ActivitySets->ActivitySet) {
+        foreach ($caseTypeXML->ActivitySets->ActivitySet as $activitySetXML) {
+          if ($activitySetXML->ActivityTypes && $activitySetXML->ActivityTypes->ActivityType) {
+            foreach ($activitySetXML->ActivityTypes->ActivityType as $activityTypeXML) {
+              $result[] = (string) $activityTypeXML->name;
+            }
           }
         }
       }
     }
-
     $result = array_unique($result);
     sort($result);
     return $result;
@@ -342,13 +342,13 @@ class CRM_Case_XMLProcessor_Process extends CRM_Case_XMLProcessor {
    */
   function getDeclaredRelationshipTypes($caseTypeXML) {
     $result = array();
-
-    if ($caseTypeXML->CaseRoles && $caseTypeXML->CaseRoles->RelationshipType) {
-      foreach ($caseTypeXML->CaseRoles->RelationshipType as $relTypeXML) {
-        $result[] = (string) $relTypeXML->name;
+    if (!empty($caseTypeXML)) {
+      if ($caseTypeXML->CaseRoles && $caseTypeXML->CaseRoles->RelationshipType) {
+        foreach ($caseTypeXML->CaseRoles->RelationshipType as $relTypeXML) {
+          $result[] = (string) $relTypeXML->name;
+        }
       }
     }
-
     $result = array_unique($result);
     sort($result);
     return $result;
diff --git a/CRM/Core/BAO/UFField.php b/CRM/Core/BAO/UFField.php
index 32045ba5f3..22bcf67496 100644
--- a/CRM/Core/BAO/UFField.php
+++ b/CRM/Core/BAO/UFField.php
@@ -348,7 +348,7 @@ WHERE cf.id IN (" . $customFieldIds . ") AND is_multiple = 1 LIMIT 0,1";
    * @static
    * @access public
    */
-  function setUFFieldStatus($customGroupId, $is_active) {
+  static function setUFFieldStatus($customGroupId, $is_active) {
     //find the profile id given custom group id
     $queryString = "SELECT civicrm_custom_field.id as custom_field_id
                         FROM   civicrm_custom_field, civicrm_custom_group
-- 
2.25.1