2017-10-04 158 views
0

我使用了一个while循环来显示Dropdown yes和no,它显示每个页面的用户权限,所以页面名称和是或否没有为每个页面下拉。PHP Posting last dynamic表格

表单显示正确,并为每个用户显示正确的数据库值作为默认值。但是,如果我将页面权限之一从“是”更改为“否”或仅更新数据库中的最后一个下拉列表,而其他页面权限则更改。任何想法我在这里失踪?

<?php 

$sql = "SELECT DISTINCT username FROM users ORDER BY username"; 
$resultusers = $connect->query($sql); 
?> 
<?php 
require ("cw/connect.php"); 


?> 
<?php 
if(isset($_POST['selectbutton'])) 
{ 
    $username = $_POST['selectuser']; 

$query = "SELECT 
users.username, users.first, users.last, users.id, permissions.PermID, permissions.PermUserID, permissions.PermPagesID, permissions.view, pages.PagesName, pages.PagesLink, pages.PagesID 
FROM users INNER JOIN permissions ON users.id = permissions.PermUserID INNER JOIN pages ON permissions.PermPagesID = pages.PagesID WHERE username = ?"; 

$stmt = mysqli_prepare($connect, $query); 

if($stmt){ 
    //Put whats to be binded from the statement so in this case we want data for this username selected 
    mysqli_stmt_bind_param($stmt,"s",$username); 

    // here add all the varibles to be pulled from database 
    mysqli_stmt_bind_result($stmt, $username, $first, $last, $id, $PermID, $PermUserID, $PermPagesID, $view, $PagesName, $PagesLink, $PagesID); 


    mysqli_stmt_execute($stmt); 

    mysqli_stmt_fetch($stmt); 




?> 


<b>First:</b><?php echo $first; ?><br> 
<b>Last:</b><?php echo $last; ?><br> 
<b>User:</b><?php echo $username; ?><br> 
<hr> 


<form action="user-permissions.php" method="post"> 
<?php 
while ($stmt->fetch()) { 

echo $PagesName; 
echo "<br>"; 
echo "<select name=\"permissionSelect\">"; 

?> 

<option value="<?php echo $view; ?>"><?php echo $view; ?></option> 
<option value="No">No</option> 
<option value="Yes">Yes</option> 

</select> 
<br><br> 
<?php 
echo "<input type=\"hidden\" name=\"PermPagesID\" value=\"".$PermPagesID."\">"; 
echo "<input type=\"hidden\" name=\"PermUserID\" value=\"".$PermUserID."\">"; 
     } 
     ?> 
<?php 
echo "<input type=\"submit\" name=\"updatep\">"; 
echo "<form>"; 


    } 

} 
?> 


<?php 
//UPDATE PERSON 
require ("cw/connect.php"); 

if(mysqli_connect_error()){ 
    echo mysqli_connect_error(); 
    exit(); 

} 
if(isset($_POST['updatep'])) 
{ 
    $view = mysqli_real_escape_string($connect,$_POST['permissionSelect']); 
    $PermPagesID = mysqli_real_escape_string($connect,$_POST['PermPagesID']); 
    $PermUserID = mysqli_real_escape_string($connect,$_POST['PermUserID']); 


$query = "UPDATE permissions SET view = ? WHERE PermPagesID = $PermPagesID AND PermUserID = $PermUserID"; 

$stmt = mysqli_prepare($connect, $query); 

if($stmt){ 
    mysqli_stmt_bind_param($stmt, "s", $view); 

    mysqli_stmt_execute($stmt); 

    mysqli_stmt_fetch($stmt); 
    echo "Updated to: ".$username; 


    }else{ 

    echo "object not created"; 

} 
} 

?> 
+1

您可以将输入的名称更改为数组,例如'name = \“PermUserID [] \”',然后循环并更新 – Hackerman

+0

正如Hackerman所说,输入中不能有重复名称,否则会被最后一个覆盖。所以你把[]添加到名称的末尾,将其转换为一个值数组,然后将其循环。 – LordNeo

回答

0

原因是因为您不能在表单元素中使用多次相同的名称。否则,只会使用最后一个。

您将需要为它们提供所有不同的名称或模仿如下的数组语法。例如,而不是:

echo '<select name="permissionSelect">'; 

用途:

echo '<select name="permissionSelect[]">'; 

然后在你的PHP脚本,你可以通过它们在foreach循环引用这些:

foreach($_POST['permissionSelect'] as $permissionSelect){ 
    //handle it here 
} 

您需要对循环内的其他名称属性也这样做。

最后,您会注意到我从代码中删除了反斜杠。这是因为PHP和HTML都可以使用单引号或双引号。尽管PHP在两者之间存在一些差异,但在大多数情况下,您只需交换它们即可。所以,如果PHP使用单引号,那么对于HTML属性使用双引号,反之亦然。

+0

也许你应该在循环之前检查数组是否为空 – Akintunde007

+0

@Akintunde是的,但是这可能应该在代码的早期完成,以检查其他必填字段是否为空。还有很多其他的改进,例如不使用'mysqli_real_escape_string'等。 – kojow7