【问题标题】:prepared mysql statements and arrays准备好的mysql语句和数组
【发布时间】: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


【解决方案1】:

在第二行代码全局$db,$tags;正在覆盖if($this->has_lead_type_selected($person['ID'],$tags)),这导致全局$tags 覆盖代码下部的$tags。所以全局变量是空的,因为它是在实际的 $tags 变量被赋予函数之前。

【讨论】:

  • 感谢您修复我的评论,我以后如何回答我自己的问题?
  • 嘿 Cody,您可以像在任何其他问题上一样简单地输入答案字段。
猜你喜欢
  • 1970-01-01
  • 2011-09-16
  • 1970-01-01
  • 2013-09-24
  • 2012-12-18
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多