adamjakab / SuiteCRM

modules/Campaigns/WizardNewsletterSave.php

Spacing +78 added lines, -78 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.
@@ -51,10 +51,10 @@ 
 
     //create new campaign bean and populate
     $campaign_focus = new Campaign();
-    if(isset($_REQUEST['record']) && $_REQUEST['record'] && !(isset($_REQUEST['campaign_id']) && $_REQUEST['campaign_id'])) {
+    if (isset($_REQUEST['record']) && $_REQUEST['record'] && !(isset($_REQUEST['campaign_id']) && $_REQUEST['campaign_id'])) {
         $campaign_focus->retrieve($_REQUEST['record']);
     }
-    else if(isset($_REQUEST['campaign_id']) && $_REQUEST['campaign_id']) {
+    else if (isset($_REQUEST['campaign_id']) && $_REQUEST['campaign_id']) {
         $campaign_focus->retrieve($_REQUEST['campaign_id']);
     }
 
@@ -63,11 +63,11 @@ 
 
     $campaign_focus = populateFromPost('', $campaign_focus);
 
-    foreach($camp_steps as $step){
-       $campaign_focus =  populate_wizard_bean_from_request($campaign_focus,$step);
+    foreach ($camp_steps as $step) {
+       $campaign_focus = populate_wizard_bean_from_request($campaign_focus, $step);
     }
 
-    switch($_REQUEST['currentstep']) {
+    switch ($_REQUEST['currentstep']) {
         case 1;
             //save here so we can link relationships
             $campaign_focus->save();
@@ -76,7 +76,7 @@ 
             break;
         case 2;
             //process subscription lists if this is a newsletter
-            if($campaign_focus->campaign_type =='NewsLetter'){
+            if ($campaign_focus->campaign_type == 'NewsLetter') {
                 $pl_list = process_subscriptions_from_request($campaign_focus->name);
 
                 $campaign_focus->load_relationship('prospectlists');
@@ -84,7 +84,7 @@ 
                 $ui_ids = array();
 
                 //for each list returned, add the list to the relationship
-                foreach($pl_list as $pl){
+                foreach ($pl_list as $pl) {
                     $campaign_focus->prospectlists->add($pl->id);
                     //populate array with id's from UI'
                     $ui_ids[] = $pl->id;
@@ -92,30 +92,30 @@ 
 
                 //now remove the lists that may have existed before, but were not specified in UI.
                 //this will enforce that Newsletters only have 3 available target lists.
-                foreach($existing_pls as $pl_del){
-                    if (!in_array($pl_del, $ui_ids)){
+                foreach ($existing_pls as $pl_del) {
+                    if (!in_array($pl_del, $ui_ids)) {
                         $campaign_focus->prospectlists->delete($campaign_focus->id, $pl_del);
                     }
                 }
-            } else{
+            } else {
                 //process target lists if this is not a newsletter
                 //remove Target Lists if defined
 
-                if(isset($_REQUEST['wiz_remove_target_list'])){
+                if (isset($_REQUEST['wiz_remove_target_list'])) {
 
                     $remove_target_strings = explode(",", $_REQUEST['wiz_remove_target_list']);
-                    foreach($remove_target_strings as $remove_trgt_string){
-                        if(!empty($remove_trgt_string)){
+                    foreach ($remove_target_strings as $remove_trgt_string) {
+                        if (!empty($remove_trgt_string)) {
                             //load relationship and add to the list
                             $campaign_focus->load_relationship('prospectlists');
-                            $campaign_focus->prospectlists->delete($campaign_focus->id,$remove_trgt_string);
+                            $campaign_focus->prospectlists->delete($campaign_focus->id, $remove_trgt_string);
                         }
                     }
                 }
 
 
                 //create new campaign tracker and save if defined
-                if(isset($_REQUEST['wiz_list_of_targets'])) {
+                if (isset($_REQUEST['wiz_list_of_targets'])) {
                     $target_strings = explode(",", $_REQUEST['wiz_list_of_targets']);
                     foreach ($target_strings as $trgt_string) {
                         $target_values = explode("@@", $trgt_string);
@@ -152,27 +152,27 @@ 
 
 
             //remove campaign trackers if defined
-            if(isset($_REQUEST['wiz_remove_tracker_list'])) {
+            if (isset($_REQUEST['wiz_remove_tracker_list'])) {
 
                 $remove_tracker_strings = explode(",", $_REQUEST['wiz_remove_tracker_list']);
-                foreach($remove_tracker_strings as $remove_trkr_string){
-                    if(!empty($remove_trkr_string)){
+                foreach ($remove_tracker_strings as $remove_trkr_string) {
+                    if (!empty($remove_trkr_string)) {
                         //load relationship and add to the list
                         $campaign_focus->load_relationship('tracked_urls');
-                        $campaign_focus->tracked_urls->delete($campaign_focus->id,$remove_trkr_string);
+                        $campaign_focus->tracked_urls->delete($campaign_focus->id, $remove_trkr_string);
                     }
                 }
             }
 
 
             //save  campaign trackers and save if defined
-            if(isset($_REQUEST['wiz_list_of_existing_trackers'])) {
+            if (isset($_REQUEST['wiz_list_of_existing_trackers'])) {
                 $tracker_strings = explode(",", $_REQUEST['wiz_list_of_existing_trackers']);
-                foreach($tracker_strings as $trkr_string){
+                foreach ($tracker_strings as $trkr_string) {
                     $tracker_values = explode("@@", $trkr_string);
                     $ct_focus = new CampaignTracker();
                     $ct_focus->retrieve($tracker_values[0]);
-                    if(!empty($ct_focus->tracker_name)){
+                    if (!empty($ct_focus->tracker_name)) {
                         $ct_focus->tracker_name = $tracker_values[1];
                         $ct_focus->is_optout = $tracker_values[2];
                         $ct_focus->tracker_url = $tracker_values[3];
@@ -187,11 +187,11 @@ 
 
 
             //create new campaign tracker and save if defined
-            if(isset($_REQUEST['wiz_list_of_trackers'])){
+            if (isset($_REQUEST['wiz_list_of_trackers'])) {
                 $tracker_strings = explode(",", $_REQUEST['wiz_list_of_trackers']);
-                foreach($tracker_strings as $trkr_string){
+                foreach ($tracker_strings as $trkr_string) {
                     $tracker_values = explode("@@", $trkr_string);
-                    if(count($tracker_values)==3){
+                    if (count($tracker_values) == 3) {
                         $ct_focus = new CampaignTracker();
                         $ct_focus->tracker_name = $tracker_values[0];
                         $ct_focus->is_optout = $tracker_values[1];
@@ -216,9 +216,9 @@ 
 
             $action = '';
             $redirectToTargetList = '';
-            if(isset($_REQUEST['wiz_direction'])){
+            if (isset($_REQUEST['wiz_direction'])) {
 
-                switch($_REQUEST['wiz_direction']) {
+                switch ($_REQUEST['wiz_direction']) {
                     case 'continue':
                     $action = 'WizardMarketing';
                         break;
@@ -230,14 +230,14 @@ 
                          $action = 'WizardMarketing';
                         break;
                 }
-            }else{
+            } else {
                 $action = 'WizardHome&record='.$campaign_focus->id;
             }
             //require_once('modules/Campaigns/WizardMarketing.php');
             $header_URL = "Location: index.php?return_module=Campaigns&module=Campaigns&action=".$action.$redirectToTargetList."&campaign_id=".$campaign_focus->id."&return_action=WizardNewsLetter&return_id=".$campaign_focus->id;
             $GLOBALS['log']->debug("about to post header URL of: $header_URL");
 
-            if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+            if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
                 $href = $matches[1];
                 SugarApplication::redirect($href);
             }
@@ -248,7 +248,7 @@ 
             break;
         case 3:
             //process subscription lists if this is a newsletter
-            if($campaign_focus->campaign_type =='NewsLetter'){
+            if ($campaign_focus->campaign_type == 'NewsLetter') {
                 $pl_list = process_subscriptions_from_request($campaign_focus->name);
 
                 $campaign_focus->load_relationship('prospectlists');
@@ -256,7 +256,7 @@ 
                 $ui_ids = array();
 
                 //for each list returned, add the list to the relationship
-                foreach($pl_list as $pl){
+                foreach ($pl_list as $pl) {
                     $campaign_focus->prospectlists->add($pl->id);
                     //populate array with id's from UI'
                     $ui_ids[] = $pl->id;
@@ -264,30 +264,30 @@ 
 
                 //now remove the lists that may have existed before, but were not specified in UI.
                 //this will enforce that Newsletters only have 3 available target lists.
-                foreach($existing_pls as $pl_del){
-                    if (!in_array($pl_del, $ui_ids)){
+                foreach ($existing_pls as $pl_del) {
+                    if (!in_array($pl_del, $ui_ids)) {
                         $campaign_focus->prospectlists->delete($campaign_focus->id, $pl_del);
                     }
                 }
-            } else{
+            } else {
                 //process target lists if this is not a newsletter
                 //remove Target Lists if defined
 
-                if(isset($_REQUEST['wiz_remove_target_list'])){
+                if (isset($_REQUEST['wiz_remove_target_list'])) {
 
                     $remove_target_strings = explode(",", $_REQUEST['wiz_remove_target_list']);
-                    foreach($remove_target_strings as $remove_trgt_string){
-                        if(!empty($remove_trgt_string)){
+                    foreach ($remove_target_strings as $remove_trgt_string) {
+                        if (!empty($remove_trgt_string)) {
                             //load relationship and add to the list
                             $campaign_focus->load_relationship('prospectlists');
-                            $campaign_focus->prospectlists->delete($campaign_focus->id,$remove_trgt_string);
+                            $campaign_focus->prospectlists->delete($campaign_focus->id, $remove_trgt_string);
                         }
                     }
                 }
 
 
                 //create new campaign tracker and save if defined
-                if(isset($_REQUEST['wiz_list_of_targets'])) {
+                if (isset($_REQUEST['wiz_list_of_targets'])) {
                     $target_strings = explode(",", $_REQUEST['wiz_list_of_targets']);
                     foreach ($target_strings as $trgt_string) {
                         $target_values = explode("@@", $trgt_string);
@@ -324,27 +324,27 @@ 
 
 
             //remove campaign trackers if defined
-            if(isset($_REQUEST['wiz_remove_tracker_list'])) {
+            if (isset($_REQUEST['wiz_remove_tracker_list'])) {
 
                 $remove_tracker_strings = explode(",", $_REQUEST['wiz_remove_tracker_list']);
-                foreach($remove_tracker_strings as $remove_trkr_string){
-                    if(!empty($remove_trkr_string)){
+                foreach ($remove_tracker_strings as $remove_trkr_string) {
+                    if (!empty($remove_trkr_string)) {
                         //load relationship and add to the list
                         $campaign_focus->load_relationship('tracked_urls');
-                        $campaign_focus->tracked_urls->delete($campaign_focus->id,$remove_trkr_string);
+                        $campaign_focus->tracked_urls->delete($campaign_focus->id, $remove_trkr_string);
                     }
                 }
             }
 
 
             //save  campaign trackers and save if defined
-            if(isset($_REQUEST['wiz_list_of_existing_trackers'])) {
+            if (isset($_REQUEST['wiz_list_of_existing_trackers'])) {
                 $tracker_strings = explode(",", $_REQUEST['wiz_list_of_existing_trackers']);
-                foreach($tracker_strings as $trkr_string){
+                foreach ($tracker_strings as $trkr_string) {
                     $tracker_values = explode("@@", $trkr_string);
                     $ct_focus = new CampaignTracker();
                     $ct_focus->retrieve($tracker_values[0]);
-                    if(!empty($ct_focus->tracker_name)){
+                    if (!empty($ct_focus->tracker_name)) {
                         $ct_focus->tracker_name = $tracker_values[1];
                         $ct_focus->is_optout = $tracker_values[2];
                         $ct_focus->tracker_url = $tracker_values[3];
@@ -359,11 +359,11 @@ 
 
 
             //create new campaign tracker and save if defined
-            if(isset($_REQUEST['wiz_list_of_trackers'])){
+            if (isset($_REQUEST['wiz_list_of_trackers'])) {
                 $tracker_strings = explode(",", $_REQUEST['wiz_list_of_trackers']);
-                foreach($tracker_strings as $trkr_string){
+                foreach ($tracker_strings as $trkr_string) {
                     $tracker_values = explode("@@", $trkr_string);
-                    if(count($tracker_values)==3){
+                    if (count($tracker_values) == 3) {
                         $ct_focus = new CampaignTracker();
                         $ct_focus->tracker_name = $tracker_values[0];
                         $ct_focus->is_optout = $tracker_values[1];
@@ -388,9 +388,9 @@ 
 
             $action = '';
             $redirectToTargetList = '';
-            if(isset($_REQUEST['wiz_direction'])){
+            if (isset($_REQUEST['wiz_direction'])) {
 
-                switch($_REQUEST['wiz_direction']) {
+                switch ($_REQUEST['wiz_direction']) {
                     case 'continue':
                         $action = 'WizardMarketing';
                         break;
@@ -402,13 +402,13 @@ 
                         $action = 'WizardHome';
                         break;
                 }
-            }else{
+            } else {
                 $action = 'WizardHome&record='.$campaign_focus->id;
             }
             //require_once('modules/Campaigns/WizardMarketing.php');
             $header_URL = "Location: index.php?return_module=Campaigns&module=Campaigns&action=".$action.$redirectToTargetList."&campaign_id=".$campaign_focus->id."&record=".$campaign_focus->id;
             $GLOBALS['log']->debug("about to post header URL of: $header_URL");
-            if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+            if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
                 $href = $matches[1];
                 SugarApplication::redirect($href);
             }
@@ -425,12 +425,12 @@ 
  * This function will populate the passed in bean with the post variables
  * that contain the specified prefix
  */
-function populate_wizard_bean_from_request($bean,$prefix){
-    foreach($_REQUEST as $key=> $val){
+function populate_wizard_bean_from_request($bean, $prefix) {
+    foreach ($_REQUEST as $key=> $val) {
      $key = trim($key);
-     if((strstr($key, $prefix )) && (strpos($key, $prefix )== 0)){
-          $field  =substr($key, strlen($prefix)) ;
-          if(isset($_REQUEST[$key]) && !empty($_REQUEST[$key])){
+     if ((strstr($key, $prefix)) && (strpos($key, $prefix) == 0)) {
+          $field = substr($key, strlen($prefix));
+          if (isset($_REQUEST[$key]) && !empty($_REQUEST[$key])) {
               //echo "prefix is $prefix, field is $field,    key is $key,   and value is $val<br>";
               $value = $_REQUEST[$key];
               $bean->$field = $value;
@@ -447,34 +447,34 @@ 
  * If no prospect lists have been specified, then it will create one for you.  A total of 3 prospect lists
  * will be created for you (Subscription, Unsubscription, and test)
  */
-function process_subscriptions_from_request($campaign_name){
+function process_subscriptions_from_request($campaign_name) {
     global $mod_strings;
     $pl_list = array();
 
     //process default target list
     $create_new = true;
     $pl_subs = new ProspectList($campaign_name);
-    if(!empty($_REQUEST['wiz_step3_subscription_list_id'])){
+    if (!empty($_REQUEST['wiz_step3_subscription_list_id'])) {
         //if subscription list is specified then attach
         $pl_subs->retrieve($_REQUEST['wiz_step3_subscription_list_id']);
         //check to see name matches the bean, if not, then the user has chosen to create new bean
-        if($pl_subs->name == $_REQUEST['wiz_step3_subscription_name']){
+        if ($pl_subs->name == $_REQUEST['wiz_step3_subscription_name']) {
             $pl_list[] = $pl_subs;
             $create_new = false;
        }
 
     }
     //create new bio if one was not retrieved successfully
-    if($create_new){
+    if ($create_new) {
         //use default name if one has not been specified
-        $name = $campaign_name . " ".$mod_strings['LBL_SUBSCRIPTION_LIST'];
-        if(isset($_REQUEST['wiz_step3_subscription_name']) && !empty($_REQUEST['wiz_step3_subscription_name'])){
+        $name = $campaign_name." ".$mod_strings['LBL_SUBSCRIPTION_LIST'];
+        if (isset($_REQUEST['wiz_step3_subscription_name']) && !empty($_REQUEST['wiz_step3_subscription_name'])) {
             $name = $_REQUEST['wiz_step3_subscription_name'];
         }
         //if subscription list is not specified then create and attach default one
         $pl_subs->name = $name;
         $pl_subs->list_type = 'default';
-        $pl_subs->assigned_user_id= $GLOBALS['current_user']->id;
+        $pl_subs->assigned_user_id = $GLOBALS['current_user']->id;
         $pl_subs->save();
         $pl_list[] = $pl_subs;
     }
@@ -482,27 +482,27 @@ 
     //process exempt target list
     $create_new = true;
     $pl_un_subs = new ProspectList();
-    if(!empty($_REQUEST['wiz_step3_unsubscription_list_id'])){
+    if (!empty($_REQUEST['wiz_step3_unsubscription_list_id'])) {
         //if unsubscription list is specified then attach
         $pl_un_subs->retrieve($_REQUEST['wiz_step3_unsubscription_list_id']);
         //check to see name matches the bean, if not, then the user has chosen to create new bean
-        if($pl_un_subs->name == $_REQUEST['wiz_step3_unsubscription_name']){
+        if ($pl_un_subs->name == $_REQUEST['wiz_step3_unsubscription_name']) {
             $pl_list[] = $pl_un_subs;
             $create_new = false;
        }
 
     }
     //create new bean if one was not retrieved successfully
-    if($create_new){
+    if ($create_new) {
         //use default name if one has not been specified
-        $name = $campaign_name . " ".$mod_strings['LBL_UNSUBSCRIPTION_LIST'];
-        if(isset($_REQUEST['wiz_step3_unsubscription_name']) && !empty($_REQUEST['wiz_step3_unsubscription_name'])){
+        $name = $campaign_name." ".$mod_strings['LBL_UNSUBSCRIPTION_LIST'];
+        if (isset($_REQUEST['wiz_step3_unsubscription_name']) && !empty($_REQUEST['wiz_step3_unsubscription_name'])) {
             $name = $_REQUEST['wiz_step3_unsubscription_name'];
         }
         //if unsubscription list is not specified then create and attach default one
         $pl_un_subs->name = $name;
         $pl_un_subs->list_type = 'exempt';
-        $pl_un_subs->assigned_user_id= $GLOBALS['current_user']->id;
+        $pl_un_subs->assigned_user_id = $GLOBALS['current_user']->id;
         $pl_un_subs->save();
         $pl_list[] = $pl_un_subs;
     }
@@ -510,26 +510,26 @@ 
     //process test target list
     $pl_test = new ProspectList();
     $create_new = true;
-    if(!empty($_REQUEST['wiz_step3_test_list_id'])){
+    if (!empty($_REQUEST['wiz_step3_test_list_id'])) {
         //if test list is specified then attach
         $pl_test->retrieve($_REQUEST['wiz_step3_test_list_id']);
         //check to see name matches the bean, if not, then the user has chosen to create new bean
-        if($pl_test->name == $_REQUEST['wiz_step3_test_name']){
+        if ($pl_test->name == $_REQUEST['wiz_step3_test_name']) {
             $pl_list[] = $pl_test;
             $create_new = false;
         }
     }
     //create new bio if one was not retrieved successfully
-    if($create_new){
+    if ($create_new) {
         //use default name if one has not been specified
-        $name = $campaign_name . " ".$mod_strings['LBL_TEST_LIST'];
-        if(isset($_REQUEST['wiz_step3_test_name']) && !empty($_REQUEST['wiz_step3_test_name'])){
+        $name = $campaign_name." ".$mod_strings['LBL_TEST_LIST'];
+        if (isset($_REQUEST['wiz_step3_test_name']) && !empty($_REQUEST['wiz_step3_test_name'])) {
             $name = $_REQUEST['wiz_step3_test_name'];
         }
         //if test list is not specified then create and attach default one
         $pl_test->name = $name;
         $pl_test->list_type = 'test';
-        $pl_test->assigned_user_id= $GLOBALS['current_user']->id;
+        $pl_test->assigned_user_id = $GLOBALS['current_user']->id;
         $pl_test->save();
         $pl_list[] = $pl_test;
     }

modules/Campaigns/WizardNewsletter.php

Spacing +140 added lines, -140 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.
@@ -64,10 +64,10 @@ 
 /*************** GENERAL SETUP WORK **********/
 
 $focus = new Campaign();
-if(isset($_REQUEST['record'])) {
+if (isset($_REQUEST['record'])) {
     $focus->retrieve($_REQUEST['record']);
 }
-if(isset($_REQUEST['isDuplicate']) && $_REQUEST['isDuplicate'] == 'true') {
+if (isset($_REQUEST['isDuplicate']) && $_REQUEST['isDuplicate'] == 'true') {
     $focus->id = "";
 }
 global $theme;
@@ -78,9 +78,9 @@ 
 
 $GLOBALS['log']->info("Campaign NewsLetter Wizard");
 
-if( (isset($_REQUEST['wizardtype'])  && $_REQUEST['wizardtype']==1)  ||  ($focus->campaign_type=='NewsLetter')){
+if ((isset($_REQUEST['wizardtype']) && $_REQUEST['wizardtype'] == 1) || ($focus->campaign_type == 'NewsLetter')) {
     echo getClassicModuleTitle($mod_strings['LBL_MODULE_NAME'], array($mod_strings['LBL_NEWSLETTER WIZARD_TITLE'].$focus->name), true, false);
-}else{
+} else {
     echo getClassicModuleTitle($mod_strings['LBL_MODULE_NAME'], array($mod_strings['LBL_CAMPAIGN'].$focus->name), true, false);   
 }
 
@@ -104,26 +104,26 @@ 
 $sqs_objects = array('parent_name' => $qsd->getQSParent(), 
                     'assigned_user_name' => $qsd->getQSUser(),
                     //'prospect_list_name' => getProspectListQSObjects(),
-                    'test_name' => getProspectListQSObjects('prospect_list_type_test', 'test_name','wiz_step3_test_name_id'),
-                    'unsubscription_name' => getProspectListQSObjects('prospect_list_type_exempt', 'unsubscription_name','wiz_step3_unsubscription_name_id'),
-                    'subscription_name' => getProspectListQSObjects('prospect_list_type_default', 'subscription_name','wiz_step3_subscription_name_id'),
+                    'test_name' => getProspectListQSObjects('prospect_list_type_test', 'test_name', 'wiz_step3_test_name_id'),
+                    'unsubscription_name' => getProspectListQSObjects('prospect_list_type_exempt', 'unsubscription_name', 'wiz_step3_unsubscription_name_id'),
+                    'subscription_name' => getProspectListQSObjects('prospect_list_type_default', 'subscription_name', 'wiz_step3_subscription_name_id'),
                     );
                     
 
-$quicksearch_js = '<script type="text/javascript" language="javascript">sqs_objects = ' . $json->encode($sqs_objects) . '</script>';
+$quicksearch_js = '<script type="text/javascript" language="javascript">sqs_objects = '.$json->encode($sqs_objects).'</script>';
 
 $ss->assign("JAVASCRIPT", $quicksearch_js);
 
 
 //set the campaign type based on wizardtype value from request object
 $campaign_type = 'newsletter';
-if( (isset($_REQUEST['wizardtype'])  && $_REQUEST['wizardtype']==1)  ||  ($focus->campaign_type=='NewsLetter')){
+if ((isset($_REQUEST['wizardtype']) && $_REQUEST['wizardtype'] == 1) || ($focus->campaign_type == 'NewsLetter')) {
     $campaign_type = 'newsletter';
     $ss->assign("CAMPAIGN_DIAGNOSTIC_LINK", diagnose());    
-}elseif( (isset($_REQUEST['wizardtype'])  && $_REQUEST['wizardtype']==2)  || ($focus->campaign_type=='Email') ){
+}elseif ((isset($_REQUEST['wizardtype']) && $_REQUEST['wizardtype'] == 2) || ($focus->campaign_type == 'Email')) {
     $campaign_type = 'email';
     $ss->assign("CAMPAIGN_DIAGNOSTIC_LINK", diagnose());
-}else{
+} else {
     $campaign_type = 'general';
 }
 
@@ -143,7 +143,7 @@ 
 
 //set default values
 $ss->assign("CALENDAR_LANG", "en");
-$ss->assign("USER_DATEFORMAT", '('. $timedate->get_user_date_format().')');
+$ss->assign("USER_DATEFORMAT", '('.$timedate->get_user_date_format().')');
 $ss->assign("CALENDAR_DATEFORMAT", $timedate->get_cal_date_format());
 $ss->assign("CAMP_DATE_ENTERED", $focus->date_entered);
 $ss->assign("CAMP_DATE_MODIFIED", $focus->date_modified);
@@ -170,50 +170,50 @@ 
 
 $focus->list_view_parse_additional_sections($ss);
 
-$ss->assign("ASSIGNED_USER_ID", $focus->assigned_user_id );
+$ss->assign("ASSIGNED_USER_ID", $focus->assigned_user_id);
 
-if((!isset($focus->status)) && (!isset($focus->id)))
+if ((!isset($focus->status)) && (!isset($focus->id)))
     $ss->assign("STATUS_OPTIONS", get_select_options_with_id($app_list_strings['campaign_status_dom'], 'Planning'));
 else
     $ss->assign("STATUS_OPTIONS", get_select_options_with_id($app_list_strings['campaign_status_dom'], $focus->status));
 
 //hide frequency options if this is not a newsletter
-if($campaign_type == 'newsletter'){
+if ($campaign_type == 'newsletter') {
     $ss->assign("HIDE_FREQUENCY_IF_NEWSLETTER", "Select");
     $ss->assign("FREQUENCY_LABEL", $mod_strings['LBL_CAMPAIGN_FREQUENCY']);
-    if((!isset($focus->frequency)) && (!isset($focus->id))){
+    if ((!isset($focus->frequency)) && (!isset($focus->id))) {
         $ss->assign("FREQ_OPTIONS", get_select_options_with_id($app_list_strings['newsletter_frequency_dom'], 'Monthly'));
-    }else{
+    } else {
         $ss->assign("FREQ_OPTIONS", get_select_options_with_id($app_list_strings['newsletter_frequency_dom'], $focus->frequency));
     }
-}else{
+} else {
     $ss->assign("HIDE_FREQUENCY_IF_NEWSLETTER", "input type='hidden'");
     $ss->assign("FREQUENCY_LABEL", '&nbsp;');
 }
 global $current_user;
 require_once('modules/Currencies/ListCurrency.php');
 $currency = new ListCurrency();
-if(isset($focus->currency_id) && !empty($focus->currency_id)){
+if (isset($focus->currency_id) && !empty($focus->currency_id)) {
     $selectCurrency = $currency->getSelectOptions($focus->currency_id);
     $ss->assign("CURRENCY", $selectCurrency);
 }
-else if($current_user->getPreference('currency') && !isset($focus->id))
+else if ($current_user->getPreference('currency') && !isset($focus->id))
 {
     $selectCurrency = $currency->getSelectOptions($current_user->getPreference('currency'));
     $ss->assign("CURRENCY", $selectCurrency);
-}else{
+} else {
 
     $selectCurrency = $currency->getSelectOptions();
     $ss->assign("CURRENCY", $selectCurrency);
 
 }
 global $current_user;
-if(is_admin($current_user) && $_REQUEST['module'] != 'DynamicLayout' && !empty($_SESSION['editinplace'])){  
+if (is_admin($current_user) && $_REQUEST['module'] != 'DynamicLayout' && !empty($_SESSION['editinplace'])) {  
     $record = '';
-    if(!empty($_REQUEST['record'])){
-        $record =   $_REQUEST['record'];
+    if (!empty($_REQUEST['record'])) {
+        $record = $_REQUEST['record'];
     }
-    $ss->assign("ADMIN_EDIT","<a href='index.php?action=index&module=DynamicLayout&from_action=".$_REQUEST['action'] ."&from_module=".$_REQUEST['module'] ."&record=".$record. "'>".SugarThemeRegistry::current()->getImage("EditLayout","border='0' align='bottom'",null,null,'.gif',$mod_strings['LBL_EDIT_LAYOUT'])."</a>");
+    $ss->assign("ADMIN_EDIT", "<a href='index.php?action=index&module=DynamicLayout&from_action=".$_REQUEST['action']."&from_module=".$_REQUEST['module']."&record=".$record."'>".SugarThemeRegistry::current()->getImage("EditLayout", "border='0' align='bottom'", null, null, '.gif', $mod_strings['LBL_EDIT_LAYOUT'])."</a>");
 
 }
 
@@ -226,55 +226,55 @@ 
 
 //fill out the campaign type dropdown based on type of campaign being created
 $ss->assign("campaign_type", $campaign_type);
-if($campaign_type == 'general'){
+if ($campaign_type == 'general') {
     //get regular campaign dom object and strip out entries for email and newsletter
     $myTypeOptionsArr = array();
     $OptionsArr = $app_list_strings['campaign_type_dom'];
-    foreach($OptionsArr as $key=>$val){
-        if($val =='Newsletter' || $val =='Email' || $val =='' ){
+    foreach ($OptionsArr as $key=>$val) {
+        if ($val == 'Newsletter' || $val == 'Email' || $val == '') {
             //do not add   
-        }else{
+        } else {
             $myTypeOptionsArr[$key] = $val;
         }
     }
     
     //now create select option html without the newsletter/email, or blank ('') options
-    $type_option_html =' ';
+    $type_option_html = ' ';
     $selected = false;
-    foreach($myTypeOptionsArr as $optionKey=>$optionName){
+    foreach ($myTypeOptionsArr as $optionKey=>$optionName) {
         //if the selected flag is set to true, then just populate
-        if($selected){
-            $type_option_html .="<option value='$optionKey' >$optionName</option>";
-        }else{//if not selected yet, check to see if this option should be selected
+        if ($selected) {
+            $type_option_html .= "<option value='$optionKey' >$optionName</option>";
+        } else {//if not selected yet, check to see if this option should be selected
             //if the campaign type is not empty, then select the retrieved type
-            if(!empty($focus->campaign_type)){
+            if (!empty($focus->campaign_type)) {
                 //check to see if key matches campaign type
-                if($optionKey == $focus->campaign_type){
+                if ($optionKey == $focus->campaign_type) {
                     //mark as selected
-                    $type_option_html .="<option value='$optionKey' selected>$optionName</option>";
+                    $type_option_html .= "<option value='$optionKey' selected>$optionName</option>";
                     //mark as selected for next time
-                    $selected=true;
-                }else{
+                    $selected = true;
+                } else {
                     //key does not match, just populate
-                    $type_option_html .="<option value='$optionKey' >$optionName</option>";
+                    $type_option_html .= "<option value='$optionKey' >$optionName</option>";
                 }
-            }else{
+            } else {
             //since the campaign type is empty, then select first one                
-                $type_option_html .="<option value='$optionKey' selected>$optionName</option>";    
+                $type_option_html .= "<option value='$optionKey' selected>$optionName</option>";    
                 //mark as selected for next time
-                $selected=true;
+                $selected = true;
             }
         }    
     }
     //assign the modified dropdown for general campaign creation
     $ss->assign("CAMPAIGN_TYPE_OPTIONS", $type_option_html);
     $ss->assign("SHOULD_TYPE_BE_DISABLED", "select");    
-}elseif($campaign_type == 'email'){
+}elseif ($campaign_type == 'email') {
     //Assign Email as type of campaign being created an disable the select widget
     $ss->assign("CAMPAIGN_TYPE_OPTIONS", $mod_strings['LBL_EMAIL']);
     $ss->assign("SHOULD_TYPE_BE_DISABLED", "input type='hidden' value='Email'");
     $ss->assign("HIDE_CAMPAIGN_TYPE", true);
-}else{
+} else {
     //Assign NewsLetter as type of campaign being created an disable the select widget
     $ss->assign("CAMPAIGN_TYPE_OPTIONS", $mod_strings['LBL_NEWSLETTER']);
     $ss->assign("SHOULD_TYPE_BE_DISABLED", "input type='hidden' value='NewsLetter'");
@@ -291,33 +291,33 @@ 
 $focus->load_relationship('tracked_urls');
 
 $trkr_lists = $focus->tracked_urls->get();
-$trkr_html ='';    
-$ss->assign('TRACKER_COUNT',count($trkr_lists));
-if(count($trkr_lists)>0){
+$trkr_html = '';    
+$ss->assign('TRACKER_COUNT', count($trkr_lists));
+if (count($trkr_lists) > 0) {
 global $odd_bg, $even_bg, $hilite_bg;
     
     $trkr_count = 0;
 //create the html to create tracker table
-    foreach($trkr_lists as $trkr_id){
+    foreach ($trkr_lists as $trkr_id) {
         $ct_focus = new CampaignTracker();
         $ct_focus->retrieve($trkr_id);
-      if(isset($ct_focus->tracker_name) && !empty($ct_focus->tracker_name)){
-            if($ct_focus->is_optout){$opt = 'checked';}else{$opt = '';}
-            $trkr_html .= "<div id='existing_trkr".$trkr_count."'> <table width='100%' border='0' cellspacing='0' cellpadding='0'>" ;
-            $trkr_html .= "<tr class='evenListRowS1'><td width='15%'><input name='wiz_step3_is_optout".$trkr_count."' title='".$mod_strings['LBL_EDIT_OPT_OUT'] . $trkr_count ."' id='existing_is_optout". $trkr_count ."' class='checkbox' type='checkbox' $opt  /><input name='wiz_step3_id".$trkr_count."' value='".$ct_focus->id."' id='existing_tracker_id". $trkr_count ."'type='hidden''/></td>";
-            $trkr_html .= "<td width='40%'> <input id='existing_tracker_name". $trkr_count ."' type='text' size='20' maxlength='255' name='wiz_step3_tracker_name". $trkr_count ."' title='".$mod_strings['LBL_EDIT_TRACKER_NAME']. $trkr_count ."' value='".$ct_focus->tracker_name."' ></td>";
-            $trkr_html .= "<td width='40%'><input type='text' size='60' maxlength='255' name='wiz_step3_tracker_url". $trkr_count ."' title='".$mod_strings['LBL_EDIT_TRACKER_URL']. $trkr_count ."' id='existing_tracker_url". $trkr_count ."' value='".$ct_focus->tracker_url."' ></td>";
+      if (isset($ct_focus->tracker_name) && !empty($ct_focus->tracker_name)) {
+            if ($ct_focus->is_optout) {$opt = 'checked'; } else {$opt = ''; }
+            $trkr_html .= "<div id='existing_trkr".$trkr_count."'> <table width='100%' border='0' cellspacing='0' cellpadding='0'>";
+            $trkr_html .= "<tr class='evenListRowS1'><td width='15%'><input name='wiz_step3_is_optout".$trkr_count."' title='".$mod_strings['LBL_EDIT_OPT_OUT'].$trkr_count."' id='existing_is_optout".$trkr_count."' class='checkbox' type='checkbox' $opt  /><input name='wiz_step3_id".$trkr_count."' value='".$ct_focus->id."' id='existing_tracker_id".$trkr_count."'type='hidden''/></td>";
+            $trkr_html .= "<td width='40%'> <input id='existing_tracker_name".$trkr_count."' type='text' size='20' maxlength='255' name='wiz_step3_tracker_name".$trkr_count."' title='".$mod_strings['LBL_EDIT_TRACKER_NAME'].$trkr_count."' value='".$ct_focus->tracker_name."' ></td>";
+            $trkr_html .= "<td width='40%'><input type='text' size='60' maxlength='255' name='wiz_step3_tracker_url".$trkr_count."' title='".$mod_strings['LBL_EDIT_TRACKER_URL'].$trkr_count."' id='existing_tracker_url".$trkr_count."' value='".$ct_focus->tracker_url."' ></td>";
             $trkr_html .= "<td><a href='#' onclick=\"javascript:remove_existing_tracker('existing_trkr".$trkr_count."','".$ct_focus->id."'); \" >  ";
             $trkr_html .= SugarThemeRegistry::current()->getImage('delete_inline', "border='0'  align='absmiddle'", 12, 12, ".gif", $mod_strings['LBL_DELETE'])."</a></td></tr></table></div>";
 
 
 
       }
-      $trkr_count =$trkr_count+1;
+      $trkr_count = $trkr_count + 1;
     }
     
     $trkr_html .= "<div id='no_trackers'></div>";
-    }else{
+    } else {
         $trkr_html .= "<div id='no_trackers'><table width='100%' border='0' cellspacing='0' cellpadding='0'><tr class='evenListRowS1'><td>".$mod_strings['LBL_NONE']."</td></tr></table></div>";
     }
     $ss->assign('EXISTING_TRACKERS', $trkr_html);
@@ -396,7 +396,7 @@ 
 $targetList = BeanFactory::getBean('ProspectLists')->get_full_list();
 
 $targetListData = array();
-foreach($targetList as $prospectLst) {
+foreach ($targetList as $prospectLst) {
     $nxt = array(
         'id' => $prospectLst->id,
         'name' => $prospectLst->name,
@@ -425,29 +425,29 @@ 
 
 $prospect_lists = $focus->prospectlists->get();
 
-if((isset($_REQUEST['wizardtype']) && $_REQUEST['wizardtype'] ==1) || ($focus->campaign_type=='NewsLetter')){
+if ((isset($_REQUEST['wizardtype']) && $_REQUEST['wizardtype'] == 1) || ($focus->campaign_type == 'NewsLetter')) {
  //this is a newsletter type campaign, fill in subscription values   
 
 //if prospect lists are returned, then iterate through and populate form values
-if(count($prospect_lists)>0){
+if (count($prospect_lists) > 0) {
     
-    foreach($prospect_lists as $pl_id){
+    foreach ($prospect_lists as $pl_id) {
     //retrieve prospect list
      $pl = new ProspectList();   
      $pl->retrieve($pl_id);
 
-      if(isset($pl->list_type) && !empty($pl->list_type)){
+      if (isset($pl->list_type) && !empty($pl->list_type)) {
          //assign values based on type
-         if(($pl->list_type == 'default') || ($pl->list_type == 'seed')){            
+         if (($pl->list_type == 'default') || ($pl->list_type == 'seed')) {            
             $ss->assign('SUBSCRIPTION_ID', $pl->id);
             $ss->assign('SUBSCRIPTION_NAME', $pl->name);
          };
-         if($pl->list_type == 'exempt'){
+         if ($pl->list_type == 'exempt') {
             $ss->assign('UNSUBSCRIPTION_ID', $pl->id);
             $ss->assign('UNSUBSCRIPTION_NAME', $pl->name);
          
          };
-         if($pl->list_type == 'test'){
+         if ($pl->list_type == 'test') {
             $ss->assign('TEST_ID', $pl->id);
             $ss->assign('TEST_NAME', $pl->name);
          
@@ -459,73 +459,73 @@ 
 
 
 
-}else{
+} else {
     //this is not a newlsetter campaign, so fill in target list table
     //create array for javascript, this will help to display the option text, not the value
-    $dom_txt =' ';
-    foreach($app_list_strings['prospect_list_type_dom'] as $key=>$val){
-        $dom_txt .="if(trgt_type_text =='$key'){trgt_type_text='$val';}";
+    $dom_txt = ' ';
+    foreach ($app_list_strings['prospect_list_type_dom'] as $key=>$val) {
+        $dom_txt .= "if(trgt_type_text =='$key'){trgt_type_text='$val';}";
     }
     $ss->assign("PL_DOM_STMT", $dom_txt);
     $trgt_count = 0;
     $trgt_html = ' ';
-    if(count($prospect_lists)>0){
+    if (count($prospect_lists) > 0) {
 
-        foreach($prospect_lists as $pl_id){
+        foreach ($prospect_lists as $pl_id) {
             //retrieve prospect list
             $pl = new ProspectList();
             $pl_focus = $pl->retrieve($pl_id);
-            $trgt_html .= "<div id='existing_trgt".$trgt_count."'> <table class='tabDetailViewDL2' width='100%'>" ;
-            $trgt_html .= "<td width='100' style=\"width:25%\"> <input id='existing_target_name". $trgt_count ."' type='hidden' type='text' size='60' maxlength='255' name='existing_target_name". $trgt_count ."'  value='". $pl_focus->name."' ><a href=\"index.php?module=ProspectLists&action=DetailView&record=" . $pl_focus->id . "\" target=\"_blank\" title=\"" . $mod_strings['LBL_OPEN_IN_NEW_WINDOW'] . "\">". $pl_focus->name."</a></td>";
+            $trgt_html .= "<div id='existing_trgt".$trgt_count."'> <table class='tabDetailViewDL2' width='100%'>";
+            $trgt_html .= "<td width='100' style=\"width:25%\"> <input id='existing_target_name".$trgt_count."' type='hidden' type='text' size='60' maxlength='255' name='existing_target_name".$trgt_count."'  value='".$pl_focus->name."' ><a href=\"index.php?module=ProspectLists&action=DetailView&record=".$pl_focus->id."\" target=\"_blank\" title=\"".$mod_strings['LBL_OPEN_IN_NEW_WINDOW']."\">".$pl_focus->name."</a></td>";
             $trgt_html .= "<td width='100' style=\"width:25%\">".$pl_focus->get_entry_count()."</td>";
-            $trgt_html .= "<td width='100' style=\"width:25%\"><input type='hidden' size='60' maxlength='255' name='existing_tracker_list_type". $trgt_count ."'   id='existing_tracker_list_type". $trgt_count ."' value='".$pl_focus->list_type."' >".$app_list_strings['prospect_list_type_dom'][$pl_focus->list_type];
-            $trgt_html .= "<input type='hidden' name='added_target_id". $trgt_count ."' id='added_target_id". $trgt_count ."' value='". $pl_focus->id ."' ></td>";
+            $trgt_html .= "<td width='100' style=\"width:25%\"><input type='hidden' size='60' maxlength='255' name='existing_tracker_list_type".$trgt_count."'   id='existing_tracker_list_type".$trgt_count."' value='".$pl_focus->list_type."' >".$app_list_strings['prospect_list_type_dom'][$pl_focus->list_type];
+            $trgt_html .= "<input type='hidden' name='added_target_id".$trgt_count."' id='added_target_id".$trgt_count."' value='".$pl_focus->id."' ></td>";
             $trgt_html .= "<td width='100' style=\"width:25%\"><a href='#' onclick=\"javascript:remove_existing_target('existing_trgt".$trgt_count."','".$pl_focus->id."'); \" >  ";
             $trgt_html .= SugarThemeRegistry::current()->getImage('delete_inline', "border='0' align='absmiddle'", 12, 12, ".gif", $mod_strings['LBL_DELETE'])."</a></td></tr></table></div>";
 
-            $trgt_count =$trgt_count +1;
+            $trgt_count = $trgt_count + 1;
         }
 
         $trgt_html  .= "<div id='no_targets'></div>";
-    }else{
+    } else {
         $trgt_html  .= "<div id='no_targets'><table width='100%' border='0' cellspacing='0' cellpadding='0'><tr class='evenListRowS1'><td>".$mod_strings['LBL_NONE']."</td></tr></table></div>";
 
     }
-    $ss->assign('EXISTING_TARGETS', $trgt_html );
+    $ss->assign('EXISTING_TARGETS', $trgt_html);
 
 }
 
     
 /**************************** WIZARD UI DIV Stuff *******************/
 $mrkt_string = $mod_strings['LBL_NAVIGATION_MENU_MARKETING'];
-if(!empty($focus->id)){
+if (!empty($focus->id)) {
     $mrkt_url = "<a  href='index.php?action=WizardMarketing&module=Campaigns&return_module=Campaigns&return_action=WizardHome";
     $mrkt_url .= "&return_id=".$focus->id."&campaign_id=".$focus->id;
-    $mrkt_url .= "'>". $mrkt_string."</a>";
+    $mrkt_url .= "'>".$mrkt_string."</a>";
     $mrkt_string = $mrkt_url;
 }
     $summ_url = $mod_strings['LBL_NAVIGATION_MENU_SUMMARY'];
-    if(!empty($focus->id)){
+    if (!empty($focus->id)) {
         $summ_url = "<a  href='index.php?action=WizardHome&module=Campaigns";
         $summ_url .= "&return_id=".$focus->id."&record=".$focus->id;
-        $summ_url .= "'> ". $mod_strings['LBL_NAVIGATION_MENU_SUMMARY']."</a>";
+        $summ_url .= "'> ".$mod_strings['LBL_NAVIGATION_MENU_SUMMARY']."</a>";
     } 
    
 
 
-$script_to_call ='';
-    if (!empty($focus->id)){
+$script_to_call = '';
+    if (!empty($focus->id)) {
         $maxStep = 2;
         $script_to_call = "link_navs(1, {$maxStep});";
-        if(isset($_REQUEST['direct_step']) and !empty($_REQUEST['direct_step'])){
-            $directStep = (int) $_REQUEST['direct_step'];
-            if($directStep < 1) {
+        if (isset($_REQUEST['direct_step']) and !empty($_REQUEST['direct_step'])) {
+            $directStep = (int)$_REQUEST['direct_step'];
+            if ($directStep < 1) {
                 $directStep = 1;
             }
-            if($directStep > $maxStep) {
+            if ($directStep > $maxStep) {
                 $directStep = $maxStep;
             }
-            $script_to_call .='   direct(' . $directStep . ');';
+            $script_to_call .= '   direct('.$directStep.');';
         }
     } 
     $ss->assign("HILITE_ALL", $script_to_call);
@@ -565,56 +565,56 @@ 
 
 //Create the html to fill in the wizard steps
 
-if($campaign_type == 'general'){
+if ($campaign_type == 'general') {
     $steps = create_campaign_steps();
 
-    foreach($steps as $key => $step) {
+    foreach ($steps as $key => $step) {
         $_steps[$key] = false;
     }
-    $ss->assign('NAV_ITEMS',create_wiz_menu_items($_steps,'campaign',$mrkt_string,$summ_url, 'dotlist'));
-    $ss->assign('HIDE_CONTINUE','hidden');
+    $ss->assign('NAV_ITEMS', create_wiz_menu_items($_steps, 'campaign', $mrkt_string, $summ_url, 'dotlist'));
+    $ss->assign('HIDE_CONTINUE', 'hidden');
 
-}elseif($campaign_type == 'email'){
+}elseif ($campaign_type == 'email') {
     $steps = create_email_steps();
-    if($focus->id) {
-        $summ_url = "index.php?action=WizardHome&module=Campaigns&return_id=" . $focus->id . "&record=" . $focus->id;
+    if ($focus->id) {
+        $summ_url = "index.php?action=WizardHome&module=Campaigns&return_id=".$focus->id."&record=".$focus->id;
     }
     else {
         $summ_url = false;
     }
-    foreach($steps as $key => $step) {
+    foreach ($steps as $key => $step) {
         $_steps[$key] = false;
     }
     $campaign_id = $focus->id;
     $marketing_id = isset($_REQUEST['marketing_id']) && $_REQUEST['marketing_id'] ? $_REQUEST['marketing_id'] : null;
     $template_id = isset($_REQUEST['template_id']) && $_REQUEST['template_id'] ? $_REQUEST['template_id'] : null;
-    $ss->assign('NAV_ITEMS',create_wiz_menu_items($_steps,'email',$mrkt_string,$summ_url, 'dotlist', $campaign_id, $marketing_id, $template_id));
-    $ss->assign('HIDE_CONTINUE','submit');
-}else{
+    $ss->assign('NAV_ITEMS', create_wiz_menu_items($_steps, 'email', $mrkt_string, $summ_url, 'dotlist', $campaign_id, $marketing_id, $template_id));
+    $ss->assign('HIDE_CONTINUE', 'submit');
+} else {
     $steps = create_newsletter_steps();
 
-    if($focus->id) {
-        $summ_url = "index.php?action=WizardHome&module=Campaigns&return_id=" . $focus->id . "&record=" . $focus->id;
+    if ($focus->id) {
+        $summ_url = "index.php?action=WizardHome&module=Campaigns&return_id=".$focus->id."&record=".$focus->id;
     }
     else {
         $summ_url = false;
     }
-    foreach($steps as $key => $step) {
+    foreach ($steps as $key => $step) {
         $_steps[$key] = false;
     }
-    $ss->assign('NAV_ITEMS',create_wiz_menu_items($_steps,'newsletter',$mrkt_string,$summ_url, 'dotlist'));
-    $ss->assign('HIDE_CONTINUE','submit');
+    $ss->assign('NAV_ITEMS', create_wiz_menu_items($_steps, 'newsletter', $mrkt_string, $summ_url, 'dotlist'));
+    $ss->assign('HIDE_CONTINUE', 'submit');
 }
 
 $ss->assign('TOTAL_STEPS', count($steps));
-$sshtml = create_wiz_step_divs($steps,$ss);
-$ss->assign('STEPS',$sshtml);
+$sshtml = create_wiz_step_divs($steps, $ss);
+$ss->assign('STEPS', $sshtml);
      	   	
 
 /**************************** FINAL END OF PAGE UI Stuff *******************/
 
-if(isset($_REQUEST['wizardtype'])) {
-    switch($_REQUEST['wizardtype']) {
+if (isset($_REQUEST['wizardtype'])) {
+    switch ($_REQUEST['wizardtype']) {
         case '1':
             $ss->assign('campaign_type', 'NewsLetter');
             break;
@@ -629,12 +629,12 @@ 
 
 $ss->display(file_exists('custom/modules/Campaigns/tpls/WizardNewsletter.tpl') ? 'custom/modules/Campaigns/tpls/WizardNewsletter.tpl' : 'modules/Campaigns/tpls/WizardNewsletter.tpl');
 
-if(!$focus->id) {
+if (!$focus->id) {
     unset($_SESSION['campaignWizard'][$campaign_id]['defaultSelectedMarketingId']);
 }
 
 
-function create_newsletter_steps(){
+function create_newsletter_steps() {
     global $mod_strings;
     $steps[$mod_strings['LBL_NAVIGATION_MENU_GEN1']]          = file_exists('custom/modules/Campaigns/tpls/WizardCampaignHeader.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignHeader.tpl' : 'modules/Campaigns/tpls/WizardCampaignHeader.tpl';
     //$steps[$mod_strings['LBL_NAVIGATION_MENU_GEN2']]          = file_exists('custom/modules/Campaigns/tpls/WizardCampaignBudget.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignBudget.tpl' : 'modules/Campaigns/tpls/WizardCampaignBudget.tpl';
@@ -643,54 +643,54 @@ 
     return  $steps;
 }
 
-function create_campaign_steps(){
+function create_campaign_steps() {
     global $mod_strings;
     $steps[$mod_strings['LBL_NAVIGATION_MENU_GEN1']]          = file_exists('custom/modules/Campaigns/tpls/WizardCampaignHeader.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignHeader.tpl' : 'modules/Campaigns/tpls/WizardCampaignHeader.tpl';
     $steps[$mod_strings['LBL_NAVIGATION_MENU_GEN2']]          = file_exists('custom/modules/Campaigns/tpls/WizardCampaignBudget.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignBudget.tpl' : 'modules/Campaigns/tpls/WizardCampaignBudget.tpl';
-    $steps[$mod_strings['LBL_TARGET_LISTS']]                   = file_exists('custom/modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl' : 'modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl';
+    $steps[$mod_strings['LBL_TARGET_LISTS']] = file_exists('custom/modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl' : 'modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl';
     return  $steps;
 }
 
-function create_email_steps(){
+function create_email_steps() {
     global $mod_strings;
-    $steps[$mod_strings['LBL_NAVIGATION_MENU_GEN1']]          = file_exists('custom/modules/Campaigns/tpls/WizardCampaignHeader.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignHeader.tpl' : 'modules/Campaigns/tpls/WizardCampaignHeader.tpl';
+    $steps[$mod_strings['LBL_NAVIGATION_MENU_GEN1']] = file_exists('custom/modules/Campaigns/tpls/WizardCampaignHeader.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignHeader.tpl' : 'modules/Campaigns/tpls/WizardCampaignHeader.tpl';
     //$steps[$mod_strings['LBL_NAVIGATION_MENU_GEN2']]          = file_exists('custom/modules/Campaigns/tpls/WizardCampaignBudget.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignBudget.tpl' : 'modules/Campaigns/tpls/WizardCampaignBudget.tpl';
     //$steps[$mod_strings['LBL_NAVIGATION_MENU_TRACKERS']]      = file_exists('custom/modules/Campaigns/tpls/WizardCampaignTracker.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignTracker.tpl' : 'modules/Campaigns/tpls/WizardCampaignTracker.tpl';
-    $steps[$mod_strings['LBL_TARGET_LISTS']]                   = file_exists('custom/modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl' : 'modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl';
+    $steps[$mod_strings['LBL_TARGET_LISTS']] = file_exists('custom/modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl') ? 'custom/modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl' : 'modules/Campaigns/tpls/WizardCampaignTargetListForNonNewsLetter.tpl';
     return  $steps;
 }
 
 
-function create_wiz_step_divs($steps,$ss){
+function create_wiz_step_divs($steps, $ss) {
     $step_html = '';
-    if(isset($steps)  && !empty($steps)){
-        $i=1;
-        foreach($steps as $name=>$step){
-            $step_html .="<p><div id='step$i'>";
+    if (isset($steps) && !empty($steps)) {
+        $i = 1;
+        foreach ($steps as $name=>$step) {
+            $step_html .= "<p><div id='step$i'>";
             $step_html .= $ss->fetch($step);
-            $step_html .="</div></p>";
-            $i = $i+1;
+            $step_html .= "</div></p>";
+            $i = $i + 1;
         }    
     }
     return $step_html;
 }
 
-function create_wiz_menu_items($steps,$type,$mrkt_string,$summ_url, $view = null, $campaign_id = null, $marketing_id = null, $template_id = null){
+function create_wiz_menu_items($steps, $type, $mrkt_string, $summ_url, $view = null, $campaign_id = null, $marketing_id = null, $template_id = null) {
 
     global $mod_strings;
 
 
-    if($view == 'dotlist') {
+    if ($view == 'dotlist') {
 
         include_once 'modules/Campaigns/DotListWizardMenu.php';
 
-        if($type!='campaign') {
+        if ($type != 'campaign') {
             $templateURLForProgressBar = false;
             if ($campaign_id && $marketing_id && $template_id) {
                 $templateURLForProgressBar = "index.php?action=WizardMarketing&module=Campaigns&return_module=Campaigns&return_action=WizardHome&return_id={$campaign_id}&campaign_id={$campaign_id}&jump=2&marketing_id={$marketing_id}&record={$marketing_id}&campaign_type=Email&template_id={$template_id}";
             }
 
-            if(preg_match('/\bhref=\'([^\']*)/', $mrkt_string, $matches)) {
+            if (preg_match('/\bhref=\'([^\']*)/', $mrkt_string, $matches)) {
                 $templateURLForProgressBar = $matches[1];
             }
 
@@ -700,8 +700,8 @@ 
         if ($type == 'newsletter' || $type == 'email') {
 
             preg_match('/\bhref=\'([^\']*)/', $mrkt_string, $matches);
-            if(isset($matches[1])) {
-                $marketingLink = $matches[1] . ($matches[1] ? '&jump=2' : false);
+            if (isset($matches[1])) {
+                $marketingLink = $matches[1].($matches[1] ? '&jump=2' : false);
             }
             else {
                 $marketingLink = false;
@@ -719,19 +719,19 @@ 
     }
     else {
         $nav_html = '<table border="0" cellspacing="0" cellpadding="0" width="100%" >';
-        if(isset($steps)  && !empty($steps)){
-            $i=1;
-            foreach($steps as $name=>$step){
+        if (isset($steps) && !empty($steps)) {
+            $i = 1;
+            foreach ($steps as $name=>$step) {
                 $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step$i'>$name</div></td></tr>";
-                $i=$i+1;
+                $i = $i + 1;
             }
         }
-        if($type == 'newsletter'  ||  $type == 'email'){
-            $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step'".($i+1).">$mrkt_string</div></td></tr>";
-            $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step'".($i+2).">".$mod_strings['LBL_NAVIGATION_MENU_SEND_EMAIL']."</div></li>";
-            $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step'".($i+3).">".$summ_url."</div></td></tr>";
-        }else{
-            $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step'".($i+1).">".$summ_url."</div></td></tr>";
+        if ($type == 'newsletter' || $type == 'email') {
+            $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step'".($i + 1).">$mrkt_string</div></td></tr>";
+            $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step'".($i + 2).">".$mod_strings['LBL_NAVIGATION_MENU_SEND_EMAIL']."</div></li>";
+            $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step'".($i + 3).">".$summ_url."</div></td></tr>";
+        } else {
+            $nav_html .= "<tr><td scope='row' nowrap><div id='nav_step'".($i + 1).">".$summ_url."</div></td></tr>";
         }
 
         $nav_html .= '</table>';

modules/Campaigns/WebToLeadCapture.php

Spacing +43 added lines, -43 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.
@@ -47,7 +47,7 @@ 
 
 $mod_strings = return_module_language($sugar_config['default_language'], 'Leads');
 
-$app_list_strings['record_type_module'] = array('Contact'=>'Contacts', 'Account'=>'Accounts', 'Opportunity'=>'Opportunities', 'Case'=>'Cases', 'Note'=>'Notes', 'Call'=>'Calls', 'Email'=>'Emails', 'Meeting'=>'Meetings', 'Task'=>'Tasks', 'Lead'=>'Leads','Bug'=>'Bugs',
+$app_list_strings['record_type_module'] = array('Contact'=>'Contacts', 'Account'=>'Accounts', 'Opportunity'=>'Opportunities', 'Case'=>'Cases', 'Note'=>'Notes', 'Call'=>'Calls', 'Email'=>'Emails', 'Meeting'=>'Meetings', 'Task'=>'Tasks', 'Lead'=>'Leads', 'Bug'=>'Bugs',
 
 );
 
@@ -61,18 +61,18 @@ 
 if (isset($_POST['campaign_id']) && !empty($_POST['campaign_id'])) {
 	    //adding the client ip address
 	    $_POST['client_id_address'] = query_client_ip();
-		$campaign_id=$_POST['campaign_id'];
+		$campaign_id = $_POST['campaign_id'];
 		$campaign = new Campaign();
 		$camp_query  = "select name,id from campaigns where id='$campaign_id'";
 		$camp_query .= " and deleted=0";
-        $camp_result=$campaign->db->query($camp_query);
+        $camp_result = $campaign->db->query($camp_query);
         $camp_data = $campaign->db->fetchByAssoc($camp_result);
         // Bug 41292 - have to select marketing_id for new lead
         $db = DBManagerFactory::getInstance();
         $marketing = new EmailMarketing();
         $marketing_query = $marketing->create_new_list_query(
                 'date_start desc, date_modified desc',
-                "campaign_id = '{$campaign_id}' and status = 'active' and date_start < " . $db->convert('', 'today'),
+                "campaign_id = '{$campaign_id}' and status = 'active' and date_start < ".$db->convert('', 'today'),
                 array('id')
         );
         $marketing_result = $db->limitQuery($marketing_query, 0, 1, true);
@@ -83,22 +83,22 @@ 
 			$current_user->retrieve($_REQUEST['assigned_user_id']);
 		} 
 
-	    if(isset($camp_data) && $camp_data != null ){
+	    if (isset($camp_data) && $camp_data != null) {
 			$leadForm = new LeadFormBase();
             $lead = new Lead();
 			$prefix = '';
-			if(!empty($_POST['prefix'])){
+			if (!empty($_POST['prefix'])) {
 				$prefix = $_POST['prefix'];
 			}
 
-			if(empty($lead->id)) {
+			if (empty($lead->id)) {
                 $lead->id = create_guid();
                 $lead->new_with_id = true;
             }
             $GLOBALS['check_notify'] = true;
 
             //bug: 47574 - make sure, that webtolead_email1 field has same required attribute as email1 field
-            if(isset($lead->required_fields['email1'])){
+            if (isset($lead->required_fields['email1'])) {
                 $lead->required_fields['webtolead_email1'] = $lead->required_fields['email1'];
             }
             
@@ -114,7 +114,7 @@ 
             // checkRequired needs a major overhaul before it works for web to lead forms.
             $lead = $leadForm->handleSave($prefix, false, false, false, $lead);
             
-			if(!empty($lead)){
+			if (!empty($lead)) {
 				
 	            //create campaign log
 	            $camplog = new CampaignLog();
@@ -123,9 +123,9 @@ 
 	            $camplog->related_type = $lead->module_dir;
 	            $camplog->activity_type = "lead";
 	            $camplog->target_type = $lead->module_dir;
-	            $campaign_log->activity_date=$timedate->now();
-	            $camplog->target_id    = $lead->id;
-                if(isset($marketing_data['id']))
+	            $campaign_log->activity_date = $timedate->now();
+	            $camplog->target_id = $lead->id;
+                if (isset($marketing_data['id']))
                 {
                     $camplog->marketing_id = $marketing_data['id'];
                 }
@@ -155,7 +155,7 @@ 
                 
 		        $lead->load_relationship('campaigns');
 		        $lead->campaigns->add($camplog->id);
-                if(!empty($GLOBALS['check_notify'])) {
+                if (!empty($GLOBALS['check_notify'])) {
                     $lead->save($GLOBALS['check_notify']);
                 }
                 else {
@@ -164,47 +164,47 @@ 
             }
 
             //in case there are forms out there still using email_opt_out
-            if(isset($_POST['webtolead_email_opt_out']) || isset($_POST['email_opt_out'])){
+            if (isset($_POST['webtolead_email_opt_out']) || isset($_POST['email_opt_out'])) {
                     
-                if(isset ($lead->email1) && !empty($lead->email1)){
+                if (isset ($lead->email1) && !empty($lead->email1)) {
                     $sea = new SugarEmailAddress();
-                    $sea->AddUpdateEmailAddress($lead->email1,0,1);
+                    $sea->AddUpdateEmailAddress($lead->email1, 0, 1);
                 }   
-                if(isset ($lead->email2) && !empty($lead->email2)){
+                if (isset ($lead->email2) && !empty($lead->email2)) {
                     $sea = new SugarEmailAddress();
-                    $sea->AddUpdateEmailAddress($lead->email2,0,1);
+                    $sea->AddUpdateEmailAddress($lead->email2, 0, 1);
                     
                 }
             }              
-			if(isset($_POST['redirect_url']) && !empty($_POST['redirect_url'])){
+			if (isset($_POST['redirect_url']) && !empty($_POST['redirect_url'])) {
 			    // Get the redirect url, and make sure the query string is not too long
 		        $redirect_url = $_POST['redirect_url'];
 		        $query_string = '';
 				$first_char = '&';
-				if(strpos($redirect_url, '?') === FALSE){
+				if (strpos($redirect_url, '?') === FALSE) {
 					$first_char = '?';
 				}
 				$first_iteration = true;
 				$get_and_post = array_merge($_GET, $_POST);
-				foreach($get_and_post as $param => $value) {
+				foreach ($get_and_post as $param => $value) {
 
-					if($param == 'redirect_url' && $param == 'submit')
+					if ($param == 'redirect_url' && $param == 'submit')
 						continue;
 					
-					if($first_iteration){
+					if ($first_iteration) {
 						$first_iteration = false;
 						$query_string .= $first_char;
 					}
-					else{
+					else {
 						$query_string .= "&";
 					}
 					$query_string .= "{$param}=".urlencode($value);
 				}
-				if(empty($lead)) {
-					if($first_iteration){
+				if (empty($lead)) {
+					if ($first_iteration) {
 						$query_string .= $first_char;
 					}
-					else{
+					else {
 						$query_string .= "&";
 					}
 					$query_string .= "error=1";
@@ -215,25 +215,25 @@ 
 
 				// Check if the headers have been sent, or if the redirect url is greater than 2083 characters (IE max URL length)
 				//   and use a javascript form submission if that is the case.
-			    if(headers_sent() || strlen($redirect_url) > 2083){
-    				echo '<html ' . get_language_header() . '><head><title>SugarCRM</title></head><body>';
-    				echo '<form name="redirect" action="' .$_POST['redirect_url']. '" method="GET">';
+			    if (headers_sent() || strlen($redirect_url) > 2083) {
+    				echo '<html '.get_language_header().'><head><title>SugarCRM</title></head><body>';
+    				echo '<form name="redirect" action="'.$_POST['redirect_url'].'" method="GET">';
     
-    				foreach($_POST as $param => $value) {
-    					if($param != 'redirect_url' ||$param != 'submit') {
+    				foreach ($_POST as $param => $value) {
+    					if ($param != 'redirect_url' || $param != 'submit') {
     						echo '<input type="hidden" name="'.$param.'" value="'.$value.'">';
     					}
     				}
-    				if(empty($lead)) {
+    				if (empty($lead)) {
     					echo '<input type="hidden" name="error" value="1">';
     				}
     				echo '</form><script language="javascript" type="text/javascript">document.redirect.submit();</script>';
     				echo '</body></html>';
     			}
-				else{
+				else {
     				$header_URL = "Location: {$redirect_url}";
 
-					if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+					if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
 						$href = $matches[1];
 						SugarApplication::redirect($href);
 					}
@@ -244,29 +244,29 @@ 
     				die();
 			    }
 			}
-			else{
+			else {
 				echo $mod_strings['LBL_THANKS_FOR_SUBMITTING'];
 			}
 			sugar_cleanup();
 			// die to keep code from running into redirect case below
 			die();
 	    }
-	   else{
+	   else {
 	  	  echo $mod_strings['LBL_SERVER_IS_CURRENTLY_UNAVAILABLE'];
 	  }
 }
 
 if (!empty($_POST['redirect'])) {
-    if(headers_sent()){
-    	echo '<html ' . get_language_header() . '><head><title>SugarCRM</title></head><body>';
-    	echo '<form name="redirect" action="' .$_POST['redirect']. '" method="GET">';
+    if (headers_sent()) {
+    	echo '<html '.get_language_header().'><head><title>SugarCRM</title></head><body>';
+    	echo '<form name="redirect" action="'.$_POST['redirect'].'" method="GET">';
     	echo '</form><script language="javascript" type="text/javascript">document.redirect.submit();</script>';
     	echo '</body></html>';
     }
-    else{
+    else {
     	$header_URL = "Location: {$_POST['redirect']}";
 
-		if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+		if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
 			$href = $matches[1];
 			SugarApplication::redirect($href);
 		}

modules/Campaigns/WebToPersonCapture.php

Spacing +2 added lines, -2 removed lines

@@ -237,7 +237,7 @@ 
             } else {
                 $header_URL = "Location: {$redirect_url}";
 
-                if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+                if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
                     $href = $matches[1];
                     SugarApplication::redirect($href);
                 }
@@ -273,7 +273,7 @@ 
     } else {
         $header_URL = "Location: {$_POST['redirect']}";
 
-        if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+        if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
             $href = $matches[1];
             SugarApplication::redirect($href);
         }

modules/Campaigns/WizardCampaignSave.php

Spacing +5 added lines, -5 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.
@@ -42,7 +42,7 @@ 
 
 function getTemplateValidationMessages($templateId) {
     $msgs = array();
-    if(!$templateId) {
+    if (!$templateId) {
         $msgs[] = 'LBL_NO_SELECTED_TEMPLATE';
     }
     else {
@@ -64,7 +64,7 @@ 
 $campaignId = $db->quote($_POST['campaignId']);
 $marketingId = $db->quote($_POST['marketingId']);
 $func = isset($_REQUEST['func']) ? $_REQUEST['func'] : null;
-if($func == 'getTemplateValidation') {
+if ($func == 'getTemplateValidation') {
     if (!empty($_POST['templateId'])) {
         $templateId = $db->quote($_POST['templateId']);
     }
@@ -88,7 +88,7 @@ 
     if (!$marketingId) {
         if (!empty($_SESSION['campaignWizard'][$campaignId]['defaultSelectedMarketingId']) && $func != 'createEmailMarketing') {
             $marketingId = $_SESSION['campaignWizard'][$campaignId]['defaultSelectedMarketingId'];
-        } else if($func != 'createEmailMarketing') {
+        } else if ($func != 'createEmailMarketing') {
             $marketing = new EmailMarketing();
             $marketing->save();
             $marketingId = $marketing->id;
@@ -107,7 +107,7 @@ 
     if (!empty($_POST['templateId'])) {
         $marketing->template_id = $templateId;
     }
-    if($func != 'createEmailMarketing') {
+    if ($func != 'createEmailMarketing') {
         $marketing->save();
     }
 

modules/Campaigns/WebToLeadCreation.php

Spacing +48 added lines, -48 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.
@@ -51,28 +51,28 @@ 
 
 require_once('modules/Campaigns/utils.php');
 
-global $mod_strings, $app_list_strings, $app_strings, $current_user, $import_bean_map,$import_file_name, $theme;
+global $mod_strings, $app_list_strings, $app_strings, $current_user, $import_bean_map, $import_file_name, $theme;
 
-$xtpl=new XTemplate ('modules/Campaigns/WebToLeadCreation.html');
+$xtpl = new XTemplate('modules/Campaigns/WebToLeadCreation.html');
 $xtpl->assign("MOD", $mod_strings);
 $xtpl->assign("APP", $app_strings);
-if(isset($_REQUEST['module']))
+if (isset($_REQUEST['module']))
 {
     $xtpl->assign("MODULE", $_REQUEST['module']);
 }
-if(isset($_REQUEST['return_module']))
+if (isset($_REQUEST['return_module']))
 {
     $xtpl->assign("RETURN_MODULE", $_REQUEST['return_module']);
 }
-if(isset($_REQUEST['return_id']))
+if (isset($_REQUEST['return_id']))
 {
     $xtpl->assign("RETURN_ID", $_REQUEST['return_id']);
 }
-if(isset($_REQUEST['return_id']))
+if (isset($_REQUEST['return_id']))
 {
     $xtpl->assign("RETURN_ACTION", $_REQUEST['return_action']);
 }
-if(isset($_REQUEST['record']))
+if (isset($_REQUEST['record']))
 {
     $xtpl->assign("RECORD", $_REQUEST['record']);
 }
@@ -86,21 +86,21 @@ 
 
 $beanList = filterFieldsFromBeans($subclasses);
 
-$xtpl->assign("BEAN_LIST",json_encode($beanList));
+$xtpl->assign("BEAN_LIST", json_encode($beanList));
 
 $personTypeList = "<select id='personTypeSelect'>";
-if(count($beanList) > 0)
+if (count($beanList) > 0)
 {
-    $count=0;
-    foreach($beanList as $b)
+    $count = 0;
+    foreach ($beanList as $b)
     {
-        $personTypeList.="<option value='".$count."'>".$b->name."</option>";
+        $personTypeList .= "<option value='".$count."'>".$b->name."</option>";
         $count++;
     }
 }
 else
 {
-    $personTypeList.="<option value='noPerson'>No matching types</option>";
+    $personTypeList .= "<option value='noPerson'>No matching types</option>";
 }
 
 $site_url = $sugar_config['site_url'];
@@ -127,18 +127,18 @@ 
 		),
 );
 $encoded_users_popup_request_data = $json->encode($popup_request_data);
-$xtpl->assign('encoded_campaigns_popup_request_data' , $json->encode($popup_request_data));
+$xtpl->assign('encoded_campaigns_popup_request_data', $json->encode($popup_request_data));
 
 $field_defs_js = "var field_defs = {'Contacts':[";
 
-$xtpl->assign("WEB_POST_URL",$web_post_url);
+$xtpl->assign("WEB_POST_URL", $web_post_url);
 
 if (empty($focus->assigned_user_id) && empty($focus->id))  $focus->assigned_user_id = $current_user->id;
 if (empty($focus->assigned_name) && empty($focus->id))  $focus->assigned_user_name = $current_user->user_name;
 $xtpl->assign("ASSIGNED_USER_NAME", $focus->assigned_user_name);
-$xtpl->assign("ASSIGNED_USER_ID", $focus->assigned_user_id );
+$xtpl->assign("ASSIGNED_USER_ID", $focus->assigned_user_id);
 
-$xtpl->assign("REDIRECT_URL_DEFAULT",'http://');
+$xtpl->assign("REDIRECT_URL_DEFAULT", 'http://');
 
 $xtpl->parse("main");
 $xtpl->out("main");
@@ -148,71 +148,71 @@ 
 {
     global $app_strings;
     $formattedBeans = array();
-    foreach($beans as $b)
+    foreach ($beans as $b)
     {
         $formattedFields = array();
 //bug: 47574 - make sure, that webtolead_email1 field has same required attribute as email1 field
-        if(isset($b->field_defs['webtolead_email1']) && isset($b->field_defs['email1']) && isset($b->field_defs['email1']['required'])){
+        if (isset($b->field_defs['webtolead_email1']) && isset($b->field_defs['email1']) && isset($b->field_defs['email1']['required'])) {
             $b->field_defs['webtolead_email1']['required'] = $b->field_defs['email1']['required'];
         }
 
-        foreach($b->field_defs as $field_def)
+        foreach ($b->field_defs as $field_def)
         {
             $email_fields = false;
-            if($field_def['name']== 'email1' || $field_def['name']== 'email2')
+            if ($field_def['name'] == 'email1' || $field_def['name'] == 'email2')
             {
                 $email_fields = true;
             }
-            if($field_def['name']!= 'account_name'){
-                if( ( $field_def['type'] == 'relate' && empty($field_def['custom_type']) )
-                    || $field_def['type'] == 'assigned_user_name' || $field_def['type'] =='link' || $field_def['type'] =='function'
-                    || (isset($field_def['source'])  && $field_def['source']=='non-db' && !$email_fields) || $field_def['type'] == 'id')
+            if ($field_def['name'] != 'account_name') {
+                if (($field_def['type'] == 'relate' && empty($field_def['custom_type']))
+                    || $field_def['type'] == 'assigned_user_name' || $field_def['type'] == 'link' || $field_def['type'] == 'function'
+                    || (isset($field_def['source']) && $field_def['source'] == 'non-db' && !$email_fields) || $field_def['type'] == 'id')
                 {
                     continue;
                 }
             }
-            if($field_def['name']== 'deleted' || $field_def['name']=='converted' || $field_def['name']=='date_entered'
-                || $field_def['name']== 'date_modified' || $field_def['name']=='modified_user_id'
-                || $field_def['name']=='assigned_user_id' || $field_def['name']=='created_by'
-                || $field_def['name']=='team_id')
+            if ($field_def['name'] == 'deleted' || $field_def['name'] == 'converted' || $field_def['name'] == 'date_entered'
+                || $field_def['name'] == 'date_modified' || $field_def['name'] == 'modified_user_id'
+                || $field_def['name'] == 'assigned_user_id' || $field_def['name'] == 'created_by'
+                || $field_def['name'] == 'team_id')
             {
                 continue;
             }
 
             //If the field is hidden in the studio settings, then do not show
-            if(isset($field_def['studio']) && isset($field_def['studio']['editview']) && $field_def['studio']['editview']=== false )
+            if (isset($field_def['studio']) && isset($field_def['studio']['editview']) && $field_def['studio']['editview'] === false)
             {
                 continue;
             }
 
 
-            $field_def['vname'] = preg_replace('/:$/','',translate($field_def['vname'], $b->module_dir));
+            $field_def['vname'] = preg_replace('/:$/', '', translate($field_def['vname'], $b->module_dir));
 
             //$cols_name = "{'".$field_def['vname']."'}";
             $col_arr = array();
-            if((isset($field_def['required']) && $field_def['required'] != null && $field_def['required'] != 0)
-                || $field_def['name']=='last_name'
-            ){
-                $cols_name=$field_def['vname'].' '.$app_strings['LBL_REQUIRED_SYMBOL'];
-                $col_arr[0]=$cols_name;
-                $col_arr[1]=$field_def['name'];
-                $col_arr[2]=true;
+            if ((isset($field_def['required']) && $field_def['required'] != null && $field_def['required'] != 0)
+                || $field_def['name'] == 'last_name'
+            ) {
+                $cols_name = $field_def['vname'].' '.$app_strings['LBL_REQUIRED_SYMBOL'];
+                $col_arr[0] = $cols_name;
+                $col_arr[1] = $field_def['name'];
+                $col_arr[2] = true;
             }
-            else{
-                $cols_name=$field_def['vname'];
-                $col_arr[0]=$cols_name;
-                $col_arr[1]=$field_def['name'];
+            else {
+                $cols_name = $field_def['vname'];
+                $col_arr[0] = $cols_name;
+                $col_arr[1] = $field_def['name'];
             }
-            if (! in_array($cols_name, $formattedFields))
+            if (!in_array($cols_name, $formattedFields))
             {
-                array_push($formattedFields,$col_arr);
+                array_push($formattedFields, $col_arr);
             }
         }
 
         $holder = new stdClass();
         $holder->name = $b->object_name;
         $holder->fields = $formattedFields;
-        $holder->moduleKnownAs = translate($b->module_name,'LBL_MODULE_NAME');
+        $holder->moduleKnownAs = translate($b->module_name, 'LBL_MODULE_NAME');
         $holder->moduleDir = $b->module_dir;
         $holder->moduleName = $b->module_name;
         $formattedBeans[] = $holder;
@@ -229,10 +229,10 @@ 
 function getListOfExtendingClasses($superclass)
 {
     $subclasses = array();
-    foreach($GLOBALS['moduleList'] as $mod)
+    foreach ($GLOBALS['moduleList'] as $mod)
     {
         $item = BeanFactory::getBean($mod);
-        if($item && is_subclass_of($item,$superclass))
+        if ($item && is_subclass_of($item, $superclass))
             $subclasses[] = $item;
     }
     return $subclasses;

modules/Campaigns/WizardEmailSetupSave.php

Spacing +15 added lines, -15 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.
@@ -60,22 +60,22 @@ 
     //name is used as key in post, it is also used in creation of summary page for wizard,
     //so let's clean up the posting so we can reuse the save functionality for inbound emails and
     //from existing save.php's  
-    foreach($camp_steps as $step){
+    foreach ($camp_steps as $step) {
         clean_up_post($step);
     }
 /**************************** Save general Email Setup  *****************************/
 
 //we do not need to track location if location type is not set
-if(isset($_POST['tracking_entities_location_type'])) {
+if (isset($_POST['tracking_entities_location_type'])) {
     if ($_POST['tracking_entities_location_type'] != '2') {
         unset($_POST['tracking_entities_location']);
         unset($_POST['tracking_entities_location_type']);
     }
 }
 //if the check box is empty, then set it to 0
-if(!isset($_POST['mail_smtpauth_req'])) { $_POST['mail_smtpauth_req'] = 0; }
+if (!isset($_POST['mail_smtpauth_req'])) { $_POST['mail_smtpauth_req'] = 0; }
 //default ssl use to false
-if(!isset($_POST['mail_smtpssl'])) { $_POST['mail_smtpssl'] = 0; }
+if (!isset($_POST['mail_smtpssl'])) { $_POST['mail_smtpssl'] = 0; }
 //reuse existing saveconfig functionality
 $focus->saveConfig();
 
@@ -83,7 +83,7 @@ 
 
 /**************************** Add New Monitored Box  *****************************/
 //perform this if the option to create new mail box has been checked
-if(isset($_REQUEST['wiz_new_mbox']) && ($_REQUEST['wiz_new_mbox']=='1')){
+if (isset($_REQUEST['wiz_new_mbox']) && ($_REQUEST['wiz_new_mbox'] == '1')) {
     
    //Populate the Request variables that inboundemail expects
     $_REQUEST['mark_read'] = 1;
@@ -96,12 +96,12 @@ 
     require_once('modules/InboundEmail/Save.php');    
 
 }
-    if (!empty($_REQUEST['error'])){
+    if (!empty($_REQUEST['error'])) {
             //an error was found during inbound save.  This means the save was allowed but the inbound box had problems, return user to wizard
             //and display error message
             $header_URL = "Location: index.php?action=WizardEmailSetup&module=Campaigns&error=true";
 
-        if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+        if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
             $href = $matches[1];
             SugarApplication::redirect($href);
         }
@@ -109,11 +109,11 @@ 
             header($headers);
         }
 
-    }else{
+    } else {
         //set navigation details
         $header_URL = "Location: index.php?action=index&module=Campaigns";
 
-        if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+        if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
             $href = $matches[1];
             SugarApplication::redirect($href);
         }
@@ -129,18 +129,18 @@ 
  * which does not expect the prefix, and still use the generic create summary functionality in wizard, which
  * does expect the prefix.  
  */
-function clean_up_post($prefix){
+function clean_up_post($prefix) {
 
     foreach ($_REQUEST as $key => $val) {
-              if((strstr($key, $prefix )) && (strpos($key, $prefix )== 0)){
-              $newkey  =substr($key, strlen($prefix)) ;
+              if ((strstr($key, $prefix)) && (strpos($key, $prefix) == 0)) {
+              $newkey = substr($key, strlen($prefix));
               $_REQUEST[$newkey] = $val;
          }               
     }
 
     foreach ($_POST as $key => $val) {
-              if((strstr($key, $prefix )) && (strpos($key, $prefix )== 0)){
-              $newkey  =substr($key, strlen($prefix)) ;
+              if ((strstr($key, $prefix)) && (strpos($key, $prefix) == 0)) {
+              $newkey = substr($key, strlen($prefix));
               $_POST[$newkey] = $val;
               
          }               

modules/Campaigns/DotListWizardMenu.php

Spacing +3 added lines, -3 removed lines

@@ -23,14 +23,14 @@
 
     private function getWizardMenuItemHTML($i, $label, $link = false)
     {
-        if($i >= 4) {
+        if ($i >= 4) {
             parse_str($link, $args);
-            if(empty($args['marketing_id'])) {
+            if (empty($args['marketing_id'])) {
                 $link = false;
             }
         }
 
-        if($link != false) {
+        if ($link != false) {
             $html = '<li id="nav_step'.$i.'" class="nav-steps" data-nav-step="'.$i.'" data-nav-url="'.$link.'"><div>'.$label.'</div></li>';
         } else {
             $html = '<li id="nav_step'.$i.'" class="nav-steps" data-nav-step="'.$i.'"  data-nav-url=""><div>'.$label.'</div></li>';

modules/Campaigns/WizardMarketingSave.php

Spacing +40 added lines, -40 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.
@@ -53,18 +53,18 @@ 
 $master = 'save';
 if (isset($_REQUEST['wiz_home_next_step']) && !empty($_REQUEST['wiz_home_next_step'])) {
     
-    if($_REQUEST['wiz_home_next_step']==3){
+    if ($_REQUEST['wiz_home_next_step'] == 3) {
         //user has chosen to save and schedule this campaign for email
         $master = 'send';
-    }elseif($_REQUEST['wiz_home_next_step']==2){
+    }elseif ($_REQUEST['wiz_home_next_step'] == 2) {
         //user has chosen to save and send this campaign in test mode
         $master = 'test';
-    }else{
+    } else {
         //user has chosen to simply save
-        $master  = 'save';        
+        $master = 'save';        
     }
         
-}else{
+} else {
      //default to just saving and exiting wizard
      $master = 'save';   
 }
@@ -78,11 +78,11 @@ 
     $marketing->retrieve($_REQUEST['record']);
 }
 else {
-    if(!empty($_SESSION['campaignWizard'][$_REQUEST['campaign_id']]['defaultSelectedMarketingId'])) {
+    if (!empty($_SESSION['campaignWizard'][$_REQUEST['campaign_id']]['defaultSelectedMarketingId'])) {
         $marketing->retrieve($_SESSION['campaignWizard'][$_REQUEST['campaign_id']]['defaultSelectedMarketingId']);
     }
 }
-if(!$marketing->ACLAccess('Save')){
+if (!$marketing->ACLAccess('Save')) {
         ACLController::displayNoAccess(true);
         sugar_cleanup(true);
 }
@@ -95,40 +95,40 @@ 
 }
 
     foreach ($_REQUEST as $key => $val) {
-              if((strstr($key, $prefix )) && (strpos($key, $prefix )== 0)){
-              $newkey  =substr($key, strlen($prefix)) ;
+              if ((strstr($key, $prefix)) && (strpos($key, $prefix) == 0)) {
+              $newkey = substr($key, strlen($prefix));
               $_REQUEST[$newkey] = $val;
          }               
     }
 
     foreach ($_REQUEST as $key => $val) {
-              if((strstr($key, $prefix )) && (strpos($key, $prefix )== 0)){
-              $newkey  =substr($key, strlen($prefix)) ;
+              if ((strstr($key, $prefix)) && (strpos($key, $prefix) == 0)) {
+              $newkey = substr($key, strlen($prefix));
               $_REQUEST[$newkey] = $val;
          }               
     }
 
-if(!empty($_REQUEST['meridiem'])){
-    $_REQUEST['time_start'] = $timedate->merge_time_meridiem($_REQUEST['time_start'],$timedate->get_time_format(), $_REQUEST['meridiem']);
+if (!empty($_REQUEST['meridiem'])) {
+    $_REQUEST['time_start'] = $timedate->merge_time_meridiem($_REQUEST['time_start'], $timedate->get_time_format(), $_REQUEST['meridiem']);
 }
 
-if(empty($_REQUEST['time_start'])) {
-  $_REQUEST['date_start'] = $_REQUEST['date_start'] . ' 00:00';	
+if (empty($_REQUEST['time_start'])) {
+  $_REQUEST['date_start'] = $_REQUEST['date_start'].' 00:00';	
 } else {
-  $_REQUEST['date_start'] = $_REQUEST['date_start'] . ' ' . $_REQUEST['time_start'];
+  $_REQUEST['date_start'] = $_REQUEST['date_start'].' '.$_REQUEST['time_start'];
 }
 
-foreach($marketing->column_fields as $field)
+foreach ($marketing->column_fields as $field)
 {
     if ($field == 'all_prospect_lists') {
-        if(isset($_REQUEST[$field]) && $_REQUEST[$field]='on' )
+        if (isset($_REQUEST[$field]) && $_REQUEST[$field] = 'on')
         {
             $marketing->$field = 1;
         } else {
             $marketing->$field = 0;         
         }
-    }else {
-        if(isset($_REQUEST[$field]))
+    } else {
+        if (isset($_REQUEST[$field]))
         {
             $value = $_REQUEST[$field];
             $marketing->$field = trim($value);
@@ -136,9 +136,9 @@ 
     }
 }
 
-foreach($marketing->additional_column_fields as $field)
+foreach ($marketing->additional_column_fields as $field)
 {
-    if(isset($_REQUEST[$field]))
+    if (isset($_REQUEST[$field]))
     {
         $value = $_REQUEST[$field];
         $marketing->$field = $value;
@@ -150,8 +150,8 @@ 
 
 //add prospect lists to campaign.
 $marketing->load_relationship('prospectlists');
-$prospectlists=$marketing->prospectlists->get();
-if ($marketing->all_prospect_lists==1) {
+$prospectlists = $marketing->prospectlists->get();
+if ($marketing->all_prospect_lists == 1) {
     //remove all related prospect lists.
     if (!empty($prospectlists)) {
         $marketing->prospectlists->delete($marketing->id);
@@ -160,7 +160,7 @@ 
     if (isset($_REQUEST['message_for']) && is_array($_REQUEST['message_for'])) {
         foreach ($_REQUEST['message_for'] as $prospect_list_id) {
             
-            $key=array_search($prospect_list_id,$prospectlists);
+            $key = array_search($prospect_list_id, $prospectlists);
             if ($key === null or $key === false) {
                 $marketing->prospectlists->add($prospect_list_id);          
             } else {
@@ -169,7 +169,7 @@ 
         }
         if (count($prospectlists) != 0) {
             foreach ($prospectlists as $key=>$list_id) {
-                $marketing->prospectlists->delete($marketing->id,$list_id);             
+                $marketing->prospectlists->delete($marketing->id, $list_id);             
             }   
         }
     }
@@ -179,21 +179,21 @@ 
 $mass[] = $marketing->id; 
 //if sending an email was chosen, set all the needed variables for queuing campaign
 
-if($master !='save'){
-    $_REQUEST['mass']= $mass;
-    $_POST['mass']=$mass;
-    $_REQUEST['record'] =$marketing->campaign_id;
-    $_POST['record']=$marketing->campaign_id;
+if ($master != 'save') {
+    $_REQUEST['mass'] = $mass;
+    $_POST['mass'] = $mass;
+    $_REQUEST['record'] = $marketing->campaign_id;
+    $_POST['record'] = $marketing->campaign_id;
     $_REQUEST['mode'] = $master;
      $_POST['mode'] = $master; 
-     $_REQUEST['from_wiz']= 'true';
+     $_REQUEST['from_wiz'] = 'true';
     require_once('modules/Campaigns/QueueCampaign.php');
 }
 
-if(isset($_REQUEST['show_wizard_summary']) && $_REQUEST['show_wizard_summary']) {
+if (isset($_REQUEST['show_wizard_summary']) && $_REQUEST['show_wizard_summary']) {
 
-    if( (isset($_REQUEST['sendMarketingEmailTest']) && $_REQUEST['sendMarketingEmailTest']) ||
-        (isset($_REQUEST['sendMarketingEmailSchedule']) && $_REQUEST['sendMarketingEmailSchedule']) ) {
+    if ((isset($_REQUEST['sendMarketingEmailTest']) && $_REQUEST['sendMarketingEmailTest']) ||
+        (isset($_REQUEST['sendMarketingEmailSchedule']) && $_REQUEST['sendMarketingEmailSchedule'])) {
         // set correct post variables..
         $_POST['module'] = 'Campaigns';
         $_POST['action'] = 'QueueCampaign';
@@ -205,10 +205,10 @@ 
         $_POST['direct_step'] = '1';
         //$_POST['campaign_id'] = '';
         $_POST['wiz_mass'] = $marketing->id;
-        if(isset($_REQUEST['sendMarketingEmailTest']) && $_REQUEST['sendMarketingEmailTest']) {
+        if (isset($_REQUEST['sendMarketingEmailTest']) && $_REQUEST['sendMarketingEmailTest']) {
             $_POST['mode'] = 'test';
         }
-        else if(isset($_REQUEST['sendMarketingEmailSchedule']) && $_REQUEST['sendMarketingEmailSchedule']) {
+        else if (isset($_REQUEST['sendMarketingEmailSchedule']) && $_REQUEST['sendMarketingEmailSchedule']) {
             $_POST['mode'] = 'send';
             $_SESSION['msg'] = 'LBL_EMAILS_SCHEDULED';
         }
@@ -222,7 +222,7 @@ 
         exit;
     }
 
-    $header_URL = "Location: index.php?action=WizardMarketing&module=Campaigns&return_module=Campaigns&return_action=WizardHome&return_id=" . $marketing->campaign_id . "&campaign_id=" . $marketing->campaign_id . "&jump=3&marketing_id=" . $marketing->id;
+    $header_URL = "Location: index.php?action=WizardMarketing&module=Campaigns&return_module=Campaigns&return_action=WizardHome&return_id=".$marketing->campaign_id."&campaign_id=".$marketing->campaign_id."&jump=3&marketing_id=".$marketing->id;
 }
 else {
     $header_URL = "Location: index.php?action=WizardHome&module=Campaigns&record=".$marketing->campaign_id;
@@ -230,7 +230,7 @@ 
 
 $GLOBALS['log']->debug("about to post header URL of: $header_URL");
 
-if(preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
+if (preg_match('/\s*Location:\s*(.*)$/', $header_URL, $matches)) {
     $href = $matches[1];
     SugarApplication::redirect($href);
 }