【发布时间】:2017-09-25 19:01:22
【问题描述】:
今天早些时候解决了这个问题。在第二行代码global $db,$tags;正在覆盖if($this->has_lead_type_selected($person['ID'],$tags)),这导致全局 $tags 在代码的下部覆盖 $tags。所以全局变量是空的,因为它是在实际的 $tags 变量被赋予函数之前。
当 HTML 表单提交到我们的 REST API 时,它会发送名为“lead_type”的数据,这些数据只是用于识别正在发送的潜在客户的标签。
用户从标签云中选择这些标签。如果表单使用这些标签之一 (lead_type) 提交给 API,并且我们的任何用户配置文件都匹配这些标签(他们在标签云中选择)。用户会收到一条短信通知他们。
所有内容都发布到数据库表中,API 工作,但每个用户仍然会收到短信,即使他们没有匹配的标签。如果我注释掉该行(我将在下面显示其余代码),则会向所有人发送一条短信。如果我不加注释,则不会向任何人发送 SMS。
if($this->has_lead_type_selected($person['ID'],$tags))
这是代码的流程。
}
private function has_lead_type_selected($user_id,$tags){
global $db,$tags;
$lead_types = explode(',',$tags);
$user_lead_types = $db
->where('user_id',$user_id)
->where('lead_type_id', $lead_types, 'IN')
->get('user_lead_types');
return sizeof($user_lead_types) > 0;
}
//Get lead types from API post and create $tags
$lead_types = $this->request['leadData']['lead_types'];
$strTags = array();
if(!empty($lead_types))
$strTags = explode(',',$lead_types);
$tags = '';
$lead_types_objects = $db->where('lead_type', $strTags,'IN')->get('lead_types');
foreach($lead_types_objects as $l)
{
if($tags=='')
$tags = $l['id'];
else
$tags.=',' .$l['id'];
}
如果表单标签与用户的云标签匹配,我们会向他们发送短信。
global $sid,$token;
$client = new Twilio\Rest\Client($sid, $token);
$content_data = [
"leadname" => $posted_name,
"leadzipcode" => $posted_zipcode,
"leadphone" => $posted_phone,
"leademail" => $posted_email,
"leadtags" => $lead_types
];
//Replace Content
foreach($content_data as $index => $value){
$lead_sms_template = str_replace("|".$index."|", $value, $lead_sms_template);
}
// Step 5: Loop over all our friends. $number is a phone number above, and
// $name is the name next to it
foreach ($people as $person) {
try{
//commented temporarily -- uncommented below to try to solve issue of texting everyone still
if($this->has_lead_type_selected($person['ID'],$tags))
{
$number = $person['phone_no'];
$name = $person['first_name']. ' '. $person['last_name'];
感谢您的帮助。
【问题讨论】:
-
has_lead_type_selected 应该检查用户是否选择了一个或所有选择的标签?
-
今天早些时候解决了这个问题。在代码的第 2 行 global $db,$tags;覆盖 if($this->has_lead_type_selected($person['ID'],$tags)) 导致全局 $tags 覆盖代码下部的 $tags。所以全局变量是空的,因为它是在实际的 $tags 变量被赋予函数之前。
标签: php arrays mysqli prepared-statement