adamjakab / SuiteCRM

include/phpmailer/language/phpmailer.lang-ar.php

Indentation +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 $PHPMAILER_LANG['mailer_not_supported'] = ' برنامج الإرسال غير مدعوم.';
 $PHPMAILER_LANG['provide_address']      = 'يجب توفير عنوان البريد الإلكتروني لمستلم واحد على الأقل.';
 $PHPMAILER_LANG['recipients_failed']    = 'خطأ SMTP: الأخطاء التالية ' .
-                                          'فشل في الارسال لكل من : ';
+                                            'فشل في الارسال لكل من : ';
 $PHPMAILER_LANG['signing']              = 'خطأ في التوقيع: ';
 $PHPMAILER_LANG['smtp_connect_failed']  = 'SMTP Connect() غير ممكن.';
 $PHPMAILER_LANG['smtp_error']           = 'خطأ على مستوى الخادم SMTP: ';

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 $PHPMAILER_LANG['invalid_address']      = 'الإرسال غير ممكن لأن عنوان البريد الإلكتروني غير صالح.';
 $PHPMAILER_LANG['mailer_not_supported'] = ' برنامج الإرسال غير مدعوم.';
 $PHPMAILER_LANG['provide_address']      = 'يجب توفير عنوان البريد الإلكتروني لمستلم واحد على الأقل.';
-$PHPMAILER_LANG['recipients_failed']    = 'خطأ SMTP: الأخطاء التالية ' .
+$PHPMAILER_LANG['recipients_failed']    = 'خطأ SMTP: الأخطاء التالية '.
                                           'فشل في الارسال لكل من : ';
 $PHPMAILER_LANG['signing']              = 'خطأ في التوقيع: ';
 $PHPMAILER_LANG['smtp_connect_failed']  = 'SMTP Connect() غير ممكن.';

include/formbase.php

Indentation +122 added lines, -122 removed lines

@@ -54,14 +54,14 @@ 
  */
 function checkRequired($prefix, $required)
 {
-	foreach($required as $key)
-	{
-		if(!isset($_POST[$prefix.$key]) || number_empty($_POST[$prefix.$key]))
-		{
-			return false;
-		}
-	}
-	return true;
+    foreach($required as $key)
+    {
+        if(!isset($_POST[$prefix.$key]) || number_empty($_POST[$prefix.$key]))
+        {
+            return false;
+        }
+    }
+    return true;
 }
 
 /**
@@ -75,16 +75,16 @@ 
  */
 function populateFromPost($prefix, &$focus, $skipRetrieve = false, $checkACL = false)
 {
-	global $current_user;
+    global $current_user;
 
-	if(!empty($_REQUEST[$prefix.'record']) && !$skipRetrieve)
-		$focus->retrieve($_REQUEST[$prefix.'record']);
+    if(!empty($_REQUEST[$prefix.'record']) && !$skipRetrieve)
+        $focus->retrieve($_REQUEST[$prefix.'record']);
 
-	if(!empty($_POST['assigned_user_id']) && 
-	    ($focus->assigned_user_id != $_POST['assigned_user_id']) && 
-	    ($_POST['assigned_user_id'] != $current_user->id)) {
-		$GLOBALS['check_notify'] = true;
-	}
+    if(!empty($_POST['assigned_user_id']) && 
+        ($focus->assigned_user_id != $_POST['assigned_user_id']) && 
+        ($_POST['assigned_user_id'] != $current_user->id)) {
+        $GLOBALS['check_notify'] = true;
+    }
     if(isset($_POST['dup_checked']) && $_POST['dup_checked'] && isset($_POST['id']) && $_POST['id'] != '' ){
         $focus->new_with_id = true;
     }
@@ -106,15 +106,15 @@ 
         $relatedFields[$def['id_name']] = $field;
     }
 
-	foreach($focus->field_defs as $field=>$def) {
+    foreach($focus->field_defs as $field=>$def) {
         if ( $field == 'id' && !empty($focus->id) ) {
             // Don't try and overwrite the ID
             continue;
         }
 
 
-	    $type = !empty($def['custom_type']) ? $def['custom_type'] : $def['type'];
-		$sf = $sfh->getSugarField($type);
+        $type = !empty($def['custom_type']) ? $def['custom_type'] : $def['type'];
+        $sf = $sfh->getSugarField($type);
         if($sf != null){
             $sf->save($focus, $_POST, $field, $def, $prefix);
         } else {
@@ -152,15 +152,15 @@ 
 			$focus->$field = '';
 		}
 */
-	}
+    }
 
-	foreach($focus->additional_column_fields as $field) {
-		if(isset($_POST[$prefix.$field])) {
-			$value = $_POST[$prefix.$field];
-			$focus->$field = $value;
-		}
-	}
-	return $focus;
+    foreach($focus->additional_column_fields as $field) {
+        if(isset($_POST[$prefix.$field])) {
+            $value = $_POST[$prefix.$field];
+            $focus->$field = $value;
+        }
+    }
+    return $focus;
 }
 
 function add_hidden_elements($key, $value) {
@@ -182,73 +182,73 @@ 
 
 function getPostToForm($ignore='', $isRegularExpression=false)
 {
-	$fields = '';
-	if(!empty($ignore) && $isRegularExpression) {
-		foreach ($_POST as $key=>$value){
-			if(!preg_match($ignore, $key)) {
+    $fields = '';
+    if(!empty($ignore) && $isRegularExpression) {
+        foreach ($_POST as $key=>$value){
+            if(!preg_match($ignore, $key)) {
                                 $fields .= add_hidden_elements($key, $value);
-			}
-		}	
-	} else {
-		foreach ($_POST as $key=>$value){
-			if($key != $ignore) {
+            }
+        }	
+    } else {
+        foreach ($_POST as $key=>$value){
+            if($key != $ignore) {
                                 $fields .= add_hidden_elements($key, $value);
-			}
-		}
-	}
-	return $fields;
+            }
+        }
+    }
+    return $fields;
 }
 
 function getGetToForm($ignore='', $usePostAsAuthority = false)
 {
-	$fields = '';
-	foreach ($_GET as $key=>$value)
-	{
-		if($key != $ignore){
-			if(!$usePostAsAuthority || !isset($_POST[$key])){
-				$fields.= "<input type='hidden' name='$key' value='$value'>";
-			}
-		}
-	}
-	return $fields;
+    $fields = '';
+    foreach ($_GET as $key=>$value)
+    {
+        if($key != $ignore){
+            if(!$usePostAsAuthority || !isset($_POST[$key])){
+                $fields.= "<input type='hidden' name='$key' value='$value'>";
+            }
+        }
+    }
+    return $fields;
 
 }
 function getAnyToForm($ignore='', $usePostAsAuthority = false)
 {
-	$fields = getPostToForm($ignore);
-	$fields .= getGetToForm($ignore, $usePostAsAuthority);
-	return $fields;
+    $fields = getPostToForm($ignore);
+    $fields .= getGetToForm($ignore, $usePostAsAuthority);
+    return $fields;
 
 }
 
 function handleRedirect($return_id='', $return_module='', $additionalFlags = false)
 {
-	if(isset($_REQUEST['return_url']) && $_REQUEST['return_url'] != "")
-	{
-		header("Location: ". $_REQUEST['return_url']);
-		exit;
-	}
-
-	$url = buildRedirectURL($return_id, $return_module);
-	header($url);
-	exit;	
+    if(isset($_REQUEST['return_url']) && $_REQUEST['return_url'] != "")
+    {
+        header("Location: ". $_REQUEST['return_url']);
+        exit;
+    }
+
+    $url = buildRedirectURL($return_id, $return_module);
+    header($url);
+    exit;	
 }
 
 //eggsurplus: abstract to simplify unit testing
 function buildRedirectURL($return_id='', $return_module='') 
 {
     if(isset($_REQUEST['return_module']) && $_REQUEST['return_module'] != "")
-	{
-		$return_module = $_REQUEST['return_module'];
-	}
-	else
-	{
-		$return_module = $return_module;
-	}
-	if(isset($_REQUEST['return_action']) && $_REQUEST['return_action'] != "")
-	{
+    {
+        $return_module = $_REQUEST['return_module'];
+    }
+    else
+    {
+        $return_module = $return_module;
+    }
+    if(isset($_REQUEST['return_action']) && $_REQUEST['return_action'] != "")
+    {
 	    
-	   //if we are doing a "Close and Create New"
+        //if we are doing a "Close and Create New"
         if(isCloseAndCreateNewPressed())
         {
             $return_action = "EditView";    
@@ -257,40 +257,40 @@ 
             
             // Meeting Integration
             if(isset($_REQUEST['meetingIntegrationFlag']) && $_REQUEST['meetingIntegrationFlag'] == 1) {
-            	$additionalFlags = array('meetingIntegrationShowForm' => '1');
+                $additionalFlags = array('meetingIntegrationShowForm' => '1');
             }
             // END Meeting Integration
         } 
-		// if we create a new record "Save", we want to redirect to the DetailView
-		else if(isset($_REQUEST['action']) && $_REQUEST['action'] == "Save" 
-			&& $_REQUEST['return_module'] != 'Activities'
-			&& $_REQUEST['return_module'] != 'Home' 
-			&& $_REQUEST['return_module'] != 'Forecasts' 
-			&& $_REQUEST['return_module'] != 'Calendar'
-			&& $_REQUEST['return_module'] != 'MailMerge'
-			) 
-			{
-			    $return_action = 'DetailView';
-			} elseif($_REQUEST['return_module'] == 'Activities' || $_REQUEST['return_module'] == 'Calendar') {
-			$return_module = $_REQUEST['module'];
-			$return_action = $_REQUEST['return_action']; 
-			// wp: return action needs to be set for one-click close in task list
-		} 
-		else 
-		{
-			// if we "Cancel", we go back to the list view.
-			$return_action = $_REQUEST['return_action'];
-		}
-	}
-	else
-	{
-		$return_action = "DetailView";
-	}
+        // if we create a new record "Save", we want to redirect to the DetailView
+        else if(isset($_REQUEST['action']) && $_REQUEST['action'] == "Save" 
+            && $_REQUEST['return_module'] != 'Activities'
+            && $_REQUEST['return_module'] != 'Home' 
+            && $_REQUEST['return_module'] != 'Forecasts' 
+            && $_REQUEST['return_module'] != 'Calendar'
+            && $_REQUEST['return_module'] != 'MailMerge'
+            ) 
+            {
+                $return_action = 'DetailView';
+            } elseif($_REQUEST['return_module'] == 'Activities' || $_REQUEST['return_module'] == 'Calendar') {
+            $return_module = $_REQUEST['module'];
+            $return_action = $_REQUEST['return_action']; 
+            // wp: return action needs to be set for one-click close in task list
+        } 
+        else 
+        {
+            // if we "Cancel", we go back to the list view.
+            $return_action = $_REQUEST['return_action'];
+        }
+    }
+    else
+    {
+        $return_action = "DetailView";
+    }
 	
-	if(isset($_REQUEST['return_id']) && $_REQUEST['return_id'] != "")
-	{
-		$return_id = $_REQUEST['return_id'];
-	}
+    if(isset($_REQUEST['return_id']) && $_REQUEST['return_id'] != "")
+    {
+        $return_id = $_REQUEST['return_id'];
+    }
 
     $add = "";
     if(isset($additionalFlags) && !empty($additionalFlags)) {
@@ -320,16 +320,16 @@ 
             return "Location: $url";
         }
     } else {
-    	$standard = "action=$return_action&module=$return_module&record=$return_id&isDuplicate=true&return_module=$return_module&return_action=$return_action&status=$status";
+        $standard = "action=$return_action&module=$return_module&record=$return_id&isDuplicate=true&return_module=$return_module&return_action=$return_action&status=$status";
         $url="index.php?{$standard}{$add}";
         if(!empty($_REQUEST['ajax_load']))
         {
             $ajax_ret = array(
-                 'content' => "<script>SUGAR.ajaxUI.loadContent('$url');</script>\n",
-                 'menu' => array(
-                     'module' => $return_module,
-                     'label' => translate($return_module),
-                 ),
+                    'content' => "<script>SUGAR.ajaxUI.loadContent('$url');</script>\n",
+                    'menu' => array(
+                        'module' => $return_module,
+                        'label' => translate($return_module),
+                    ),
             );
             $json = getJSONobj();
             echo $json->encode($ajax_ret);
@@ -341,21 +341,21 @@ 
 
 function getLikeForEachWord($fieldname, $value, $minsize=4)
 {
-	$value = trim($value);
-	$values = explode(' ',$value);
-	$ret = '';
-	foreach($values as $val)
-	{
-		if(strlen($val) >= $minsize)
-		{
-			if(!empty($ret))
-			{
-				$ret .= ' or';
-			}
-			$ret .= ' '. $fieldname . ' LIKE %'.$val.'%';
-		}
+    $value = trim($value);
+    $values = explode(' ',$value);
+    $ret = '';
+    foreach($values as $val)
+    {
+        if(strlen($val) >= $minsize)
+        {
+            if(!empty($ret))
+            {
+                $ret .= ' or';
+            }
+            $ret .= ' '. $fieldname . ' LIKE %'.$val.'%';
+        }
 
-	}
+    }
 
 
 }

Spacing +69 added lines, -69 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -54,9 +54,9 @@ 
  */
 function checkRequired($prefix, $required)
 {
-	foreach($required as $key)
+	foreach ($required as $key)
 	{
-		if(!isset($_POST[$prefix.$key]) || number_empty($_POST[$prefix.$key]))
+		if (!isset($_POST[$prefix.$key]) || number_empty($_POST[$prefix.$key]))
 		{
 			return false;
 		}
@@ -77,15 +77,15 @@ 
 {
 	global $current_user;
 
-	if(!empty($_REQUEST[$prefix.'record']) && !$skipRetrieve)
+	if (!empty($_REQUEST[$prefix.'record']) && !$skipRetrieve)
 		$focus->retrieve($_REQUEST[$prefix.'record']);
 
-	if(!empty($_POST['assigned_user_id']) && 
+	if (!empty($_POST['assigned_user_id']) && 
 	    ($focus->assigned_user_id != $_POST['assigned_user_id']) && 
 	    ($_POST['assigned_user_id'] != $current_user->id)) {
 		$GLOBALS['check_notify'] = true;
 	}
-    if(isset($_POST['dup_checked']) && $_POST['dup_checked'] && isset($_POST['id']) && $_POST['id'] != '' ){
+    if (isset($_POST['dup_checked']) && $_POST['dup_checked'] && isset($_POST['id']) && $_POST['id'] != '') {
         $focus->new_with_id = true;
     }
     require_once('include/SugarFields/SugarFieldHandler.php');
@@ -106,8 +106,8 @@ 
         $relatedFields[$def['id_name']] = $field;
     }
 
-	foreach($focus->field_defs as $field=>$def) {
-        if ( $field == 'id' && !empty($focus->id) ) {
+	foreach ($focus->field_defs as $field=>$def) {
+        if ($field == 'id' && !empty($focus->id)) {
             // Don't try and overwrite the ID
             continue;
         }
@@ -115,7 +115,7 @@ 
 
 	    $type = !empty($def['custom_type']) ? $def['custom_type'] : $def['type'];
 		$sf = $sfh->getSugarField($type);
-        if($sf != null){
+        if ($sf != null) {
             $sf->save($focus, $_POST, $field, $def, $prefix);
         } else {
             $GLOBALS['log']->fatal("Field '$field' does not have a SugarField handler");
@@ -154,8 +154,8 @@ 
 */
 	}
 
-	foreach($focus->additional_column_fields as $field) {
-		if(isset($_POST[$prefix.$field])) {
+	foreach ($focus->additional_column_fields as $field) {
+		if (isset($_POST[$prefix.$field])) {
 			$value = $_POST[$prefix.$field];
 			$focus->$field = $value;
 		}
@@ -180,18 +180,18 @@ 
 }
 
 
-function getPostToForm($ignore='', $isRegularExpression=false)
+function getPostToForm($ignore = '', $isRegularExpression = false)
 {
 	$fields = '';
-	if(!empty($ignore) && $isRegularExpression) {
-		foreach ($_POST as $key=>$value){
-			if(!preg_match($ignore, $key)) {
+	if (!empty($ignore) && $isRegularExpression) {
+		foreach ($_POST as $key=>$value) {
+			if (!preg_match($ignore, $key)) {
                                 $fields .= add_hidden_elements($key, $value);
 			}
 		}	
 	} else {
-		foreach ($_POST as $key=>$value){
-			if($key != $ignore) {
+		foreach ($_POST as $key=>$value) {
+			if ($key != $ignore) {
                                 $fields .= add_hidden_elements($key, $value);
 			}
 		}
@@ -199,21 +199,21 @@ 
 	return $fields;
 }
 
-function getGetToForm($ignore='', $usePostAsAuthority = false)
+function getGetToForm($ignore = '', $usePostAsAuthority = false)
 {
 	$fields = '';
 	foreach ($_GET as $key=>$value)
 	{
-		if($key != $ignore){
-			if(!$usePostAsAuthority || !isset($_POST[$key])){
-				$fields.= "<input type='hidden' name='$key' value='$value'>";
+		if ($key != $ignore) {
+			if (!$usePostAsAuthority || !isset($_POST[$key])) {
+				$fields .= "<input type='hidden' name='$key' value='$value'>";
 			}
 		}
 	}
 	return $fields;
 
 }
-function getAnyToForm($ignore='', $usePostAsAuthority = false)
+function getAnyToForm($ignore = '', $usePostAsAuthority = false)
 {
 	$fields = getPostToForm($ignore);
 	$fields .= getGetToForm($ignore, $usePostAsAuthority);
@@ -221,11 +221,11 @@ 
 
 }
 
-function handleRedirect($return_id='', $return_module='', $additionalFlags = false)
+function handleRedirect($return_id = '', $return_module = '', $additionalFlags = false)
 {
-	if(isset($_REQUEST['return_url']) && $_REQUEST['return_url'] != "")
+	if (isset($_REQUEST['return_url']) && $_REQUEST['return_url'] != "")
 	{
-		header("Location: ". $_REQUEST['return_url']);
+		header("Location: ".$_REQUEST['return_url']);
 		exit;
 	}
 
@@ -235,9 +235,9 @@ 
 }
 
 //eggsurplus: abstract to simplify unit testing
-function buildRedirectURL($return_id='', $return_module='') 
+function buildRedirectURL($return_id = '', $return_module = '') 
 {
-    if(isset($_REQUEST['return_module']) && $_REQUEST['return_module'] != "")
+    if (isset($_REQUEST['return_module']) && $_REQUEST['return_module'] != "")
 	{
 		$return_module = $_REQUEST['return_module'];
 	}
@@ -245,24 +245,24 @@ 
 	{
 		$return_module = $return_module;
 	}
-	if(isset($_REQUEST['return_action']) && $_REQUEST['return_action'] != "")
+	if (isset($_REQUEST['return_action']) && $_REQUEST['return_action'] != "")
 	{
 	    
 	   //if we are doing a "Close and Create New"
-        if(isCloseAndCreateNewPressed())
+        if (isCloseAndCreateNewPressed())
         {
             $return_action = "EditView";    
             $isDuplicate = "true";        
             $status = "";
             
             // Meeting Integration
-            if(isset($_REQUEST['meetingIntegrationFlag']) && $_REQUEST['meetingIntegrationFlag'] == 1) {
+            if (isset($_REQUEST['meetingIntegrationFlag']) && $_REQUEST['meetingIntegrationFlag'] == 1) {
             	$additionalFlags = array('meetingIntegrationShowForm' => '1');
             }
             // END Meeting Integration
         } 
 		// if we create a new record "Save", we want to redirect to the DetailView
-		else if(isset($_REQUEST['action']) && $_REQUEST['action'] == "Save" 
+		else if (isset($_REQUEST['action']) && $_REQUEST['action'] == "Save" 
 			&& $_REQUEST['return_module'] != 'Activities'
 			&& $_REQUEST['return_module'] != 'Home' 
 			&& $_REQUEST['return_module'] != 'Forecasts' 
@@ -271,7 +271,7 @@ 
 			) 
 			{
 			    $return_action = 'DetailView';
-			} elseif($_REQUEST['return_module'] == 'Activities' || $_REQUEST['return_module'] == 'Calendar') {
+			} elseif ($_REQUEST['return_module'] == 'Activities' || $_REQUEST['return_module'] == 'Calendar') {
 			$return_module = $_REQUEST['module'];
 			$return_action = $_REQUEST['return_action']; 
 			// wp: return action needs to be set for one-click close in task list
@@ -287,25 +287,25 @@ 
 		$return_action = "DetailView";
 	}
 	
-	if(isset($_REQUEST['return_id']) && $_REQUEST['return_id'] != "")
+	if (isset($_REQUEST['return_id']) && $_REQUEST['return_id'] != "")
 	{
 		$return_id = $_REQUEST['return_id'];
 	}
 
     $add = "";
-    if(isset($additionalFlags) && !empty($additionalFlags)) {
-        foreach($additionalFlags as $k => $v) {
+    if (isset($additionalFlags) && !empty($additionalFlags)) {
+        foreach ($additionalFlags as $k => $v) {
             $add .= "&{$k}={$v}";
         }
     }
     
     if (!isset($isDuplicate) || !$isDuplicate)
     {
-        $url="index.php?action=$return_action&module=$return_module&record=$return_id&return_module=$return_module&return_action=$return_action{$add}";
-        if(isset($_REQUEST['offset']) && empty($_REQUEST['duplicateSave'])) {
+        $url = "index.php?action=$return_action&module=$return_module&record=$return_id&return_module=$return_module&return_action=$return_action{$add}";
+        if (isset($_REQUEST['offset']) && empty($_REQUEST['duplicateSave'])) {
             $url .= "&offset=".$_REQUEST['offset'];
         }
-        if(!empty($_REQUEST['ajax_load']))
+        if (!empty($_REQUEST['ajax_load']))
         {
             $ajax_ret = array(
                 'content' => "<script>SUGAR.ajaxUI.loadContent('$url');</script>\n",
@@ -321,8 +321,8 @@ 
         }
     } else {
     	$standard = "action=$return_action&module=$return_module&record=$return_id&isDuplicate=true&return_module=$return_module&return_action=$return_action&status=$status";
-        $url="index.php?{$standard}{$add}";
-        if(!empty($_REQUEST['ajax_load']))
+        $url = "index.php?{$standard}{$add}";
+        if (!empty($_REQUEST['ajax_load']))
         {
             $ajax_ret = array(
                  'content' => "<script>SUGAR.ajaxUI.loadContent('$url');</script>\n",
@@ -339,20 +339,20 @@ 
     }
 }
 
-function getLikeForEachWord($fieldname, $value, $minsize=4)
+function getLikeForEachWord($fieldname, $value, $minsize = 4)
 {
 	$value = trim($value);
-	$values = explode(' ',$value);
+	$values = explode(' ', $value);
 	$ret = '';
-	foreach($values as $val)
+	foreach ($values as $val)
 	{
-		if(strlen($val) >= $minsize)
+		if (strlen($val) >= $minsize)
 		{
-			if(!empty($ret))
+			if (!empty($ret))
 			{
 				$ret .= ' or';
 			}
-			$ret .= ' '. $fieldname . ' LIKE %'.$val.'%';
+			$ret .= ' '.$fieldname.' LIKE %'.$val.'%';
 		}
 
 	}
@@ -373,36 +373,36 @@ 
  * @see include/generic/Save2.php
  */
 
-function add_prospects_to_prospect_list($parent_id,$child_id)
+function add_prospects_to_prospect_list($parent_id, $child_id)
 {
-    $focus=BeanFactory::getBean('Prospects');
-    if(is_array($child_id)){
+    $focus = BeanFactory::getBean('Prospects');
+    if (is_array($child_id)) {
         $uids = $child_id;
     }
-    else{
+    else {
         $uids = array($child_id);
     }
 
     $relationship = '';
-    foreach($focus->get_linked_fields() as $field => $def) {
+    foreach ($focus->get_linked_fields() as $field => $def) {
         if ($focus->load_relationship($field)) {
-            if ( $focus->$field->getRelatedModuleName() == 'ProspectLists' ) {
+            if ($focus->$field->getRelatedModuleName() == 'ProspectLists') {
                 $relationship = $field;
                 break;
             }
         }
     }
 
-    if ( $relationship != '' ) {
-        foreach ( $uids as $id) {
+    if ($relationship != '') {
+        foreach ($uids as $id) {
             $focus->retrieve($id);
             $focus->load_relationship($relationship);
-            $focus->prospect_lists->add( $parent_id );
+            $focus->prospect_lists->add($parent_id);
         }
     }
 }
 
-function add_to_prospect_list($query_panel,$parent_module,$parent_type,$parent_id,$child_id,$link_attribute,$link_type,$parent)
+function add_to_prospect_list($query_panel, $parent_module, $parent_type, $parent_id, $child_id, $link_attribute, $link_type, $parent)
 {
     $GLOBALS['log']->debug('add_prospects_to_prospect_list:parameters:'.$query_panel);
     $GLOBALS['log']->debug('add_prospects_to_prospect_list:parameters:'.$parent_module);
@@ -419,21 +419,21 @@ 
     }
     $focus = new $parent_type();
     $focus->retrieve($parent_id);
-    if(empty($focus->id)) {
+    if (empty($focus->id)) {
         return false;
     }
-    if(empty($parent)) {
+    if (empty($parent)) {
         return false;
     }
 
     //if link_type is default then load relationship once and add all the child ids.
-    $relationship_attribute=$link_attribute;
+    $relationship_attribute = $link_attribute;
 
     //find all prospects based on the query
 
     $subpanel = new SubPanelTiles($parent, $parent->module_dir);
-    $thisPanel=$subpanel->subpanel_definitions->load_subpanel($query_panel);
-    if(empty($thisPanel)) {
+    $thisPanel = $subpanel->subpanel_definitions->load_subpanel($query_panel);
+    if (empty($thisPanel)) {
         return false;
     }
 
@@ -443,12 +443,12 @@ 
         $thisPanel->_instance_properties['function_parameters']['EMAIL_MARKETING_ID_VALUE'] = $_REQUEST['marketing_id'];
     }
 
-    $result = SugarBean::get_union_related_list($parent, '', '', '', 0, -99,-99,'', $thisPanel);
+    $result = SugarBean::get_union_related_list($parent, '', '', '', 0, -99, -99, '', $thisPanel);
 
-    if(!empty($result['list'])) {
-        foreach($result['list'] as $object) {
+    if (!empty($result['list'])) {
+        foreach ($result['list'] as $object) {
             if ($link_type != 'default') {
-                $relationship_attribute=strtolower($object->$link_attribute);
+                $relationship_attribute = strtolower($object->$link_attribute);
             }
             $GLOBALS['log']->debug('add_prospects_to_prospect_list:relationship_attribute:'.$relationship_attribute);
             // load relationship for the first time or on change of relationship atribute.
@@ -462,7 +462,7 @@ 
 }
 
 //Link rows returned by a report to parent record.
-function save_from_report($report_id,$parent_id, $module_name, $relationship_attr_name) {
+function save_from_report($report_id, $parent_id, $module_name, $relationship_attr_name) {
     global $beanFiles;
     global $beanList;
 
@@ -472,7 +472,7 @@ 
     $GLOBALS['log']->debug("Save2:Module Name=".$module_name);
     $GLOBALS['log']->debug("Save2:Relationship Attribute Name=".$relationship_attr_name);
 
-    $GLOBALS['log']->debug("Save2:Bean Name=" . $module_name);
+    $GLOBALS['log']->debug("Save2:Bean Name=".$module_name);
     $focus = BeanFactory::newBean($module_name);
 
     $focus->retrieve($parent_id);
@@ -481,7 +481,7 @@ 
     //fetch report definition.
     global $current_language, $report_modules, $modules_report;
 
-    $mod_strings = return_module_language($current_language,"Reports");
+    $mod_strings = return_module_language($current_language, "Reports");
 
 
     $saved = new SavedReport();
@@ -498,7 +498,7 @@ 
     $result = $report->db->query($sql);
 
     $reportBean = BeanFactory::newBean($saved->module);
-    while($row = $report->db->fetchByAssoc($result))
+    while ($row = $report->db->fetchByAssoc($result))
     {
         $reportBean->id = $row['primaryid'];
         $focus->$relationship_attr_name->add($reportBean);

Braces +10 added lines, -11 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -77,8 +79,9 @@ 
 {
 	global $current_user;
 
-	if(!empty($_REQUEST[$prefix.'record']) && !$skipRetrieve)
-		$focus->retrieve($_REQUEST[$prefix.'record']);
+	if(!empty($_REQUEST[$prefix.'record']) && !$skipRetrieve) {
+			$focus->retrieve($_REQUEST[$prefix.'record']);
+	}
 
 	if(!empty($_POST['assigned_user_id']) && 
 	    ($focus->assigned_user_id != $_POST['assigned_user_id']) && 
@@ -240,8 +243,7 @@ 
     if(isset($_REQUEST['return_module']) && $_REQUEST['return_module'] != "")
 	{
 		$return_module = $_REQUEST['return_module'];
-	}
-	else
+	} else
 	{
 		$return_module = $return_module;
 	}
@@ -275,14 +277,12 @@ 
 			$return_module = $_REQUEST['module'];
 			$return_action = $_REQUEST['return_action']; 
 			// wp: return action needs to be set for one-click close in task list
-		} 
-		else 
+		} else 
 		{
 			// if we "Cancel", we go back to the list view.
 			$return_action = $_REQUEST['return_action'];
 		}
-	}
-	else
+	} else
 	{
 		$return_action = "DetailView";
 	}
@@ -378,8 +378,7 @@ 
     $focus=BeanFactory::getBean('Prospects');
     if(is_array($child_id)){
         $uids = $child_id;
-    }
-    else{
+    } else{
         $uids = array($child_id);
     }
 

Doc Comments +6 added lines

@@ -235,6 +235,9 @@ 
 }
 
 //eggsurplus: abstract to simplify unit testing
+/**
+ * @return string
+ */
 function buildRedirectURL($return_id='', $return_module='') 
 {
     if(isset($_REQUEST['return_module']) && $_REQUEST['return_module'] != "")
@@ -339,6 +342,9 @@ 
     }
 }
 
+/**
+ * @param string $fieldname
+ */
 function getLikeForEachWord($fieldname, $value, $minsize=4)
 {
 	$value = trim($value);

include/Popups/PopupSmarty.php

Braces +45 added lines, -25 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -128,8 +130,9 @@ 
         foreach($this->displayColumns as $name => $params) {
             $this->displayColumns[$name]['width'] = round($this->displayColumns[$name]['width'] / $adjustment, 2);
             // figure out which contextMenu objectsTypes are required
-            if(!empty($params['contextMenu']['objectType']))
-                $contextMenuObjectsTypes[$params['contextMenu']['objectType']] = true;
+            if(!empty($params['contextMenu']['objectType'])) {
+                            $contextMenuObjectsTypes[$params['contextMenu']['objectType']] = true;
+            }
         }
 		$this->th->ss->assign('displayColumns', $this->displayColumns);
 
@@ -147,10 +150,17 @@ 
 		$this->th->ss->assign('searchForm', $this->searchForm->display(false));
         //rrs
 
-		if($this->export) $this->th->ss->assign('exportLink', $this->buildExportLink());
+		if($this->export) {
+		    $this->th->ss->assign('exportLink', $this->buildExportLink());
+		}
 		$this->th->ss->assign('quickViewLinks', $this->quickViewLinks);
-		if($this->mailMerge) $this->th->ss->assign('mergeLink', $this->buildMergeLink()); // still check for mailmerge access
-		if($this->mergeduplicates) $this->th->ss->assign('mergedupLink', $this->buildMergeDuplicatesLink());
+		if($this->mailMerge) {
+		    $this->th->ss->assign('mergeLink', $this->buildMergeLink());
+		}
+		// still check for mailmerge access
+		if($this->mergeduplicates) {
+		    $this->th->ss->assign('mergedupLink', $this->buildMergeDuplicatesLink());
+		}
 
 
 		if (!empty($_REQUEST['mode']) && strtoupper($_REQUEST['mode']) == 'MULTISELECT') {
@@ -301,7 +311,7 @@ 
 
 		if(file_exists('custom/modules/'.$this->module.'/metadata/metafiles.php')){
 			require('custom/modules/'.$this->module.'/metadata/metafiles.php');
-		}elseif(file_exists('modules/'.$this->module.'/metadata/metafiles.php')){
+		} elseif(file_exists('modules/'.$this->module.'/metadata/metafiles.php')){
 			require('modules/'.$this->module.'/metadata/metafiles.php');
 		}
 
@@ -321,21 +331,24 @@ 
 		$displayColumns = array();
 		if(!empty($_REQUEST['displayColumns'])) {
 		    foreach(explode('|', $_REQUEST['displayColumns']) as $num => $col) {
-		        if(!empty($listViewDefs[$this->module][$col]))
-		            $displayColumns[$col] = $this->listviewdefs[$this->module][$col];
+		        if(!empty($listViewDefs[$this->module][$col])) {
+		        		            $displayColumns[$col] = $this->listviewdefs[$this->module][$col];
+		        }
 		    }
-		}
-		else {
+		} else {
 		    foreach($this->listviewdefs[$this->module] as $col => $para) {
-		        if(!empty($para['default']) && $para['default'])
-		            $displayColumns[$col] = $para;
+		        if(!empty($para['default']) && $para['default']) {
+		        		            $displayColumns[$col] = $para;
+		        }
 		    }
 		}
 		$params['massupdate'] = true;
 		if(!empty($_REQUEST['orderBy'])) {
 		    $params['orderBy'] = $_REQUEST['orderBy'];
 		    $params['overrideOrder'] = true;
-		    if(!empty($_REQUEST['sortOrder'])) $params['sortOrder'] = $_REQUEST['sortOrder'];
+		    if(!empty($_REQUEST['sortOrder'])) {
+		        $params['sortOrder'] = $_REQUEST['sortOrder'];
+		    }
 		}
 
 		$lv->displayColumns = $displayColumns;
@@ -453,7 +466,9 @@ 
 
         // Need to include the default whereStatement
 		if(!empty($this->_popupMeta['whereStatement'])){
-            if(!empty($where))$where .= ' AND ';
+            if(!empty($where)) {
+                $where .= ' AND ';
+            }
             $where .= $this->_popupMeta['whereStatement'];
 		}
 
@@ -472,11 +487,13 @@ 
 				$this->formData[] = array('field' => $data);
 				$value = '';
 				$this->customFieldDefs[$data['name']]= $data;
-				if(!empty($_REQUEST[$data['name']]))
-	            	$value = $_REQUEST[$data['name']];
+				if(!empty($_REQUEST[$data['name']])) {
+					            	$value = $_REQUEST[$data['name']];
+				}
 	            $this->customFieldDefs[$data['name']]['value'] = $value;
-			}else
-				$this->formData[] = array('field' => array('name'=>$data));
+			} else {
+							$this->formData[] = array('field' => array('name'=>$data));
+			}
 		}
 		$this->fieldDefs = array();
 		if($this->seed){
@@ -487,13 +504,15 @@ 
 	            //if we have a relate type then reset to name so that we end up with a textbox
 	            //rather than a select button
 	            $this->fieldDefs[$name]['name'] = $this->fieldDefs[$name]['name'];
-	            if($this->fieldDefs[$name]['type'] == 'relate')
-	            	$this->fieldDefs[$name]['type'] = 'name';
+	            if($this->fieldDefs[$name]['type'] == 'relate') {
+	            	            	$this->fieldDefs[$name]['type'] = 'name';
+	            }
 	            if(isset($this->fieldDefs[$name]['options']) && isset($GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']])) {
 	                $this->fieldDefs[$name]['options'] = $GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']]; // fill in enums
 	            }
-	            if(!empty($_REQUEST[$name]))
-	            	$value = $_REQUEST[$name];
+	            if(!empty($_REQUEST[$name])) {
+	            	            	$value = $_REQUEST[$name];
+	            }
 	            $this->fieldDefs[$name]['value'] = $value;
 	        }
 		}
@@ -540,8 +559,9 @@ 
 EOQ;
 		// if metadata contains custom inputs for the quickcreate
 		if(!empty($this->_popupMeta['customInput']) && is_array($this->_popupMeta['customInput'])) {
-			foreach($this->_popupMeta['customInput'] as $key => $value)
-				$formSave .= '<input type="hidden" name="' . $key . '" value="'. $value .'">\n';
+			foreach($this->_popupMeta['customInput'] as $key => $value) {
+							$formSave .= '<input type="hidden" name="' . $key . '" value="'. $value .'">\n';
+			}
 		}
 
 

Doc Comments +4 added lines

@@ -70,6 +70,9 @@ 
     var $module;
     var $massUpdateData = '';
 
+	/**
+	 * @param string $module
+	 */
 	function PopupSmarty($seed, $module){
 		parent::ListViewSmarty();
 		$this->th = new TemplateHandler();
@@ -108,6 +111,7 @@ 
      * @param file file Template file to use
      * @param data array from ListViewData
      * @param html_var string the corresponding html var in xtpl per row
+     * @param string $htmlVar
      *
      */
 	function process($file, $data, $htmlVar) {

Indentation +279 added lines, -279 removed lines

@@ -45,13 +45,13 @@ 
 define("NUM_COLS", 2);
 class PopupSmarty extends ListViewSmarty{
 
-	var $contextMenus = false;
-	var $export = false;
-	var $mailmerge = false;
-	var $mergeduplicates = false;
-	var $quickViewLinks = false;
-	var $multiSelect = false;
-	var $headerTpl;
+    var $contextMenus = false;
+    var $export = false;
+    var $mailmerge = false;
+    var $mergeduplicates = false;
+    var $quickViewLinks = false;
+    var $multiSelect = false;
+    var $headerTpl;
     var $footerTpl;
     var $th;
     var $tpl;
@@ -70,19 +70,19 @@ 
     var $module;
     var $massUpdateData = '';
 
-	public function __construct($seed, $module){
-		parent::__construct();
-		$this->th = new TemplateHandler();
-		$this->th->loadSmarty();
-		$this->seed = $seed;
-		$this->view = 'Popup';
-		$this->module = $module;
-		$this->searchForm = new SearchForm($this->seed, $this->module);
-		$this->th->deleteTemplate($module, $this->view);
+    public function __construct($seed, $module){
+        parent::__construct();
+        $this->th = new TemplateHandler();
+        $this->th->loadSmarty();
+        $this->seed = $seed;
+        $this->view = 'Popup';
+        $this->module = $module;
+        $this->searchForm = new SearchForm($this->seed, $this->module);
+        $this->th->deleteTemplate($module, $this->view);
         $this->headerTpl = 'include/Popups/tpls/header.tpl';
         $this->footerTpl = 'include/Popups/tpls/footer.tpl';
 
-	}
+    }
 
     /**
      * Assign several arrow image attributes to TemplateHandler smarty. Such as width, height, etc.
@@ -101,7 +101,7 @@ 
         $this->th->ss->assign('arrowAlt', translate('LBL_SORT'));
     }
 
-	/**
+    /**
      * Processes the request. Calls ListViewData process. Also assigns all lang strings, export links,
      * This is called from ListViewDisplay
      *
@@ -110,13 +110,13 @@ 
      * @param html_var string the corresponding html var in xtpl per row
      *
      */
-	function process($file, $data, $htmlVar) {
+    function process($file, $data, $htmlVar) {
 
-		global $odd_bg, $even_bg, $hilite_bg, $click_bg, $app_strings;
-		parent::process($file, $data, $htmlVar);
+        global $odd_bg, $even_bg, $hilite_bg, $click_bg, $app_strings;
+        parent::process($file, $data, $htmlVar);
 
-		$this->tpl = $file;
-		$this->data = $data;
+        $this->tpl = $file;
+        $this->data = $data;
 
         $totalWidth = 0;
         foreach($this->displayColumns as $name => $params) {
@@ -131,47 +131,47 @@ 
             if(!empty($params['contextMenu']['objectType']))
                 $contextMenuObjectsTypes[$params['contextMenu']['objectType']] = true;
         }
-		$this->th->ss->assign('displayColumns', $this->displayColumns);
+        $this->th->ss->assign('displayColumns', $this->displayColumns);
 
 
-		$this->th->ss->assign('bgHilite', $hilite_bg);
-		$this->th->ss->assign('colCount', count($this->displayColumns) + 1);
-		$this->th->ss->assign('htmlVar', strtoupper($htmlVar));
-		$this->th->ss->assign('moduleString', $this->moduleString);
+        $this->th->ss->assign('bgHilite', $hilite_bg);
+        $this->th->ss->assign('colCount', count($this->displayColumns) + 1);
+        $this->th->ss->assign('htmlVar', strtoupper($htmlVar));
+        $this->th->ss->assign('moduleString', $this->moduleString);
         $this->th->ss->assign('editLinkString', $GLOBALS['app_strings']['LBL_EDIT_BUTTON']);
         $this->th->ss->assign('viewLinkString', $GLOBALS['app_strings']['LBL_VIEW_BUTTON']);
 
         //rrs
         $this->searchForm->parsedView = 'popup_query_form';
         $this->searchForm->displayType = 'popupView';
-		$this->th->ss->assign('searchForm', $this->searchForm->display(false));
+        $this->th->ss->assign('searchForm', $this->searchForm->display(false));
         //rrs
 
-		if($this->export) $this->th->ss->assign('exportLink', $this->buildExportLink());
-		$this->th->ss->assign('quickViewLinks', $this->quickViewLinks);
-		if($this->mailMerge) $this->th->ss->assign('mergeLink', $this->buildMergeLink()); // still check for mailmerge access
-		if($this->mergeduplicates) $this->th->ss->assign('mergedupLink', $this->buildMergeDuplicatesLink());
+        if($this->export) $this->th->ss->assign('exportLink', $this->buildExportLink());
+        $this->th->ss->assign('quickViewLinks', $this->quickViewLinks);
+        if($this->mailMerge) $this->th->ss->assign('mergeLink', $this->buildMergeLink()); // still check for mailmerge access
+        if($this->mergeduplicates) $this->th->ss->assign('mergedupLink', $this->buildMergeDuplicatesLink());
 
 
-		if (!empty($_REQUEST['mode']) && strtoupper($_REQUEST['mode']) == 'MULTISELECT') {
-			$this->multiSelect = true;
-		}
-		// handle save checks and stuff
-		if($this->multiSelect) {
-			$this->th->ss->assign('selectedObjectsSpan', $this->buildSelectedObjectsSpan());
-			$this->th->ss->assign('multiSelectData', $this->getMultiSelectData());
-			$this->th->ss->assign('MODE', "<input type='hidden' name='mode' value='MultiSelect'>");
+        if (!empty($_REQUEST['mode']) && strtoupper($_REQUEST['mode']) == 'MULTISELECT') {
+            $this->multiSelect = true;
+        }
+        // handle save checks and stuff
+        if($this->multiSelect) {
+            $this->th->ss->assign('selectedObjectsSpan', $this->buildSelectedObjectsSpan());
+            $this->th->ss->assign('multiSelectData', $this->getMultiSelectData());
+            $this->th->ss->assign('MODE', "<input type='hidden' name='mode' value='MultiSelect'>");
             $pageTotal = $this->data['pageData']['offsets']['next'] - $this->data['pageData']['offsets']['current'];
             if($this->data['pageData']['offsets']['next'] < 0){ // If we are on the last page, 'next' is -1, which means we have to have a custom calculation
                 $pageTotal = $this->data['pageData']['offsets']['total'] - $this->data['pageData']['offsets']['current'];
             }
-    		$this->th->ss->assign('selectLink', $this->buildSelectLink('select_link', $this->data['pageData']['offsets']['total'], $pageTotal));
-		}
+            $this->th->ss->assign('selectLink', $this->buildSelectLink('select_link', $this->data['pageData']['offsets']['total'], $pageTotal));
+        }
 
-		$this->processArrows($data['pageData']['ordering']);
-		$this->th->ss->assign('prerow', $this->multiSelect);
-		$this->th->ss->assign('rowColor', array('oddListRow', 'evenListRow'));
-		$this->th->ss->assign('bgColor', array($odd_bg, $even_bg));
+        $this->processArrows($data['pageData']['ordering']);
+        $this->th->ss->assign('prerow', $this->multiSelect);
+        $this->th->ss->assign('rowColor', array('oddListRow', 'evenListRow'));
+        $this->th->ss->assign('bgColor', array($odd_bg, $even_bg));
         $this->th->ss->assign('contextMenus', $this->contextMenus);
 
 
@@ -191,12 +191,12 @@ 
 
             // arrow image attributes
             $this->processArrowVars();
-	}
+    }
 
-	/*
+    /*
 	 * Display the Smarty template.  Here we are using the TemplateHandler for caching per the module.
 	 */
-	function display($end = true) {
+    function display($end = true) {
         global $app_strings;
 
         if(!is_file(sugar_cached("jsLanguage/{$GLOBALS['current_language']}.js"))) {
@@ -206,8 +206,8 @@ 
         $jsLang = getVersionedScript("cache/jsLanguage/{$GLOBALS['current_language']}.js",  $GLOBALS['sugar_config']['js_lang_version']);
 
         $this->th->ss->assign('data', $this->data['data']);
-		$this->data['pageData']['offsets']['lastOffsetOnPage'] = $this->data['pageData']['offsets']['current'] + count($this->data['data']);
-		$this->th->ss->assign('pageData', $this->data['pageData']);
+        $this->data['pageData']['offsets']['lastOffsetOnPage'] = $this->data['pageData']['offsets']['current'] + count($this->data['data']);
+        $this->th->ss->assign('pageData', $this->data['pageData']);
 
         $navStrings = array('next' => $GLOBALS['app_strings']['LNK_LIST_NEXT'],
                             'previous' => $GLOBALS['app_strings']['LNK_LIST_PREVIOUS'],
@@ -217,128 +217,128 @@ 
         $this->th->ss->assign('navStrings', $navStrings);
 
 
-		$associated_row_data = array();
-
-		//C.L. - Bug 44324 - Override the NAME entry to not display salutation so that the data returned from the popup can be searched on correctly
-		$searchNameOverride = !empty($this->seed) && $this->seed instanceof Person && (isset($this->data['data'][0]['FIRST_NAME']) && isset($this->data['data'][0]['LAST_NAME'])) ? true : false;
-
-		global $locale;
-		foreach($this->data['data'] as $val)
-		{
-			$associated_row_data[$val['ID']] = $val;
-			if($searchNameOverride)
-			{
-			   $associated_row_data[$val['ID']]['NAME'] = $locale->getLocaleFormattedName($val['FIRST_NAME'], $val['LAST_NAME']);
-			}
-		}
-		$is_show_fullname = showFullName() ? 1 : 0;
-		$json = getJSONobj();
-		$this->th->ss->assign('jsLang', $jsLang);
-		$this->th->ss->assign('lang', substr($GLOBALS['current_language'], 0, 2));
+        $associated_row_data = array();
+
+        //C.L. - Bug 44324 - Override the NAME entry to not display salutation so that the data returned from the popup can be searched on correctly
+        $searchNameOverride = !empty($this->seed) && $this->seed instanceof Person && (isset($this->data['data'][0]['FIRST_NAME']) && isset($this->data['data'][0]['LAST_NAME'])) ? true : false;
+
+        global $locale;
+        foreach($this->data['data'] as $val)
+        {
+            $associated_row_data[$val['ID']] = $val;
+            if($searchNameOverride)
+            {
+                $associated_row_data[$val['ID']]['NAME'] = $locale->getLocaleFormattedName($val['FIRST_NAME'], $val['LAST_NAME']);
+            }
+        }
+        $is_show_fullname = showFullName() ? 1 : 0;
+        $json = getJSONobj();
+        $this->th->ss->assign('jsLang', $jsLang);
+        $this->th->ss->assign('lang', substr($GLOBALS['current_language'], 0, 2));
         $this->th->ss->assign('headerTpl', $this->headerTpl);
         $this->th->ss->assign('footerTpl', $this->footerTpl);
         $this->th->ss->assign('ASSOCIATED_JAVASCRIPT_DATA', 'var associated_javascript_data = '.$json->encode($associated_row_data). '; var is_show_fullname = '.$is_show_fullname.';');
-		$this->th->ss->assign('module', $this->seed->module_dir);
-		$request_data = empty($_REQUEST['request_data']) ? '' : $_REQUEST['request_data'];
-
-		$this->th->ss->assign('request_data', $request_data);
-		$this->th->ss->assign('fields', $this->fieldDefs);
-		$this->th->ss->assign('formData', $this->formData);
-		$this->th->ss->assign('APP', $GLOBALS['app_strings']);
-		$this->th->ss->assign('MOD', $GLOBALS['mod_strings']);
+        $this->th->ss->assign('module', $this->seed->module_dir);
+        $request_data = empty($_REQUEST['request_data']) ? '' : $_REQUEST['request_data'];
+
+        $this->th->ss->assign('request_data', $request_data);
+        $this->th->ss->assign('fields', $this->fieldDefs);
+        $this->th->ss->assign('formData', $this->formData);
+        $this->th->ss->assign('APP', $GLOBALS['app_strings']);
+        $this->th->ss->assign('MOD', $GLOBALS['mod_strings']);
         if (isset($this->_popupMeta['create']['createButton']))
-		{
-           $this->_popupMeta['create']['createButton'] = translate($this->_popupMeta['create']['createButton']);
+        {
+            $this->_popupMeta['create']['createButton'] = translate($this->_popupMeta['create']['createButton']);
         }
-		$this->th->ss->assign('popupMeta', $this->_popupMeta);
+        $this->th->ss->assign('popupMeta', $this->_popupMeta);
         $this->th->ss->assign('current_query', base64_encode(serialize($_REQUEST)));
-		$this->th->ss->assign('customFields', $this->customFieldDefs);
-		$this->th->ss->assign('numCols', NUM_COLS);
-		$this->th->ss->assign('massUpdateData', $this->massUpdateData);
-		$this->th->ss->assign('sugarVersion', $GLOBALS['sugar_version']);
+        $this->th->ss->assign('customFields', $this->customFieldDefs);
+        $this->th->ss->assign('numCols', NUM_COLS);
+        $this->th->ss->assign('massUpdateData', $this->massUpdateData);
+        $this->th->ss->assign('sugarVersion', $GLOBALS['sugar_version']);
         $this->th->ss->assign('should_process', $this->should_process);
 
-		if($this->_create){
-			$this->th->ss->assign('ADDFORM', $this->getQuickCreate());//$this->_getAddForm());
-			$this->th->ss->assign('ADDFORMHEADER', $this->_getAddFormHeader());
-			$this->th->ss->assign('object_name', $this->seed->object_name);
-		}
-		$this->th->ss->assign('LIST_HEADER', get_form_header($GLOBALS['mod_strings']['LBL_LIST_FORM_TITLE'], '', false));
-		$this->th->ss->assign('SEARCH_FORM_HEADER', get_form_header($GLOBALS['mod_strings']['LBL_SEARCH_FORM_TITLE'], '', false));
-		$str = $this->th->displayTemplate($this->seed->module_dir, $this->view, $this->tpl);
-		return $str;
-	}
-
-	/*
+        if($this->_create){
+            $this->th->ss->assign('ADDFORM', $this->getQuickCreate());//$this->_getAddForm());
+            $this->th->ss->assign('ADDFORMHEADER', $this->_getAddFormHeader());
+            $this->th->ss->assign('object_name', $this->seed->object_name);
+        }
+        $this->th->ss->assign('LIST_HEADER', get_form_header($GLOBALS['mod_strings']['LBL_LIST_FORM_TITLE'], '', false));
+        $this->th->ss->assign('SEARCH_FORM_HEADER', get_form_header($GLOBALS['mod_strings']['LBL_SEARCH_FORM_TITLE'], '', false));
+        $str = $this->th->displayTemplate($this->seed->module_dir, $this->view, $this->tpl);
+        return $str;
+    }
+
+    /*
 	 * Setup up the smarty template. we added an extra step here to add the order by from the popupdefs.
 	 */
-	function setup($seed, $file = null, $where = null, $params = Array(), $offset = 0, $limit = -1, $filter_fields = Array(), $id_field = 'id') {
-		$args = func_get_args();
-		return call_user_func_array(array($this, '_setup'), $args);
-	}
-	function _setup($file) {
-
-	    if(isset($this->_popupMeta)){
-			if(isset($this->_popupMeta['create']['formBase'])) {
-				require_once('modules/' . $this->seed->module_dir . '/' . $this->_popupMeta['create']['formBase']);
-				$this->_create = true;
-			}
-		}
-	    if(!empty($this->_popupMeta['create'])){
-			$formBase = new $this->_popupMeta['create']['formBaseClass']();
-			if(isset($_REQUEST['doAction']) && $_REQUEST['doAction'] == 'save')
-			{
-				//If it's a new record, set useRequired to false
-				$useRequired = empty($_REQUEST['id']) ? false : true;
-				$formBase->handleSave('', false, $useRequired);
-			}
-		}
-
-		$params = array();
-		if(!empty($this->_popupMeta['orderBy'])){
-			$params['orderBy'] = $this->_popupMeta['orderBy'];
-		}
-
-		if(file_exists('custom/modules/'.$this->module.'/metadata/metafiles.php')){
-			require('custom/modules/'.$this->module.'/metadata/metafiles.php');
-		}elseif(file_exists('modules/'.$this->module.'/metadata/metafiles.php')){
-			require('modules/'.$this->module.'/metadata/metafiles.php');
-		}
-
-		if(!empty($metafiles[$this->module]['searchfields'])) {
-			require($metafiles[$this->module]['searchfields']);
-		} elseif(file_exists('modules/'.$this->module.'/metadata/SearchFields.php')) {
-			require('modules/'.$this->module.'/metadata/SearchFields.php');
-	    }
+    function setup($seed, $file = null, $where = null, $params = Array(), $offset = 0, $limit = -1, $filter_fields = Array(), $id_field = 'id') {
+        $args = func_get_args();
+        return call_user_func_array(array($this, '_setup'), $args);
+    }
+    function _setup($file) {
+
+        if(isset($this->_popupMeta)){
+            if(isset($this->_popupMeta['create']['formBase'])) {
+                require_once('modules/' . $this->seed->module_dir . '/' . $this->_popupMeta['create']['formBase']);
+                $this->_create = true;
+            }
+        }
+        if(!empty($this->_popupMeta['create'])){
+            $formBase = new $this->_popupMeta['create']['formBaseClass']();
+            if(isset($_REQUEST['doAction']) && $_REQUEST['doAction'] == 'save')
+            {
+                //If it's a new record, set useRequired to false
+                $useRequired = empty($_REQUEST['id']) ? false : true;
+                $formBase->handleSave('', false, $useRequired);
+            }
+        }
+
+        $params = array();
+        if(!empty($this->_popupMeta['orderBy'])){
+            $params['orderBy'] = $this->_popupMeta['orderBy'];
+        }
+
+        if(file_exists('custom/modules/'.$this->module.'/metadata/metafiles.php')){
+            require('custom/modules/'.$this->module.'/metadata/metafiles.php');
+        }elseif(file_exists('modules/'.$this->module.'/metadata/metafiles.php')){
+            require('modules/'.$this->module.'/metadata/metafiles.php');
+        }
+
+        if(!empty($metafiles[$this->module]['searchfields'])) {
+            require($metafiles[$this->module]['searchfields']);
+        } elseif(file_exists('modules/'.$this->module.'/metadata/SearchFields.php')) {
+            require('modules/'.$this->module.'/metadata/SearchFields.php');
+        }
         $this->searchdefs[$this->module]['templateMeta']['maxColumns'] = 2;
         $this->searchdefs[$this->module]['templateMeta']['widths']['label'] = 10;
         $this->searchdefs[$this->module]['templateMeta']['widths']['field'] = 30;
 
         $this->searchForm->view = 'PopupSearchForm';
-		$this->searchForm->setup($this->searchdefs, $searchFields, 'SearchFormGenericAdvanced.tpl', 'advanced_search', $this->listviewdefs);
-
-		$lv = new ListViewSmarty();
-		$displayColumns = array();
-		if(!empty($_REQUEST['displayColumns'])) {
-		    foreach(explode('|', $_REQUEST['displayColumns']) as $num => $col) {
-		        if(!empty($listViewDefs[$this->module][$col]))
-		            $displayColumns[$col] = $this->listviewdefs[$this->module][$col];
-		    }
-		}
-		else {
-		    foreach($this->listviewdefs[$this->module] as $col => $para) {
-		        if(!empty($para['default']) && $para['default'])
-		            $displayColumns[$col] = $para;
-		    }
-		}
-		$params['massupdate'] = true;
-		if(!empty($_REQUEST['orderBy'])) {
-		    $params['orderBy'] = $_REQUEST['orderBy'];
-		    $params['overrideOrder'] = true;
-		    if(!empty($_REQUEST['sortOrder'])) $params['sortOrder'] = $_REQUEST['sortOrder'];
-		}
-
-		$lv->displayColumns = $displayColumns;
+        $this->searchForm->setup($this->searchdefs, $searchFields, 'SearchFormGenericAdvanced.tpl', 'advanced_search', $this->listviewdefs);
+
+        $lv = new ListViewSmarty();
+        $displayColumns = array();
+        if(!empty($_REQUEST['displayColumns'])) {
+            foreach(explode('|', $_REQUEST['displayColumns']) as $num => $col) {
+                if(!empty($listViewDefs[$this->module][$col]))
+                    $displayColumns[$col] = $this->listviewdefs[$this->module][$col];
+            }
+        }
+        else {
+            foreach($this->listviewdefs[$this->module] as $col => $para) {
+                if(!empty($para['default']) && $para['default'])
+                    $displayColumns[$col] = $para;
+            }
+        }
+        $params['massupdate'] = true;
+        if(!empty($_REQUEST['orderBy'])) {
+            $params['orderBy'] = $_REQUEST['orderBy'];
+            $params['overrideOrder'] = true;
+            if(!empty($_REQUEST['sortOrder'])) $params['sortOrder'] = $_REQUEST['sortOrder'];
+        }
+
+        $lv->displayColumns = $displayColumns;
         $this->searchForm->lv = $lv;
         $this->searchForm->displaySavedSearch = false;
 
@@ -351,25 +351,25 @@ 
         $this->should_process = true;
 
         if(isset($params['export'])) {
-          $this->export = $params['export'];
+            $this->export = $params['export'];
         }
         if(!empty($params['multiSelectPopup'])) {
-		  $this->multi_select_popup = $params['multiSelectPopup'];
+            $this->multi_select_popup = $params['multiSelectPopup'];
         }
-		if(!empty($params['massupdate']) && $params['massupdate'] != false) {
-			$this->show_mass_update_form = true;
-			$this->mass = new MassUpdate();
-			$this->mass->setSugarBean($this->seed);
-			if(!empty($params['handleMassupdate']) || !isset($params['handleMassupdate'])) {
+        if(!empty($params['massupdate']) && $params['massupdate'] != false) {
+            $this->show_mass_update_form = true;
+            $this->mass = new MassUpdate();
+            $this->mass->setSugarBean($this->seed);
+            if(!empty($params['handleMassupdate']) || !isset($params['handleMassupdate'])) {
                 $this->mass->handleMassUpdate();
             }
-		}
+        }
 
         // create filter fields based off of display columns
         if(empty($this->filter_fields) || $this->mergeDisplayColumns) {
             foreach($this->displayColumns as $columnName => $def) {
-               $this->filter_fields[strtolower($columnName)] = true;
-               if(!empty($def['related_fields'])) {
+                $this->filter_fields[strtolower($columnName)] = true;
+                if(!empty($def['related_fields'])) {
                     foreach($def['related_fields'] as $field) {
                         //id column is added by query construction function. This addition creates duplicates
                         //and causes issues in oracle. #10165
@@ -379,12 +379,12 @@ 
                     }
                 }
                 if (!empty($this->seed->field_defs[strtolower($columnName)]['db_concat_fields'])) {
-	                foreach($this->seed->field_defs[strtolower($columnName)]['db_concat_fields'] as $index=>$field){
-	                    if(!isset($this->filter_fields[strtolower($field)]) || !$this->filter_fields[strtolower($field)])
-	                    {
-	                        $this->filter_fields[strtolower($field)] = true;
-	                    }
-	                }
+                    foreach($this->seed->field_defs[strtolower($columnName)]['db_concat_fields'] as $index=>$field){
+                        if(!isset($this->filter_fields[strtolower($field)]) || !$this->filter_fields[strtolower($field)])
+                        {
+                            $this->filter_fields[strtolower($field)] = true;
+                        }
+                    }
                 }
             }
             foreach ($this->searchColumns as $columnName => $def )
@@ -418,119 +418,119 @@ 
         }
 
 
-		if (!empty($_REQUEST['query']) || (!empty($GLOBALS['sugar_config']['save_query']) && $GLOBALS['sugar_config']['save_query'] != 'populate_only')) {
-			$data = $this->lvd->getListViewData($this->seed, $searchWhere, 0, -1, $this->filter_fields, $params, 'id');
-		} else {
-			$this->should_process = false;
-			$data = array(
-				'data'=>array(),
-			    'pageData'=>array(
-			    	'bean'=>array('moduleDir'=>$this->seed->module_dir),
-					'ordering'=>'',
-					'offsets'=>array('total'=>0,'next'=>0,'current'=>0),
-				),
-			);
-		}
+        if (!empty($_REQUEST['query']) || (!empty($GLOBALS['sugar_config']['save_query']) && $GLOBALS['sugar_config']['save_query'] != 'populate_only')) {
+            $data = $this->lvd->getListViewData($this->seed, $searchWhere, 0, -1, $this->filter_fields, $params, 'id');
+        } else {
+            $this->should_process = false;
+            $data = array(
+                'data'=>array(),
+                'pageData'=>array(
+                    'bean'=>array('moduleDir'=>$this->seed->module_dir),
+                    'ordering'=>'',
+                    'offsets'=>array('total'=>0,'next'=>0,'current'=>0),
+                ),
+            );
+        }
 
         $this->fillDisplayColumnsWithVardefs();
 
-		$this->process($file, $data, $this->seed->object_name);
-	}
+        $this->process($file, $data, $this->seed->object_name);
+    }
 
-	/*
+    /*
 	 * Return the where clause as per the REQUEST.
 	 */
-	function _get_where_clause()
-	{
-		$where = '';
-		$where_clauses = $this->searchForm->generateSearchWhere(true, $this->seed->module_dir);
-
-		// Bug 43452 - FG - Changed the way generated Where array is imploded into the string.
-		//                  Now it's imploding in the same way view.list.php do.
-		if (count($where_clauses) > 0 ) {
-		    $where = '( ' . implode(' and ', $where_clauses) . ' )';
+    function _get_where_clause()
+    {
+        $where = '';
+        $where_clauses = $this->searchForm->generateSearchWhere(true, $this->seed->module_dir);
+
+        // Bug 43452 - FG - Changed the way generated Where array is imploded into the string.
+        //                  Now it's imploding in the same way view.list.php do.
+        if (count($where_clauses) > 0 ) {
+            $where = '( ' . implode(' and ', $where_clauses) . ' )';
         }
 
         // Need to include the default whereStatement
-		if(!empty($this->_popupMeta['whereStatement'])){
+        if(!empty($this->_popupMeta['whereStatement'])){
             if(!empty($where))$where .= ' AND ';
             $where .= $this->_popupMeta['whereStatement'];
-		}
+        }
 
-		return $where;
-	}
+        return $where;
+    }
 
-	/*
+    /*
 	 * Generate the data for the search form on the header of the Popup.
 	 */
-		function _build_field_defs(){
-		$this->formData = array();
-		$this->customFieldDefs = array();
-		foreach($this->searchdefs[$this->module]['layout']['advanced_search'] as $data){
-			if(is_array($data)){
-
-				$this->formData[] = array('field' => $data);
-				$value = '';
-				$this->customFieldDefs[$data['name']]= $data;
-				if(!empty($_REQUEST[$data['name']]))
-	            	$value = $_REQUEST[$data['name']];
-	            $this->customFieldDefs[$data['name']]['value'] = $value;
-			}else
-				$this->formData[] = array('field' => array('name'=>$data));
-		}
-		$this->fieldDefs = array();
-		if($this->seed){
-			$this->seed->fill_in_additional_detail_fields();
-
-	        foreach($this->seed->toArray() as $name => $value) {
-	            $this->fieldDefs[$name] = $this->seed->field_defs[$name];
-	            //if we have a relate type then reset to name so that we end up with a textbox
-	            //rather than a select button
-	            $this->fieldDefs[$name]['name'] = $this->fieldDefs[$name]['name'];
-	            if($this->fieldDefs[$name]['type'] == 'relate')
-	            	$this->fieldDefs[$name]['type'] = 'name';
-	            if(isset($this->fieldDefs[$name]['options']) && isset($GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']])) {
-	                $this->fieldDefs[$name]['options'] = $GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']]; // fill in enums
-	            }
-	            if(!empty($_REQUEST[$name]))
-	            	$value = $_REQUEST[$name];
-	            $this->fieldDefs[$name]['value'] = $value;
-	        }
-		}
-	}
-
-	function _getAddForm(){
-		$addform = '';
+        function _build_field_defs(){
+        $this->formData = array();
+        $this->customFieldDefs = array();
+        foreach($this->searchdefs[$this->module]['layout']['advanced_search'] as $data){
+            if(is_array($data)){
+
+                $this->formData[] = array('field' => $data);
+                $value = '';
+                $this->customFieldDefs[$data['name']]= $data;
+                if(!empty($_REQUEST[$data['name']]))
+                    $value = $_REQUEST[$data['name']];
+                $this->customFieldDefs[$data['name']]['value'] = $value;
+            }else
+                $this->formData[] = array('field' => array('name'=>$data));
+        }
+        $this->fieldDefs = array();
+        if($this->seed){
+            $this->seed->fill_in_additional_detail_fields();
+
+            foreach($this->seed->toArray() as $name => $value) {
+                $this->fieldDefs[$name] = $this->seed->field_defs[$name];
+                //if we have a relate type then reset to name so that we end up with a textbox
+                //rather than a select button
+                $this->fieldDefs[$name]['name'] = $this->fieldDefs[$name]['name'];
+                if($this->fieldDefs[$name]['type'] == 'relate')
+                    $this->fieldDefs[$name]['type'] = 'name';
+                if(isset($this->fieldDefs[$name]['options']) && isset($GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']])) {
+                    $this->fieldDefs[$name]['options'] = $GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']]; // fill in enums
+                }
+                if(!empty($_REQUEST[$name]))
+                    $value = $_REQUEST[$name];
+                $this->fieldDefs[$name]['value'] = $value;
+            }
+        }
+    }
+
+    function _getAddForm(){
+        $addform = '';
         if(!$this->seed->ACLAccess('save')){
             return;
         }
-		if(!empty($this->_popupMeta['create'])){
-			$formBase = new $this->_popupMeta['create']['formBaseClass']();
+        if(!empty($this->_popupMeta['create'])){
+            $formBase = new $this->_popupMeta['create']['formBaseClass']();
 
 
 
-				// TODO: cleanup the construction of $addform
-				$prefix = empty($this->_popupMeta['create']['getFormBodyParams'][0]) ? '' : $this->_popupMeta['create']['getFormBodyParams'][0];
-				$mod = empty($this->_popupMeta['create']['getFormBodyParams'][1]) ? '' : $this->_popupMeta['create']['getFormBodyParams'][1];
-				$formBody = empty($this->_popupMeta['create']['getFormBodyParams'][2]) ? '' : $this->_popupMeta['create']['getFormBodyParams'][2];
+                // TODO: cleanup the construction of $addform
+                $prefix = empty($this->_popupMeta['create']['getFormBodyParams'][0]) ? '' : $this->_popupMeta['create']['getFormBodyParams'][0];
+                $mod = empty($this->_popupMeta['create']['getFormBodyParams'][1]) ? '' : $this->_popupMeta['create']['getFormBodyParams'][1];
+                $formBody = empty($this->_popupMeta['create']['getFormBodyParams'][2]) ? '' : $this->_popupMeta['create']['getFormBodyParams'][2];
 
-				$getFormMethod = (empty($this->_popupMeta['create']['getFormMethod']) ? 'getFormBody' : $this->_popupMeta['create']['getFormMethod']);
-				$formbody = $formBase->$getFormMethod($prefix, $mod, $formBody);
+                $getFormMethod = (empty($this->_popupMeta['create']['getFormMethod']) ? 'getFormBody' : $this->_popupMeta['create']['getFormMethod']);
+                $formbody = $formBase->$getFormMethod($prefix, $mod, $formBody);
 
-				$addform = '<table><tr><td nowrap="nowrap" valign="top">'
-					. str_replace('<br>', '</td><td nowrap="nowrap" valign="top">&nbsp;', $formbody)
-					. '</td></tr></table>'
-					. '<input type="hidden" name="action" value="Popup" />';
+                $addform = '<table><tr><td nowrap="nowrap" valign="top">'
+                    . str_replace('<br>', '</td><td nowrap="nowrap" valign="top">&nbsp;', $formbody)
+                    . '</td></tr></table>'
+                    . '<input type="hidden" name="action" value="Popup" />';
 
-			return $addform;
-		}
-	}
+            return $addform;
+        }
+    }
 
-	function _getAddFormHeader(){
-		$lbl_save_button_title = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_TITLE'];
-		$lbl_save_button_key = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_KEY'];
-		$lbl_save_button_label = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_LABEL'];
-		$module_dir = $this->seed->module_dir;
+    function _getAddFormHeader(){
+        $lbl_save_button_title = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_TITLE'];
+        $lbl_save_button_key = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_KEY'];
+        $lbl_save_button_label = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_LABEL'];
+        $module_dir = $this->seed->module_dir;
 $formSave = <<<EOQ
 			<input type="hidden" name="create" value="true">
 			<input type="hidden" name="popup" value="true">
@@ -538,21 +538,21 @@ 
 			<input type="hidden" name="return_module" value="$module_dir">
 			<input type="hidden" name="return_action" value="Popup">
 EOQ;
-		// if metadata contains custom inputs for the quickcreate
-		if(!empty($this->_popupMeta['customInput']) && is_array($this->_popupMeta['customInput'])) {
-			foreach($this->_popupMeta['customInput'] as $key => $value)
-				$formSave .= '<input type="hidden" name="' . $key . '" value="'. $value .'">\n';
-		}
-
-
-		$addformheader = get_form_header(translate($this->_popupMeta['create']['createButton']), $formSave, false);
-		return $addformheader;
-	}
-
-	function getQuickCreate(){
-		require_once("include/EditView/PopupQuickCreate.php");
-		$qc = new PopupQuickCreate($this->module);
-		return $qc->process($this->module);
-	}
+        // if metadata contains custom inputs for the quickcreate
+        if(!empty($this->_popupMeta['customInput']) && is_array($this->_popupMeta['customInput'])) {
+            foreach($this->_popupMeta['customInput'] as $key => $value)
+                $formSave .= '<input type="hidden" name="' . $key . '" value="'. $value .'">\n';
+        }
+
+
+        $addformheader = get_form_header(translate($this->_popupMeta['create']['createButton']), $formSave, false);
+        return $addformheader;
+    }
+
+    function getQuickCreate(){
+        require_once("include/EditView/PopupQuickCreate.php");
+        $qc = new PopupQuickCreate($this->module);
+        return $qc->process($this->module);
+    }
 }
 ?>

Spacing +77 added lines, -77 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -43,7 +43,7 @@ 
 require_once('include/TemplateHandler/TemplateHandler.php');
 require_once('include/SearchForm/SearchForm2.php');
 define("NUM_COLS", 2);
-class PopupSmarty extends ListViewSmarty{
+class PopupSmarty extends ListViewSmarty {
 
 	var $contextMenus = false;
 	var $export = false;
@@ -70,7 +70,7 @@ 
     var $module;
     var $massUpdateData = '';
 
-	public function __construct($seed, $module){
+	public function __construct($seed, $module) {
 		parent::__construct();
 		$this->th = new TemplateHandler();
 		$this->th->loadSmarty();
@@ -91,9 +91,9 @@ 
      */
     function processArrowVars()
     {
-        $pathParts = pathinfo(SugarThemeRegistry::current()->getImageURL('arrow.gif',false));
+        $pathParts = pathinfo(SugarThemeRegistry::current()->getImageURL('arrow.gif', false));
 
-        list($width,$height) = getimagesize($pathParts['dirname'].'/'.$pathParts['basename']);
+        list($width, $height) = getimagesize($pathParts['dirname'].'/'.$pathParts['basename']);
 
         $this->th->ss->assign('arrowExt', $pathParts['extension']);
         $this->th->ss->assign('arrowWidth', $width);
@@ -119,16 +119,16 @@ 
 		$this->data = $data;
 
         $totalWidth = 0;
-        foreach($this->displayColumns as $name => $params) {
+        foreach ($this->displayColumns as $name => $params) {
             $totalWidth += $params['width'];
         }
         $adjustment = $totalWidth / 100;
 
         $contextMenuObjectsTypes = array();
-        foreach($this->displayColumns as $name => $params) {
+        foreach ($this->displayColumns as $name => $params) {
             $this->displayColumns[$name]['width'] = round($this->displayColumns[$name]['width'] / $adjustment, 2);
             // figure out which contextMenu objectsTypes are required
-            if(!empty($params['contextMenu']['objectType']))
+            if (!empty($params['contextMenu']['objectType']))
                 $contextMenuObjectsTypes[$params['contextMenu']['objectType']] = true;
         }
 		$this->th->ss->assign('displayColumns', $this->displayColumns);
@@ -147,22 +147,22 @@ 
 		$this->th->ss->assign('searchForm', $this->searchForm->display(false));
         //rrs
 
-		if($this->export) $this->th->ss->assign('exportLink', $this->buildExportLink());
+		if ($this->export) $this->th->ss->assign('exportLink', $this->buildExportLink());
 		$this->th->ss->assign('quickViewLinks', $this->quickViewLinks);
-		if($this->mailMerge) $this->th->ss->assign('mergeLink', $this->buildMergeLink()); // still check for mailmerge access
-		if($this->mergeduplicates) $this->th->ss->assign('mergedupLink', $this->buildMergeDuplicatesLink());
+		if ($this->mailMerge) $this->th->ss->assign('mergeLink', $this->buildMergeLink()); // still check for mailmerge access
+		if ($this->mergeduplicates) $this->th->ss->assign('mergedupLink', $this->buildMergeDuplicatesLink());
 
 
 		if (!empty($_REQUEST['mode']) && strtoupper($_REQUEST['mode']) == 'MULTISELECT') {
 			$this->multiSelect = true;
 		}
 		// handle save checks and stuff
-		if($this->multiSelect) {
+		if ($this->multiSelect) {
 			$this->th->ss->assign('selectedObjectsSpan', $this->buildSelectedObjectsSpan());
 			$this->th->ss->assign('multiSelectData', $this->getMultiSelectData());
 			$this->th->ss->assign('MODE', "<input type='hidden' name='mode' value='MultiSelect'>");
             $pageTotal = $this->data['pageData']['offsets']['next'] - $this->data['pageData']['offsets']['current'];
-            if($this->data['pageData']['offsets']['next'] < 0){ // If we are on the last page, 'next' is -1, which means we have to have a custom calculation
+            if ($this->data['pageData']['offsets']['next'] < 0) { // If we are on the last page, 'next' is -1, which means we have to have a custom calculation
                 $pageTotal = $this->data['pageData']['offsets']['total'] - $this->data['pageData']['offsets']['current'];
             }
     		$this->th->ss->assign('selectLink', $this->buildSelectLink('select_link', $this->data['pageData']['offsets']['total'], $pageTotal));
@@ -175,10 +175,10 @@ 
         $this->th->ss->assign('contextMenus', $this->contextMenus);
 
 
-        if($this->contextMenus && !empty($contextMenuObjectsTypes)) {
+        if ($this->contextMenus && !empty($contextMenuObjectsTypes)) {
             $script = '';
             $cm = new contextMenu();
-            foreach($contextMenuObjectsTypes as $type => $value) {
+            foreach ($contextMenuObjectsTypes as $type => $value) {
                 $cm->loadFromFile($type);
                 $script .= $cm->getScript();
                 $cm->menuItems = array(); // clear menuItems out
@@ -199,11 +199,11 @@ 
 	function display($end = true) {
         global $app_strings;
 
-        if(!is_file(sugar_cached("jsLanguage/{$GLOBALS['current_language']}.js"))) {
+        if (!is_file(sugar_cached("jsLanguage/{$GLOBALS['current_language']}.js"))) {
             require_once('include/language/jsLanguage.php');
             jsLanguage::createAppStringsCache($GLOBALS['current_language']);
         }
-        $jsLang = getVersionedScript("cache/jsLanguage/{$GLOBALS['current_language']}.js",  $GLOBALS['sugar_config']['js_lang_version']);
+        $jsLang = getVersionedScript("cache/jsLanguage/{$GLOBALS['current_language']}.js", $GLOBALS['sugar_config']['js_lang_version']);
 
         $this->th->ss->assign('data', $this->data['data']);
 		$this->data['pageData']['offsets']['lastOffsetOnPage'] = $this->data['pageData']['offsets']['current'] + count($this->data['data']);
@@ -223,10 +223,10 @@ 
 		$searchNameOverride = !empty($this->seed) && $this->seed instanceof Person && (isset($this->data['data'][0]['FIRST_NAME']) && isset($this->data['data'][0]['LAST_NAME'])) ? true : false;
 
 		global $locale;
-		foreach($this->data['data'] as $val)
+		foreach ($this->data['data'] as $val)
 		{
 			$associated_row_data[$val['ID']] = $val;
-			if($searchNameOverride)
+			if ($searchNameOverride)
 			{
 			   $associated_row_data[$val['ID']]['NAME'] = $locale->getLocaleFormattedName($val['FIRST_NAME'], $val['LAST_NAME']);
 			}
@@ -237,7 +237,7 @@ 
 		$this->th->ss->assign('lang', substr($GLOBALS['current_language'], 0, 2));
         $this->th->ss->assign('headerTpl', $this->headerTpl);
         $this->th->ss->assign('footerTpl', $this->footerTpl);
-        $this->th->ss->assign('ASSOCIATED_JAVASCRIPT_DATA', 'var associated_javascript_data = '.$json->encode($associated_row_data). '; var is_show_fullname = '.$is_show_fullname.';');
+        $this->th->ss->assign('ASSOCIATED_JAVASCRIPT_DATA', 'var associated_javascript_data = '.$json->encode($associated_row_data).'; var is_show_fullname = '.$is_show_fullname.';');
 		$this->th->ss->assign('module', $this->seed->module_dir);
 		$request_data = empty($_REQUEST['request_data']) ? '' : $_REQUEST['request_data'];
 
@@ -258,8 +258,8 @@ 
 		$this->th->ss->assign('sugarVersion', $GLOBALS['sugar_version']);
         $this->th->ss->assign('should_process', $this->should_process);
 
-		if($this->_create){
-			$this->th->ss->assign('ADDFORM', $this->getQuickCreate());//$this->_getAddForm());
+		if ($this->_create) {
+			$this->th->ss->assign('ADDFORM', $this->getQuickCreate()); //$this->_getAddForm());
 			$this->th->ss->assign('ADDFORMHEADER', $this->_getAddFormHeader());
 			$this->th->ss->assign('object_name', $this->seed->object_name);
 		}
@@ -278,15 +278,15 @@ 
 	}
 	function _setup($file) {
 
-	    if(isset($this->_popupMeta)){
-			if(isset($this->_popupMeta['create']['formBase'])) {
-				require_once('modules/' . $this->seed->module_dir . '/' . $this->_popupMeta['create']['formBase']);
+	    if (isset($this->_popupMeta)) {
+			if (isset($this->_popupMeta['create']['formBase'])) {
+				require_once('modules/'.$this->seed->module_dir.'/'.$this->_popupMeta['create']['formBase']);
 				$this->_create = true;
 			}
 		}
-	    if(!empty($this->_popupMeta['create'])){
+	    if (!empty($this->_popupMeta['create'])) {
 			$formBase = new $this->_popupMeta['create']['formBaseClass']();
-			if(isset($_REQUEST['doAction']) && $_REQUEST['doAction'] == 'save')
+			if (isset($_REQUEST['doAction']) && $_REQUEST['doAction'] == 'save')
 			{
 				//If it's a new record, set useRequired to false
 				$useRequired = empty($_REQUEST['id']) ? false : true;
@@ -295,19 +295,19 @@ 
 		}
 
 		$params = array();
-		if(!empty($this->_popupMeta['orderBy'])){
+		if (!empty($this->_popupMeta['orderBy'])) {
 			$params['orderBy'] = $this->_popupMeta['orderBy'];
 		}
 
-		if(file_exists('custom/modules/'.$this->module.'/metadata/metafiles.php')){
+		if (file_exists('custom/modules/'.$this->module.'/metadata/metafiles.php')) {
 			require('custom/modules/'.$this->module.'/metadata/metafiles.php');
-		}elseif(file_exists('modules/'.$this->module.'/metadata/metafiles.php')){
+		}elseif (file_exists('modules/'.$this->module.'/metadata/metafiles.php')) {
 			require('modules/'.$this->module.'/metadata/metafiles.php');
 		}
 
-		if(!empty($metafiles[$this->module]['searchfields'])) {
+		if (!empty($metafiles[$this->module]['searchfields'])) {
 			require($metafiles[$this->module]['searchfields']);
-		} elseif(file_exists('modules/'.$this->module.'/metadata/SearchFields.php')) {
+		} elseif (file_exists('modules/'.$this->module.'/metadata/SearchFields.php')) {
 			require('modules/'.$this->module.'/metadata/SearchFields.php');
 	    }
         $this->searchdefs[$this->module]['templateMeta']['maxColumns'] = 2;
@@ -319,23 +319,23 @@ 
 
 		$lv = new ListViewSmarty();
 		$displayColumns = array();
-		if(!empty($_REQUEST['displayColumns'])) {
-		    foreach(explode('|', $_REQUEST['displayColumns']) as $num => $col) {
-		        if(!empty($listViewDefs[$this->module][$col]))
+		if (!empty($_REQUEST['displayColumns'])) {
+		    foreach (explode('|', $_REQUEST['displayColumns']) as $num => $col) {
+		        if (!empty($listViewDefs[$this->module][$col]))
 		            $displayColumns[$col] = $this->listviewdefs[$this->module][$col];
 		    }
 		}
 		else {
-		    foreach($this->listviewdefs[$this->module] as $col => $para) {
-		        if(!empty($para['default']) && $para['default'])
+		    foreach ($this->listviewdefs[$this->module] as $col => $para) {
+		        if (!empty($para['default']) && $para['default'])
 		            $displayColumns[$col] = $para;
 		    }
 		}
 		$params['massupdate'] = true;
-		if(!empty($_REQUEST['orderBy'])) {
+		if (!empty($_REQUEST['orderBy'])) {
 		    $params['orderBy'] = $_REQUEST['orderBy'];
 		    $params['overrideOrder'] = true;
-		    if(!empty($_REQUEST['sortOrder'])) $params['sortOrder'] = $_REQUEST['sortOrder'];
+		    if (!empty($_REQUEST['sortOrder'])) $params['sortOrder'] = $_REQUEST['sortOrder'];
 		}
 
 		$lv->displayColumns = $displayColumns;
@@ -350,27 +350,27 @@ 
 
         $this->should_process = true;
 
-        if(isset($params['export'])) {
+        if (isset($params['export'])) {
           $this->export = $params['export'];
         }
-        if(!empty($params['multiSelectPopup'])) {
+        if (!empty($params['multiSelectPopup'])) {
 		  $this->multi_select_popup = $params['multiSelectPopup'];
         }
-		if(!empty($params['massupdate']) && $params['massupdate'] != false) {
+		if (!empty($params['massupdate']) && $params['massupdate'] != false) {
 			$this->show_mass_update_form = true;
 			$this->mass = new MassUpdate();
 			$this->mass->setSugarBean($this->seed);
-			if(!empty($params['handleMassupdate']) || !isset($params['handleMassupdate'])) {
+			if (!empty($params['handleMassupdate']) || !isset($params['handleMassupdate'])) {
                 $this->mass->handleMassUpdate();
             }
 		}
 
         // create filter fields based off of display columns
-        if(empty($this->filter_fields) || $this->mergeDisplayColumns) {
-            foreach($this->displayColumns as $columnName => $def) {
+        if (empty($this->filter_fields) || $this->mergeDisplayColumns) {
+            foreach ($this->displayColumns as $columnName => $def) {
                $this->filter_fields[strtolower($columnName)] = true;
-               if(!empty($def['related_fields'])) {
-                    foreach($def['related_fields'] as $field) {
+               if (!empty($def['related_fields'])) {
+                    foreach ($def['related_fields'] as $field) {
                         //id column is added by query construction function. This addition creates duplicates
                         //and causes issues in oracle. #10165
                         if ($field != 'id') {
@@ -379,15 +379,15 @@ 
                     }
                 }
                 if (!empty($this->seed->field_defs[strtolower($columnName)]['db_concat_fields'])) {
-	                foreach($this->seed->field_defs[strtolower($columnName)]['db_concat_fields'] as $index=>$field){
-	                    if(!isset($this->filter_fields[strtolower($field)]) || !$this->filter_fields[strtolower($field)])
+	                foreach ($this->seed->field_defs[strtolower($columnName)]['db_concat_fields'] as $index=>$field) {
+	                    if (!isset($this->filter_fields[strtolower($field)]) || !$this->filter_fields[strtolower($field)])
 	                    {
 	                        $this->filter_fields[strtolower($field)] = true;
 	                    }
 	                }
                 }
             }
-            foreach ($this->searchColumns as $columnName => $def )
+            foreach ($this->searchColumns as $columnName => $def)
             {
                 $this->filter_fields[strtolower($columnName)] = true;
             }
@@ -403,13 +403,13 @@ 
          * By default bean's create_new_list_query function loads fields displayed on the page or used in the search
          * add fields used to populate forms from _viewdefs :: field_to_name_array to retrive from db
          */
-        if ( isset($_REQUEST['field_to_name']) && $_REQUEST['field_to_name'] )
+        if (isset($_REQUEST['field_to_name']) && $_REQUEST['field_to_name'])
         {
             $_REQUEST['field_to_name'] = is_array($_REQUEST['field_to_name']) ? $_REQUEST['field_to_name'] : array($_REQUEST['field_to_name']);
-            foreach ( $_REQUEST['field_to_name'] as $add_field )
+            foreach ($_REQUEST['field_to_name'] as $add_field)
             {
                 $add_field = strtolower($add_field);
-                if ( $add_field != 'id' && !isset($this->filter_fields[$add_field]) && isset($this->seed->field_defs[$add_field]) )
+                if ($add_field != 'id' && !isset($this->filter_fields[$add_field]) && isset($this->seed->field_defs[$add_field]))
                 {
                     $this->filter_fields[$add_field] = true;
                 }
@@ -427,7 +427,7 @@ 
 			    'pageData'=>array(
 			    	'bean'=>array('moduleDir'=>$this->seed->module_dir),
 					'ordering'=>'',
-					'offsets'=>array('total'=>0,'next'=>0,'current'=>0),
+					'offsets'=>array('total'=>0, 'next'=>0, 'current'=>0),
 				),
 			);
 		}
@@ -447,13 +447,13 @@ 
 
 		// Bug 43452 - FG - Changed the way generated Where array is imploded into the string.
 		//                  Now it's imploding in the same way view.list.php do.
-		if (count($where_clauses) > 0 ) {
-		    $where = '( ' . implode(' and ', $where_clauses) . ' )';
+		if (count($where_clauses) > 0) {
+		    $where = '( '.implode(' and ', $where_clauses).' )';
         }
 
         // Need to include the default whereStatement
-		if(!empty($this->_popupMeta['whereStatement'])){
-            if(!empty($where))$where .= ' AND ';
+		if (!empty($this->_popupMeta['whereStatement'])) {
+            if (!empty($where))$where .= ' AND ';
             $where .= $this->_popupMeta['whereStatement'];
 		}
 
@@ -463,48 +463,48 @@ 
 	/*
 	 * Generate the data for the search form on the header of the Popup.
 	 */
-		function _build_field_defs(){
+		function _build_field_defs() {
 		$this->formData = array();
 		$this->customFieldDefs = array();
-		foreach($this->searchdefs[$this->module]['layout']['advanced_search'] as $data){
-			if(is_array($data)){
+		foreach ($this->searchdefs[$this->module]['layout']['advanced_search'] as $data) {
+			if (is_array($data)) {
 
 				$this->formData[] = array('field' => $data);
 				$value = '';
-				$this->customFieldDefs[$data['name']]= $data;
-				if(!empty($_REQUEST[$data['name']]))
+				$this->customFieldDefs[$data['name']] = $data;
+				if (!empty($_REQUEST[$data['name']]))
 	            	$value = $_REQUEST[$data['name']];
 	            $this->customFieldDefs[$data['name']]['value'] = $value;
-			}else
+			} else
 				$this->formData[] = array('field' => array('name'=>$data));
 		}
 		$this->fieldDefs = array();
-		if($this->seed){
+		if ($this->seed) {
 			$this->seed->fill_in_additional_detail_fields();
 
-	        foreach($this->seed->toArray() as $name => $value) {
+	        foreach ($this->seed->toArray() as $name => $value) {
 	            $this->fieldDefs[$name] = $this->seed->field_defs[$name];
 	            //if we have a relate type then reset to name so that we end up with a textbox
 	            //rather than a select button
 	            $this->fieldDefs[$name]['name'] = $this->fieldDefs[$name]['name'];
-	            if($this->fieldDefs[$name]['type'] == 'relate')
+	            if ($this->fieldDefs[$name]['type'] == 'relate')
 	            	$this->fieldDefs[$name]['type'] = 'name';
-	            if(isset($this->fieldDefs[$name]['options']) && isset($GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']])) {
+	            if (isset($this->fieldDefs[$name]['options']) && isset($GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']])) {
 	                $this->fieldDefs[$name]['options'] = $GLOBALS['app_list_strings'][$this->fieldDefs[$name]['options']]; // fill in enums
 	            }
-	            if(!empty($_REQUEST[$name]))
+	            if (!empty($_REQUEST[$name]))
 	            	$value = $_REQUEST[$name];
 	            $this->fieldDefs[$name]['value'] = $value;
 	        }
 		}
 	}
 
-	function _getAddForm(){
+	function _getAddForm() {
 		$addform = '';
-        if(!$this->seed->ACLAccess('save')){
+        if (!$this->seed->ACLAccess('save')) {
             return;
         }
-		if(!empty($this->_popupMeta['create'])){
+		if (!empty($this->_popupMeta['create'])) {
 			$formBase = new $this->_popupMeta['create']['formBaseClass']();
 
 
@@ -526,7 +526,7 @@ 
 		}
 	}
 
-	function _getAddFormHeader(){
+	function _getAddFormHeader() {
 		$lbl_save_button_title = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_TITLE'];
 		$lbl_save_button_key = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_KEY'];
 		$lbl_save_button_label = $GLOBALS['app_strings']['LBL_SAVE_BUTTON_LABEL'];
@@ -539,9 +539,9 @@ 
 			<input type="hidden" name="return_action" value="Popup">
 EOQ;
 		// if metadata contains custom inputs for the quickcreate
-		if(!empty($this->_popupMeta['customInput']) && is_array($this->_popupMeta['customInput'])) {
-			foreach($this->_popupMeta['customInput'] as $key => $value)
-				$formSave .= '<input type="hidden" name="' . $key . '" value="'. $value .'">\n';
+		if (!empty($this->_popupMeta['customInput']) && is_array($this->_popupMeta['customInput'])) {
+			foreach ($this->_popupMeta['customInput'] as $key => $value)
+				$formSave .= '<input type="hidden" name="'.$key.'" value="'.$value.'">\n';
 		}
 
 
@@ -549,7 +549,7 @@ 
 		return $addformheader;
 	}
 
-	function getQuickCreate(){
+	function getQuickCreate() {
 		require_once("include/EditView/PopupQuickCreate.php");
 		$qc = new PopupQuickCreate($this->module);
 		return $qc->process($this->module);

include/language/jsLanguage.php

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -62,7 +62,7 @@ 
 EOQ;
         
         $cacheDir = create_cache_directory('jsLanguage/');
-        if($fh = @sugar_fopen($cacheDir . $lang . '.js', "w")){
+        if ($fh = @sugar_fopen($cacheDir.$lang.'.js', "w")) {
             fputs($fh, $str);
             fclose($fh);
         }
@@ -74,16 +74,16 @@ 
         // cn: bug 8242 - non-US langpack chokes
         $mod_strings = return_module_language($lang, $moduleDir);
         $mod_strings_encoded = $json->encode($mod_strings);
-        $str = "SUGAR.language.setLanguage('" . $moduleDir . "', " . $mod_strings_encoded . ");";
+        $str = "SUGAR.language.setLanguage('".$moduleDir."', ".$mod_strings_encoded.");";
         
-        $cacheDir = create_cache_directory('jsLanguage/' . $moduleDir . '/');
+        $cacheDir = create_cache_directory('jsLanguage/'.$moduleDir.'/');
         
-        if($fh = @fopen($cacheDir . $lang . '.js', "w")){
+        if ($fh = @fopen($cacheDir.$lang.'.js', "w")) {
             fputs($fh, $str);
             fclose($fh);
         }
 
-        if($return) {
+        if ($return) {
             return $str;
         }
     }

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

include/SubPanel/SubPanelSearchDisplay.php

Indentation +3 added lines, -3 removed lines

@@ -46,12 +46,12 @@ 
 
 if(empty($_REQUEST['module']))
 {
-	die("'module' was not defined");
+    die("'module' was not defined");
 }
 
 if(!isset($beanList[$_REQUEST['module']]))
 {
-	die("'".$_REQUEST['module']."' is not defined in \$beanList");
+    die("'".$_REQUEST['module']."' is not defined in \$beanList");
 }
 
 if (!isset($_REQUEST['subpanel'])) {
@@ -67,7 +67,7 @@ 
 include('include/SubPanel/SubPanel.php');
 $layout_def_key = '';
 if(!empty($_REQUEST['layout_def_key'])){
-	$layout_def_key = $_REQUEST['layout_def_key'];
+    $layout_def_key = $_REQUEST['layout_def_key'];
 }
 
 $subpanel_object = new CustomSubPanel($module, $record, $subpanel,null, $layout_def_key, $collection);

Spacing +6 added lines, -6 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -38,18 +38,18 @@ 
  * display the words  "Powered by SugarCRM" and "Supercharged by SuiteCRM".
  ********************************************************************************/
 
-ini_set('display_errors',1);
+ini_set('display_errors', 1);
 
 global $beanList;
 global $beanFiles;
 
 
-if(empty($_REQUEST['module']))
+if (empty($_REQUEST['module']))
 {
 	die("'module' was not defined");
 }
 
-if(!isset($beanList[$_REQUEST['module']]))
+if (!isset($beanList[$_REQUEST['module']]))
 {
 	die("'".$_REQUEST['module']."' is not defined in \$beanList");
 }
@@ -66,11 +66,11 @@ 
 
 include('include/SubPanel/SubPanel.php');
 $layout_def_key = '';
-if(!empty($_REQUEST['layout_def_key'])){
+if (!empty($_REQUEST['layout_def_key'])) {
 	$layout_def_key = $_REQUEST['layout_def_key'];
 }
 
-$subpanel_object = new CustomSubPanel($module, $record, $subpanel,null, $layout_def_key, $collection);
+$subpanel_object = new CustomSubPanel($module, $record, $subpanel, null, $layout_def_key, $collection);
 
 echo $subpanel_object->getSearchForm();
 

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

include/SubPanel/SubPanelDefinitions.php

Braces +60 added lines, -39 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if (! defined ( 'sugarEntry' ) || ! sugarEntry) die ( 'Not A Valid Entry Point' ) ;
+if (! defined ( 'sugarEntry' ) || ! sugarEntry) {
+    die ( 'Not A Valid Entry Point' ) ;
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -85,7 +87,9 @@ 
 
 		$this->_instance_properties = $instance_properties ;
 
-		if(isset($instance_properties['collection_list' ])) $this->base_collection_list = $instance_properties['collection_list' ];
+		if(isset($instance_properties['collection_list' ])) {
+		    $this->base_collection_list = $instance_properties['collection_list' ];
+		}
 
 		if(!empty($collections) && isset($instance_properties['collection_list' ])){
 			foreach($instance_properties['collection_list' ] as $cname => $value){
@@ -277,7 +281,9 @@ 
 				}
 			}
 			// if it's empty just dump out as there is nothing to process.
-			if(empty($this->sub_subpanels)) return false;
+			if(empty($this->sub_subpanels)) {
+			    return false;
+			}
 			//Sync displayed list fields across the subpanels
 			$display_fields = $this->getDisplayFieldsFromCollection($this->sub_subpanels);
 			$query_fields = array();
@@ -291,10 +297,11 @@ 
 					if (isset($def['vname']) && isset($def['width']))
 					{
 						$index++;
-						if(!empty($def['alias']))
-							$listFieldMap[$key][$def['alias']] = $field;
-						else
-							$listFieldMap[$key][$field] = $field;
+						if(!empty($def['alias'])) {
+													$listFieldMap[$key][$def['alias']] = $field;
+						} else {
+													$listFieldMap[$key][$field] = $field;
+						}
 						if (!isset($display_fields[$def['vname']]))
 						{
 							if(sizeof($display_fields) > $index)
@@ -332,13 +339,11 @@ 
 					if (isset($subpanel->panel_definition['list_fields'][$field]))
 					{
 						$list_fields[$field] = $subpanel->panel_definition['list_fields'][$field];
-					}
-					else if ($list_key != $field && isset($subpanel->panel_definition['list_fields'][$list_key]))
+					} else if ($list_key != $field && isset($subpanel->panel_definition['list_fields'][$list_key]))
 					{
 						$list_fields[$list_key] = $subpanel->panel_definition['list_fields'][$list_key];
 
-					}
-					else {
+					} else {
 						$list_fields[$field] = $display_fields[$vname];
 					}
 				}
@@ -347,8 +352,7 @@ 
 					if (isset($subpanel->panel_definition['list_fields'][$field]))
 					{
 						$list_fields[$field] = $subpanel->panel_definition['list_fields'][$field];
-					}
-					else {
+					} else {
 						$list_fields[$field] = $def;
 					}
 				}
@@ -624,18 +628,22 @@ 
 		global $modListHeader ;
 		global $modules_exempt_from_availability_check ;
 
-		if (isset ( $this->_visible_tabs_array ))
-			return $this->_visible_tabs_array ;
+		if (isset ( $this->_visible_tabs_array )) {
+					return $this->_visible_tabs_array ;
+		}
 
-		if (empty($modListHeader))
-		    $modListHeader = query_module_access_list($GLOBALS['current_user']);
+		if (empty($modListHeader)) {
+				    $modListHeader = query_module_access_list($GLOBALS['current_user']);
+		}
 
 		$this->_visible_tabs_array = array ( ) ; // bug 16820 - make sure this is an array for the later ksort
 
-		if (isset ( $this->layout_defs [ 'subpanel_setup' ] )) // bug 17434 - belts-and-braces - check that we have some subpanels first
+		if (isset ( $this->layout_defs [ 'subpanel_setup' ] )) {
+		    // bug 17434 - belts-and-braces - check that we have some subpanels first
 		{
 			//retrieve list of hidden subpanels
 			$hidden_panels = $this->get_hidden_subpanels();
+		}
 
 			//activities is a special use case in that if it is hidden,
 			//then the history tab should be hidden too.
@@ -648,8 +656,9 @@ 
 			{
 				//exclude if this subpanel is hidden from admin screens
                 $module = $key;
-                if ( isset($values_array['module']) )
-                    $module = strtolower($values_array['module']);
+                if ( isset($values_array['module']) ) {
+                                    $module = strtolower($values_array['module']);
+                }
 				 if ($hidden_panels && is_array($hidden_panels) && (in_array($module, $hidden_panels) || array_key_exists($module, $hidden_panels)) ){
 				 	//this panel is hidden, skip it
 				 	continue;
@@ -696,8 +705,9 @@ 
 	 */
 	function load_subpanel ( $name , $reload = false , $original_only = false, $search_query = '', $collections = array() )
 	{
-		if (!is_dir('modules/' . $this->layout_defs [ 'subpanel_setup' ][ strtolower ( $name ) ] [ 'module' ]))
-			return false;
+		if (!is_dir('modules/' . $this->layout_defs [ 'subpanel_setup' ][ strtolower ( $name ) ] [ 'module' ])) {
+					return false;
+		}
 
 		$subpanel = new aSubPanel ( $name, $this->layout_defs [ 'subpanel_setup' ] [ strtolower ( $name ) ], $this->_focus, $reload, $original_only, $search_query, $collections ) ;
 
@@ -720,16 +730,19 @@ 
 
 		if (empty ( $this->layout_defs ) || $reload || (! empty ( $layout_def_key ) && ! isset ( $layout_defs [ $layout_def_key ] )))
 		{
-			if (file_exists ( 'modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php' ))
-				require ('modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php') ;
+			if (file_exists ( 'modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php' )) {
+							require ('modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php') ;
+			}
 
-			if (! $original_only && file_exists ( 'custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php' ))
-				require ('custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php') ;
+			if (! $original_only && file_exists ( 'custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php' )) {
+							require ('custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php') ;
+			}
 
-			if (! empty ( $layout_def_key ))
-				$this->layout_defs = $layout_defs [ $layout_def_key ] ;
-			else
-				$this->layout_defs = $layout_defs [ $this->_focus->module_dir ] ;
+			if (! empty ( $layout_def_key )) {
+							$this->layout_defs = $layout_defs [ $layout_def_key ] ;
+			} else {
+							$this->layout_defs = $layout_defs [ $this->_focus->module_dir ] ;
+			}
 
 		}
 
@@ -785,11 +798,15 @@ 
 		foreach($modules_to_check as $mod_name){
 
 			//skip if module name is not in bean list, otherwise get the bean class name
-			if(!isset($beanList[$mod_name])) continue;
+			if(!isset($beanList[$mod_name])) {
+			    continue;
+			}
 			$class = $beanList[$mod_name];
 
 			//skip if class name is not in file list, otherwise require the bean file and create new class
-			if(!isset($beanFiles[$class]) || !file_exists($beanFiles[$class])) continue;
+			if(!isset($beanFiles[$class]) || !file_exists($beanFiles[$class])) {
+			    continue;
+			}
 
 			//retrieve subpanels for this bean
 			require_once($beanFiles[$class]);
@@ -803,15 +820,20 @@ 
 			foreach( $sub_tabs as $panel_key){
 				$panel_key = strtolower($panel_key);
                 $panel_module = $panel_key;
-                if ( isset($spd->layout_defs['subpanel_setup'][$panel_key]['module']) )
-                    $panel_module = strtolower($spd->layout_defs['subpanel_setup'][$panel_key]['module']);
+                if ( isset($spd->layout_defs['subpanel_setup'][$panel_key]['module']) ) {
+                                    $panel_module = strtolower($spd->layout_defs['subpanel_setup'][$panel_key]['module']);
+                }
                 //if module_only flag is set, only if it is also in module array
-				if($return_tab_modules_only && !array_key_exists($panel_module, $modules_to_check)) continue;
+				if($return_tab_modules_only && !array_key_exists($panel_module, $modules_to_check)) {
+				    continue;
+				}
 				$panel_key_name = $panel_module;
 
 				//group_by_key_name is set to true, then array will hold an entry for each
 				//subpanel, with the module name prepended in the key
-				if($group_by_module) $panel_key_name = $class.'_'.$panel_key_name;
+				if($group_by_module) {
+				    $panel_key_name = $class.'_'.$panel_key_name;
+				}
 				//add panel name to subpanel array
 				$spd_arr[$panel_key_name] = $panel_module;
 			}
@@ -864,12 +886,11 @@ 
 					}
 
 
-				}else{
+				} else{
 					//no settings found, return empty
 					return $hidden_subpanels;
 				}
-			}
-			else
+			} else
 			{	//no settings found, return empty
 				return $hidden_subpanels;
 			}

Doc Comments +8 added lines

@@ -417,11 +417,19 @@
 	}
 
 	//get value of a property defined at the panel instance level.
+
+	/**
+	 * @param string $name
+	 */
 	function get_inst_prop_value ( $name )
 	{
 		return isset($this->_instance_properties[$name]) ? $this->_instance_properties [ $name ] : null;
 	}
 	//get value of a property defined at the panel definition level.
+
+	/**
+	 * @param string $name
+	 */
 	function get_def_prop_value ( $name )
 	{
 		if (isset ( $this->panel_definition [ $name ] ))

Indentation +709 added lines, -709 removed lines

@@ -54,13 +54,13 @@ 
 class aSubPanel
 {
 
-	var $name ;
-	var $_instance_properties ;
+    var $name ;
+    var $_instance_properties ;
 
-	var $mod_strings ;
-	var $panel_definition ;
-	var $sub_subpanels ;
-	var $parent_bean ;
+    var $mod_strings ;
+    var $panel_definition ;
+    var $sub_subpanels ;
+    var $parent_bean ;
 
     /**
      * Can we display this subpanel?
@@ -72,80 +72,80 @@ 
      */
     var $canDisplay = true;
 
-	//module's table name and column fields.
-	var $table_name ;
-	var $db_fields ;
-	var $bean_name ;
-	var $template_instance ;
-
-	var $search_query;
-	var $base_collection_list = array();
-
-	function __construct ($name , $instance_properties , $parent_bean , $reload = false , $original_only = false, $search_query = '', $collections = array() ){
-
-		$this->_instance_properties = $instance_properties ;
-
-		if(isset($instance_properties['collection_list' ])) $this->base_collection_list = $instance_properties['collection_list' ];
-
-		if(!empty($collections) && isset($instance_properties['collection_list' ])){
-			foreach($instance_properties['collection_list' ] as $cname => $value){
-				if(!in_array($value['module'], $collections)){
-					unset($instance_properties['collection_list'][$cname]);
-				}
-			}
-		}
-		if (!$this->isCollection()){
-			$table = strtolower($instance_properties['module']);
-			$search_query = str_replace('meetings',$table,$search_query);
-		}
-
-		$this->search_query = $search_query;
-		$this->name = $name ;
-		$this->parent_bean = $parent_bean ;
-
-		//set language
-		global $current_language ;
-		if (! isset ( $parent_bean->mbvardefs ))
-		{
-			$mod_strings = return_module_language ( $current_language, $parent_bean->module_dir ) ;
-		}
-		$this->mod_strings = $mod_strings ;
+    //module's table name and column fields.
+    var $table_name ;
+    var $db_fields ;
+    var $bean_name ;
+    var $template_instance ;
+
+    var $search_query;
+    var $base_collection_list = array();
+
+    function __construct ($name , $instance_properties , $parent_bean , $reload = false , $original_only = false, $search_query = '', $collections = array() ){
+
+        $this->_instance_properties = $instance_properties ;
+
+        if(isset($instance_properties['collection_list' ])) $this->base_collection_list = $instance_properties['collection_list' ];
+
+        if(!empty($collections) && isset($instance_properties['collection_list' ])){
+            foreach($instance_properties['collection_list' ] as $cname => $value){
+                if(!in_array($value['module'], $collections)){
+                    unset($instance_properties['collection_list'][$cname]);
+                }
+            }
+        }
+        if (!$this->isCollection()){
+            $table = strtolower($instance_properties['module']);
+            $search_query = str_replace('meetings',$table,$search_query);
+        }
+
+        $this->search_query = $search_query;
+        $this->name = $name ;
+        $this->parent_bean = $parent_bean ;
+
+        //set language
+        global $current_language ;
+        if (! isset ( $parent_bean->mbvardefs ))
+        {
+            $mod_strings = return_module_language ( $current_language, $parent_bean->module_dir ) ;
+        }
+        $this->mod_strings = $mod_strings ;
 
         if ($this->isCollection ())
-		{
-			$this->canDisplay = $this->load_sub_subpanels () ; //load sub-panel definition.
-		} else
-		{
-			if (!is_dir('modules/' . $this->_instance_properties [ 'module' ])){
-				_pstack_trace();
-			}
-			$def_path = 'modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'subpanel_name' ] . '.php' ;
-
-			$orig_exists = is_file($def_path);
-			$loaded = false;
-			if ($orig_exists)
-			{
-				require ($def_path);
-				$loaded = true;
-			}
-			if (is_file("custom/$def_path") && (!$original_only  || !$orig_exists))
-			{
-				require ("custom/$def_path");
-				$loaded = true;
-			}
-
-			if (! $original_only && isset ( $this->_instance_properties [ 'override_subpanel_name' ] ) && file_exists ( 'custom/modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'override_subpanel_name' ] . '.php' ))
-			{
-				$cust_def_path = 'custom/modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'override_subpanel_name' ] . '.php' ;
-				require ($cust_def_path) ;
-				$loaded = true;
-			}
-
-			if (!$loaded)
-			{
-				$GLOBALS['log']->fatal("Failed to load original or custom subpanel data for $name in $def_path");
+        {
+            $this->canDisplay = $this->load_sub_subpanels () ; //load sub-panel definition.
+        } else
+        {
+            if (!is_dir('modules/' . $this->_instance_properties [ 'module' ])){
+                _pstack_trace();
+            }
+            $def_path = 'modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'subpanel_name' ] . '.php' ;
+
+            $orig_exists = is_file($def_path);
+            $loaded = false;
+            if ($orig_exists)
+            {
+                require ($def_path);
+                $loaded = true;
+            }
+            if (is_file("custom/$def_path") && (!$original_only  || !$orig_exists))
+            {
+                require ("custom/$def_path");
+                $loaded = true;
+            }
+
+            if (! $original_only && isset ( $this->_instance_properties [ 'override_subpanel_name' ] ) && file_exists ( 'custom/modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'override_subpanel_name' ] . '.php' ))
+            {
+                $cust_def_path = 'custom/modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'override_subpanel_name' ] . '.php' ;
+                require ($cust_def_path) ;
+                $loaded = true;
+            }
+
+            if (!$loaded)
+            {
+                $GLOBALS['log']->fatal("Failed to load original or custom subpanel data for $name in $def_path");
                 $this->canDisplay = false;
-			}
+            }
 
             // load module info from the module's bean file
             $this->load_module_info();
@@ -156,10 +156,10 @@ 
             if (isset($subpanel_layout) && is_array($subpanel_layout)) {
                 $this->set_panel_definition($subpanel_layout);
 
-			}
-		}
+            }
+        }
 
-	}
+    }
 
     /**
      * is the sub panel default hidden?
@@ -176,64 +176,64 @@ 
     }
 
 
-	function distinct_query ()
-	{
-		if (isset ( $this->_instance_properties [ 'get_distinct_data' ] ))
-		{
-			return !empty($this->_instance_properties['get_distinct_data']) ? true : false;
-		}
-		return false ;
-	}
-
-	//return the translated header value.
-	function get_title ()
-	{
-		if (empty ( $this->mod_strings [ $this->_instance_properties [ 'title_key' ] ] ))
-		{
-			return translate ( $this->_instance_properties [ 'title_key' ], $this->_instance_properties [ 'module' ] ) ;
-		}
-		return $this->mod_strings [ $this->_instance_properties [ 'title_key' ] ] ;
-	}
-
-	//return the definition of buttons. looks for buttons in 2 locations.
-	function get_buttons ()
-	{
-		$buttons = array ( ) ;
-		if (isset ( $this->_instance_properties [ 'top_buttons' ] ))
-		{
-			//this will happen only in the case of sub-panels with multiple sources(activities).
-			$buttons = $this->_instance_properties [ 'top_buttons' ] ;
-		} else
-		{
-			$buttons = $this->panel_definition [ 'top_buttons' ] ;
-		}
-
-		// permissions. hide SubPanelTopComposeEmailButton from activities if email module is disabled.
-		//only email is  being tested becuase other submodules in activites/history such as notes, tasks, meetings and calls cannot be disabled.
-		//as of today these are the only 2 sub-panels that use the union clause.
-		$mod_name = $this->get_module_name () ;
-		if ($mod_name == 'Activities' || $mod_name == 'History')
-		{
-			global $modListHeader ;
-			global $modules_exempt_from_availability_check ;
-			if (isset ( $modListHeader ) && (! (array_key_exists ( 'Emails', $modListHeader ) or array_key_exists ( 'Emails', $modules_exempt_from_availability_check ))))
-			{
-				foreach ( $buttons as $key => $button )
-				{
-					foreach ( $button as $property => $value )
-					{
-						if ($value === 'SubPanelTopComposeEmailButton' || $value === 'SubPanelTopArchiveEmailButton')
-						{
-							//remove this button from the array.
-							unset ( $buttons [ $key ] ) ;
-						}
-					}
-				}
-			}
-		}
-
-		return $buttons ;
-	}
+    function distinct_query ()
+    {
+        if (isset ( $this->_instance_properties [ 'get_distinct_data' ] ))
+        {
+            return !empty($this->_instance_properties['get_distinct_data']) ? true : false;
+        }
+        return false ;
+    }
+
+    //return the translated header value.
+    function get_title ()
+    {
+        if (empty ( $this->mod_strings [ $this->_instance_properties [ 'title_key' ] ] ))
+        {
+            return translate ( $this->_instance_properties [ 'title_key' ], $this->_instance_properties [ 'module' ] ) ;
+        }
+        return $this->mod_strings [ $this->_instance_properties [ 'title_key' ] ] ;
+    }
+
+    //return the definition of buttons. looks for buttons in 2 locations.
+    function get_buttons ()
+    {
+        $buttons = array ( ) ;
+        if (isset ( $this->_instance_properties [ 'top_buttons' ] ))
+        {
+            //this will happen only in the case of sub-panels with multiple sources(activities).
+            $buttons = $this->_instance_properties [ 'top_buttons' ] ;
+        } else
+        {
+            $buttons = $this->panel_definition [ 'top_buttons' ] ;
+        }
+
+        // permissions. hide SubPanelTopComposeEmailButton from activities if email module is disabled.
+        //only email is  being tested becuase other submodules in activites/history such as notes, tasks, meetings and calls cannot be disabled.
+        //as of today these are the only 2 sub-panels that use the union clause.
+        $mod_name = $this->get_module_name () ;
+        if ($mod_name == 'Activities' || $mod_name == 'History')
+        {
+            global $modListHeader ;
+            global $modules_exempt_from_availability_check ;
+            if (isset ( $modListHeader ) && (! (array_key_exists ( 'Emails', $modListHeader ) or array_key_exists ( 'Emails', $modules_exempt_from_availability_check ))))
+            {
+                foreach ( $buttons as $key => $button )
+                {
+                    foreach ( $button as $property => $value )
+                    {
+                        if ($value === 'SubPanelTopComposeEmailButton' || $value === 'SubPanelTopArchiveEmailButton')
+                        {
+                            //remove this button from the array.
+                            unset ( $buttons [ $key ] ) ;
+                        }
+                    }
+                }
+            }
+        }
+
+        return $buttons ;
+    }
 
 
     /**
@@ -244,327 +244,327 @@ 
      * @return bool         True by default if the subpanel was loaded.  Will return false if none in the collection are
      *                      allowed by the current user.
      */
-	function load_sub_subpanels ()
-	{
-
-		global $modListHeader ;
-		// added a check for security of tabs to see if an user has access to them
-		// this prevents passing an "unseen" tab to the query string and pulling up its contents
-		if (! isset ( $modListHeader ))
-		{
-			global $current_user ;
-			if (isset ( $current_user ))
-			{
-				$modListHeader = query_module_access_list ( $current_user ) ;
-			}
-		}
-
-		//by default all the activities modules are exempt, so hiding them won't affect their appearance unless the 'activity' subpanel itself is hidden.
-		//add email to the list temporarily so it is not affected in activities subpanel
-		global $modules_exempt_from_availability_check ;
-		$modules_exempt_from_availability_check['Emails'] = 'Emails';
-
-		$listFieldMap = array();
-
-		if (empty ( $this->sub_subpanels ))
-		{
-			$panels = $this->get_inst_prop_value ( 'collection_list' ) ;
-			foreach ( $panels as $panel => $properties )
-			{
-				if (array_key_exists ( $properties [ 'module' ], $modListHeader ) or array_key_exists ( $properties [ 'module' ], $modules_exempt_from_availability_check ))
-				{
-					$this->sub_subpanels [ $panel ] = new aSubPanel ( $panel, $properties, $this->parent_bean, false, false, $this->search_query ) ;
-				}
-			}
-			// if it's empty just dump out as there is nothing to process.
-			if(empty($this->sub_subpanels)) return false;
-			//Sync displayed list fields across the subpanels
-			$display_fields = $this->getDisplayFieldsFromCollection($this->sub_subpanels);
-			$query_fields = array();
-			foreach ( $this->sub_subpanels as $key => $subpanel )
-			{
-				$list_fields = $subpanel->get_list_fields();
-				$listFieldMap[$key] = array();
-				$index = 0;
-				foreach($list_fields as $field => $def)
-				{
-					if (isset($def['vname']) && isset($def['width']))
-					{
-						$index++;
-						if(!empty($def['alias']))
-							$listFieldMap[$key][$def['alias']] = $field;
-						else
-							$listFieldMap[$key][$field] = $field;
-						if (!isset($display_fields[$def['vname']]))
-						{
-							if(sizeof($display_fields) > $index)
-							{
-								//Try to insert the new field in an order that makes sense
-								$start = array_slice($display_fields, 0, $index);
-								$end = array_slice($display_fields, $index);
-								$display_fields = array_merge(
-									$start,
-									array($def['vname'] => array('name' => $field, 'vname' => $def['vname'], 'width' => $def['width'] )),
-									$end
-								);
-							} else
-							{
-								$display_fields[$def['vname']] = array(
-									'name' => empty($def['alias']) ? $field : $def['alias'],
-									'vname' => $def['vname'],
-									'width' => $def['width'],
-								);
-							}
-						}
-					} else {
-						$query_fields[$field] = $def;
-					}
-				}
-			}
-			foreach ( $this->sub_subpanels as $key => $subpanel )
-			{
-				$list_fields = array();
-				foreach($display_fields as $vname => $def)
-				{
-					$field = $def['name'];
-					$list_key = isset($listFieldMap[$key][$field]) ? $listFieldMap[$key][$field] : $field;
-
-					if (isset($subpanel->panel_definition['list_fields'][$field]))
-					{
-						$list_fields[$field] = $subpanel->panel_definition['list_fields'][$field];
-					}
-					else if ($list_key != $field && isset($subpanel->panel_definition['list_fields'][$list_key]))
-					{
-						$list_fields[$list_key] = $subpanel->panel_definition['list_fields'][$list_key];
-
-					}
-					else {
-						$list_fields[$field] = $display_fields[$vname];
-					}
-				}
-				foreach($query_fields as $field => $def)
-				{
-					if (isset($subpanel->panel_definition['list_fields'][$field]))
-					{
-						$list_fields[$field] = $subpanel->panel_definition['list_fields'][$field];
-					}
-					else {
-						$list_fields[$field] = $def;
-					}
-				}
-				$subpanel->panel_definition['list_fields'] = $list_fields;
-			}
-		}
-
-		return true;
-	}
-
-	protected function getDisplayFieldsFromCollection($sub_subpanels)
-	{
-		$display_fields = array();
-		foreach ($sub_subpanels as $key => $subpanel )
-		{
-			$list_fields = $subpanel->get_list_fields();
-			$index = 0;
-			foreach($list_fields as $field => $def)
-			{
-				if (isset($def['vname']) && isset($def['width']))
-				{
-					$index++;
-					if (!isset($display_fields[$def['vname']]))
-					{
-						if(sizeof($display_fields) > $index)
-						{
-							//Try to insert the new field in an order that makes sense
-							$start = array_slice($display_fields, 0, $index);
-							$end = array_slice($display_fields, $index);
-							$display_fields = array_merge(
-								$start,
-								array($def['vname'] => array('name' => $field, 'vname' => $def['vname'], 'width' => $def['width'] )),
-								$end
-							);
-						} else
-						{
-							$display_fields[$def['vname']] = array(
-								'name' => $field,
-								'vname' => $def['vname'],
-								'width' => $def['width'],
-							);
-						}
-					}
-				}
-			}
-		}
-	}
-
-	function isDatasourceFunction ()
-	{
-		if (strpos ( $this->get_inst_prop_value ( 'get_subpanel_data' ), 'function' ) === false)
-		{
-			return false ;
-		}
-		return true ;
-	}
+    function load_sub_subpanels ()
+    {
+
+        global $modListHeader ;
+        // added a check for security of tabs to see if an user has access to them
+        // this prevents passing an "unseen" tab to the query string and pulling up its contents
+        if (! isset ( $modListHeader ))
+        {
+            global $current_user ;
+            if (isset ( $current_user ))
+            {
+                $modListHeader = query_module_access_list ( $current_user ) ;
+            }
+        }
+
+        //by default all the activities modules are exempt, so hiding them won't affect their appearance unless the 'activity' subpanel itself is hidden.
+        //add email to the list temporarily so it is not affected in activities subpanel
+        global $modules_exempt_from_availability_check ;
+        $modules_exempt_from_availability_check['Emails'] = 'Emails';
+
+        $listFieldMap = array();
+
+        if (empty ( $this->sub_subpanels ))
+        {
+            $panels = $this->get_inst_prop_value ( 'collection_list' ) ;
+            foreach ( $panels as $panel => $properties )
+            {
+                if (array_key_exists ( $properties [ 'module' ], $modListHeader ) or array_key_exists ( $properties [ 'module' ], $modules_exempt_from_availability_check ))
+                {
+                    $this->sub_subpanels [ $panel ] = new aSubPanel ( $panel, $properties, $this->parent_bean, false, false, $this->search_query ) ;
+                }
+            }
+            // if it's empty just dump out as there is nothing to process.
+            if(empty($this->sub_subpanels)) return false;
+            //Sync displayed list fields across the subpanels
+            $display_fields = $this->getDisplayFieldsFromCollection($this->sub_subpanels);
+            $query_fields = array();
+            foreach ( $this->sub_subpanels as $key => $subpanel )
+            {
+                $list_fields = $subpanel->get_list_fields();
+                $listFieldMap[$key] = array();
+                $index = 0;
+                foreach($list_fields as $field => $def)
+                {
+                    if (isset($def['vname']) && isset($def['width']))
+                    {
+                        $index++;
+                        if(!empty($def['alias']))
+                            $listFieldMap[$key][$def['alias']] = $field;
+                        else
+                            $listFieldMap[$key][$field] = $field;
+                        if (!isset($display_fields[$def['vname']]))
+                        {
+                            if(sizeof($display_fields) > $index)
+                            {
+                                //Try to insert the new field in an order that makes sense
+                                $start = array_slice($display_fields, 0, $index);
+                                $end = array_slice($display_fields, $index);
+                                $display_fields = array_merge(
+                                    $start,
+                                    array($def['vname'] => array('name' => $field, 'vname' => $def['vname'], 'width' => $def['width'] )),
+                                    $end
+                                );
+                            } else
+                            {
+                                $display_fields[$def['vname']] = array(
+                                    'name' => empty($def['alias']) ? $field : $def['alias'],
+                                    'vname' => $def['vname'],
+                                    'width' => $def['width'],
+                                );
+                            }
+                        }
+                    } else {
+                        $query_fields[$field] = $def;
+                    }
+                }
+            }
+            foreach ( $this->sub_subpanels as $key => $subpanel )
+            {
+                $list_fields = array();
+                foreach($display_fields as $vname => $def)
+                {
+                    $field = $def['name'];
+                    $list_key = isset($listFieldMap[$key][$field]) ? $listFieldMap[$key][$field] : $field;
+
+                    if (isset($subpanel->panel_definition['list_fields'][$field]))
+                    {
+                        $list_fields[$field] = $subpanel->panel_definition['list_fields'][$field];
+                    }
+                    else if ($list_key != $field && isset($subpanel->panel_definition['list_fields'][$list_key]))
+                    {
+                        $list_fields[$list_key] = $subpanel->panel_definition['list_fields'][$list_key];
+
+                    }
+                    else {
+                        $list_fields[$field] = $display_fields[$vname];
+                    }
+                }
+                foreach($query_fields as $field => $def)
+                {
+                    if (isset($subpanel->panel_definition['list_fields'][$field]))
+                    {
+                        $list_fields[$field] = $subpanel->panel_definition['list_fields'][$field];
+                    }
+                    else {
+                        $list_fields[$field] = $def;
+                    }
+                }
+                $subpanel->panel_definition['list_fields'] = $list_fields;
+            }
+        }
+
+        return true;
+    }
+
+    protected function getDisplayFieldsFromCollection($sub_subpanels)
+    {
+        $display_fields = array();
+        foreach ($sub_subpanels as $key => $subpanel )
+        {
+            $list_fields = $subpanel->get_list_fields();
+            $index = 0;
+            foreach($list_fields as $field => $def)
+            {
+                if (isset($def['vname']) && isset($def['width']))
+                {
+                    $index++;
+                    if (!isset($display_fields[$def['vname']]))
+                    {
+                        if(sizeof($display_fields) > $index)
+                        {
+                            //Try to insert the new field in an order that makes sense
+                            $start = array_slice($display_fields, 0, $index);
+                            $end = array_slice($display_fields, $index);
+                            $display_fields = array_merge(
+                                $start,
+                                array($def['vname'] => array('name' => $field, 'vname' => $def['vname'], 'width' => $def['width'] )),
+                                $end
+                            );
+                        } else
+                        {
+                            $display_fields[$def['vname']] = array(
+                                'name' => $field,
+                                'vname' => $def['vname'],
+                                'width' => $def['width'],
+                            );
+                        }
+                    }
+                }
+            }
+        }
+    }
+
+    function isDatasourceFunction ()
+    {
+        if (strpos ( $this->get_inst_prop_value ( 'get_subpanel_data' ), 'function' ) === false)
+        {
+            return false ;
+        }
+        return true ;
+    }
 
     /**
      * Test to see if the sub panels defs contain a collection
      *
      * @return bool
      */
-	function isCollection ()
-	{
-		return ($this->get_inst_prop_value ( 'type' ) == 'collection');
-	}
-
-	//get value of a property defined at the panel instance level.
-	function get_inst_prop_value ( $name )
-	{
-		return isset($this->_instance_properties[$name]) ? $this->_instance_properties [ $name ] : null;
-	}
-	//get value of a property defined at the panel definition level.
-	function get_def_prop_value ( $name )
-	{
-		if (isset ( $this->panel_definition [ $name ] ))
-		{
-			return $this->panel_definition [ $name ] ;
-		} else
-		{
-			return null ;
-		}
-	}
-
-	//if datasource is of the type function then return the function name
-	//else return the value as is.
-	function get_function_parameters ()
-	{
-		$parameters = array ( ) ;
-		if ($this->isDatasourceFunction ())
-		{
-			$parameters = $this->get_inst_prop_value ( 'function_parameters' ) ;
-		}
-		return $parameters ;
-	}
-
-	function get_data_source_name ( $check_set_subpanel_data = false )
-	{
-		$prop_value = null ;
-		if ($check_set_subpanel_data)
-		{
-			$prop_value = $this->get_inst_prop_value ( 'set_subpanel_data' ) ;
-		}
-		if (! empty ( $prop_value ))
-		{
-			return $prop_value ;
-		} else
-		{
-			//fall back to default behavior.
-		}
-		if ($this->isDatasourceFunction ())
-		{
-			return (substr_replace ( $this->get_inst_prop_value ( 'get_subpanel_data' ), '', 0, 9 )) ;
-		} else
-		{
-			return $this->get_inst_prop_value ( 'get_subpanel_data' ) ;
-		}
-	}
-
-	//returns the where clause for the query.
-	function get_where ()
-	{
-		if($this->get_def_prop_value ( 'where' ) != '' && $this->search_query != ''){
-			return $this->get_def_prop_value ( 'where' ).' AND '.$this->search_query;
-		} else if($this->search_query != ''){
-			return $this->search_query;
-		}
-		return $this->get_def_prop_value ( 'where' ) ;
-	}
-
-	function is_fill_in_additional_fields ()
-	{
-		// do both. inst_prop returns values from metadata/subpaneldefs.php and def_prop returns from subpanel/default.php
-		$temp = $this->get_inst_prop_value ( 'fill_in_additional_fields' ) || $this->get_def_prop_value ( 'fill_in_additional_fields' ) ;
-		return $temp ;
-	}
-
-	function get_list_fields ()
-	{
-		if (isset ( $this->panel_definition [ 'list_fields' ] ))
-		{
-			return $this->panel_definition [ 'list_fields' ] ;
-		} else
-		{
-			return array ( ) ;
-		}
-	}
-
-	function get_module_name ()
-	{
-		return $this->get_inst_prop_value ( 'module' ) ;
-	}
-
-	function get_name ()
-	{
-		return $this->name ;
-	}
-
-	//load subpanel module's table name and column fields.
-	function load_module_info ()
-	{
-		global $beanList ;
-		global $beanFiles ;
-
-		$module_name = $this->get_module_name () ;
-		if (! empty ( $module_name ))
-		{
-
-			$bean_name = $beanList [ $this->get_module_name () ] ;
-
-			$this->bean_name = $bean_name ;
-
-			include_once ($beanFiles [ $bean_name ]) ;
-			$this->template_instance = new $bean_name ( ) ;
-			$this->template_instance->force_load_details = true ;
-			$this->table_name = $this->template_instance->table_name ;
-			//$this->db_fields=$this->template_instance->column_fields;
-		}
-	}
-	//this function is to be used only with sub-panels that are based
-	//on collections.
-	function get_header_panel_def ()
-	{
-		if (! empty ( $this->sub_subpanels ))
-		{
-			if (! empty ( $this->_instance_properties [ 'header_definition_from_subpanel' ] ) && ! empty ( $this->sub_subpanels [ $this->_instance_properties [ 'header_definition_from_subpanel' ] ] ))
-			{
-				return $this->sub_subpanels [ $this->_instance_properties [ 'header_definition_from_subpanel' ] ] ;
-			} else
-			{
-				$display_fields = array();
-				//If we are not pulling from a specific subpanel, create a list of all list fields and use that.
-				foreach($this->sub_subpanels as $subpanel)
-				{
-					$list_fields = $subpanel->get_list_fields();
-					foreach($list_fields as $field => $def)
-					{
-
-					}
-				}
-
-				reset ( $this->sub_subpanels ) ;
-				return current ( $this->sub_subpanels ) ;
-			}
-		}
-		return null ;
-	}
-
-	/**
-	 * Returns an array of current properties of the class.
-	 * It will simply give the class name for instances of classes.
-	 */
-	function _to_array ()
-	{
-		return array ( '_instance_properties' => $this->_instance_properties , 'db_fields' => $this->db_fields , 'mod_strings' => $this->mod_strings , 'name' => $this->name , 'panel_definition' => $this->panel_definition , 'parent_bean' => get_class ( $this->parent_bean ) , 'sub_subpanels' => $this->sub_subpanels , 'table_name' => $this->table_name , 'template_instance' => get_class ( $this->template_instance ) ) ;
-	}
+    function isCollection ()
+    {
+        return ($this->get_inst_prop_value ( 'type' ) == 'collection');
+    }
+
+    //get value of a property defined at the panel instance level.
+    function get_inst_prop_value ( $name )
+    {
+        return isset($this->_instance_properties[$name]) ? $this->_instance_properties [ $name ] : null;
+    }
+    //get value of a property defined at the panel definition level.
+    function get_def_prop_value ( $name )
+    {
+        if (isset ( $this->panel_definition [ $name ] ))
+        {
+            return $this->panel_definition [ $name ] ;
+        } else
+        {
+            return null ;
+        }
+    }
+
+    //if datasource is of the type function then return the function name
+    //else return the value as is.
+    function get_function_parameters ()
+    {
+        $parameters = array ( ) ;
+        if ($this->isDatasourceFunction ())
+        {
+            $parameters = $this->get_inst_prop_value ( 'function_parameters' ) ;
+        }
+        return $parameters ;
+    }
+
+    function get_data_source_name ( $check_set_subpanel_data = false )
+    {
+        $prop_value = null ;
+        if ($check_set_subpanel_data)
+        {
+            $prop_value = $this->get_inst_prop_value ( 'set_subpanel_data' ) ;
+        }
+        if (! empty ( $prop_value ))
+        {
+            return $prop_value ;
+        } else
+        {
+            //fall back to default behavior.
+        }
+        if ($this->isDatasourceFunction ())
+        {
+            return (substr_replace ( $this->get_inst_prop_value ( 'get_subpanel_data' ), '', 0, 9 )) ;
+        } else
+        {
+            return $this->get_inst_prop_value ( 'get_subpanel_data' ) ;
+        }
+    }
+
+    //returns the where clause for the query.
+    function get_where ()
+    {
+        if($this->get_def_prop_value ( 'where' ) != '' && $this->search_query != ''){
+            return $this->get_def_prop_value ( 'where' ).' AND '.$this->search_query;
+        } else if($this->search_query != ''){
+            return $this->search_query;
+        }
+        return $this->get_def_prop_value ( 'where' ) ;
+    }
+
+    function is_fill_in_additional_fields ()
+    {
+        // do both. inst_prop returns values from metadata/subpaneldefs.php and def_prop returns from subpanel/default.php
+        $temp = $this->get_inst_prop_value ( 'fill_in_additional_fields' ) || $this->get_def_prop_value ( 'fill_in_additional_fields' ) ;
+        return $temp ;
+    }
+
+    function get_list_fields ()
+    {
+        if (isset ( $this->panel_definition [ 'list_fields' ] ))
+        {
+            return $this->panel_definition [ 'list_fields' ] ;
+        } else
+        {
+            return array ( ) ;
+        }
+    }
+
+    function get_module_name ()
+    {
+        return $this->get_inst_prop_value ( 'module' ) ;
+    }
+
+    function get_name ()
+    {
+        return $this->name ;
+    }
+
+    //load subpanel module's table name and column fields.
+    function load_module_info ()
+    {
+        global $beanList ;
+        global $beanFiles ;
+
+        $module_name = $this->get_module_name () ;
+        if (! empty ( $module_name ))
+        {
+
+            $bean_name = $beanList [ $this->get_module_name () ] ;
+
+            $this->bean_name = $bean_name ;
+
+            include_once ($beanFiles [ $bean_name ]) ;
+            $this->template_instance = new $bean_name ( ) ;
+            $this->template_instance->force_load_details = true ;
+            $this->table_name = $this->template_instance->table_name ;
+            //$this->db_fields=$this->template_instance->column_fields;
+        }
+    }
+    //this function is to be used only with sub-panels that are based
+    //on collections.
+    function get_header_panel_def ()
+    {
+        if (! empty ( $this->sub_subpanels ))
+        {
+            if (! empty ( $this->_instance_properties [ 'header_definition_from_subpanel' ] ) && ! empty ( $this->sub_subpanels [ $this->_instance_properties [ 'header_definition_from_subpanel' ] ] ))
+            {
+                return $this->sub_subpanels [ $this->_instance_properties [ 'header_definition_from_subpanel' ] ] ;
+            } else
+            {
+                $display_fields = array();
+                //If we are not pulling from a specific subpanel, create a list of all list fields and use that.
+                foreach($this->sub_subpanels as $subpanel)
+                {
+                    $list_fields = $subpanel->get_list_fields();
+                    foreach($list_fields as $field => $def)
+                    {
+
+                    }
+                }
+
+                reset ( $this->sub_subpanels ) ;
+                return current ( $this->sub_subpanels ) ;
+            }
+        }
+        return null ;
+    }
+
+    /**
+     * Returns an array of current properties of the class.
+     * It will simply give the class name for instances of classes.
+     */
+    function _to_array ()
+    {
+        return array ( '_instance_properties' => $this->_instance_properties , 'db_fields' => $this->db_fields , 'mod_strings' => $this->mod_strings , 'name' => $this->name , 'panel_definition' => $this->panel_definition , 'parent_bean' => get_class ( $this->parent_bean ) , 'sub_subpanels' => $this->sub_subpanels , 'table_name' => $this->table_name , 'template_instance' => get_class ( $this->template_instance ) ) ;
+    }
 
     /**
      * Sets definition of the subpanel
@@ -582,301 +582,301 @@ 
 class SubPanelDefinitions
 {
 
-	var $_focus ;
-	var $_visible_tabs_array ;
-	var $panels ;
-	var $layout_defs ;
-
-	/**
-	 * Enter description here...
-	 *
-	 * @param BEAN $focus - this is the bean you want to get the data from
-	 * @param STRING $layout_def_key - if you wish to use a layout_def defined in the default metadata/subpaneldefs.php that is not keyed off of $bean->module_dir pass in the key here
-	 * @param ARRAY $layout_def_override - if you wish to override the default loaded layout defs you pass them in here.
-	 * @return SubPanelDefinitions
-	 */
-	function __construct ( $focus , $layout_def_key = '' , $layout_def_override = '' )
-	{
-		$this->_focus = $focus ;
-		if (! empty ( $layout_def_override ))
-		{
-			$this->layout_defs = $layout_def_override ;
-
-		} else
-		{
-			$this->open_layout_defs ( false, $layout_def_key ) ;
-		}
-	}
-
-	/**
-	 * This function returns an ordered list of all "tabs", actually subpanels, for this module
-	 * The source list is obtained from the subpanel layout contained in the layout_defs for this module,
-	 * found either in the modules metadata/subpaneldefs.php file, or in the modules custom/.../Ext/Layoutdefs/layoutdefs.ext.php file
-	 * and filtered through an ACL check.
-	 * Note that the keys for the resulting array of tabs are in practice the name of the underlying source relationship for the subpanel
-	 * So for example, the key for a custom module's subpanel with Accounts might be 'one_one_accounts', as generated by the Studio Relationship Editor
-	 * Although OOB module subpanels have keys such as 'accounts', which might on the face of it appear to be a reference to the related module, in fact 'accounts' is still the relationship name
-	 * @param boolean 	Optional - include the subpanel title label in the return array (false)
-	 * @return array	All tabs that pass an ACL check
-	 */
-	function get_available_tabs ($FromGetModuleSubpanels=false)
-	{
-		global $modListHeader ;
-		global $modules_exempt_from_availability_check ;
-
-		if (isset ( $this->_visible_tabs_array ))
-			return $this->_visible_tabs_array ;
-
-		if (empty($modListHeader))
-		    $modListHeader = query_module_access_list($GLOBALS['current_user']);
-
-		$this->_visible_tabs_array = array ( ) ; // bug 16820 - make sure this is an array for the later ksort
-
-		if (isset ( $this->layout_defs [ 'subpanel_setup' ] )) // bug 17434 - belts-and-braces - check that we have some subpanels first
-		{
-			//retrieve list of hidden subpanels
-			$hidden_panels = $this->get_hidden_subpanels();
-
-			//activities is a special use case in that if it is hidden,
-			//then the history tab should be hidden too.
-			if(!empty($hidden_panels) && is_array($hidden_panels) && in_array('activities',$hidden_panels)){
-				//add history to list hidden_panels
-				$hidden_panels['history'] = 'history';
-			}
-
-			foreach ( $this->layout_defs [ 'subpanel_setup' ] as $key => $values_array )
-			{
-				//exclude if this subpanel is hidden from admin screens
+    var $_focus ;
+    var $_visible_tabs_array ;
+    var $panels ;
+    var $layout_defs ;
+
+    /**
+     * Enter description here...
+     *
+     * @param BEAN $focus - this is the bean you want to get the data from
+     * @param STRING $layout_def_key - if you wish to use a layout_def defined in the default metadata/subpaneldefs.php that is not keyed off of $bean->module_dir pass in the key here
+     * @param ARRAY $layout_def_override - if you wish to override the default loaded layout defs you pass them in here.
+     * @return SubPanelDefinitions
+     */
+    function __construct ( $focus , $layout_def_key = '' , $layout_def_override = '' )
+    {
+        $this->_focus = $focus ;
+        if (! empty ( $layout_def_override ))
+        {
+            $this->layout_defs = $layout_def_override ;
+
+        } else
+        {
+            $this->open_layout_defs ( false, $layout_def_key ) ;
+        }
+    }
+
+    /**
+     * This function returns an ordered list of all "tabs", actually subpanels, for this module
+     * The source list is obtained from the subpanel layout contained in the layout_defs for this module,
+     * found either in the modules metadata/subpaneldefs.php file, or in the modules custom/.../Ext/Layoutdefs/layoutdefs.ext.php file
+     * and filtered through an ACL check.
+     * Note that the keys for the resulting array of tabs are in practice the name of the underlying source relationship for the subpanel
+     * So for example, the key for a custom module's subpanel with Accounts might be 'one_one_accounts', as generated by the Studio Relationship Editor
+     * Although OOB module subpanels have keys such as 'accounts', which might on the face of it appear to be a reference to the related module, in fact 'accounts' is still the relationship name
+     * @param boolean 	Optional - include the subpanel title label in the return array (false)
+     * @return array	All tabs that pass an ACL check
+     */
+    function get_available_tabs ($FromGetModuleSubpanels=false)
+    {
+        global $modListHeader ;
+        global $modules_exempt_from_availability_check ;
+
+        if (isset ( $this->_visible_tabs_array ))
+            return $this->_visible_tabs_array ;
+
+        if (empty($modListHeader))
+            $modListHeader = query_module_access_list($GLOBALS['current_user']);
+
+        $this->_visible_tabs_array = array ( ) ; // bug 16820 - make sure this is an array for the later ksort
+
+        if (isset ( $this->layout_defs [ 'subpanel_setup' ] )) // bug 17434 - belts-and-braces - check that we have some subpanels first
+        {
+            //retrieve list of hidden subpanels
+            $hidden_panels = $this->get_hidden_subpanels();
+
+            //activities is a special use case in that if it is hidden,
+            //then the history tab should be hidden too.
+            if(!empty($hidden_panels) && is_array($hidden_panels) && in_array('activities',$hidden_panels)){
+                //add history to list hidden_panels
+                $hidden_panels['history'] = 'history';
+            }
+
+            foreach ( $this->layout_defs [ 'subpanel_setup' ] as $key => $values_array )
+            {
+                //exclude if this subpanel is hidden from admin screens
                 $module = $key;
                 if ( isset($values_array['module']) )
                     $module = strtolower($values_array['module']);
-				 if ($hidden_panels && is_array($hidden_panels) && (in_array($module, $hidden_panels) || array_key_exists($module, $hidden_panels)) ){
-				 	//this panel is hidden, skip it
-				 	continue;
-				 }
-
-				// make sure the module attribute is set, else none of this works...
-				if ( !isset($values_array [ 'module' ])) {
-					$GLOBALS['log']->debug("SubPanelDefinitions->get_available_tabs(): no module defined in subpaneldefs for '$key' =>" . var_export($values_array,true) . " - ingoring subpanel defintion") ;
-					continue;
-				}
-
-				//check permissions.
-				$exempt = array_key_exists ( $values_array [ 'module' ], $modules_exempt_from_availability_check ) ;
-				$ok = $exempt || ( (! ACLController::moduleSupportsACL ( $values_array [ 'module' ] ) || ACLController::checkAccess ( $values_array [ 'module' ], 'list', true ) ) ) ;
-
-				$GLOBALS [ 'log' ]->debug ( "SubPanelDefinitions->get_available_tabs(): " . $key . "= " . ( $exempt ? "exempt " : "not exempt " .( $ok ? " ACL OK" : "" ) ) ) ;
-
-				if ( $ok )
-				{
-					while ( ! empty ( $this->_visible_tabs_array [ $values_array [ 'order' ] ] ) )
-					{
-						$values_array [ 'order' ] ++ ;
-					}
-
-					$this->_visible_tabs_array [ $values_array ['order'] ] = ($FromGetModuleSubpanels) ? array($key=>$values_array['title_key']) : $key ;
-				}
-			}
-		}
-
-		ksort ( $this->_visible_tabs_array ) ;
-		return $this->_visible_tabs_array ;
-	}
-
-	/**
-	 * Load the definition of the a sub-panel.
-	 * Also the sub-panel is added to an array of sub-panels.
-	 * use of reload has been deprecated, since the subpanel is initialized every time.
+                    if ($hidden_panels && is_array($hidden_panels) && (in_array($module, $hidden_panels) || array_key_exists($module, $hidden_panels)) ){
+                        //this panel is hidden, skip it
+                        continue;
+                    }
+
+                // make sure the module attribute is set, else none of this works...
+                if ( !isset($values_array [ 'module' ])) {
+                    $GLOBALS['log']->debug("SubPanelDefinitions->get_available_tabs(): no module defined in subpaneldefs for '$key' =>" . var_export($values_array,true) . " - ingoring subpanel defintion") ;
+                    continue;
+                }
+
+                //check permissions.
+                $exempt = array_key_exists ( $values_array [ 'module' ], $modules_exempt_from_availability_check ) ;
+                $ok = $exempt || ( (! ACLController::moduleSupportsACL ( $values_array [ 'module' ] ) || ACLController::checkAccess ( $values_array [ 'module' ], 'list', true ) ) ) ;
+
+                $GLOBALS [ 'log' ]->debug ( "SubPanelDefinitions->get_available_tabs(): " . $key . "= " . ( $exempt ? "exempt " : "not exempt " .( $ok ? " ACL OK" : "" ) ) ) ;
+
+                if ( $ok )
+                {
+                    while ( ! empty ( $this->_visible_tabs_array [ $values_array [ 'order' ] ] ) )
+                    {
+                        $values_array [ 'order' ] ++ ;
+                    }
+
+                    $this->_visible_tabs_array [ $values_array ['order'] ] = ($FromGetModuleSubpanels) ? array($key=>$values_array['title_key']) : $key ;
+                }
+            }
+        }
+
+        ksort ( $this->_visible_tabs_array ) ;
+        return $this->_visible_tabs_array ;
+    }
+
+    /**
+     * Load the definition of the a sub-panel.
+     * Also the sub-panel is added to an array of sub-panels.
+     * use of reload has been deprecated, since the subpanel is initialized every time.
      *
      * @param string $name              The name of the sub-panel to reload
      * @param boolean $reload           Reload the sub-panel (unused)
      * @param boolean $original_only    Only load the original sub-panel and no custom ones
      * @return boolean|aSubPanel        Returns aSubPanel object or boolean false if one is not found or it can't be
      *      displayed due to ACL reasons.
-	 */
-	function load_subpanel ( $name , $reload = false , $original_only = false, $search_query = '', $collections = array() )
-	{
-		if (!is_dir('modules/' . $this->layout_defs [ 'subpanel_setup' ][ strtolower ( $name ) ] [ 'module' ]))
-			return false;
+     */
+    function load_subpanel ( $name , $reload = false , $original_only = false, $search_query = '', $collections = array() )
+    {
+        if (!is_dir('modules/' . $this->layout_defs [ 'subpanel_setup' ][ strtolower ( $name ) ] [ 'module' ]))
+            return false;
 
-		$subpanel = new aSubPanel ( $name, $this->layout_defs [ 'subpanel_setup' ] [ strtolower ( $name ) ], $this->_focus, $reload, $original_only, $search_query, $collections ) ;
+        $subpanel = new aSubPanel ( $name, $this->layout_defs [ 'subpanel_setup' ] [ strtolower ( $name ) ], $this->_focus, $reload, $original_only, $search_query, $collections ) ;
 
-		// only return the subpanel object if we can display it.
-		if($subpanel->canDisplay == true) {
-			return $subpanel;
-		}
+        // only return the subpanel object if we can display it.
+        if($subpanel->canDisplay == true) {
+            return $subpanel;
+        }
 
-		// by default return false so we don't show anything if it's not required.
-		return false;
-	}
+        // by default return false so we don't show anything if it's not required.
+        return false;
+    }
 
-	/**
-	 * Load the layout def file and associate the definition with a variable in the file.
-	 */
-	function open_layout_defs ( $reload = false , $layout_def_key = '' , $original_only = false )
-	{
-		$layout_defs [ $this->_focus->module_dir ] = array ( ) ;
-		$layout_defs [ $layout_def_key ] = array ( ) ;
+    /**
+     * Load the layout def file and associate the definition with a variable in the file.
+     */
+    function open_layout_defs ( $reload = false , $layout_def_key = '' , $original_only = false )
+    {
+        $layout_defs [ $this->_focus->module_dir ] = array ( ) ;
+        $layout_defs [ $layout_def_key ] = array ( ) ;
 
-		if (empty ( $this->layout_defs ) || $reload || (! empty ( $layout_def_key ) && ! isset ( $layout_defs [ $layout_def_key ] )))
-		{
-			if (file_exists ( 'modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php' ))
-				require ('modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php') ;
+        if (empty ( $this->layout_defs ) || $reload || (! empty ( $layout_def_key ) && ! isset ( $layout_defs [ $layout_def_key ] )))
+        {
+            if (file_exists ( 'modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php' ))
+                require ('modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php') ;
 
-			if (! $original_only && file_exists ( 'custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php' ))
-				require ('custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php') ;
+            if (! $original_only && file_exists ( 'custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php' ))
+                require ('custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php') ;
 
-			if (! empty ( $layout_def_key ))
-				$this->layout_defs = $layout_defs [ $layout_def_key ] ;
-			else
-				$this->layout_defs = $layout_defs [ $this->_focus->module_dir ] ;
+            if (! empty ( $layout_def_key ))
+                $this->layout_defs = $layout_defs [ $layout_def_key ] ;
+            else
+                $this->layout_defs = $layout_defs [ $this->_focus->module_dir ] ;
 
-		}
+        }
 
-	}
+    }
 
-	/**
-	 * Removes a tab from the list of loaded tabs.
-	 * Returns true if successful, false otherwise.
-	 * Hint: Used by Campaign's DetailView.
-	 */
-	function exclude_tab ( $tab_name )
-	{
-		$result = false ;
-		//unset layout definition
-		if (! empty ( $this->layout_defs [ 'subpanel_setup' ] [ $tab_name ] ))
-		{
-			unset ( $this->layout_defs [ 'subpanel_setup' ] [ $tab_name ] ) ;
-		}
-		//unset instance from _visible_tab_array
-		if (! empty ( $this->_visible_tabs_array ))
-		{
-			$key = array_search ( $tab_name, $this->_visible_tabs_array ) ;
-			if ($key !== false)
-			{
-				unset ( $this->_visible_tabs_array [ $key ] ) ;
-			}
-		}
-		return $result ;
-	}
-
-
-	/**
-	 * return all available subpanels that belong to the list of tab modules.  You can optionally return all
-	 * available subpanels, and also optionally group by module (prepends the key with the bean class name).
-	 */
-	function get_all_subpanels( $return_tab_modules_only = true, $group_by_module = false )
-	{
-		global $moduleList, $beanFiles, $beanList, $module;
+    /**
+     * Removes a tab from the list of loaded tabs.
+     * Returns true if successful, false otherwise.
+     * Hint: Used by Campaign's DetailView.
+     */
+    function exclude_tab ( $tab_name )
+    {
+        $result = false ;
+        //unset layout definition
+        if (! empty ( $this->layout_defs [ 'subpanel_setup' ] [ $tab_name ] ))
+        {
+            unset ( $this->layout_defs [ 'subpanel_setup' ] [ $tab_name ] ) ;
+        }
+        //unset instance from _visible_tab_array
+        if (! empty ( $this->_visible_tabs_array ))
+        {
+            $key = array_search ( $tab_name, $this->_visible_tabs_array ) ;
+            if ($key !== false)
+            {
+                unset ( $this->_visible_tabs_array [ $key ] ) ;
+            }
+        }
+        return $result ;
+    }
+
+
+    /**
+     * return all available subpanels that belong to the list of tab modules.  You can optionally return all
+     * available subpanels, and also optionally group by module (prepends the key with the bean class name).
+     */
+    function get_all_subpanels( $return_tab_modules_only = true, $group_by_module = false )
+    {
+        global $moduleList, $beanFiles, $beanList, $module;
 
-		//use tab controller function to get module list with named keys
-		require_once("modules/MySettings/TabController.php");
-		$modules_to_check = TabController::get_key_array($moduleList);
+        //use tab controller function to get module list with named keys
+        require_once("modules/MySettings/TabController.php");
+        $modules_to_check = TabController::get_key_array($moduleList);
 
-		//change case to match subpanel processing later on
-		$modules_to_check = array_change_key_case($modules_to_check);
+        //change case to match subpanel processing later on
+        $modules_to_check = array_change_key_case($modules_to_check);
         // Append on the CampaignLog module, because that is where the subpanels point, not directly to Campaigns
         $modules_to_check['campaignlog'] = "CampaignLog";
 
 
-		$spd = '';
-		$spd_arr = array();
-		//iterate through modules and build subpanel array
-		foreach($modules_to_check as $mod_name){
+        $spd = '';
+        $spd_arr = array();
+        //iterate through modules and build subpanel array
+        foreach($modules_to_check as $mod_name){
 
-			//skip if module name is not in bean list, otherwise get the bean class name
-			if(!isset($beanList[$mod_name])) continue;
-			$class = $beanList[$mod_name];
+            //skip if module name is not in bean list, otherwise get the bean class name
+            if(!isset($beanList[$mod_name])) continue;
+            $class = $beanList[$mod_name];
 
-			//skip if class name is not in file list, otherwise require the bean file and create new class
-			if(!isset($beanFiles[$class]) || !file_exists($beanFiles[$class])) continue;
+            //skip if class name is not in file list, otherwise require the bean file and create new class
+            if(!isset($beanFiles[$class]) || !file_exists($beanFiles[$class])) continue;
 
-			//retrieve subpanels for this bean
-			require_once($beanFiles[$class]);
-			$bean_class = new $class();
+            //retrieve subpanels for this bean
+            require_once($beanFiles[$class]);
+            $bean_class = new $class();
 
-			//create new subpanel definition instance and get list of tabs
-			$spd = new SubPanelDefinitions($bean_class) ;
-			$sub_tabs = $spd->get_available_tabs();
+            //create new subpanel definition instance and get list of tabs
+            $spd = new SubPanelDefinitions($bean_class) ;
+            $sub_tabs = $spd->get_available_tabs();
 
-			//add each subpanel to array of total subpanles
-			foreach( $sub_tabs as $panel_key){
-				$panel_key = strtolower($panel_key);
+            //add each subpanel to array of total subpanles
+            foreach( $sub_tabs as $panel_key){
+                $panel_key = strtolower($panel_key);
                 $panel_module = $panel_key;
                 if ( isset($spd->layout_defs['subpanel_setup'][$panel_key]['module']) )
                     $panel_module = strtolower($spd->layout_defs['subpanel_setup'][$panel_key]['module']);
                 //if module_only flag is set, only if it is also in module array
-				if($return_tab_modules_only && !array_key_exists($panel_module, $modules_to_check)) continue;
-				$panel_key_name = $panel_module;
-
-				//group_by_key_name is set to true, then array will hold an entry for each
-				//subpanel, with the module name prepended in the key
-				if($group_by_module) $panel_key_name = $class.'_'.$panel_key_name;
-				//add panel name to subpanel array
-				$spd_arr[$panel_key_name] = $panel_module;
-			}
-		}
-		return 	$spd_arr;
-	}
-
-	/*
+                if($return_tab_modules_only && !array_key_exists($panel_module, $modules_to_check)) continue;
+                $panel_key_name = $panel_module;
+
+                //group_by_key_name is set to true, then array will hold an entry for each
+                //subpanel, with the module name prepended in the key
+                if($group_by_module) $panel_key_name = $class.'_'.$panel_key_name;
+                //add panel name to subpanel array
+                $spd_arr[$panel_key_name] = $panel_module;
+            }
+        }
+        return 	$spd_arr;
+    }
+
+    /*
 	 * save array of hidden panels to mysettings category in config table
 	 */
-	function set_hidden_subpanels($panels){
-		$administration = new Administration();
-		$serialized = base64_encode(serialize($panels));
-		$administration->saveSetting('MySettings', 'hide_subpanels', $serialized);
-	}
+    function set_hidden_subpanels($panels){
+        $administration = new Administration();
+        $serialized = base64_encode(serialize($panels));
+        $administration->saveSetting('MySettings', 'hide_subpanels', $serialized);
+    }
 
-	/*
+    /*
 	 * retrieve hidden subpanels
 	 */
-	function get_hidden_subpanels(){
-		global $moduleList;
-
-		//create variable as static to minimize queries
-		static $hidden_subpanels = null;
-
-		// if the static value is not already cached, then retrieve it.
-		if(empty($hidden_subpanels))
-		{
-
-			//create Administration object and retrieve any settings for panels
-			$administration = new Administration();
-			$administration->retrieveSettings('MySettings');
-
-			if(isset($administration->settings) && isset($administration->settings['MySettings_hide_subpanels'])){
-				$hidden_subpanels = $administration->settings['MySettings_hide_subpanels'];
-				$hidden_subpanels = trim($hidden_subpanels);
-
-				//make sure serialized string is not empty
-				if (!empty($hidden_subpanels)){
-					//decode and unserialize to retrieve the array
-					$hidden_subpanels = base64_decode($hidden_subpanels);
-					$hidden_subpanels = unserialize($hidden_subpanels);
-
-					//Ensure modules saved in the preferences exist.
-					//get user preference
-					//unserialize and add to array if not empty
-					$pref_hidden = array();
-					foreach($pref_hidden as $id => $pref_hidden_panel) {
-						$hidden_subpanels[] = $pref_hidden_panel;
-					}
-
-
-				}else{
-					//no settings found, return empty
-					return $hidden_subpanels;
-				}
-			}
-			else
-			{	//no settings found, return empty
-				return $hidden_subpanels;
-			}
-		}
-
-		return $hidden_subpanels;
-	}
+    function get_hidden_subpanels(){
+        global $moduleList;
+
+        //create variable as static to minimize queries
+        static $hidden_subpanels = null;
+
+        // if the static value is not already cached, then retrieve it.
+        if(empty($hidden_subpanels))
+        {
+
+            //create Administration object and retrieve any settings for panels
+            $administration = new Administration();
+            $administration->retrieveSettings('MySettings');
+
+            if(isset($administration->settings) && isset($administration->settings['MySettings_hide_subpanels'])){
+                $hidden_subpanels = $administration->settings['MySettings_hide_subpanels'];
+                $hidden_subpanels = trim($hidden_subpanels);
+
+                //make sure serialized string is not empty
+                if (!empty($hidden_subpanels)){
+                    //decode and unserialize to retrieve the array
+                    $hidden_subpanels = base64_decode($hidden_subpanels);
+                    $hidden_subpanels = unserialize($hidden_subpanels);
+
+                    //Ensure modules saved in the preferences exist.
+                    //get user preference
+                    //unserialize and add to array if not empty
+                    $pref_hidden = array();
+                    foreach($pref_hidden as $id => $pref_hidden_panel) {
+                        $hidden_subpanels[] = $pref_hidden_panel;
+                    }
+
+
+                }else{
+                    //no settings found, return empty
+                    return $hidden_subpanels;
+                }
+            }
+            else
+            {	//no settings found, return empty
+                return $hidden_subpanels;
+            }
+        }
+
+        return $hidden_subpanels;
+    }
 
 
 }

Spacing +212 added lines, -212 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if (! defined ( 'sugarEntry' ) || ! sugarEntry) die ( 'Not A Valid Entry Point' ) ;
+if (!defined('sugarEntry') || !sugarEntry) die ('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -54,13 +54,13 @@ 
 class aSubPanel
 {
 
-	var $name ;
-	var $_instance_properties ;
+	var $name;
+	var $_instance_properties;
 
-	var $mod_strings ;
-	var $panel_definition ;
-	var $sub_subpanels ;
-	var $parent_bean ;
+	var $mod_strings;
+	var $panel_definition;
+	var $sub_subpanels;
+	var $parent_bean;
 
     /**
      * Can we display this subpanel?
@@ -73,53 +73,53 @@ 
     var $canDisplay = true;
 
 	//module's table name and column fields.
-	var $table_name ;
-	var $db_fields ;
-	var $bean_name ;
-	var $template_instance ;
+	var $table_name;
+	var $db_fields;
+	var $bean_name;
+	var $template_instance;
 
 	var $search_query;
 	var $base_collection_list = array();
 
-	function __construct ($name , $instance_properties , $parent_bean , $reload = false , $original_only = false, $search_query = '', $collections = array() ){
+	function __construct($name, $instance_properties, $parent_bean, $reload = false, $original_only = false, $search_query = '', $collections = array()) {
 
-		$this->_instance_properties = $instance_properties ;
+		$this->_instance_properties = $instance_properties;
 
-		if(isset($instance_properties['collection_list' ])) $this->base_collection_list = $instance_properties['collection_list' ];
+		if (isset($instance_properties['collection_list'])) $this->base_collection_list = $instance_properties['collection_list'];
 
-		if(!empty($collections) && isset($instance_properties['collection_list' ])){
-			foreach($instance_properties['collection_list' ] as $cname => $value){
-				if(!in_array($value['module'], $collections)){
+		if (!empty($collections) && isset($instance_properties['collection_list'])) {
+			foreach ($instance_properties['collection_list'] as $cname => $value) {
+				if (!in_array($value['module'], $collections)) {
 					unset($instance_properties['collection_list'][$cname]);
 				}
 			}
 		}
-		if (!$this->isCollection()){
+		if (!$this->isCollection()) {
 			$table = strtolower($instance_properties['module']);
-			$search_query = str_replace('meetings',$table,$search_query);
+			$search_query = str_replace('meetings', $table, $search_query);
 		}
 
 		$this->search_query = $search_query;
-		$this->name = $name ;
-		$this->parent_bean = $parent_bean ;
+		$this->name = $name;
+		$this->parent_bean = $parent_bean;
 
 		//set language
-		global $current_language ;
-		if (! isset ( $parent_bean->mbvardefs ))
+		global $current_language;
+		if (!isset ($parent_bean->mbvardefs))
 		{
-			$mod_strings = return_module_language ( $current_language, $parent_bean->module_dir ) ;
+			$mod_strings = return_module_language($current_language, $parent_bean->module_dir);
 		}
-		$this->mod_strings = $mod_strings ;
+		$this->mod_strings = $mod_strings;
 
-        if ($this->isCollection ())
+        if ($this->isCollection())
 		{
-			$this->canDisplay = $this->load_sub_subpanels () ; //load sub-panel definition.
+			$this->canDisplay = $this->load_sub_subpanels(); //load sub-panel definition.
 		} else
 		{
-			if (!is_dir('modules/' . $this->_instance_properties [ 'module' ])){
+			if (!is_dir('modules/'.$this->_instance_properties ['module'])) {
 				_pstack_trace();
 			}
-			$def_path = 'modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'subpanel_name' ] . '.php' ;
+			$def_path = 'modules/'.$this->_instance_properties ['module'].'/metadata/subpanels/'.$this->_instance_properties ['subpanel_name'].'.php';
 
 			$orig_exists = is_file($def_path);
 			$loaded = false;
@@ -128,16 +128,16 @@ 
 				require ($def_path);
 				$loaded = true;
 			}
-			if (is_file("custom/$def_path") && (!$original_only  || !$orig_exists))
+			if (is_file("custom/$def_path") && (!$original_only || !$orig_exists))
 			{
 				require ("custom/$def_path");
 				$loaded = true;
 			}
 
-			if (! $original_only && isset ( $this->_instance_properties [ 'override_subpanel_name' ] ) && file_exists ( 'custom/modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'override_subpanel_name' ] . '.php' ))
+			if (!$original_only && isset ($this->_instance_properties ['override_subpanel_name']) && file_exists('custom/modules/'.$this->_instance_properties ['module'].'/metadata/subpanels/'.$this->_instance_properties ['override_subpanel_name'].'.php'))
 			{
-				$cust_def_path = 'custom/modules/' . $this->_instance_properties [ 'module' ] . '/metadata/subpanels/' . $this->_instance_properties [ 'override_subpanel_name' ] . '.php' ;
-				require ($cust_def_path) ;
+				$cust_def_path = 'custom/modules/'.$this->_instance_properties ['module'].'/metadata/subpanels/'.$this->_instance_properties ['override_subpanel_name'].'.php';
+				require ($cust_def_path);
 				$loaded = true;
 			}
 
@@ -168,7 +168,7 @@ 
      */
     public function isDefaultHidden()
     {
-        if(isset($this->_instance_properties['default_hidden']) && $this->_instance_properties['default_hidden'] == true) {
+        if (isset($this->_instance_properties['default_hidden']) && $this->_instance_properties['default_hidden'] == true) {
             return true;
         }
 
@@ -176,63 +176,63 @@ 
     }
 
 
-	function distinct_query ()
+	function distinct_query()
 	{
-		if (isset ( $this->_instance_properties [ 'get_distinct_data' ] ))
+		if (isset ($this->_instance_properties ['get_distinct_data']))
 		{
 			return !empty($this->_instance_properties['get_distinct_data']) ? true : false;
 		}
-		return false ;
+		return false;
 	}
 
 	//return the translated header value.
-	function get_title ()
+	function get_title()
 	{
-		if (empty ( $this->mod_strings [ $this->_instance_properties [ 'title_key' ] ] ))
+		if (empty ($this->mod_strings [$this->_instance_properties ['title_key']]))
 		{
-			return translate ( $this->_instance_properties [ 'title_key' ], $this->_instance_properties [ 'module' ] ) ;
+			return translate($this->_instance_properties ['title_key'], $this->_instance_properties ['module']);
 		}
-		return $this->mod_strings [ $this->_instance_properties [ 'title_key' ] ] ;
+		return $this->mod_strings [$this->_instance_properties ['title_key']];
 	}
 
 	//return the definition of buttons. looks for buttons in 2 locations.
-	function get_buttons ()
+	function get_buttons()
 	{
-		$buttons = array ( ) ;
-		if (isset ( $this->_instance_properties [ 'top_buttons' ] ))
+		$buttons = array( );
+		if (isset ($this->_instance_properties ['top_buttons']))
 		{
 			//this will happen only in the case of sub-panels with multiple sources(activities).
-			$buttons = $this->_instance_properties [ 'top_buttons' ] ;
+			$buttons = $this->_instance_properties ['top_buttons'];
 		} else
 		{
-			$buttons = $this->panel_definition [ 'top_buttons' ] ;
+			$buttons = $this->panel_definition ['top_buttons'];
 		}
 
 		// permissions. hide SubPanelTopComposeEmailButton from activities if email module is disabled.
 		//only email is  being tested becuase other submodules in activites/history such as notes, tasks, meetings and calls cannot be disabled.
 		//as of today these are the only 2 sub-panels that use the union clause.
-		$mod_name = $this->get_module_name () ;
+		$mod_name = $this->get_module_name();
 		if ($mod_name == 'Activities' || $mod_name == 'History')
 		{
-			global $modListHeader ;
-			global $modules_exempt_from_availability_check ;
-			if (isset ( $modListHeader ) && (! (array_key_exists ( 'Emails', $modListHeader ) or array_key_exists ( 'Emails', $modules_exempt_from_availability_check ))))
+			global $modListHeader;
+			global $modules_exempt_from_availability_check;
+			if (isset ($modListHeader) && (!(array_key_exists('Emails', $modListHeader) or array_key_exists('Emails', $modules_exempt_from_availability_check))))
 			{
-				foreach ( $buttons as $key => $button )
+				foreach ($buttons as $key => $button)
 				{
-					foreach ( $button as $property => $value )
+					foreach ($button as $property => $value)
 					{
 						if ($value === 'SubPanelTopComposeEmailButton' || $value === 'SubPanelTopArchiveEmailButton')
 						{
 							//remove this button from the array.
-							unset ( $buttons [ $key ] ) ;
+							unset ($buttons [$key]);
 						}
 					}
 				}
 			}
 		}
 
-		return $buttons ;
+		return $buttons;
 	}
 
 
@@ -244,67 +244,67 @@ 
      * @return bool         True by default if the subpanel was loaded.  Will return false if none in the collection are
      *                      allowed by the current user.
      */
-	function load_sub_subpanels ()
+	function load_sub_subpanels()
 	{
 
-		global $modListHeader ;
+		global $modListHeader;
 		// added a check for security of tabs to see if an user has access to them
 		// this prevents passing an "unseen" tab to the query string and pulling up its contents
-		if (! isset ( $modListHeader ))
+		if (!isset ($modListHeader))
 		{
-			global $current_user ;
-			if (isset ( $current_user ))
+			global $current_user;
+			if (isset ($current_user))
 			{
-				$modListHeader = query_module_access_list ( $current_user ) ;
+				$modListHeader = query_module_access_list($current_user);
 			}
 		}
 
 		//by default all the activities modules are exempt, so hiding them won't affect their appearance unless the 'activity' subpanel itself is hidden.
 		//add email to the list temporarily so it is not affected in activities subpanel
-		global $modules_exempt_from_availability_check ;
+		global $modules_exempt_from_availability_check;
 		$modules_exempt_from_availability_check['Emails'] = 'Emails';
 
 		$listFieldMap = array();
 
-		if (empty ( $this->sub_subpanels ))
+		if (empty ($this->sub_subpanels))
 		{
-			$panels = $this->get_inst_prop_value ( 'collection_list' ) ;
-			foreach ( $panels as $panel => $properties )
+			$panels = $this->get_inst_prop_value('collection_list');
+			foreach ($panels as $panel => $properties)
 			{
-				if (array_key_exists ( $properties [ 'module' ], $modListHeader ) or array_key_exists ( $properties [ 'module' ], $modules_exempt_from_availability_check ))
+				if (array_key_exists($properties ['module'], $modListHeader) or array_key_exists($properties ['module'], $modules_exempt_from_availability_check))
 				{
-					$this->sub_subpanels [ $panel ] = new aSubPanel ( $panel, $properties, $this->parent_bean, false, false, $this->search_query ) ;
+					$this->sub_subpanels [$panel] = new aSubPanel($panel, $properties, $this->parent_bean, false, false, $this->search_query);
 				}
 			}
 			// if it's empty just dump out as there is nothing to process.
-			if(empty($this->sub_subpanels)) return false;
+			if (empty($this->sub_subpanels)) return false;
 			//Sync displayed list fields across the subpanels
 			$display_fields = $this->getDisplayFieldsFromCollection($this->sub_subpanels);
 			$query_fields = array();
-			foreach ( $this->sub_subpanels as $key => $subpanel )
+			foreach ($this->sub_subpanels as $key => $subpanel)
 			{
 				$list_fields = $subpanel->get_list_fields();
 				$listFieldMap[$key] = array();
 				$index = 0;
-				foreach($list_fields as $field => $def)
+				foreach ($list_fields as $field => $def)
 				{
 					if (isset($def['vname']) && isset($def['width']))
 					{
 						$index++;
-						if(!empty($def['alias']))
+						if (!empty($def['alias']))
 							$listFieldMap[$key][$def['alias']] = $field;
 						else
 							$listFieldMap[$key][$field] = $field;
 						if (!isset($display_fields[$def['vname']]))
 						{
-							if(sizeof($display_fields) > $index)
+							if (sizeof($display_fields) > $index)
 							{
 								//Try to insert the new field in an order that makes sense
 								$start = array_slice($display_fields, 0, $index);
 								$end = array_slice($display_fields, $index);
 								$display_fields = array_merge(
 									$start,
-									array($def['vname'] => array('name' => $field, 'vname' => $def['vname'], 'width' => $def['width'] )),
+									array($def['vname'] => array('name' => $field, 'vname' => $def['vname'], 'width' => $def['width'])),
 									$end
 								);
 							} else
@@ -321,10 +321,10 @@ 
 					}
 				}
 			}
-			foreach ( $this->sub_subpanels as $key => $subpanel )
+			foreach ($this->sub_subpanels as $key => $subpanel)
 			{
 				$list_fields = array();
-				foreach($display_fields as $vname => $def)
+				foreach ($display_fields as $vname => $def)
 				{
 					$field = $def['name'];
 					$list_key = isset($listFieldMap[$key][$field]) ? $listFieldMap[$key][$field] : $field;
@@ -342,7 +342,7 @@ 
 						$list_fields[$field] = $display_fields[$vname];
 					}
 				}
-				foreach($query_fields as $field => $def)
+				foreach ($query_fields as $field => $def)
 				{
 					if (isset($subpanel->panel_definition['list_fields'][$field]))
 					{
@@ -362,25 +362,25 @@ 
 	protected function getDisplayFieldsFromCollection($sub_subpanels)
 	{
 		$display_fields = array();
-		foreach ($sub_subpanels as $key => $subpanel )
+		foreach ($sub_subpanels as $key => $subpanel)
 		{
 			$list_fields = $subpanel->get_list_fields();
 			$index = 0;
-			foreach($list_fields as $field => $def)
+			foreach ($list_fields as $field => $def)
 			{
 				if (isset($def['vname']) && isset($def['width']))
 				{
 					$index++;
 					if (!isset($display_fields[$def['vname']]))
 					{
-						if(sizeof($display_fields) > $index)
+						if (sizeof($display_fields) > $index)
 						{
 							//Try to insert the new field in an order that makes sense
 							$start = array_slice($display_fields, 0, $index);
 							$end = array_slice($display_fields, $index);
 							$display_fields = array_merge(
 								$start,
-								array($def['vname'] => array('name' => $field, 'vname' => $def['vname'], 'width' => $def['width'] )),
+								array($def['vname'] => array('name' => $field, 'vname' => $def['vname'], 'width' => $def['width'])),
 								$end
 							);
 						} else
@@ -397,13 +397,13 @@ 
 		}
 	}
 
-	function isDatasourceFunction ()
+	function isDatasourceFunction()
 	{
-		if (strpos ( $this->get_inst_prop_value ( 'get_subpanel_data' ), 'function' ) === false)
+		if (strpos($this->get_inst_prop_value('get_subpanel_data'), 'function') === false)
 		{
-			return false ;
+			return false;
 		}
-		return true ;
+		return true;
 	}
 
     /**
@@ -411,159 +411,159 @@ 
      *
      * @return bool
      */
-	function isCollection ()
+	function isCollection()
 	{
-		return ($this->get_inst_prop_value ( 'type' ) == 'collection');
+		return ($this->get_inst_prop_value('type') == 'collection');
 	}
 
 	//get value of a property defined at the panel instance level.
-	function get_inst_prop_value ( $name )
+	function get_inst_prop_value($name)
 	{
-		return isset($this->_instance_properties[$name]) ? $this->_instance_properties [ $name ] : null;
+		return isset($this->_instance_properties[$name]) ? $this->_instance_properties [$name] : null;
 	}
 	//get value of a property defined at the panel definition level.
-	function get_def_prop_value ( $name )
+	function get_def_prop_value($name)
 	{
-		if (isset ( $this->panel_definition [ $name ] ))
+		if (isset ($this->panel_definition [$name]))
 		{
-			return $this->panel_definition [ $name ] ;
+			return $this->panel_definition [$name];
 		} else
 		{
-			return null ;
+			return null;
 		}
 	}
 
 	//if datasource is of the type function then return the function name
 	//else return the value as is.
-	function get_function_parameters ()
+	function get_function_parameters()
 	{
-		$parameters = array ( ) ;
-		if ($this->isDatasourceFunction ())
+		$parameters = array( );
+		if ($this->isDatasourceFunction())
 		{
-			$parameters = $this->get_inst_prop_value ( 'function_parameters' ) ;
+			$parameters = $this->get_inst_prop_value('function_parameters');
 		}
-		return $parameters ;
+		return $parameters;
 	}
 
-	function get_data_source_name ( $check_set_subpanel_data = false )
+	function get_data_source_name($check_set_subpanel_data = false)
 	{
-		$prop_value = null ;
+		$prop_value = null;
 		if ($check_set_subpanel_data)
 		{
-			$prop_value = $this->get_inst_prop_value ( 'set_subpanel_data' ) ;
+			$prop_value = $this->get_inst_prop_value('set_subpanel_data');
 		}
-		if (! empty ( $prop_value ))
+		if (!empty ($prop_value))
 		{
-			return $prop_value ;
+			return $prop_value;
 		} else
 		{
 			//fall back to default behavior.
 		}
-		if ($this->isDatasourceFunction ())
+		if ($this->isDatasourceFunction())
 		{
-			return (substr_replace ( $this->get_inst_prop_value ( 'get_subpanel_data' ), '', 0, 9 )) ;
+			return (substr_replace($this->get_inst_prop_value('get_subpanel_data'), '', 0, 9));
 		} else
 		{
-			return $this->get_inst_prop_value ( 'get_subpanel_data' ) ;
+			return $this->get_inst_prop_value('get_subpanel_data');
 		}
 	}
 
 	//returns the where clause for the query.
-	function get_where ()
+	function get_where()
 	{
-		if($this->get_def_prop_value ( 'where' ) != '' && $this->search_query != ''){
-			return $this->get_def_prop_value ( 'where' ).' AND '.$this->search_query;
-		} else if($this->search_query != ''){
+		if ($this->get_def_prop_value('where') != '' && $this->search_query != '') {
+			return $this->get_def_prop_value('where').' AND '.$this->search_query;
+		} else if ($this->search_query != '') {
 			return $this->search_query;
 		}
-		return $this->get_def_prop_value ( 'where' ) ;
+		return $this->get_def_prop_value('where');
 	}
 
-	function is_fill_in_additional_fields ()
+	function is_fill_in_additional_fields()
 	{
 		// do both. inst_prop returns values from metadata/subpaneldefs.php and def_prop returns from subpanel/default.php
-		$temp = $this->get_inst_prop_value ( 'fill_in_additional_fields' ) || $this->get_def_prop_value ( 'fill_in_additional_fields' ) ;
-		return $temp ;
+		$temp = $this->get_inst_prop_value('fill_in_additional_fields') || $this->get_def_prop_value('fill_in_additional_fields');
+		return $temp;
 	}
 
-	function get_list_fields ()
+	function get_list_fields()
 	{
-		if (isset ( $this->panel_definition [ 'list_fields' ] ))
+		if (isset ($this->panel_definition ['list_fields']))
 		{
-			return $this->panel_definition [ 'list_fields' ] ;
+			return $this->panel_definition ['list_fields'];
 		} else
 		{
-			return array ( ) ;
+			return array( );
 		}
 	}
 
-	function get_module_name ()
+	function get_module_name()
 	{
-		return $this->get_inst_prop_value ( 'module' ) ;
+		return $this->get_inst_prop_value('module');
 	}
 
-	function get_name ()
+	function get_name()
 	{
-		return $this->name ;
+		return $this->name;
 	}
 
 	//load subpanel module's table name and column fields.
-	function load_module_info ()
+	function load_module_info()
 	{
-		global $beanList ;
-		global $beanFiles ;
+		global $beanList;
+		global $beanFiles;
 
-		$module_name = $this->get_module_name () ;
-		if (! empty ( $module_name ))
+		$module_name = $this->get_module_name();
+		if (!empty ($module_name))
 		{
 
-			$bean_name = $beanList [ $this->get_module_name () ] ;
+			$bean_name = $beanList [$this->get_module_name()];
 
-			$this->bean_name = $bean_name ;
+			$this->bean_name = $bean_name;
 
-			include_once ($beanFiles [ $bean_name ]) ;
-			$this->template_instance = new $bean_name ( ) ;
-			$this->template_instance->force_load_details = true ;
-			$this->table_name = $this->template_instance->table_name ;
+			include_once ($beanFiles [$bean_name]);
+			$this->template_instance = new $bean_name( );
+			$this->template_instance->force_load_details = true;
+			$this->table_name = $this->template_instance->table_name;
 			//$this->db_fields=$this->template_instance->column_fields;
 		}
 	}
 	//this function is to be used only with sub-panels that are based
 	//on collections.
-	function get_header_panel_def ()
+	function get_header_panel_def()
 	{
-		if (! empty ( $this->sub_subpanels ))
+		if (!empty ($this->sub_subpanels))
 		{
-			if (! empty ( $this->_instance_properties [ 'header_definition_from_subpanel' ] ) && ! empty ( $this->sub_subpanels [ $this->_instance_properties [ 'header_definition_from_subpanel' ] ] ))
+			if (!empty ($this->_instance_properties ['header_definition_from_subpanel']) && !empty ($this->sub_subpanels [$this->_instance_properties ['header_definition_from_subpanel']]))
 			{
-				return $this->sub_subpanels [ $this->_instance_properties [ 'header_definition_from_subpanel' ] ] ;
+				return $this->sub_subpanels [$this->_instance_properties ['header_definition_from_subpanel']];
 			} else
 			{
 				$display_fields = array();
 				//If we are not pulling from a specific subpanel, create a list of all list fields and use that.
-				foreach($this->sub_subpanels as $subpanel)
+				foreach ($this->sub_subpanels as $subpanel)
 				{
 					$list_fields = $subpanel->get_list_fields();
-					foreach($list_fields as $field => $def)
+					foreach ($list_fields as $field => $def)
 					{
 
 					}
 				}
 
-				reset ( $this->sub_subpanels ) ;
-				return current ( $this->sub_subpanels ) ;
+				reset($this->sub_subpanels);
+				return current($this->sub_subpanels);
 			}
 		}
-		return null ;
+		return null;
 	}
 
 	/**
 	 * Returns an array of current properties of the class.
 	 * It will simply give the class name for instances of classes.
 	 */
-	function _to_array ()
+	function _to_array()
 	{
-		return array ( '_instance_properties' => $this->_instance_properties , 'db_fields' => $this->db_fields , 'mod_strings' => $this->mod_strings , 'name' => $this->name , 'panel_definition' => $this->panel_definition , 'parent_bean' => get_class ( $this->parent_bean ) , 'sub_subpanels' => $this->sub_subpanels , 'table_name' => $this->table_name , 'template_instance' => get_class ( $this->template_instance ) ) ;
+		return array('_instance_properties' => $this->_instance_properties, 'db_fields' => $this->db_fields, 'mod_strings' => $this->mod_strings, 'name' => $this->name, 'panel_definition' => $this->panel_definition, 'parent_bean' => get_class($this->parent_bean), 'sub_subpanels' => $this->sub_subpanels, 'table_name' => $this->table_name, 'template_instance' => get_class($this->template_instance));
 	}
 
     /**
@@ -582,10 +582,10 @@ 
 class SubPanelDefinitions
 {
 
-	var $_focus ;
-	var $_visible_tabs_array ;
-	var $panels ;
-	var $layout_defs ;
+	var $_focus;
+	var $_visible_tabs_array;
+	var $panels;
+	var $layout_defs;
 
 	/**
 	 * Enter description here...
@@ -595,16 +595,16 @@ 
 	 * @param ARRAY $layout_def_override - if you wish to override the default loaded layout defs you pass them in here.
 	 * @return SubPanelDefinitions
 	 */
-	function __construct ( $focus , $layout_def_key = '' , $layout_def_override = '' )
+	function __construct($focus, $layout_def_key = '', $layout_def_override = '')
 	{
-		$this->_focus = $focus ;
-		if (! empty ( $layout_def_override ))
+		$this->_focus = $focus;
+		if (!empty ($layout_def_override))
 		{
-			$this->layout_defs = $layout_def_override ;
+			$this->layout_defs = $layout_def_override;
 
 		} else
 		{
-			$this->open_layout_defs ( false, $layout_def_key ) ;
+			$this->open_layout_defs(false, $layout_def_key);
 		}
 	}
 
@@ -619,68 +619,68 @@ 
 	 * @param boolean 	Optional - include the subpanel title label in the return array (false)
 	 * @return array	All tabs that pass an ACL check
 	 */
-	function get_available_tabs ($FromGetModuleSubpanels=false)
+	function get_available_tabs($FromGetModuleSubpanels = false)
 	{
-		global $modListHeader ;
-		global $modules_exempt_from_availability_check ;
+		global $modListHeader;
+		global $modules_exempt_from_availability_check;
 
-		if (isset ( $this->_visible_tabs_array ))
-			return $this->_visible_tabs_array ;
+		if (isset ($this->_visible_tabs_array))
+			return $this->_visible_tabs_array;
 
 		if (empty($modListHeader))
 		    $modListHeader = query_module_access_list($GLOBALS['current_user']);
 
-		$this->_visible_tabs_array = array ( ) ; // bug 16820 - make sure this is an array for the later ksort
+		$this->_visible_tabs_array = array( ); // bug 16820 - make sure this is an array for the later ksort
 
-		if (isset ( $this->layout_defs [ 'subpanel_setup' ] )) // bug 17434 - belts-and-braces - check that we have some subpanels first
+		if (isset ($this->layout_defs ['subpanel_setup'])) // bug 17434 - belts-and-braces - check that we have some subpanels first
 		{
 			//retrieve list of hidden subpanels
 			$hidden_panels = $this->get_hidden_subpanels();
 
 			//activities is a special use case in that if it is hidden,
 			//then the history tab should be hidden too.
-			if(!empty($hidden_panels) && is_array($hidden_panels) && in_array('activities',$hidden_panels)){
+			if (!empty($hidden_panels) && is_array($hidden_panels) && in_array('activities', $hidden_panels)) {
 				//add history to list hidden_panels
 				$hidden_panels['history'] = 'history';
 			}
 
-			foreach ( $this->layout_defs [ 'subpanel_setup' ] as $key => $values_array )
+			foreach ($this->layout_defs ['subpanel_setup'] as $key => $values_array)
 			{
 				//exclude if this subpanel is hidden from admin screens
                 $module = $key;
-                if ( isset($values_array['module']) )
+                if (isset($values_array['module']))
                     $module = strtolower($values_array['module']);
-				 if ($hidden_panels && is_array($hidden_panels) && (in_array($module, $hidden_panels) || array_key_exists($module, $hidden_panels)) ){
+				 if ($hidden_panels && is_array($hidden_panels) && (in_array($module, $hidden_panels) || array_key_exists($module, $hidden_panels))) {
 				 	//this panel is hidden, skip it
 				 	continue;
 				 }
 
 				// make sure the module attribute is set, else none of this works...
-				if ( !isset($values_array [ 'module' ])) {
-					$GLOBALS['log']->debug("SubPanelDefinitions->get_available_tabs(): no module defined in subpaneldefs for '$key' =>" . var_export($values_array,true) . " - ingoring subpanel defintion") ;
+				if (!isset($values_array ['module'])) {
+					$GLOBALS['log']->debug("SubPanelDefinitions->get_available_tabs(): no module defined in subpaneldefs for '$key' =>".var_export($values_array, true)." - ingoring subpanel defintion");
 					continue;
 				}
 
 				//check permissions.
-				$exempt = array_key_exists ( $values_array [ 'module' ], $modules_exempt_from_availability_check ) ;
-				$ok = $exempt || ( (! ACLController::moduleSupportsACL ( $values_array [ 'module' ] ) || ACLController::checkAccess ( $values_array [ 'module' ], 'list', true ) ) ) ;
+				$exempt = array_key_exists($values_array ['module'], $modules_exempt_from_availability_check);
+				$ok = $exempt || ((!ACLController::moduleSupportsACL($values_array ['module']) || ACLController::checkAccess($values_array ['module'], 'list', true)));
 
-				$GLOBALS [ 'log' ]->debug ( "SubPanelDefinitions->get_available_tabs(): " . $key . "= " . ( $exempt ? "exempt " : "not exempt " .( $ok ? " ACL OK" : "" ) ) ) ;
+				$GLOBALS ['log']->debug("SubPanelDefinitions->get_available_tabs(): ".$key."= ".($exempt ? "exempt " : "not exempt ".($ok ? " ACL OK" : "")));
 
-				if ( $ok )
+				if ($ok)
 				{
-					while ( ! empty ( $this->_visible_tabs_array [ $values_array [ 'order' ] ] ) )
+					while (!empty ($this->_visible_tabs_array [$values_array ['order']]))
 					{
-						$values_array [ 'order' ] ++ ;
+						$values_array ['order']++;
 					}
 
-					$this->_visible_tabs_array [ $values_array ['order'] ] = ($FromGetModuleSubpanels) ? array($key=>$values_array['title_key']) : $key ;
+					$this->_visible_tabs_array [$values_array ['order']] = ($FromGetModuleSubpanels) ? array($key=>$values_array['title_key']) : $key;
 				}
 			}
 		}
 
-		ksort ( $this->_visible_tabs_array ) ;
-		return $this->_visible_tabs_array ;
+		ksort($this->_visible_tabs_array);
+		return $this->_visible_tabs_array;
 	}
 
 	/**
@@ -694,15 +694,15 @@ 
      * @return boolean|aSubPanel        Returns aSubPanel object or boolean false if one is not found or it can't be
      *      displayed due to ACL reasons.
 	 */
-	function load_subpanel ( $name , $reload = false , $original_only = false, $search_query = '', $collections = array() )
+	function load_subpanel($name, $reload = false, $original_only = false, $search_query = '', $collections = array())
 	{
-		if (!is_dir('modules/' . $this->layout_defs [ 'subpanel_setup' ][ strtolower ( $name ) ] [ 'module' ]))
+		if (!is_dir('modules/'.$this->layout_defs ['subpanel_setup'][strtolower($name)] ['module']))
 			return false;
 
-		$subpanel = new aSubPanel ( $name, $this->layout_defs [ 'subpanel_setup' ] [ strtolower ( $name ) ], $this->_focus, $reload, $original_only, $search_query, $collections ) ;
+		$subpanel = new aSubPanel($name, $this->layout_defs ['subpanel_setup'] [strtolower($name)], $this->_focus, $reload, $original_only, $search_query, $collections);
 
 		// only return the subpanel object if we can display it.
-		if($subpanel->canDisplay == true) {
+		if ($subpanel->canDisplay == true) {
 			return $subpanel;
 		}
 
@@ -713,23 +713,23 @@ 
 	/**
 	 * Load the layout def file and associate the definition with a variable in the file.
 	 */
-	function open_layout_defs ( $reload = false , $layout_def_key = '' , $original_only = false )
+	function open_layout_defs($reload = false, $layout_def_key = '', $original_only = false)
 	{
-		$layout_defs [ $this->_focus->module_dir ] = array ( ) ;
-		$layout_defs [ $layout_def_key ] = array ( ) ;
+		$layout_defs [$this->_focus->module_dir] = array( );
+		$layout_defs [$layout_def_key] = array( );
 
-		if (empty ( $this->layout_defs ) || $reload || (! empty ( $layout_def_key ) && ! isset ( $layout_defs [ $layout_def_key ] )))
+		if (empty ($this->layout_defs) || $reload || (!empty ($layout_def_key) && !isset ($layout_defs [$layout_def_key])))
 		{
-			if (file_exists ( 'modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php' ))
-				require ('modules/' . $this->_focus->module_dir . '/metadata/subpaneldefs.php') ;
+			if (file_exists('modules/'.$this->_focus->module_dir.'/metadata/subpaneldefs.php'))
+				require ('modules/'.$this->_focus->module_dir.'/metadata/subpaneldefs.php');
 
-			if (! $original_only && file_exists ( 'custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php' ))
-				require ('custom/modules/' . $this->_focus->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php') ;
+			if (!$original_only && file_exists('custom/modules/'.$this->_focus->module_dir.'/Ext/Layoutdefs/layoutdefs.ext.php'))
+				require ('custom/modules/'.$this->_focus->module_dir.'/Ext/Layoutdefs/layoutdefs.ext.php');
 
-			if (! empty ( $layout_def_key ))
-				$this->layout_defs = $layout_defs [ $layout_def_key ] ;
+			if (!empty ($layout_def_key))
+				$this->layout_defs = $layout_defs [$layout_def_key];
 			else
-				$this->layout_defs = $layout_defs [ $this->_focus->module_dir ] ;
+				$this->layout_defs = $layout_defs [$this->_focus->module_dir];
 
 		}
 
@@ -740,24 +740,24 @@ 
 	 * Returns true if successful, false otherwise.
 	 * Hint: Used by Campaign's DetailView.
 	 */
-	function exclude_tab ( $tab_name )
+	function exclude_tab($tab_name)
 	{
-		$result = false ;
+		$result = false;
 		//unset layout definition
-		if (! empty ( $this->layout_defs [ 'subpanel_setup' ] [ $tab_name ] ))
+		if (!empty ($this->layout_defs ['subpanel_setup'] [$tab_name]))
 		{
-			unset ( $this->layout_defs [ 'subpanel_setup' ] [ $tab_name ] ) ;
+			unset ($this->layout_defs ['subpanel_setup'] [$tab_name]);
 		}
 		//unset instance from _visible_tab_array
-		if (! empty ( $this->_visible_tabs_array ))
+		if (!empty ($this->_visible_tabs_array))
 		{
-			$key = array_search ( $tab_name, $this->_visible_tabs_array ) ;
+			$key = array_search($tab_name, $this->_visible_tabs_array);
 			if ($key !== false)
 			{
-				unset ( $this->_visible_tabs_array [ $key ] ) ;
+				unset ($this->_visible_tabs_array [$key]);
 			}
 		}
-		return $result ;
+		return $result;
 	}
 
 
@@ -765,7 +765,7 @@ 
 	 * return all available subpanels that belong to the list of tab modules.  You can optionally return all
 	 * available subpanels, and also optionally group by module (prepends the key with the bean class name).
 	 */
-	function get_all_subpanels( $return_tab_modules_only = true, $group_by_module = false )
+	function get_all_subpanels($return_tab_modules_only = true, $group_by_module = false)
 	{
 		global $moduleList, $beanFiles, $beanList, $module;
 
@@ -782,36 +782,36 @@ 
 		$spd = '';
 		$spd_arr = array();
 		//iterate through modules and build subpanel array
-		foreach($modules_to_check as $mod_name){
+		foreach ($modules_to_check as $mod_name) {
 
 			//skip if module name is not in bean list, otherwise get the bean class name
-			if(!isset($beanList[$mod_name])) continue;
+			if (!isset($beanList[$mod_name])) continue;
 			$class = $beanList[$mod_name];
 
 			//skip if class name is not in file list, otherwise require the bean file and create new class
-			if(!isset($beanFiles[$class]) || !file_exists($beanFiles[$class])) continue;
+			if (!isset($beanFiles[$class]) || !file_exists($beanFiles[$class])) continue;
 
 			//retrieve subpanels for this bean
 			require_once($beanFiles[$class]);
 			$bean_class = new $class();
 
 			//create new subpanel definition instance and get list of tabs
-			$spd = new SubPanelDefinitions($bean_class) ;
+			$spd = new SubPanelDefinitions($bean_class);
 			$sub_tabs = $spd->get_available_tabs();
 
 			//add each subpanel to array of total subpanles
-			foreach( $sub_tabs as $panel_key){
+			foreach ($sub_tabs as $panel_key) {
 				$panel_key = strtolower($panel_key);
                 $panel_module = $panel_key;
-                if ( isset($spd->layout_defs['subpanel_setup'][$panel_key]['module']) )
+                if (isset($spd->layout_defs['subpanel_setup'][$panel_key]['module']))
                     $panel_module = strtolower($spd->layout_defs['subpanel_setup'][$panel_key]['module']);
                 //if module_only flag is set, only if it is also in module array
-				if($return_tab_modules_only && !array_key_exists($panel_module, $modules_to_check)) continue;
+				if ($return_tab_modules_only && !array_key_exists($panel_module, $modules_to_check)) continue;
 				$panel_key_name = $panel_module;
 
 				//group_by_key_name is set to true, then array will hold an entry for each
 				//subpanel, with the module name prepended in the key
-				if($group_by_module) $panel_key_name = $class.'_'.$panel_key_name;
+				if ($group_by_module) $panel_key_name = $class.'_'.$panel_key_name;
 				//add panel name to subpanel array
 				$spd_arr[$panel_key_name] = $panel_module;
 			}
@@ -822,7 +822,7 @@ 
 	/*
 	 * save array of hidden panels to mysettings category in config table
 	 */
-	function set_hidden_subpanels($panels){
+	function set_hidden_subpanels($panels) {
 		$administration = new Administration();
 		$serialized = base64_encode(serialize($panels));
 		$administration->saveSetting('MySettings', 'hide_subpanels', $serialized);
@@ -831,26 +831,26 @@ 
 	/*
 	 * retrieve hidden subpanels
 	 */
-	function get_hidden_subpanels(){
+	function get_hidden_subpanels() {
 		global $moduleList;
 
 		//create variable as static to minimize queries
 		static $hidden_subpanels = null;
 
 		// if the static value is not already cached, then retrieve it.
-		if(empty($hidden_subpanels))
+		if (empty($hidden_subpanels))
 		{
 
 			//create Administration object and retrieve any settings for panels
 			$administration = new Administration();
 			$administration->retrieveSettings('MySettings');
 
-			if(isset($administration->settings) && isset($administration->settings['MySettings_hide_subpanels'])){
+			if (isset($administration->settings) && isset($administration->settings['MySettings_hide_subpanels'])) {
 				$hidden_subpanels = $administration->settings['MySettings_hide_subpanels'];
 				$hidden_subpanels = trim($hidden_subpanels);
 
 				//make sure serialized string is not empty
-				if (!empty($hidden_subpanels)){
+				if (!empty($hidden_subpanels)) {
 					//decode and unserialize to retrieve the array
 					$hidden_subpanels = base64_decode($hidden_subpanels);
 					$hidden_subpanels = unserialize($hidden_subpanels);
@@ -859,12 +859,12 @@ 
 					//get user preference
 					//unserialize and add to array if not empty
 					$pref_hidden = array();
-					foreach($pref_hidden as $id => $pref_hidden_panel) {
+					foreach ($pref_hidden as $id => $pref_hidden_panel) {
 						$hidden_subpanels[] = $pref_hidden_panel;
 					}
 
 
-				}else{
+				} else {
 					//no settings found, return empty
 					return $hidden_subpanels;
 				}

include/SubPanel/SubPanel.php

Braces +20 added lines, -12 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -141,8 +143,7 @@ 
 			if(empty($widget_data['widget_class']))
 			{
 				$widget_contents .= "widget_class not defined for top subpanel buttons";
-			}
-			else
+			} else
 			{
 				$widget_contents .= $layout_manager->widgetDisplay($widget_data);
 			}
@@ -163,7 +164,7 @@ 
 
 		if(isset($this->listview)){
 			$ListView =& $this->listview;
-		}else{
+		} else{
 			$ListView = new ListView();
 		}
 		$ListView->initNewXTemplate($xTemplatePath,$this->subpanel_defs->mod_strings);
@@ -178,7 +179,9 @@ 
 		$ListView->xTemplateAssign("SUBPANEL_ID", $this->subpanel_id);
 		$ListView->xTemplateAssign("SUBPANEL_SEARCH", $this->getSearchForm());
 		$display_sps = '';
-		if($this->search_query == '' && empty($this->collections)) $display_sps = 'display:none';
+		if($this->search_query == '' && empty($this->collections)) {
+		    $display_sps = 'display:none';
+		}
 		$ListView->xTemplateAssign("DISPLAY_SPS",$display_sps);
 
 		if(is_admin($current_user) && $_REQUEST['module'] != 'DynamicLayout' && !empty($_SESSION['editinplace']))
@@ -311,10 +314,12 @@ 
   		$moduleInstaller = new ModuleInstaller();
   		$moduleInstaller->silent = true; // make sure that the ModuleInstaller->log() function doesn't echo while rebuilding the layoutdefs
   		$moduleInstaller->rebuild_layoutdefs();
-  		if (file_exists('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php'))
-  			include('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php');
-  		if (file_exists('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php'))
-  			include('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php');
+  		if (file_exists('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php')) {
+  		  			include('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php');
+  		}
+  		if (file_exists('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php')) {
+  		  			include('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php');
+  		}
   }
 
 	function get_subpanel_setup($module)
@@ -400,7 +405,9 @@ 
 
 		$where_clauses = $searchForm->generateSearchWhere(true, $seed->module_dir);
 
-		if (count($where_clauses) > 0 )$this->search_query = '('. implode(' ) AND ( ', $where_clauses) . ')';
+		if (count($where_clauses) > 0 ) {
+		    $this->search_query = '('. implode(' ) AND ( ', $where_clauses) . ')';
+		}
 		$GLOBALS['log']->info("Subpanel Where Clause: $this->search_query");
 
 		return print_r($where_clauses,true);
@@ -436,8 +443,9 @@ 
 
 			$searchForm->setup($subpanel_searchMetaData, $searchMetaData['searchFields'], 'SubpanelSearchFormGeneric.tpl', 'basic_search');
 
-			if(!empty($this->collections))
-				$searchForm->searchFields['collection'] = array();
+			if(!empty($this->collections)) {
+							$searchForm->searchFields['collection'] = array();
+			}
 
 			$searchForm->populateFromRequest();
 

Doc Comments +10 added lines

@@ -102,6 +102,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $template_file
+	 */
 	function setTemplateFile($template_file)
 	{
 		$this->template_file = $template_file;
@@ -271,6 +274,10 @@ 
   }
 
   //saves overrides for defs
+
+  /**
+   * @param string $subsection
+   */
   function saveSubPanelDefOverride( $panel, $subsection, $override){
   		global $layout_defs, $beanList;
 
@@ -406,6 +413,9 @@ 
 		return print_r($where_clauses,true);
 	}
 
+	/**
+	 * @param string $module
+	 */
 	function get_searchdefs($module)
 	{
 		$thisPanel =& $this->subpanel_defs;

Indentation +356 added lines, -356 removed lines

@@ -45,406 +45,406 @@
  */
 class SubPanel
 {
-	var $hideNewButton = false;
-	var $subpanel_id;
-	var $parent_record_id;
-	var $parent_module;  // the name of the parent module
-	var $parent_bean;  // the instantiated bean of the parent
-	var $template_file;
-	var $linked_fields;
-	var $action = 'DetailView';
-	var $show_select_button = true;
-	var $subpanel_define = null;  // contains the layout_def.php
-	var $subpanel_defs;
-	var $subpanel_query=null;
+    var $hideNewButton = false;
+    var $subpanel_id;
+    var $parent_record_id;
+    var $parent_module;  // the name of the parent module
+    var $parent_bean;  // the instantiated bean of the parent
+    var $template_file;
+    var $linked_fields;
+    var $action = 'DetailView';
+    var $show_select_button = true;
+    var $subpanel_define = null;  // contains the layout_def.php
+    var $subpanel_defs;
+    var $subpanel_query=null;
     var $layout_def_key='';
-	var $search_query='';
-	var $collections = array();
-
-	function __construct($module, $record_id, $subpanel_id, $subpanelDef, $layout_def_key='', $collections = array() )
-	{
-		global $beanList, $beanFiles, $focus, $app_strings;
-
-		$this->subpanel_defs=$subpanelDef;
-		$this->subpanel_id = $subpanel_id;
-		$this->parent_record_id = $record_id;
-		$this->parent_module = $module;
-		$this->layout_def_key = $layout_def_key;
-		$this->collections = $collections;
-
-		$this->parent_bean = $focus;
-		$result = $focus;
-
-		if(empty($result))
-		{
-			$parent_bean_name = $beanList[$module];
-			$parent_bean_file = $beanFiles[$parent_bean_name];
-			require_once($parent_bean_file);
-			$this->parent_bean = new $parent_bean_name();
-			$this->parent_bean->retrieve($this->parent_record_id);
-			$result = $this->parent_bean;
-		}
-
-		if($record_id!='fab4' && $result == null)
-		{
-			sugar_die($app_strings['ERROR_NO_RECORD']);
-		}
-		$this->buildSearchQuery();
-		if (empty($subpanelDef)) {
-			//load the subpanel by name.
-			if (!class_exists('MyClass')) {
-				require_once 'include/SubPanel/SubPanelDefinitions.php' ;
-			}
-			$panelsdef=new SubPanelDefinitions($result,$layout_def_key);
-			$subpanelDef=$panelsdef->load_subpanel($subpanel_id, false, false, $this->search_query,$collections);
-			$this->subpanel_defs=$subpanelDef;
-
-		}
-	}
-
-	function setTemplateFile($template_file)
-	{
-		$this->template_file = $template_file;
-	}
-
-	function setBeanList(&$value){
-		$this->bean_list =$value;
-	}
-
-	function setHideNewButton($value){
-		$this->hideNewButton = $value;
-	}
-
-
-	function getHeaderText( $currentModule){
-	}
-
-	function get_buttons( $panel_query=null)
-	{
-
-		$thisPanel =& $this->subpanel_defs;
-		$subpanel_def = $thisPanel->get_buttons();
-
-		if(!isset($this->listview)){
-			$this->listview = new ListView();
-		}
-		$layout_manager = $this->listview->getLayoutManager();
-		$widget_contents = '<div><table cellpadding="0" cellspacing="0"><tr>';
-		foreach($subpanel_def as $widget_data)
-		{
-			$widget_data['action'] = $_REQUEST['action'];
-			$widget_data['module'] =  $thisPanel->get_inst_prop_value('module');
-			$widget_data['focus'] = $this->parent_bean;
-			$widget_data['subpanel_definition'] = $thisPanel;
-			$widget_contents .= '<td style="padding-right: 2px; padding-bottom: 2px;">' . "\n";
-
-			if(empty($widget_data['widget_class']))
-			{
-				$widget_contents .= "widget_class not defined for top subpanel buttons";
-			}
-			else
-			{
-				$widget_contents .= $layout_manager->widgetDisplay($widget_data);
-			}
-
-			$widget_contents .= '</td>';
-		}
-
-		$widget_contents .= '</tr></table></div>';
-		return $widget_contents;
-	}
-
-
-	function ProcessSubPanelListView($xTemplatePath, &$mod_strings)
-	{
-		global $app_strings;
-		global $current_user;
-		global $sugar_config;
-
-		if(isset($this->listview)){
-			$ListView =& $this->listview;
-		}else{
-			$ListView = new ListView();
-		}
-		$ListView->initNewXTemplate($xTemplatePath,$this->subpanel_defs->mod_strings);
-		$ListView->xTemplateAssign("RETURN_URL", "&return_module=".$this->parent_module."&return_action=DetailView&return_id=".$this->parent_bean->id);
-		$ListView->xTemplateAssign("RELATED_MODULE", $this->parent_module);  // TODO: what about unions?
-		$ListView->xTemplateAssign("RECORD_ID", $this->parent_bean->id);
-		$ListView->xTemplateAssign("EDIT_INLINE_PNG", SugarThemeRegistry::current()->getImage('edit_inline','align="absmiddle"  border="0"',null,null,'.gif',$app_strings['LNK_EDIT']));
-		$ListView->xTemplateAssign("DELETE_INLINE_PNG", SugarThemeRegistry::current()->getImage('delete_inline','align="absmiddle" border="0"',null,null,'.gif',$app_strings['LBL_DELETE_INLINE']));
-		$ListView->xTemplateAssign("REMOVE_INLINE_PNG", SugarThemeRegistry::current()->getImage('delete_inline','align="absmiddle" border="0"',null,null,'.gif',$app_strings['LBL_ID_FF_REMOVE']));
-		$header_text= '';
-
-		$ListView->xTemplateAssign("SUBPANEL_ID", $this->subpanel_id);
-		$ListView->xTemplateAssign("SUBPANEL_SEARCH", $this->getSearchForm());
-		$display_sps = '';
-		if($this->search_query == '' && empty($this->collections)) $display_sps = 'display:none';
-		$ListView->xTemplateAssign("DISPLAY_SPS",$display_sps);
-
-		if(is_admin($current_user) && $_REQUEST['module'] != 'DynamicLayout' && !empty($_SESSION['editinplace']))
-		{
-			$exploded = explode('/', $xTemplatePath);
-			$file_name = $exploded[sizeof($exploded) - 1];
-			$mod_name =  $exploded[sizeof($exploded) - 2];
-			$header_text= "&nbsp;<a href='index.php?action=index&module=DynamicLayout&from_action=$file_name&from_module=$mod_name&mod_lang="
-				.$_REQUEST['module']."'>".SugarThemeRegistry::current()->getImage("EditLayout","border='0' align='bottom'",null,null,'.gif','Edit Layout')."</a>";
-		}
-		$ListView->setHeaderTitle('');
-		$ListView->setHeaderText('');
-
-		ob_start();
-
-		$ListView->is_dynamic = true;
-		$ListView->records_per_page = $sugar_config['list_max_entries_per_subpanel'] + 0;
-		if (isset($this->subpanel_defs->_instance_properties['records_per_page'])) {
-			$ListView->records_per_page = $this->subpanel_defs->_instance_properties['records_per_page'] + 0;
-		}
-		$ListView->start_link_wrapper = "javascript:showSubPanel('".$this->subpanel_id."','";
-		$ListView->subpanel_id = $this->subpanel_id;
-		$ListView->end_link_wrapper = "',true);";
-		if ( !empty($this->layout_def_key) ) {
-			$ListView->end_link_wrapper = '&layout_def_key='.$this->layout_def_key.$ListView->end_link_wrapper;
-		}
-
-		$where = '';
-		$ListView->setQuery($where, '', '', '');
-		$ListView->show_export_button = false;
-
-		//function returns the query that was used to populate sub-panel data.
-
-		$query=$ListView->process_dynamic_listview($this->parent_module, $this->parent_bean,$this->subpanel_defs);
-		$this->subpanel_query=$query;
-		$ob_contents = ob_get_contents();
-		ob_end_clean();
-		return $ob_contents;
-	}
-
-	function display()
-	{
-		$result_array = array();
-
-		$return_string = $this->ProcessSubPanelListView($this->template_file,$result_array);
-
-		print $return_string;
-	}
-
-	function getModulesWithSubpanels()
-	{
-		global $beanList;
-		$dir = dir('modules');
-		$modules = array();
-		while($entry = $dir->read())
-		{
-			if(file_exists('modules/' . $entry . '/layout_defs.php'))
-			{
-				$modules[$entry] = $entry;
-			}
-		}
-		return $modules;
-	}
-
-  static function getModuleSubpanels($module){
-  	require_once('include/SubPanel/SubPanelDefinitions.php');
-  		global $beanList, $beanFiles;
-  		if(!isset($beanList[$module])){
-  			return array();
-  		}
-
-  		$class = $beanList[$module];
-  		require_once($beanFiles[$class]);
-  		$mod = new $class();
-  		$spd = new SubPanelDefinitions($mod);
-  		$tabs = $spd->get_available_tabs(true);
-  		$ret_tabs = array();
-  		$reject_tabs = array('history'=>1, 'activities'=>1);
-  		foreach($tabs as $key=>$tab){
-  		    foreach($tab as $k=>$v){
+    var $search_query='';
+    var $collections = array();
+
+    function __construct($module, $record_id, $subpanel_id, $subpanelDef, $layout_def_key='', $collections = array() )
+    {
+        global $beanList, $beanFiles, $focus, $app_strings;
+
+        $this->subpanel_defs=$subpanelDef;
+        $this->subpanel_id = $subpanel_id;
+        $this->parent_record_id = $record_id;
+        $this->parent_module = $module;
+        $this->layout_def_key = $layout_def_key;
+        $this->collections = $collections;
+
+        $this->parent_bean = $focus;
+        $result = $focus;
+
+        if(empty($result))
+        {
+            $parent_bean_name = $beanList[$module];
+            $parent_bean_file = $beanFiles[$parent_bean_name];
+            require_once($parent_bean_file);
+            $this->parent_bean = new $parent_bean_name();
+            $this->parent_bean->retrieve($this->parent_record_id);
+            $result = $this->parent_bean;
+        }
+
+        if($record_id!='fab4' && $result == null)
+        {
+            sugar_die($app_strings['ERROR_NO_RECORD']);
+        }
+        $this->buildSearchQuery();
+        if (empty($subpanelDef)) {
+            //load the subpanel by name.
+            if (!class_exists('MyClass')) {
+                require_once 'include/SubPanel/SubPanelDefinitions.php' ;
+            }
+            $panelsdef=new SubPanelDefinitions($result,$layout_def_key);
+            $subpanelDef=$panelsdef->load_subpanel($subpanel_id, false, false, $this->search_query,$collections);
+            $this->subpanel_defs=$subpanelDef;
+
+        }
+    }
+
+    function setTemplateFile($template_file)
+    {
+        $this->template_file = $template_file;
+    }
+
+    function setBeanList(&$value){
+        $this->bean_list =$value;
+    }
+
+    function setHideNewButton($value){
+        $this->hideNewButton = $value;
+    }
+
+
+    function getHeaderText( $currentModule){
+    }
+
+    function get_buttons( $panel_query=null)
+    {
+
+        $thisPanel =& $this->subpanel_defs;
+        $subpanel_def = $thisPanel->get_buttons();
+
+        if(!isset($this->listview)){
+            $this->listview = new ListView();
+        }
+        $layout_manager = $this->listview->getLayoutManager();
+        $widget_contents = '<div><table cellpadding="0" cellspacing="0"><tr>';
+        foreach($subpanel_def as $widget_data)
+        {
+            $widget_data['action'] = $_REQUEST['action'];
+            $widget_data['module'] =  $thisPanel->get_inst_prop_value('module');
+            $widget_data['focus'] = $this->parent_bean;
+            $widget_data['subpanel_definition'] = $thisPanel;
+            $widget_contents .= '<td style="padding-right: 2px; padding-bottom: 2px;">' . "\n";
+
+            if(empty($widget_data['widget_class']))
+            {
+                $widget_contents .= "widget_class not defined for top subpanel buttons";
+            }
+            else
+            {
+                $widget_contents .= $layout_manager->widgetDisplay($widget_data);
+            }
+
+            $widget_contents .= '</td>';
+        }
+
+        $widget_contents .= '</tr></table></div>';
+        return $widget_contents;
+    }
+
+
+    function ProcessSubPanelListView($xTemplatePath, &$mod_strings)
+    {
+        global $app_strings;
+        global $current_user;
+        global $sugar_config;
+
+        if(isset($this->listview)){
+            $ListView =& $this->listview;
+        }else{
+            $ListView = new ListView();
+        }
+        $ListView->initNewXTemplate($xTemplatePath,$this->subpanel_defs->mod_strings);
+        $ListView->xTemplateAssign("RETURN_URL", "&return_module=".$this->parent_module."&return_action=DetailView&return_id=".$this->parent_bean->id);
+        $ListView->xTemplateAssign("RELATED_MODULE", $this->parent_module);  // TODO: what about unions?
+        $ListView->xTemplateAssign("RECORD_ID", $this->parent_bean->id);
+        $ListView->xTemplateAssign("EDIT_INLINE_PNG", SugarThemeRegistry::current()->getImage('edit_inline','align="absmiddle"  border="0"',null,null,'.gif',$app_strings['LNK_EDIT']));
+        $ListView->xTemplateAssign("DELETE_INLINE_PNG", SugarThemeRegistry::current()->getImage('delete_inline','align="absmiddle" border="0"',null,null,'.gif',$app_strings['LBL_DELETE_INLINE']));
+        $ListView->xTemplateAssign("REMOVE_INLINE_PNG", SugarThemeRegistry::current()->getImage('delete_inline','align="absmiddle" border="0"',null,null,'.gif',$app_strings['LBL_ID_FF_REMOVE']));
+        $header_text= '';
+
+        $ListView->xTemplateAssign("SUBPANEL_ID", $this->subpanel_id);
+        $ListView->xTemplateAssign("SUBPANEL_SEARCH", $this->getSearchForm());
+        $display_sps = '';
+        if($this->search_query == '' && empty($this->collections)) $display_sps = 'display:none';
+        $ListView->xTemplateAssign("DISPLAY_SPS",$display_sps);
+
+        if(is_admin($current_user) && $_REQUEST['module'] != 'DynamicLayout' && !empty($_SESSION['editinplace']))
+        {
+            $exploded = explode('/', $xTemplatePath);
+            $file_name = $exploded[sizeof($exploded) - 1];
+            $mod_name =  $exploded[sizeof($exploded) - 2];
+            $header_text= "&nbsp;<a href='index.php?action=index&module=DynamicLayout&from_action=$file_name&from_module=$mod_name&mod_lang="
+                .$_REQUEST['module']."'>".SugarThemeRegistry::current()->getImage("EditLayout","border='0' align='bottom'",null,null,'.gif','Edit Layout')."</a>";
+        }
+        $ListView->setHeaderTitle('');
+        $ListView->setHeaderText('');
+
+        ob_start();
+
+        $ListView->is_dynamic = true;
+        $ListView->records_per_page = $sugar_config['list_max_entries_per_subpanel'] + 0;
+        if (isset($this->subpanel_defs->_instance_properties['records_per_page'])) {
+            $ListView->records_per_page = $this->subpanel_defs->_instance_properties['records_per_page'] + 0;
+        }
+        $ListView->start_link_wrapper = "javascript:showSubPanel('".$this->subpanel_id."','";
+        $ListView->subpanel_id = $this->subpanel_id;
+        $ListView->end_link_wrapper = "',true);";
+        if ( !empty($this->layout_def_key) ) {
+            $ListView->end_link_wrapper = '&layout_def_key='.$this->layout_def_key.$ListView->end_link_wrapper;
+        }
+
+        $where = '';
+        $ListView->setQuery($where, '', '', '');
+        $ListView->show_export_button = false;
+
+        //function returns the query that was used to populate sub-panel data.
+
+        $query=$ListView->process_dynamic_listview($this->parent_module, $this->parent_bean,$this->subpanel_defs);
+        $this->subpanel_query=$query;
+        $ob_contents = ob_get_contents();
+        ob_end_clean();
+        return $ob_contents;
+    }
+
+    function display()
+    {
+        $result_array = array();
+
+        $return_string = $this->ProcessSubPanelListView($this->template_file,$result_array);
+
+        print $return_string;
+    }
+
+    function getModulesWithSubpanels()
+    {
+        global $beanList;
+        $dir = dir('modules');
+        $modules = array();
+        while($entry = $dir->read())
+        {
+            if(file_exists('modules/' . $entry . '/layout_defs.php'))
+            {
+                $modules[$entry] = $entry;
+            }
+        }
+        return $modules;
+    }
+
+    static function getModuleSubpanels($module){
+        require_once('include/SubPanel/SubPanelDefinitions.php');
+            global $beanList, $beanFiles;
+            if(!isset($beanList[$module])){
+                return array();
+            }
+
+            $class = $beanList[$module];
+            require_once($beanFiles[$class]);
+            $mod = new $class();
+            $spd = new SubPanelDefinitions($mod);
+            $tabs = $spd->get_available_tabs(true);
+            $ret_tabs = array();
+            $reject_tabs = array('history'=>1, 'activities'=>1);
+            foreach($tabs as $key=>$tab){
+                foreach($tab as $k=>$v){
                 if (! isset ( $reject_tabs [$k] )) {
                     $ret_tabs [$k] = $v;
                 }
             }
-  		}
+            }
 
-  		return $ret_tabs;
+            return $ret_tabs;
 
 
-  }
+    }
 
-  //saves overrides for defs
-  function saveSubPanelDefOverride( $panel, $subsection, $override){
-  		global $layout_defs, $beanList;
+    //saves overrides for defs
+    function saveSubPanelDefOverride( $panel, $subsection, $override){
+            global $layout_defs, $beanList;
 
-  		//save the new subpanel
-  		$name = "subpanel_layout['list_fields']";
+            //save the new subpanel
+            $name = "subpanel_layout['list_fields']";
 
-  		//bugfix: load looks for moduleName/metadata/subpanels, not moduleName/subpanels
-  		$path = 'custom/modules/'. $panel->_instance_properties['module'] . '/metadata/subpanels';
+            //bugfix: load looks for moduleName/metadata/subpanels, not moduleName/subpanels
+            $path = 'custom/modules/'. $panel->_instance_properties['module'] . '/metadata/subpanels';
 
-  		//bug# 40171: "Custom subpanels not working as expected"
-  		//each custom subpanel needs to have a unique custom def file
-  		$filename = $panel->parent_bean->object_name . "_subpanel_" . $panel->name; //bug 42262 (filename with $panel->_instance_properties['get_subpanel_data'] can create problem if had word "function" in it)
-  		$oldName1 = '_override' . $panel->parent_bean->object_name .$panel->_instance_properties['module'] . $panel->_instance_properties['subpanel_name'] ;
-  		$oldName2 = '_override' . $panel->parent_bean->object_name .$panel->_instance_properties['get_subpanel_data'] ;
-  		if (file_exists('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName1.php")){
-  		  unlink('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName1.php");
-  		}
-  		if (file_exists('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName2.php")){
-         unlink('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName2.php");
-  		}
-  		$extname = '_override'.$filename;
-  		//end of bug# 40171
+            //bug# 40171: "Custom subpanels not working as expected"
+            //each custom subpanel needs to have a unique custom def file
+            $filename = $panel->parent_bean->object_name . "_subpanel_" . $panel->name; //bug 42262 (filename with $panel->_instance_properties['get_subpanel_data'] can create problem if had word "function" in it)
+            $oldName1 = '_override' . $panel->parent_bean->object_name .$panel->_instance_properties['module'] . $panel->_instance_properties['subpanel_name'] ;
+            $oldName2 = '_override' . $panel->parent_bean->object_name .$panel->_instance_properties['get_subpanel_data'] ;
+            if (file_exists('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName1.php")){
+            unlink('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName1.php");
+            }
+            if (file_exists('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName2.php")){
+            unlink('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName2.php");
+            }
+            $extname = '_override'.$filename;
+            //end of bug# 40171
 
-  		mkdir_recursive($path, true);
-  		write_array_to_file( $name, $override,$path.'/' . $filename .'.php');
+            mkdir_recursive($path, true);
+            write_array_to_file( $name, $override,$path.'/' . $filename .'.php');
 
-  		//save the override for the layoutdef
+            //save the override for the layoutdef
         //tyoung 10.12.07 pushed panel->name to lowercase to match case in subpaneldefs.php files -
         //gave error on bad index 'module' as this override key didn't match the key in the subpaneldefs
-  		$name = "layout_defs['".  $panel->parent_bean->module_dir. "']['subpanel_setup']['" .strtolower($panel->name). "']";
+            $name = "layout_defs['".  $panel->parent_bean->module_dir. "']['subpanel_setup']['" .strtolower($panel->name). "']";
 //  	$GLOBALS['log']->debug('SubPanel.php->saveSubPanelDefOverride(): '.$name);
-  		$newValue = override_value_to_string($name, 'override_subpanel_name', $filename);
-  		mkdir_recursive('custom/Extension/modules/'. $panel->parent_bean->module_dir . '/Ext/Layoutdefs', true);
-  		$fp = sugar_fopen('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$extname.php", 'w');
-  		fwrite($fp, "<?php\n//auto-generated file DO NOT EDIT\n$newValue\n?>");
-  		fclose($fp);
-  		require_once('ModuleInstall/ModuleInstaller.php');
-  		$moduleInstaller = new ModuleInstaller();
-  		$moduleInstaller->silent = true; // make sure that the ModuleInstaller->log() function doesn't echo while rebuilding the layoutdefs
-  		$moduleInstaller->rebuild_layoutdefs();
-  		if (file_exists('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php'))
-  			include('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php');
-  		if (file_exists('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php'))
-  			include('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php');
-  }
-
-	function get_subpanel_setup($module)
-	{
-		$subpanel_setup = '';
-		$layout_defs = get_layout_defs();
-
-		if(!empty($layout_defs) && !empty($layout_defs[$module]['subpanel_setup']))
-      {
-      	$subpanel_setup = $layout_defs[$module]['subpanel_setup'];
-      }
-
-      return $subpanel_setup;
-	}
-
-	/**
-	 * Retrieve the subpanel definition from the registered layout_defs arrays.
-	 */
-	function getSubPanelDefine($module, $subpanel_id)
-	{
-		$default_subpanel_define = SubPanel::_get_default_subpanel_define($module, $subpanel_id);
-		$custom_subpanel_define = SubPanel::_get_custom_subpanel_define($module, $subpanel_id);
-
-		$subpanel_define = array_merge($default_subpanel_define, $custom_subpanel_define);
-
-		if(empty($subpanel_define))
-		{
-			print('Could not load subpanel definition for: ' . $subpanel_id);
-		}
-
-		return $subpanel_define;
-	}
-
-	function _get_custom_subpanel_define($module, $subpanel_id)
-	{
-		$ret_val = array();
-
-		if($subpanel_id != '')
-		{
-			$layout_defs = get_layout_defs();
-
-			if(!empty($layout_defs[$module]['custom_subpanel_defines'][$subpanel_id]))
-			{
-				$ret_val = $layout_defs[$module]['custom_subpanel_defines'][$subpanel_id];
-			}
-		}
+            $newValue = override_value_to_string($name, 'override_subpanel_name', $filename);
+            mkdir_recursive('custom/Extension/modules/'. $panel->parent_bean->module_dir . '/Ext/Layoutdefs', true);
+            $fp = sugar_fopen('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$extname.php", 'w');
+            fwrite($fp, "<?php\n//auto-generated file DO NOT EDIT\n$newValue\n?>");
+            fclose($fp);
+            require_once('ModuleInstall/ModuleInstaller.php');
+            $moduleInstaller = new ModuleInstaller();
+            $moduleInstaller->silent = true; // make sure that the ModuleInstaller->log() function doesn't echo while rebuilding the layoutdefs
+            $moduleInstaller->rebuild_layoutdefs();
+            if (file_exists('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php'))
+                include('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php');
+            if (file_exists('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php'))
+                include('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php');
+    }
+
+    function get_subpanel_setup($module)
+    {
+        $subpanel_setup = '';
+        $layout_defs = get_layout_defs();
+
+        if(!empty($layout_defs) && !empty($layout_defs[$module]['subpanel_setup']))
+        {
+            $subpanel_setup = $layout_defs[$module]['subpanel_setup'];
+        }
+
+        return $subpanel_setup;
+    }
+
+    /**
+     * Retrieve the subpanel definition from the registered layout_defs arrays.
+     */
+    function getSubPanelDefine($module, $subpanel_id)
+    {
+        $default_subpanel_define = SubPanel::_get_default_subpanel_define($module, $subpanel_id);
+        $custom_subpanel_define = SubPanel::_get_custom_subpanel_define($module, $subpanel_id);
+
+        $subpanel_define = array_merge($default_subpanel_define, $custom_subpanel_define);
+
+        if(empty($subpanel_define))
+        {
+            print('Could not load subpanel definition for: ' . $subpanel_id);
+        }
+
+        return $subpanel_define;
+    }
+
+    function _get_custom_subpanel_define($module, $subpanel_id)
+    {
+        $ret_val = array();
+
+        if($subpanel_id != '')
+        {
+            $layout_defs = get_layout_defs();
+
+            if(!empty($layout_defs[$module]['custom_subpanel_defines'][$subpanel_id]))
+            {
+                $ret_val = $layout_defs[$module]['custom_subpanel_defines'][$subpanel_id];
+            }
+        }
 
-		return $ret_val;
-	}
+        return $ret_val;
+    }
 
-	function _get_default_subpanel_define($module, $subpanel_id)
-	{
-		$ret_val = array();
+    function _get_default_subpanel_define($module, $subpanel_id)
+    {
+        $ret_val = array();
 
-		if($subpanel_id != '')
-		{
-	  		$layout_defs = get_layout_defs();
+        if($subpanel_id != '')
+        {
+                $layout_defs = get_layout_defs();
 
-			if(!empty($layout_defs[$subpanel_id]['default_subpanel_define']))
-			{
-				$ret_val = $layout_defs[$subpanel_id]['default_subpanel_define'];
-			}
-		}
+            if(!empty($layout_defs[$subpanel_id]['default_subpanel_define']))
+            {
+                $ret_val = $layout_defs[$subpanel_id]['default_subpanel_define'];
+            }
+        }
 
-		return $ret_val;
-	}
+        return $ret_val;
+    }
 
-	function buildSearchQuery()
-	{
-		require_once('include/SubPanel/SubPanelSearchForm.php');
+    function buildSearchQuery()
+    {
+        require_once('include/SubPanel/SubPanelSearchForm.php');
 
-		$module = 'Meetings';
+        $module = 'Meetings';
 
-		$seed = new Meeting();
+        $seed = new Meeting();
 
-		$_REQUEST['searchFormTab'] = 'basic_search';
-		$searchForm = new SubPanelSearchForm($seed, $module, $this);
+        $_REQUEST['searchFormTab'] = 'basic_search';
+        $searchForm = new SubPanelSearchForm($seed, $module, $this);
 
-		$searchMetaData = $searchForm->retrieveSearchDefs($module);
-		$searchForm->setup($searchMetaData['searchdefs'], $searchMetaData['searchFields'], 'SubpanelSearchFormGeneric.tpl', 'basic_search');
+        $searchMetaData = $searchForm->retrieveSearchDefs($module);
+        $searchForm->setup($searchMetaData['searchdefs'], $searchMetaData['searchFields'], 'SubpanelSearchFormGeneric.tpl', 'basic_search');
 
-		$searchForm->populateFromRequest();
+        $searchForm->populateFromRequest();
 
-		$where_clauses = $searchForm->generateSearchWhere(true, $seed->module_dir);
+        $where_clauses = $searchForm->generateSearchWhere(true, $seed->module_dir);
 
-		if (count($where_clauses) > 0 )$this->search_query = '('. implode(' ) AND ( ', $where_clauses) . ')';
-		$GLOBALS['log']->info("Subpanel Where Clause: $this->search_query");
+        if (count($where_clauses) > 0 )$this->search_query = '('. implode(' ) AND ( ', $where_clauses) . ')';
+        $GLOBALS['log']->info("Subpanel Where Clause: $this->search_query");
 
-		return print_r($where_clauses,true);
-	}
+        return print_r($where_clauses,true);
+    }
 
-	function get_searchdefs($module)
-	{
-		$thisPanel =& $this->subpanel_defs;
-		$subpanel_defs = $thisPanel->_instance_properties;
+    function get_searchdefs($module)
+    {
+        $thisPanel =& $this->subpanel_defs;
+        $subpanel_defs = $thisPanel->_instance_properties;
 
-		if(isset($subpanel_defs['searchdefs'])){
-			$searchdefs[$module]['layout']['basic_search'] = $subpanel_defs['searchdefs'];
-			$searchdefs[$module]['templateMeta'] = Array ('maxColumns' => 3, 'maxColumnsBasic' => 4, 'widths' => Array ( 'label' => 10, 'field' => 30 )) ;
-			return $searchdefs;
-		}
+        if(isset($subpanel_defs['searchdefs'])){
+            $searchdefs[$module]['layout']['basic_search'] = $subpanel_defs['searchdefs'];
+            $searchdefs[$module]['templateMeta'] = Array ('maxColumns' => 3, 'maxColumnsBasic' => 4, 'widths' => Array ( 'label' => 10, 'field' => 30 )) ;
+            return $searchdefs;
+        }
 
-		return false;
-	}
+        return false;
+    }
 
-	function getSearchForm()
-	{
-		require_once('include/SubPanel/SubPanelSearchForm.php');
+    function getSearchForm()
+    {
+        require_once('include/SubPanel/SubPanelSearchForm.php');
 
-		$module = 'Meetings';
+        $module = 'Meetings';
 
-		$seed = new Meeting();
+        $seed = new Meeting();
 
-		$searchForm = new SubPanelSearchForm($seed, $module, $this);
+        $searchForm = new SubPanelSearchForm($seed, $module, $this);
 
-		$searchMetaData = $searchForm->retrieveSearchDefs($module);
+        $searchMetaData = $searchForm->retrieveSearchDefs($module);
 
-		if ($subpanel_searchMetaData = $this->get_searchdefs($module)){
+        if ($subpanel_searchMetaData = $this->get_searchdefs($module)){
 
-			$searchForm->setup($subpanel_searchMetaData, $searchMetaData['searchFields'], 'SubpanelSearchFormGeneric.tpl', 'basic_search');
+            $searchForm->setup($subpanel_searchMetaData, $searchMetaData['searchFields'], 'SubpanelSearchFormGeneric.tpl', 'basic_search');
 
-			if(!empty($this->collections))
-				$searchForm->searchFields['collection'] = array();
+            if(!empty($this->collections))
+                $searchForm->searchFields['collection'] = array();
 
-			$searchForm->populateFromRequest();
+            $searchForm->populateFromRequest();
 
-			return $searchForm->display();
-		}
+            return $searchForm->display();
+        }
 
-		return '';
-	}
+        return '';
+    }
 }
 ?>

Spacing +83 added lines, -83 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -48,24 +48,24 @@ 
 	var $hideNewButton = false;
 	var $subpanel_id;
 	var $parent_record_id;
-	var $parent_module;  // the name of the parent module
-	var $parent_bean;  // the instantiated bean of the parent
+	var $parent_module; // the name of the parent module
+	var $parent_bean; // the instantiated bean of the parent
 	var $template_file;
 	var $linked_fields;
 	var $action = 'DetailView';
 	var $show_select_button = true;
-	var $subpanel_define = null;  // contains the layout_def.php
+	var $subpanel_define = null; // contains the layout_def.php
 	var $subpanel_defs;
-	var $subpanel_query=null;
-    var $layout_def_key='';
-	var $search_query='';
+	var $subpanel_query = null;
+    var $layout_def_key = '';
+	var $search_query = '';
 	var $collections = array();
 
-	function __construct($module, $record_id, $subpanel_id, $subpanelDef, $layout_def_key='', $collections = array() )
+	function __construct($module, $record_id, $subpanel_id, $subpanelDef, $layout_def_key = '', $collections = array())
 	{
 		global $beanList, $beanFiles, $focus, $app_strings;
 
-		$this->subpanel_defs=$subpanelDef;
+		$this->subpanel_defs = $subpanelDef;
 		$this->subpanel_id = $subpanel_id;
 		$this->parent_record_id = $record_id;
 		$this->parent_module = $module;
@@ -75,7 +75,7 @@ 
 		$this->parent_bean = $focus;
 		$result = $focus;
 
-		if(empty($result))
+		if (empty($result))
 		{
 			$parent_bean_name = $beanList[$module];
 			$parent_bean_file = $beanFiles[$parent_bean_name];
@@ -85,7 +85,7 @@ 
 			$result = $this->parent_bean;
 		}
 
-		if($record_id!='fab4' && $result == null)
+		if ($record_id != 'fab4' && $result == null)
 		{
 			sugar_die($app_strings['ERROR_NO_RECORD']);
 		}
@@ -93,11 +93,11 @@ 
 		if (empty($subpanelDef)) {
 			//load the subpanel by name.
 			if (!class_exists('MyClass')) {
-				require_once 'include/SubPanel/SubPanelDefinitions.php' ;
+				require_once 'include/SubPanel/SubPanelDefinitions.php';
 			}
-			$panelsdef=new SubPanelDefinitions($result,$layout_def_key);
-			$subpanelDef=$panelsdef->load_subpanel($subpanel_id, false, false, $this->search_query,$collections);
-			$this->subpanel_defs=$subpanelDef;
+			$panelsdef = new SubPanelDefinitions($result, $layout_def_key);
+			$subpanelDef = $panelsdef->load_subpanel($subpanel_id, false, false, $this->search_query, $collections);
+			$this->subpanel_defs = $subpanelDef;
 
 		}
 	}
@@ -107,38 +107,38 @@ 
 		$this->template_file = $template_file;
 	}
 
-	function setBeanList(&$value){
-		$this->bean_list =$value;
+	function setBeanList(&$value) {
+		$this->bean_list = $value;
 	}
 
-	function setHideNewButton($value){
+	function setHideNewButton($value) {
 		$this->hideNewButton = $value;
 	}
 
 
-	function getHeaderText( $currentModule){
+	function getHeaderText($currentModule) {
 	}
 
-	function get_buttons( $panel_query=null)
+	function get_buttons($panel_query = null)
 	{
 
-		$thisPanel =& $this->subpanel_defs;
+		$thisPanel = & $this->subpanel_defs;
 		$subpanel_def = $thisPanel->get_buttons();
 
-		if(!isset($this->listview)){
+		if (!isset($this->listview)) {
 			$this->listview = new ListView();
 		}
 		$layout_manager = $this->listview->getLayoutManager();
 		$widget_contents = '<div><table cellpadding="0" cellspacing="0"><tr>';
-		foreach($subpanel_def as $widget_data)
+		foreach ($subpanel_def as $widget_data)
 		{
 			$widget_data['action'] = $_REQUEST['action'];
-			$widget_data['module'] =  $thisPanel->get_inst_prop_value('module');
+			$widget_data['module'] = $thisPanel->get_inst_prop_value('module');
 			$widget_data['focus'] = $this->parent_bean;
 			$widget_data['subpanel_definition'] = $thisPanel;
-			$widget_contents .= '<td style="padding-right: 2px; padding-bottom: 2px;">' . "\n";
+			$widget_contents .= '<td style="padding-right: 2px; padding-bottom: 2px;">'."\n";
 
-			if(empty($widget_data['widget_class']))
+			if (empty($widget_data['widget_class']))
 			{
 				$widget_contents .= "widget_class not defined for top subpanel buttons";
 			}
@@ -161,33 +161,33 @@ 
 		global $current_user;
 		global $sugar_config;
 
-		if(isset($this->listview)){
-			$ListView =& $this->listview;
-		}else{
+		if (isset($this->listview)) {
+			$ListView = & $this->listview;
+		} else {
 			$ListView = new ListView();
 		}
-		$ListView->initNewXTemplate($xTemplatePath,$this->subpanel_defs->mod_strings);
+		$ListView->initNewXTemplate($xTemplatePath, $this->subpanel_defs->mod_strings);
 		$ListView->xTemplateAssign("RETURN_URL", "&return_module=".$this->parent_module."&return_action=DetailView&return_id=".$this->parent_bean->id);
-		$ListView->xTemplateAssign("RELATED_MODULE", $this->parent_module);  // TODO: what about unions?
+		$ListView->xTemplateAssign("RELATED_MODULE", $this->parent_module); // TODO: what about unions?
 		$ListView->xTemplateAssign("RECORD_ID", $this->parent_bean->id);
-		$ListView->xTemplateAssign("EDIT_INLINE_PNG", SugarThemeRegistry::current()->getImage('edit_inline','align="absmiddle"  border="0"',null,null,'.gif',$app_strings['LNK_EDIT']));
-		$ListView->xTemplateAssign("DELETE_INLINE_PNG", SugarThemeRegistry::current()->getImage('delete_inline','align="absmiddle" border="0"',null,null,'.gif',$app_strings['LBL_DELETE_INLINE']));
-		$ListView->xTemplateAssign("REMOVE_INLINE_PNG", SugarThemeRegistry::current()->getImage('delete_inline','align="absmiddle" border="0"',null,null,'.gif',$app_strings['LBL_ID_FF_REMOVE']));
-		$header_text= '';
+		$ListView->xTemplateAssign("EDIT_INLINE_PNG", SugarThemeRegistry::current()->getImage('edit_inline', 'align="absmiddle"  border="0"', null, null, '.gif', $app_strings['LNK_EDIT']));
+		$ListView->xTemplateAssign("DELETE_INLINE_PNG", SugarThemeRegistry::current()->getImage('delete_inline', 'align="absmiddle" border="0"', null, null, '.gif', $app_strings['LBL_DELETE_INLINE']));
+		$ListView->xTemplateAssign("REMOVE_INLINE_PNG", SugarThemeRegistry::current()->getImage('delete_inline', 'align="absmiddle" border="0"', null, null, '.gif', $app_strings['LBL_ID_FF_REMOVE']));
+		$header_text = '';
 
 		$ListView->xTemplateAssign("SUBPANEL_ID", $this->subpanel_id);
 		$ListView->xTemplateAssign("SUBPANEL_SEARCH", $this->getSearchForm());
 		$display_sps = '';
-		if($this->search_query == '' && empty($this->collections)) $display_sps = 'display:none';
-		$ListView->xTemplateAssign("DISPLAY_SPS",$display_sps);
+		if ($this->search_query == '' && empty($this->collections)) $display_sps = 'display:none';
+		$ListView->xTemplateAssign("DISPLAY_SPS", $display_sps);
 
-		if(is_admin($current_user) && $_REQUEST['module'] != 'DynamicLayout' && !empty($_SESSION['editinplace']))
+		if (is_admin($current_user) && $_REQUEST['module'] != 'DynamicLayout' && !empty($_SESSION['editinplace']))
 		{
 			$exploded = explode('/', $xTemplatePath);
 			$file_name = $exploded[sizeof($exploded) - 1];
-			$mod_name =  $exploded[sizeof($exploded) - 2];
-			$header_text= "&nbsp;<a href='index.php?action=index&module=DynamicLayout&from_action=$file_name&from_module=$mod_name&mod_lang="
-				.$_REQUEST['module']."'>".SugarThemeRegistry::current()->getImage("EditLayout","border='0' align='bottom'",null,null,'.gif','Edit Layout')."</a>";
+			$mod_name = $exploded[sizeof($exploded) - 2];
+			$header_text = "&nbsp;<a href='index.php?action=index&module=DynamicLayout&from_action=$file_name&from_module=$mod_name&mod_lang="
+				.$_REQUEST['module']."'>".SugarThemeRegistry::current()->getImage("EditLayout", "border='0' align='bottom'", null, null, '.gif', 'Edit Layout')."</a>";
 		}
 		$ListView->setHeaderTitle('');
 		$ListView->setHeaderText('');
@@ -202,7 +202,7 @@ 
 		$ListView->start_link_wrapper = "javascript:showSubPanel('".$this->subpanel_id."','";
 		$ListView->subpanel_id = $this->subpanel_id;
 		$ListView->end_link_wrapper = "',true);";
-		if ( !empty($this->layout_def_key) ) {
+		if (!empty($this->layout_def_key)) {
 			$ListView->end_link_wrapper = '&layout_def_key='.$this->layout_def_key.$ListView->end_link_wrapper;
 		}
 
@@ -212,8 +212,8 @@ 
 
 		//function returns the query that was used to populate sub-panel data.
 
-		$query=$ListView->process_dynamic_listview($this->parent_module, $this->parent_bean,$this->subpanel_defs);
-		$this->subpanel_query=$query;
+		$query = $ListView->process_dynamic_listview($this->parent_module, $this->parent_bean, $this->subpanel_defs);
+		$this->subpanel_query = $query;
 		$ob_contents = ob_get_contents();
 		ob_end_clean();
 		return $ob_contents;
@@ -223,7 +223,7 @@ 
 	{
 		$result_array = array();
 
-		$return_string = $this->ProcessSubPanelListView($this->template_file,$result_array);
+		$return_string = $this->ProcessSubPanelListView($this->template_file, $result_array);
 
 		print $return_string;
 	}
@@ -233,9 +233,9 @@ 
 		global $beanList;
 		$dir = dir('modules');
 		$modules = array();
-		while($entry = $dir->read())
+		while ($entry = $dir->read())
 		{
-			if(file_exists('modules/' . $entry . '/layout_defs.php'))
+			if (file_exists('modules/'.$entry.'/layout_defs.php'))
 			{
 				$modules[$entry] = $entry;
 			}
@@ -243,10 +243,10 @@ 
 		return $modules;
 	}
 
-  static function getModuleSubpanels($module){
+  static function getModuleSubpanels($module) {
   	require_once('include/SubPanel/SubPanelDefinitions.php');
   		global $beanList, $beanFiles;
-  		if(!isset($beanList[$module])){
+  		if (!isset($beanList[$module])) {
   			return array();
   		}
 
@@ -257,9 +257,9 @@ 
   		$tabs = $spd->get_available_tabs(true);
   		$ret_tabs = array();
   		$reject_tabs = array('history'=>1, 'activities'=>1);
-  		foreach($tabs as $key=>$tab){
-  		    foreach($tab as $k=>$v){
-                if (! isset ( $reject_tabs [$k] )) {
+  		foreach ($tabs as $key=>$tab) {
+  		    foreach ($tab as $k=>$v) {
+                if (!isset ($reject_tabs [$k])) {
                     $ret_tabs [$k] = $v;
                 }
             }
@@ -271,50 +271,50 @@ 
   }
 
   //saves overrides for defs
-  function saveSubPanelDefOverride( $panel, $subsection, $override){
+  function saveSubPanelDefOverride($panel, $subsection, $override) {
   		global $layout_defs, $beanList;
 
   		//save the new subpanel
   		$name = "subpanel_layout['list_fields']";
 
   		//bugfix: load looks for moduleName/metadata/subpanels, not moduleName/subpanels
-  		$path = 'custom/modules/'. $panel->_instance_properties['module'] . '/metadata/subpanels';
+  		$path = 'custom/modules/'.$panel->_instance_properties['module'].'/metadata/subpanels';
 
   		//bug# 40171: "Custom subpanels not working as expected"
   		//each custom subpanel needs to have a unique custom def file
-  		$filename = $panel->parent_bean->object_name . "_subpanel_" . $panel->name; //bug 42262 (filename with $panel->_instance_properties['get_subpanel_data'] can create problem if had word "function" in it)
-  		$oldName1 = '_override' . $panel->parent_bean->object_name .$panel->_instance_properties['module'] . $panel->_instance_properties['subpanel_name'] ;
-  		$oldName2 = '_override' . $panel->parent_bean->object_name .$panel->_instance_properties['get_subpanel_data'] ;
-  		if (file_exists('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName1.php")){
-  		  unlink('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName1.php");
+  		$filename = $panel->parent_bean->object_name."_subpanel_".$panel->name; //bug 42262 (filename with $panel->_instance_properties['get_subpanel_data'] can create problem if had word "function" in it)
+  		$oldName1 = '_override'.$panel->parent_bean->object_name.$panel->_instance_properties['module'].$panel->_instance_properties['subpanel_name'];
+  		$oldName2 = '_override'.$panel->parent_bean->object_name.$panel->_instance_properties['get_subpanel_data'];
+  		if (file_exists('custom/Extension/modules/'.$panel->parent_bean->module_dir."/Ext/Layoutdefs/$oldName1.php")) {
+  		  unlink('custom/Extension/modules/'.$panel->parent_bean->module_dir."/Ext/Layoutdefs/$oldName1.php");
   		}
-  		if (file_exists('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName2.php")){
-         unlink('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$oldName2.php");
+  		if (file_exists('custom/Extension/modules/'.$panel->parent_bean->module_dir."/Ext/Layoutdefs/$oldName2.php")) {
+         unlink('custom/Extension/modules/'.$panel->parent_bean->module_dir."/Ext/Layoutdefs/$oldName2.php");
   		}
   		$extname = '_override'.$filename;
   		//end of bug# 40171
 
   		mkdir_recursive($path, true);
-  		write_array_to_file( $name, $override,$path.'/' . $filename .'.php');
+  		write_array_to_file($name, $override, $path.'/'.$filename.'.php');
 
   		//save the override for the layoutdef
         //tyoung 10.12.07 pushed panel->name to lowercase to match case in subpaneldefs.php files -
         //gave error on bad index 'module' as this override key didn't match the key in the subpaneldefs
-  		$name = "layout_defs['".  $panel->parent_bean->module_dir. "']['subpanel_setup']['" .strtolower($panel->name). "']";
+  		$name = "layout_defs['".$panel->parent_bean->module_dir."']['subpanel_setup']['".strtolower($panel->name)."']";
 //  	$GLOBALS['log']->debug('SubPanel.php->saveSubPanelDefOverride(): '.$name);
   		$newValue = override_value_to_string($name, 'override_subpanel_name', $filename);
-  		mkdir_recursive('custom/Extension/modules/'. $panel->parent_bean->module_dir . '/Ext/Layoutdefs', true);
-  		$fp = sugar_fopen('custom/Extension/modules/'. $panel->parent_bean->module_dir . "/Ext/Layoutdefs/$extname.php", 'w');
+  		mkdir_recursive('custom/Extension/modules/'.$panel->parent_bean->module_dir.'/Ext/Layoutdefs', true);
+  		$fp = sugar_fopen('custom/Extension/modules/'.$panel->parent_bean->module_dir."/Ext/Layoutdefs/$extname.php", 'w');
   		fwrite($fp, "<?php\n//auto-generated file DO NOT EDIT\n$newValue\n?>");
   		fclose($fp);
   		require_once('ModuleInstall/ModuleInstaller.php');
   		$moduleInstaller = new ModuleInstaller();
   		$moduleInstaller->silent = true; // make sure that the ModuleInstaller->log() function doesn't echo while rebuilding the layoutdefs
   		$moduleInstaller->rebuild_layoutdefs();
-  		if (file_exists('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php'))
-  			include('modules/'.  $panel->parent_bean->module_dir . '/layout_defs.php');
-  		if (file_exists('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php'))
-  			include('custom/modules/'.  $panel->parent_bean->module_dir . '/Ext/Layoutdefs/layoutdefs.ext.php');
+  		if (file_exists('modules/'.$panel->parent_bean->module_dir.'/layout_defs.php'))
+  			include('modules/'.$panel->parent_bean->module_dir.'/layout_defs.php');
+  		if (file_exists('custom/modules/'.$panel->parent_bean->module_dir.'/Ext/Layoutdefs/layoutdefs.ext.php'))
+  			include('custom/modules/'.$panel->parent_bean->module_dir.'/Ext/Layoutdefs/layoutdefs.ext.php');
   }
 
 	function get_subpanel_setup($module)
@@ -322,7 +322,7 @@ 
 		$subpanel_setup = '';
 		$layout_defs = get_layout_defs();
 
-		if(!empty($layout_defs) && !empty($layout_defs[$module]['subpanel_setup']))
+		if (!empty($layout_defs) && !empty($layout_defs[$module]['subpanel_setup']))
       {
       	$subpanel_setup = $layout_defs[$module]['subpanel_setup'];
       }
@@ -340,9 +340,9 @@ 
 
 		$subpanel_define = array_merge($default_subpanel_define, $custom_subpanel_define);
 
-		if(empty($subpanel_define))
+		if (empty($subpanel_define))
 		{
-			print('Could not load subpanel definition for: ' . $subpanel_id);
+			print('Could not load subpanel definition for: '.$subpanel_id);
 		}
 
 		return $subpanel_define;
@@ -352,11 +352,11 @@ 
 	{
 		$ret_val = array();
 
-		if($subpanel_id != '')
+		if ($subpanel_id != '')
 		{
 			$layout_defs = get_layout_defs();
 
-			if(!empty($layout_defs[$module]['custom_subpanel_defines'][$subpanel_id]))
+			if (!empty($layout_defs[$module]['custom_subpanel_defines'][$subpanel_id]))
 			{
 				$ret_val = $layout_defs[$module]['custom_subpanel_defines'][$subpanel_id];
 			}
@@ -369,11 +369,11 @@ 
 	{
 		$ret_val = array();
 
-		if($subpanel_id != '')
+		if ($subpanel_id != '')
 		{
 	  		$layout_defs = get_layout_defs();
 
-			if(!empty($layout_defs[$subpanel_id]['default_subpanel_define']))
+			if (!empty($layout_defs[$subpanel_id]['default_subpanel_define']))
 			{
 				$ret_val = $layout_defs[$subpanel_id]['default_subpanel_define'];
 			}
@@ -400,20 +400,20 @@ 
 
 		$where_clauses = $searchForm->generateSearchWhere(true, $seed->module_dir);
 
-		if (count($where_clauses) > 0 )$this->search_query = '('. implode(' ) AND ( ', $where_clauses) . ')';
+		if (count($where_clauses) > 0)$this->search_query = '('.implode(' ) AND ( ', $where_clauses).')';
 		$GLOBALS['log']->info("Subpanel Where Clause: $this->search_query");
 
-		return print_r($where_clauses,true);
+		return print_r($where_clauses, true);
 	}
 
 	function get_searchdefs($module)
 	{
-		$thisPanel =& $this->subpanel_defs;
+		$thisPanel = & $this->subpanel_defs;
 		$subpanel_defs = $thisPanel->_instance_properties;
 
-		if(isset($subpanel_defs['searchdefs'])){
+		if (isset($subpanel_defs['searchdefs'])) {
 			$searchdefs[$module]['layout']['basic_search'] = $subpanel_defs['searchdefs'];
-			$searchdefs[$module]['templateMeta'] = Array ('maxColumns' => 3, 'maxColumnsBasic' => 4, 'widths' => Array ( 'label' => 10, 'field' => 30 )) ;
+			$searchdefs[$module]['templateMeta'] = Array('maxColumns' => 3, 'maxColumnsBasic' => 4, 'widths' => Array('label' => 10, 'field' => 30));
 			return $searchdefs;
 		}
 
@@ -432,11 +432,11 @@ 
 
 		$searchMetaData = $searchForm->retrieveSearchDefs($module);
 
-		if ($subpanel_searchMetaData = $this->get_searchdefs($module)){
+		if ($subpanel_searchMetaData = $this->get_searchdefs($module)) {
 
 			$searchForm->setup($subpanel_searchMetaData, $searchMetaData['searchFields'], 'SubpanelSearchFormGeneric.tpl', 'basic_search');
 
-			if(!empty($this->collections))
+			if (!empty($this->collections))
 				$searchForm->searchFields['collection'] = array();
 
 			$searchForm->populateFromRequest();

include/SubPanel/registered_layout_defs.php

Indentation +2 added lines, -2 removed lines

@@ -49,8 +49,8 @@
 function get_layout_defs()
 {
     //TODO add global memory cache support here.  If there is an in memory cache, leverage it.
-	global $layout_defs;
-	return $layout_defs;
+    global $layout_defs;
+    return $layout_defs;
 }
 
 ?>
\ No newline at end of file

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

Braces +3 added lines, -1 removed lines

@@ -1,5 +1,7 @@
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.

include/SubPanel/SubPanelTilesTabs.php

Braces +22 added lines, -17 removed lines

@@ -1,5 +1,7 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if(!defined('sugarEntry') || !sugarEntry) {
+    die('Not A Valid Entry Point');
+}
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -111,8 +113,9 @@ 
     function _getTabs($tabs, $showTabs = true, $selectedGroup='All')
     {
         //WDong Bug: 12258 "All" tab in the middle of a record's detail view is not localized.
-        if($selectedGroup=='All')
-        	$selectedGroup=translate('LBL_TABGROUP_ALL');
+        if($selectedGroup=='All') {
+                	$selectedGroup=translate('LBL_TABGROUP_ALL');
+        }
 
     	// Set up a mapping from subpanelID, found in the $tabs list, to the source module name
     	// As the $GLOBALS['tabStructure'] array holds the Group Tabs by module name we need to efficiently convert between the two
@@ -127,8 +130,9 @@ 
             // Bug #44344 : Custom relationships under same module only show once in subpanel tabs
             // use object property instead new object to have ability run unit test (can override subpanel_definitions)
             $subpanel =  $this->subpanel_definitions->load_subpanel( $subpanelID );
-    		if ($subpanel !== false)
-    		  $moduleNames [ $subpanelID ] = $subpanel->get_module_name() ;
+    		if ($subpanel !== false) {
+    		    		  $moduleNames [ $subpanelID ] = $subpanel->get_module_name() ;
+    		}
     	}
 
     	$groups =  array () ;
@@ -138,11 +142,12 @@ 
         {
             foreach( $subModules['modules'] as $key => $subModule )
             {
-    			foreach ( $tabs as $subpanelID )
-                    if (isset($moduleNames[ $subpanelID ] ) && strcasecmp( $subModule , $moduleNames[ $subpanelID ] ) === 0)
+    			foreach ( $tabs as $subpanelID ) {
+    			                    if (isset($moduleNames[ $subpanelID ] ) && strcasecmp( $subModule , $moduleNames[ $subpanelID ] ) === 0)
                     {
                         // Bug #44344 : Custom relationships under same module only show once in subpanel tabs
                         $groups [ translate ( $mainTab ) ] [ 'modules' ] [] = $subpanelID ;
+    			}
                     	$found [ $subpanelID ] = true ;
                 	}
             }
@@ -152,8 +157,9 @@ 
 
         foreach( $tabs as $subpanelID )
         {
-        	if ( ! isset ( $found [ $subpanelID ] ) )
-	        	$groups [ translate ('LBL_TABGROUP_OTHER') ]['modules'] [] = $subpanelID ;
+        	if ( ! isset ( $found [ $subpanelID ] ) ) {
+        		        	$groups [ translate ('LBL_TABGROUP_OTHER') ]['modules'] [] = $subpanelID ;
+        	}
         }
 
         /* Move history to same tab as activities */
@@ -168,8 +174,7 @@ 
                     	/* Move hist from there to here */
                         $groups[$mainTab]['modules'] []= 'history';
                     }
-                }
-                else if(false !== ($i = array_search('history', array_map('strtolower', $group['modules']))))
+                } else if(false !== ($i = array_search('history', array_map('strtolower', $group['modules']))))
                 {
                     unset($groups[$mainTab]['modules'][$i]);
                     if(empty($groups[$mainTab]['modules']))
@@ -184,10 +189,11 @@ 
          * Note that if a tab group already exists with the name 'All',
          * it will be overwritten in this union operation.
          */
-        if(count($groups) <= 1)
-        	$groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs));
-        else
-            $groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs)) + $groups;
+        if(count($groups) <= 1) {
+                	$groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs));
+        } else {
+                    $groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs)) + $groups;
+        }
         /* Note - all $display checking and array_intersects with $tabs
          * are now redundant (thanks to GroupedTabStructure), and could
          * be removed for performance, but for now can stay to help ensure
@@ -261,8 +267,7 @@ 
             	$sugarTab->setup($sugarTabs, $otherTabs, $displayTabs, $selectedGroup);
             	$sugarTab->display();
             }
-        }
-        else
+        } else
         {
             $tabs = SubPanelTilesTabs::applyUserCustomLayoutToTabs($tabs, $selectedGroup);
 

Indentation +51 added lines, -51 removed lines

@@ -48,15 +48,15 @@ 
 class SubPanelTilesTabs extends SubPanelTiles
 {
 
-	function __construct(&$focus, $layout_def_key='', $layout_def_override = '')
-	{
+    function __construct(&$focus, $layout_def_key='', $layout_def_override = '')
+    {
 
-		$this->focus = $focus;
-		$this->id = $focus->id;
-		$this->module = $focus->module_dir;
-		$this->layout_def_key = $layout_def_key;
-		$this->subpanel_definitions = new SubPanelDefinitions($focus, $layout_def_key, $layout_def_override);
-	}
+        $this->focus = $focus;
+        $this->id = $focus->id;
+        $this->module = $focus->module_dir;
+        $this->layout_def_key = $layout_def_key;
+        $this->subpanel_definitions = new SubPanelDefinitions($focus, $layout_def_key, $layout_def_override);
+    }
 
     function getSubpanelGroupLayout($selectedGroup)
     {
@@ -77,9 +77,9 @@ 
     {
         //WDong Bug: 12258 "All" tab in the middle of a record's detail view is not localized.
         if($key=='All')
-    	{
-    		$key=translate('LBL_TABGROUP_ALL');
-    	}
+        {
+            $key=translate('LBL_TABGROUP_ALL');
+        }
         $usersCustomLayout = SubPanelTilesTabs::getSubpanelGroupLayout($key);
         if(!empty($usersCustomLayout))
         {
@@ -91,7 +91,7 @@ 
             $tabs = array_intersect($usersCustomLayout, $tabs);
             foreach($diff as $subpanel)
             {
-            	$tabs []= $subpanel;
+                $tabs []= $subpanel;
             }
         }
 
@@ -112,39 +112,39 @@ 
     {
         //WDong Bug: 12258 "All" tab in the middle of a record's detail view is not localized.
         if($selectedGroup=='All')
-        	$selectedGroup=translate('LBL_TABGROUP_ALL');
-
-    	// Set up a mapping from subpanelID, found in the $tabs list, to the source module name
-    	// As the $GLOBALS['tabStructure'] array holds the Group Tabs by module name we need to efficiently convert between the two
-    	// when constructing the subpanel tabs
-    	// Note that we can't use the very similar GroupedTabStructure class as it lacks this mapping, and logically, it is designed
-    	// for use when constructing the module by module tabs, not the subpanel tabs, as we move away from using module names to represent
-    	// subpanels, and use unique subpanel IDs instead.
-
-    	$moduleNames = array () ;
-    	foreach ( $tabs as $subpanelID )
-    	{
+            $selectedGroup=translate('LBL_TABGROUP_ALL');
+
+        // Set up a mapping from subpanelID, found in the $tabs list, to the source module name
+        // As the $GLOBALS['tabStructure'] array holds the Group Tabs by module name we need to efficiently convert between the two
+        // when constructing the subpanel tabs
+        // Note that we can't use the very similar GroupedTabStructure class as it lacks this mapping, and logically, it is designed
+        // for use when constructing the module by module tabs, not the subpanel tabs, as we move away from using module names to represent
+        // subpanels, and use unique subpanel IDs instead.
+
+        $moduleNames = array () ;
+        foreach ( $tabs as $subpanelID )
+        {
             // Bug #44344 : Custom relationships under same module only show once in subpanel tabs
             // use object property instead new object to have ability run unit test (can override subpanel_definitions)
             $subpanel =  $this->subpanel_definitions->load_subpanel( $subpanelID );
-    		if ($subpanel !== false)
-    		  $moduleNames [ $subpanelID ] = $subpanel->get_module_name() ;
-    	}
+            if ($subpanel !== false)
+                $moduleNames [ $subpanelID ] = $subpanel->get_module_name() ;
+        }
 
-    	$groups =  array () ;
-    	$found = array () ;
+        $groups =  array () ;
+        $found = array () ;
 
         foreach( $GLOBALS['tabStructure'] as $mainTab => $subModules)
         {
             foreach( $subModules['modules'] as $key => $subModule )
             {
-    			foreach ( $tabs as $subpanelID )
+                foreach ( $tabs as $subpanelID )
                     if (isset($moduleNames[ $subpanelID ] ) && strcasecmp( $subModule , $moduleNames[ $subpanelID ] ) === 0)
                     {
                         // Bug #44344 : Custom relationships under same module only show once in subpanel tabs
                         $groups [ translate ( $mainTab ) ] [ 'modules' ] [] = $subpanelID ;
-                    	$found [ $subpanelID ] = true ;
-                	}
+                        $found [ $subpanelID ] = true ;
+                    }
             }
         }
 
@@ -152,8 +152,8 @@ 
 
         foreach( $tabs as $subpanelID )
         {
-        	if ( ! isset ( $found [ $subpanelID ] ) )
-	        	$groups [ translate ('LBL_TABGROUP_OTHER') ]['modules'] [] = $subpanelID ;
+            if ( ! isset ( $found [ $subpanelID ] ) )
+                $groups [ translate ('LBL_TABGROUP_OTHER') ]['modules'] [] = $subpanelID ;
         }
 
         /* Move history to same tab as activities */
@@ -161,11 +161,11 @@ 
         {
             foreach($groups as $mainTab => $group)
             {
-            	if(in_array('activities', array_map('strtolower', $group['modules'])))
+                if(in_array('activities', array_map('strtolower', $group['modules'])))
                 {
-                	if(!in_array('history', array_map('strtolower', $group['modules'])))
+                    if(!in_array('history', array_map('strtolower', $group['modules'])))
                     {
-                    	/* Move hist from there to here */
+                        /* Move hist from there to here */
                         $groups[$mainTab]['modules'] []= 'history';
                     }
                 }
@@ -174,7 +174,7 @@ 
                     unset($groups[$mainTab]['modules'][$i]);
                     if(empty($groups[$mainTab]['modules']))
                     {
-                    	unset($groups[$mainTab]);
+                        unset($groups[$mainTab]);
                     }
                 }
             }
@@ -185,7 +185,7 @@ 
          * it will be overwritten in this union operation.
          */
         if(count($groups) <= 1)
-        	$groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs));
+            $groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs));
         else
             $groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs)) + $groups;
         /* Note - all $display checking and array_intersects with $tabs
@@ -197,14 +197,14 @@ 
         $retTabs = array();
         if($showTabs)
         {
-        	require_once('include/SubPanel/SugarTab.php');
-        	$sugarTab = new SugarTab();
+            require_once('include/SubPanel/SugarTab.php');
+            $sugarTab = new SugarTab();
 
             $displayTabs = array();
             $otherTabs = array();
 
-    	    foreach ($groups as $key=>$tab)
-    		{
+            foreach ($groups as $key=>$tab)
+            {
                 $display = false;
                 foreach($tab['modules'] as $subkey=>$subtab)
                 {
@@ -227,9 +227,9 @@ 
                     $relevantTabs = SubPanelTilesTabs::applyUserCustomLayoutToTabs($tabs, $key);
 
                     $sugarTabs[$key] = array(//'url'=>'index.php?module=' . $_REQUEST['module'] . '&record=' . $_REQUEST['record'] . '&action=' . $_REQUEST['action']. '&subpanel=' . $key.'#tabs',
-                                         //'url'=>"javascript:SUGAR.util.retrieveAndFill('index.php?to_pdf=1&module=MySettings&action=LoadTabSubpanels&loadModule={$_REQUEST['module']}&record={$_REQUEST['record']}&subpanel=$key','subpanel_list',null,null,null);",
-                                         'label'=>( !empty($tab['label']) ? $tab['label']: $key ),
-                                         'type'=>$selected);
+                                            //'url'=>"javascript:SUGAR.util.retrieveAndFill('index.php?to_pdf=1&module=MySettings&action=LoadTabSubpanels&loadModule={$_REQUEST['module']}&record={$_REQUEST['record']}&subpanel=$key','subpanel_list',null,null,null);",
+                                            'label'=>( !empty($tab['label']) ? $tab['label']: $key ),
+                                            'type'=>$selected);
 
                     $otherTabs[$key] = array('key'=>$key, 'tabs'=>array());
 
@@ -246,7 +246,7 @@ 
                         $retTabs = $orderedTabs;
                     }
                 }
-    		}
+            }
 
             if(empty($displayTabs) && !empty($otherTabs))
             {
@@ -258,8 +258,8 @@ 
             }
 
             if (!empty($sugarTabs) || !empty($otherTabs) ) {
-            	$sugarTab->setup($sugarTabs, $otherTabs, $displayTabs, $selectedGroup);
-            	$sugarTab->display();
+                $sugarTab->setup($sugarTabs, $otherTabs, $displayTabs, $selectedGroup);
+                $sugarTab->display();
             }
         }
         else
@@ -269,7 +269,7 @@ 
             $retTabs = array_intersect($tabs, array_map('strtolower', $groups[$selectedGroup]['modules']));
         }
 
-		return $retTabs;
-	}
+        return $retTabs;
+    }
 }
 ?>

Spacing +46 added lines, -46 removed lines

@@ -1,5 +1,5 @@ 
 <?php
-if(!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
+if (!defined('sugarEntry') || !sugarEntry) die('Not A Valid Entry Point');
 /*********************************************************************************
  * SugarCRM Community Edition is a customer relationship management program developed by
  * SugarCRM, Inc. Copyright (C) 2004-2013 SugarCRM Inc.
@@ -48,7 +48,7 @@ 
 class SubPanelTilesTabs extends SubPanelTiles
 {
 
-	function __construct(&$focus, $layout_def_key='', $layout_def_override = '')
+	function __construct(&$focus, $layout_def_key = '', $layout_def_override = '')
 	{
 
 		$this->focus = $focus;
@@ -64,7 +64,7 @@ 
 
         $layoutParams = $this->module;
         //WDong Bug: 12258 "All" tab in the middle of a record's detail view is not localized.
-        if($selectedGroup != translate('LBL_TABGROUP_ALL'))
+        if ($selectedGroup != translate('LBL_TABGROUP_ALL'))
         {
             $layoutParams .= ':'.$selectedGroup;
         }
@@ -73,15 +73,15 @@ 
         return $current_user->getPreference('subpanelLayout', $layoutParams);
     }
 
-    function applyUserCustomLayoutToTabs($tabs, $key='All')
+    function applyUserCustomLayoutToTabs($tabs, $key = 'All')
     {
         //WDong Bug: 12258 "All" tab in the middle of a record's detail view is not localized.
-        if($key=='All')
+        if ($key == 'All')
     	{
-    		$key=translate('LBL_TABGROUP_ALL');
+    		$key = translate('LBL_TABGROUP_ALL');
     	}
         $usersCustomLayout = SubPanelTilesTabs::getSubpanelGroupLayout($key);
-        if(!empty($usersCustomLayout))
+        if (!empty($usersCustomLayout))
         {
             /* Return elements of the custom layout
              * which occur in $tabs in unchanged order.
@@ -89,9 +89,9 @@ 
              * not included in the layout. */
             $diff = array_diff($tabs, $usersCustomLayout);
             $tabs = array_intersect($usersCustomLayout, $tabs);
-            foreach($diff as $subpanel)
+            foreach ($diff as $subpanel)
             {
-            	$tabs []= $subpanel;
+            	$tabs [] = $subpanel;
             }
         }
 
@@ -104,15 +104,15 @@ 
      * @param boolean $showTabs	Call the view code to display the generated tabs
      * @param string $selectedGroup	(Optional) Name of any selected tab (defaults to 'All')
      */
-    function getTabs($showTabs = true, $selectedGroup='') {
+    function getTabs($showTabs = true, $selectedGroup = '') {
         $args = func_get_args();
         return call_user_func_array(array($this, '_getTabs'), $args);
     }
-    function _getTabs($tabs, $showTabs = true, $selectedGroup='All')
+    function _getTabs($tabs, $showTabs = true, $selectedGroup = 'All')
     {
         //WDong Bug: 12258 "All" tab in the middle of a record's detail view is not localized.
-        if($selectedGroup=='All')
-        	$selectedGroup=translate('LBL_TABGROUP_ALL');
+        if ($selectedGroup == 'All')
+        	$selectedGroup = translate('LBL_TABGROUP_ALL');
 
     	// Set up a mapping from subpanelID, found in the $tabs list, to the source module name
     	// As the $GLOBALS['tabStructure'] array holds the Group Tabs by module name we need to efficiently convert between the two
@@ -121,58 +121,58 @@ 
     	// for use when constructing the module by module tabs, not the subpanel tabs, as we move away from using module names to represent
     	// subpanels, and use unique subpanel IDs instead.
 
-    	$moduleNames = array () ;
-    	foreach ( $tabs as $subpanelID )
+    	$moduleNames = array();
+    	foreach ($tabs as $subpanelID)
     	{
             // Bug #44344 : Custom relationships under same module only show once in subpanel tabs
             // use object property instead new object to have ability run unit test (can override subpanel_definitions)
-            $subpanel =  $this->subpanel_definitions->load_subpanel( $subpanelID );
+            $subpanel = $this->subpanel_definitions->load_subpanel($subpanelID);
     		if ($subpanel !== false)
-    		  $moduleNames [ $subpanelID ] = $subpanel->get_module_name() ;
+    		  $moduleNames [$subpanelID] = $subpanel->get_module_name();
     	}
 
-    	$groups =  array () ;
-    	$found = array () ;
+    	$groups = array();
+    	$found = array();
 
-        foreach( $GLOBALS['tabStructure'] as $mainTab => $subModules)
+        foreach ($GLOBALS['tabStructure'] as $mainTab => $subModules)
         {
-            foreach( $subModules['modules'] as $key => $subModule )
+            foreach ($subModules['modules'] as $key => $subModule)
             {
-    			foreach ( $tabs as $subpanelID )
-                    if (isset($moduleNames[ $subpanelID ] ) && strcasecmp( $subModule , $moduleNames[ $subpanelID ] ) === 0)
+    			foreach ($tabs as $subpanelID)
+                    if (isset($moduleNames[$subpanelID]) && strcasecmp($subModule, $moduleNames[$subpanelID]) === 0)
                     {
                         // Bug #44344 : Custom relationships under same module only show once in subpanel tabs
-                        $groups [ translate ( $mainTab ) ] [ 'modules' ] [] = $subpanelID ;
-                    	$found [ $subpanelID ] = true ;
+                        $groups [translate($mainTab)] ['modules'] [] = $subpanelID;
+                    	$found [$subpanelID] = true;
                 	}
             }
         }
 
         // Put all the remaining subpanels into the 'Other' tab.
 
-        foreach( $tabs as $subpanelID )
+        foreach ($tabs as $subpanelID)
         {
-        	if ( ! isset ( $found [ $subpanelID ] ) )
-	        	$groups [ translate ('LBL_TABGROUP_OTHER') ]['modules'] [] = $subpanelID ;
+        	if (!isset ($found [$subpanelID]))
+	        	$groups [translate('LBL_TABGROUP_OTHER')]['modules'] [] = $subpanelID;
         }
 
         /* Move history to same tab as activities */
-        if(in_array('history', $tabs) && in_array('activities', $tabs))
+        if (in_array('history', $tabs) && in_array('activities', $tabs))
         {
-            foreach($groups as $mainTab => $group)
+            foreach ($groups as $mainTab => $group)
             {
-            	if(in_array('activities', array_map('strtolower', $group['modules'])))
+            	if (in_array('activities', array_map('strtolower', $group['modules'])))
                 {
-                	if(!in_array('history', array_map('strtolower', $group['modules'])))
+                	if (!in_array('history', array_map('strtolower', $group['modules'])))
                     {
                     	/* Move hist from there to here */
-                        $groups[$mainTab]['modules'] []= 'history';
+                        $groups[$mainTab]['modules'] [] = 'history';
                     }
                 }
-                else if(false !== ($i = array_search('history', array_map('strtolower', $group['modules']))))
+                else if (false !== ($i = array_search('history', array_map('strtolower', $group['modules']))))
                 {
                     unset($groups[$mainTab]['modules'][$i]);
-                    if(empty($groups[$mainTab]['modules']))
+                    if (empty($groups[$mainTab]['modules']))
                     {
                     	unset($groups[$mainTab]);
                     }
@@ -184,7 +184,7 @@ 
          * Note that if a tab group already exists with the name 'All',
          * it will be overwritten in this union operation.
          */
-        if(count($groups) <= 1)
+        if (count($groups) <= 1)
         	$groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs));
         else
             $groups = array(translate('LBL_TABGROUP_ALL') => array('label' => translate('LBL_TABGROUP_ALL'), 'modules' => $tabs)) + $groups;
@@ -195,7 +195,7 @@ 
          */
 
         $retTabs = array();
-        if($showTabs)
+        if ($showTabs)
         {
         	require_once('include/SubPanel/SugarTab.php');
         	$sugarTab = new SugarTab();
@@ -206,9 +206,9 @@ 
     	    foreach ($groups as $key=>$tab)
     		{
                 $display = false;
-                foreach($tab['modules'] as $subkey=>$subtab)
+                foreach ($tab['modules'] as $subkey=>$subtab)
                 {
-                    if(in_array(strtolower($subtab), $tabs))
+                    if (in_array(strtolower($subtab), $tabs))
                     {
                         $display = true;
                         break;
@@ -217,30 +217,30 @@ 
 
                 $selected = '';
 
-                if($selectedGroup == $key)
+                if ($selectedGroup == $key)
                 {
                     $selected = 'current';
                 }
 
-                if($display)
+                if ($display)
                 {
                     $relevantTabs = SubPanelTilesTabs::applyUserCustomLayoutToTabs($tabs, $key);
 
                     $sugarTabs[$key] = array(//'url'=>'index.php?module=' . $_REQUEST['module'] . '&record=' . $_REQUEST['record'] . '&action=' . $_REQUEST['action']. '&subpanel=' . $key.'#tabs',
                                          //'url'=>"javascript:SUGAR.util.retrieveAndFill('index.php?to_pdf=1&module=MySettings&action=LoadTabSubpanels&loadModule={$_REQUEST['module']}&record={$_REQUEST['record']}&subpanel=$key','subpanel_list',null,null,null);",
-                                         'label'=>( !empty($tab['label']) ? $tab['label']: $key ),
+                                         'label'=>(!empty($tab['label']) ? $tab['label'] : $key),
                                          'type'=>$selected);
 
                     $otherTabs[$key] = array('key'=>$key, 'tabs'=>array());
 
                     $orderedTabs = array_intersect($relevantTabs, array_map('strtolower', $groups[$key]['modules']));
 
-                    foreach($orderedTabs as $subkey => $subtab)
+                    foreach ($orderedTabs as $subkey => $subtab)
                     {
                         $otherTabs[$key]['tabs'][$subkey] = array('key'=>$subtab, 'label'=>translate($this->subpanel_definitions->layout_defs['subpanel_setup'][$subtab]['title_key']));
                     }
 
-                    if($selectedGroup == $key)
+                    if ($selectedGroup == $key)
                     {
                         $displayTabs = $otherTabs[$key]['tabs'];
                         $retTabs = $orderedTabs;
@@ -248,7 +248,7 @@ 
                 }
     		}
 
-            if(empty($displayTabs) && !empty($otherTabs))
+            if (empty($displayTabs) && !empty($otherTabs))
             {
                 //WDong Bug: 12258 "All" tab in the middle of a record's detail view is not localized.
                 $selectedGroup = translate('LBL_TABGROUP_ALL');
@@ -257,7 +257,7 @@ 
                 $retTabs = array_intersect($tabs, array_map('strtolower', $groups[$selectedGroup]['modules']));
             }
 
-            if (!empty($sugarTabs) || !empty($otherTabs) ) {
+            if (!empty($sugarTabs) || !empty($otherTabs)) {
             	$sugarTab->setup($sugarTabs, $otherTabs, $displayTabs, $selectedGroup);
             	$sugarTab->display();
             }