From 773698b7e1125acc45797187e40b9e344df7c6a8 Mon Sep 17 00:00:00 2001
From: SinergiaCRM <desarrollo@sinergiacrm.org>
Date: Mon, 25 Nov 2024 12:54:17 +0000
Subject: [PATCH] Fix #10586 - Unable to send marketing emails from the wizard
 for an inactive campaign

---
 modules/Campaigns/WizardMarketing.html    | 24 +++++++++++++----------
 modules/Campaigns/WizardMarketing.php     |  1 +
 modules/Campaigns/language/en_us.lang.php |  1 +
 3 files changed, 16 insertions(+), 10 deletions(-)

diff --git a/modules/Campaigns/WizardMarketing.html b/modules/Campaigns/WizardMarketing.html
index a9bc19ef522..c37b6fb3800 100755
--- a/modules/Campaigns/WizardMarketing.html
+++ b/modules/Campaigns/WizardMarketing.html
@@ -1349,18 +1349,22 @@ <h4 class="header-4" style="border-bottom: solid 1px #eee;padding-bottom: 8px">{
 		</div>
 		<div class="template-panel-container panel-toolbar ">
 			<div class="template-container-full">
-				<div class="button-group">
-					<input type="hidden" name="wiz_home_next_step" value="1" />
+				{if $CAMPAIGN_STATUS != 'Inactive'}	
+					<div class="button-group">
+						<input type="hidden" name="wiz_home_next_step" value="1" />
 
-					<a class="btn {if $PL_DISABLED}btn-default{else}btn-primary{/if} btn-sm" style="border-radius: 0px;" {if $PL_DISABLED} {else}href="javascript:;" onclick="onScheduleClick(this, '{$CAMPAIGN_ID}', $('#marketing_select').val());"{/if}>
-						<span class="suitepicon suitepicon-action-schedule"></span> {$MOD.LBL_SEND_EMAIL}
-					</a>
+						<a class="btn {if $PL_DISABLED}btn-default{else}btn-primary{/if} btn-sm" style="border-radius: 0px;" {if $PL_DISABLED} {else}href="javascript:;" onclick="onScheduleClick(this, '{$CAMPAIGN_ID}', $('#marketing_select').val());"{/if}>
+							<span class="suitepicon suitepicon-action-schedule"></span> {$MOD.LBL_SEND_EMAIL}
+						</a>
 
-					<a class="btn {if $PL_DISABLED_TEST}btn-default{else}btn-primary{/if} btn-sm" style="border-radius: 0px;" {if $PL_DISABLED_TEST} {else}href="javascript:;"  onclick="onSendAsTestClick(this, '{$CAMPAIGN_ID}', $('#marketing_select').val());"{/if}>
-					<span class="suitepicon suitepicon-module-emails"></span> {$MOD.LBL_SEND_AS_TEST}
-					</a>
-				</div>
-				<div class="button-group-separator"></div>
+						<a class="btn {if $PL_DISABLED_TEST}btn-default{else}btn-primary{/if} btn-sm" style="border-radius: 0px;" {if $PL_DISABLED_TEST} {else}href="javascript:;"  onclick="onSendAsTestClick(this, '{$CAMPAIGN_ID}', $('#marketing_select').val());"{/if}>
+						<span class="suitepicon suitepicon-module-emails"></span> {$MOD.LBL_SEND_AS_TEST}
+						</a>
+					</div>
+					<div class="button-group-separator"></div>
+				{else}
+					<div  class="button-group"> {$MOD.LBL_CAMPAIGN_INACTIVE_WIZARD} </div>
+				{/if}					
 				<div class="button-group">
 					<a class="btn btn-primary btn-sm" href="index.php?module=Campaigns&action=DetailView&record={$CAMPAIGN_ID}" style="border-radius: 0px;">{$MOD.LBL_TODETAIL_BUTTON_TITLE}</a>
 				</div>
diff --git a/modules/Campaigns/WizardMarketing.php b/modules/Campaigns/WizardMarketing.php
index 72f5c34cfdb..1f89999cf1f 100755
--- a/modules/Campaigns/WizardMarketing.php
+++ b/modules/Campaigns/WizardMarketing.php
@@ -93,6 +93,7 @@
 }
 // handle Create $module then Cancel
 $ss->assign('CAMPAIGN_ID', $campaign_focus->id);
+$ss->assign('CAMPAIGN_STATUS', $campaign_focus->status);
 
 $seps = get_number_separators();
 $ss->assign("NUM_GRP_SEP", $seps[0]);
diff --git a/modules/Campaigns/language/en_us.lang.php b/modules/Campaigns/language/en_us.lang.php
index 134b6888b38..7466c146c13 100755
--- a/modules/Campaigns/language/en_us.lang.php
+++ b/modules/Campaigns/language/en_us.lang.php
@@ -139,6 +139,7 @@
     //error messages.
     'ERR_SENDING_NOW' => 'Messages are being delivered , please try this later.',
 
+    'LBL_CAMPAIGN_INACTIVE_WIZARD' => 'Sending buttons will not show on inactive campaigns.',
     'LBL_TRACK_ROI_BUTTON_LABEL' => 'View ROI',
     'LBL_TRACK_DELETE_BUTTON_TITLE' => 'Delete Test Entries',
     'LBL_TRACK_DELETE_BUTTON_LABEL' => 'Delete Test Entries',