2016-05-22 4 views
0

Я пытаюсь использовать флажки в форме html и отправлять их в базу данных sql по почте. но я всегда получаю 0 независимо от того, установлен ли флажок или нет.input checkbox всегда возвращает 0 в POST

это HTML форма:

<form action = "MediaList2.php" method = "POST"> 
     <li><input type="checkbox" name="gradea" value="1"> CPC for Grade A websites<br><input type='hidden' value='0' name='gradea'></li> 
     <li><input type="checkbox" name="gradeb" value="1"> CPC for Grade B websites<br><input type='hidden' value='0' name='gradeb'></li> 
     <li><input type="checkbox" name="gradec" value="1"> CPC for Grade C websites<br><input type='hidden' value='0' name='gradec'></li> 
     <li><input type="text" name="clicksno" placeholder="if you want a specific number for your clicks please mention it here ">Number of Clicks<br></li> 
     <li> Duration<input type="text" name="start_date" class="form-control" id="start_date" placeholder="yyyy-mm-dd" /> <span> TO </span> <input type="text" name="end_date" class="form-control" id="end_date" placeholder="yyyy-mm-dd" /> 
</li> 
     <li> Budget <input type ="text" name ="budget" required></li> 

      <button type= "submit">Submit</button> 
      </form> 

это мой PHP код:

try{ 
    // receiving the post params 
    $gradea= $_POST['gradea']; 
    $gradeb= $_POST['gradeb']; 
    $gradec= $_POST['gradec']; 
    $clicksno= $_POST['clicksno']; 
    $start_date = $_POST['start_date']; 
    $end_date = $_POST['end_date']; 
    $budget = $_POST['budget']; 
    if($budget > $_SESSION['totalcampaignbudget']) 
    { 
       echo "Your CPC budget should not be bigger than your total budget!"; 
       exit(0); 
    } 
    $user = $db->storeinCPC($gradea , $gradeb, $gradec, $clicksno, $start_date, $end_date, $budget, $campaignname, $email); 

и это моя storinCPC функция в PHP:

public function storeinCPC($gradea , $gradeb, $gradec, $clicksno, $start_date, $end_date, $budget, $campaignname, $email) 
    {                   // Create connection 
$conn = new mysqli($servername, $username, $password, $dbname); 
                         // Check connection 
                         if ($conn->connect_error) 
                         { 
                         die("Connection failed: " . $conn->connect_error); 
                         }                    
    $q1 = "SELECT * from CPC where tocampaign = '$campaignname' AND toperson = '$email' " ; 
    $res1 = $conn -> query($q1); 
    $count = $res1 ->num_rows; 
    if($count != null) 
    { 
     $query = "UPDATE CPC set clicksno ='$clicksno' , GradeA = '$gradea', GradeB='$gradeb', GradeC = '$gradec',budget='$budget', tocampaign='$campaignname', startdate='$start_date', enddate='$end_date', toperson='$email', updated_at = NOW() WHERE toperson = '$email' AND tocampaign = '$campaignname'"; 
    } 
    else if($count ==0) 
    { 
    $query = "INSERT INTO CPC (clicksno , GradeA, GradeB, GradeC,budget, tocampaign, startdate, enddate, toperson, created_at, updated_at) VALUES ('$clicksno', '$gradea', '$gradeb' , '$gradec', '$budget', '$campaignname', '$start_date', '$end_date', '$email', NOW(), NOW())"; 
    } 
    $result = $conn->query($query); 
    } 
+0

Ваш код открыт для SQL-инъекции. Вы действительно должны это исправить. Используйте подготовленные операторы и связанные параметры для всех пользовательских данных. – Mike

+0

Почему у вас есть ? –

ответ

2

Глядя на сингле эти строки ввода дают четкое представление о вашей основной проблеме:

<li><input type="checkbox" name="gradea" value="1"> CPC for Grade A websites<br><input type='hidden' value='0' name='gradea'></li> 

Вы заметили, что вы создали поле ввода флажок с именем gradea, и сразу после того, как создан скрытый поле ввода одного и того же имени с value='0' который перезаписывает первый?

+0

Я добавил скрытое значение, чтобы флажок отправил значение, даже если он не установлен! как я могу это сделать, если этот путь неправильный? – Neshat

+0

Что-то по строкам: '$ gradea = isset ($ _ POST ['gradea']);' сделает '$ gradea' логическим. Или, если вы хотите 1 или 0, '$ gradea = isset ($ _ POST ['gradea'])? 1: 0; '. – Mike

+1

Вы должны поместить скрытый ввод перед этим флажком. Всякий раз, когда флажок установлен, это значение переопределяет значение скрытого ввода при его отправке, иначе вместо этого будет показано скрытое значение. –

Смежные вопросы