Skip to content

Commit cd1b53f

Browse files
nathanielcabanbanalexanderabenoja
authored andcommitted
Bug Fix #8701: Campaign in webform blank value issue
1 parent dde6f1d commit cd1b53f

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

goAreacodes/goAddAreacode.php

+3-3
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161
if ($goapiaccess > 0 && $userlevel > 7) {
6262

6363
// check if already existing in whole system
64-
$astDB->where( "campaign_id", $campaign_id );
64+
/* (nat) $astDB->where( "campaign_id", $campaign_id );
6565
$astDB->where( "areacode", $areacode );
6666
$astDB->getOne( "vicidial_campaign_cid_areacodes", "campaign_id, areacode" );
6767
@@ -70,7 +70,7 @@
7070
$apiresults = array(
7171
"result" => $err_msg
7272
);
73-
} else {
73+
} else { */
7474

7575
$data = array(
7676
'campaign_id' => $campaign_id,
@@ -92,7 +92,7 @@
9292
"result" => $astDB->getLastError()
9393
);
9494
}
95-
}
95+
// (nat) }
9696
}
9797
}
9898
return $apiresults;

goCampaigns/goEditCampaign.php

+5-3
Original file line numberDiff line numberDiff line change
@@ -310,7 +310,7 @@
310310
'dial_method' => (gettype($dial_method) != NULL) ? $dial_method : $resultGet['dial_method'],
311311
'auto_dial_level' => $autoDialLevel,
312312
'dial_prefix' => $dialprefix,
313-
'web_form_address' => (!empty($webform)) ? $webform : $resultGet['web_form_address'],
313+
'web_form_address' => (!empty($webform) OR empty($webform)) ? $webform : $resultGet['web_form_address'],
314314
'campaign_script' => $campaign_script,
315315
'campaign_cid' => (!empty($campaign_cid)) ? $campaign_cid : $resultGet['campaign_cid'],
316316
'campaign_vdad_exten' => (!empty($campaign_vdad_exten)) ? $campaign_vdad_exten : $resultGet['campaign_vdad_exten'],
@@ -330,8 +330,10 @@
330330
'xferconf_b_number' => (!empty($xferconf_b_number)) ? $xferconf_b_number : $resultGet['xferconf_b_number'],
331331
//'three_way_dial_prefix' => (!empty($three_way_dial_prefix)) ? $three_way_dial_prefix : $resultGet['three_way_dial_prefix'],
332332
'three_way_dial_prefix' => $three_way_dial_prefix,
333-
'closer_campaigns' => (!empty($closer_campaigns)) ? $closer_campaigns : $resultGet['closer_campaigns'],
334-
'xfer_groups' => (!empty($xfer_groups)) ? $xfer_groups : $resultGet['xfer_groups'],
333+
//nat: made changes to the line of code below
334+
'closer_campaigns' => (!empty($closer_campaigns) OR empty($closer_campaigns)) ? $closer_campaigns : $resultGet['closer_campaigns'],
335+
//nat: made changes to the line of code below
336+
'xfer_groups' => (!empty($xfer_groups) OR empty($xfer_groups)) ? $xfer_groups : $resultGet['xfer_groups'],
335337
'survey_first_audio_file' => (!empty($survey_first_audio_file)) ? $survey_first_audio_file : $resultGet['survey_first_audio_file'],
336338
'survey_method' => (!empty($survey_method)) ? $survey_method : $resultGet['survey_method'],
337339
'survey_menu_id' => (!empty($survey_menu_id)) ? $survey_menu_id : $resultGet['survey_menu_id'],

0 commit comments

Comments
 (0)