question on php

jjjj

Tracing status from the database, how can I implement radio botton in the table under two different columns named active and inactive.

nnichols

You need to give more detail of what you are trying to do for anyone to be able to help you.

morphy_6deex

mmm


while($row = mysql_fetch_array($rs))
{
    $aSel = array("","");
    $aSel[$row["selection"]] = "checked";
    //selection field can be 1 or 0
    echo "<form>\n";
    echo "<input type=\"hidden\" value=\"".$row["id"]."\" name=\"id\">\n";
    echo "<input type=\"radio\" name=\"optSel\" value=\"0\" ".$aSel[0].">\n";
    echo "<input type=\"radio\" name=\"optSel\" value=\"1\" ".$aSel[1].">\n";
    echo "<input type=\"submit\">\n";
    echo "</form>\n";
}

is it what u need?

jjjj

Let me try to make my question clear.

I want to display a list of employer info as a table on the web page, and the data traced from MySQL db.

In the db, each employer has a attribute called status(0 or 1). I try to display the status on the web page table using two columns named <th>active and inactive. And in each row of the table, I want to use radio botton to represent active(radio botton has been selected) and inactive(radio botton not selected).

Here are my code for doing that, but it doesn't work for the radio botton.

if($query_result)//ok query
{

  echo "<table border='1' >";
  echo "<tr><th>User Detail</th>
        	<th>empID</th>
    	<th>empf_name</th>
    	<th>empl_name</th>
    	<th>emp_passwd</th>
        	<th>emp_email</th>
        	<th>emp_rank</th>
    	<th>emp_department</th>
        	<th>creatDate</th>
    	<th>active</th>
        	<th>inactive</th></tr>";
  while
  (
        $myrow  = mysql_fetch_array($query_result))

  {
    $number_rows++;
$empID = $myrow["empID"];
    $empf_name=$myrow["empf_name"];
    $empl_name=$myrow["empl_name"];
    $emp_passwd=$myrow["emp_passwd"];
    $emp_email =$myrow["emp_email"];
    $emp_rank=$myrow["emp_rank"];
    $emp_department=$myrow["emp_department"];
    $creatDate =$myrow["creatDate"];
    $status=$myrow["status"];
   //echo $query;
  echo "<tr><td><a href=\"employerdetail.php\">User Details</a></td>
             <td>$empID</td>
	 <td>$empf_name</td>
	 <td>$empl_name</td>
	 <td>$emp_passwd</td>
	 <td>$emp_email</td>
	 <td>$emp_rank</td>
	 <td>$emp_department</td>
	 <td>$creatDate</td>";
//<!-----Active & Inactive------>

if($status==1)
{
	echo "<td><input type='radio' name=status checked></td>
             <td><input type='radio' name=status> </td>";
}
else
{
	echo "<td><input type='radio' name=status></td>
	 <td><input type='radio' name=status checked></td>";
}	

    echo "</tr>";
    # make sure that we recieved some data from our query 
    $num_of_rows = mysql_num_rows ($query_result);       

   }//end while

echo "</table>";

print "<font color=red>There are $number_rows registered HR manager(s) listed above.</font>";
}//end if

can you give me any suggestion on this code?
thank you,

nnichols

The reason your code is not doing what you want it to is simple -

you have named all the radio buttons as being part of the same group, and as only one member of a group can be selected, only the last one has a selected value.

Change it so that you append the "empID" to the name of the radio button group. Then it should work fine.

Hope this helps 😉

P.S. you seem to be working out the number of rows twice with your counter - $somevar++ and also with mysql_num_rows(). There is no need to do it twice.

jjjj

my radio botton works fine now! thank you

new question is coming up:

I want to change the status of some employers by just selecting the radio botton. After clicking the update botton in this wed page, the employers' status have been changed.

the question is:
how can I change the status value(0 or 1) in the db by selecting the radio botton in my web page?

nnichols

One way to tackle this is to use an <IFRAME> and lots of little forms.

For each row you print in the table put a <FORM> around it and include the empID in a hidden form field. Set the forms action to the name of your little update script and set it's target to your hidden <IFRAME>. Then on your radio buttons add an onChange event like this -

onChange=this.form.submit()

That way when you change the radio button that is selected it will submit the form and run your update script.

You could also do this by including the update script in the top of your main script and just get the little forms to submit to the current page - $PHP_SELF, making sure you run the update before you pull your data back out of the db.

If you want more help, be sure to post your existing code and put it in [ PHP][/code].

Hope this helps 😉

jjjj

I have tried to write my code by following your suggestion, but it does work. After I change the active button and click the Update button on the web page, it can't find next page I was expecting.

I know there are some errors in my code, but I really don't know
what it supposts to be. Can you wake up me by some suggestions?

I don't know this is correct or not <form action="'.$PHP_SELF.'" method="post">
how can use the variable to check the Update condition in the update script. I just put like
if($active)
{
$query="update...
}
else if($inactive)
{
$query="update...
}

Here is the exactly code for the main page
$dbName ="";
$dbPass ="";
$dbSite="emp_profile";

$conn = @mysql_connect("localhost",$dbName,$dbPass)
or die("Could not connect to localhost");

mysql_select_db($dbSite, $conn)
or die("Could not select database");

include ("updateActive111.php");

print "<FORM ACTION=\"'. $PHP_SELF .'\" METHOD=\"POST\"><input type='submit' name='submit' value ='Update'></FORM>";

//print $query;
/**
Make connection to database
Select database to build into and from
/
$query = "select from emp_profile";

$query_result = mysql_query ($query);
$number_rows = 0;

if($query_result)//ok query
{

  echo "<table border='1' >";
  echo "<tr><th>User Detail</th>
        	<th>empID</th>
    	<th>empf_name</th>
    	<th>empl_name</th>
    	<th>emp_passwd</th>
        	<th>emp_email</th>
        	<th>emp_rank</th>
    	<th>emp_department</th>
        	<th>creatDate</th>
    	<th>active</th>
        	<th>inactive</th></tr>";

while
(
$myrow = mysql_fetch_array($query_result))

/************************************/

Here is the code of the update script

$dbName ="";
$dbPass ="";
$dbSite="emp_profile";

$conn = @mysql_connect("localhost",$dbName,$dbPass)
or die("Could not connect to localhost");

mysql_select_db($dbSite, $conn)
or die("Could not select database");

//print $query;
/**
Make connection to database
Select database to build into and from
*/

if($active)
{

    $query ="UPDATE emp_profile SET empID='$empID',"
  . " empf_name='$empf_name', empl_name ='$empl_name',"
  . " emp_passwd='$emp_passwd',emp_email='$emp_email',"
  . " emp_department='$emp_department', creatDate='$creatDate',"     
  . " status='1'"
  . " WHERE empID = '$empID' AND emp_passwd ='$emp_passwd'";

}
else if($inactive)
{

    $query ="UPDATE emp_profile SET empID='$empID',"
  . " empf_name='$empf_name', empl_name ='$empl_name',"
  . " emp_passwd='$emp_passwd',emp_email='$emp_email',"
  . " emp_department='$emp_department', creatDate='$creatDate',"     
  . " status='0'"
  . " WHERE empID = '$empID' AND emp_passwd

='$emp_passwd'";
}

nnichols

I haven't run this code, 'cause I've got no db to run it against, but it should be ok. Maybe some simple syntax errors.

<?PHP

$dbName =""; 
$dbPass =""; 
$dbSite="emp_profile"; 

$conn = @mysql_connect("localhost",$dbName,$dbPass) 
or die("Could not connect to localhost"); 

mysql_select_db($dbSite, $conn) 
or die("Could not select database"); 

include ("updateActive111.php"); 

//print $query;
/**
* Make connection to database
* Select database to build into and from
*/
$query = "select * from emp_profile";

$query_result = mysql_query ($query);

# get number of rows returned by our query 
$num_of_rows = mysql_num_rows ($query_result); 

if($query_result)//ok query
{

echo "<table border='1' >";
echo "<tr><th>User Detail</th>
<th>empID</th>
<th>empf_name</th>
<th>empl_name</th>
<th>emp_passwd</th>
<th>emp_email</th>
<th>emp_rank</th>
<th>emp_department</th>
<th>creatDate</th>
<th>active</th>
<th>inactive</th></tr>";

while ($myrow = mysql_fetch_array($query_result)) {

$empID = $myrow["empID"]; 
$empf_name=$myrow["empf_name"]; 
$empl_name=$myrow["empl_name"]; 
$emp_passwd=$myrow["emp_passwd"]; 
$emp_email =$myrow["emp_email"]; 
$emp_rank=$myrow["emp_rank"]; 
$emp_department=$myrow["emp_department"]; 
$creatDate =$myrow["creatDate"]; 
$status=$myrow["status"]; 

//echo $query; 
echo "<form method=POST action=\"$PHP_SELF\">";
// add hidden form field for empID - so we know which emp to UPDATE
echo "<input type=\"hidden\" name=\"update_emp_ID\" value=\"$empID\">

echo "<tr><td><a href=\"employerdetail.php\">User Details</a></td> 
<td>$empID</td> 
<td>$empf_name</td> 
<td>$empl_name</td> 
<td>$emp_passwd</td> 
<td>$emp_email</td> 
<td>$emp_rank</td> 
<td>$emp_department</td> 
<td>$creatDate</td>"; 
//<!-----Active & Inactive------> 
$onclick = "onClick=\"this.form.submit();\"";
if($status==1) 
{ 
echo "<td><input type=\"radio\" name=\"status\" value=\"1\" checked></td> 
<td><input type=\"radio\" name=\"status\" value=\"0\" $onclick></td>"; 
} 
else 
{ 
echo "<td><input type=\"radio\" name=\"status\" value=\"1\" $onclick></td> 
<td><input type=\"radio\" name=\"status\" value=\"0\" checked></td>"; 
} 

echo "</tr>";

echo "</form>";

}//end while 
echo "</table>"; 


print "<font color=red>There are $num_of_rows registered HR manager(s) listed above.</font>"; 
}//end if

/************************************/

//Here is the code of the update script

$dbName ="";
$dbPass ="";
$dbSite="emp_profile";

$conn = @mysql_connect("localhost",$dbName,$dbPass)
or die("Could not connect to localhost");

mysql_select_db($dbSite, $conn)
or die("Could not select database");

if($_POST['status'])
{
$status = $_POST['status'];
$emp_id = $_POST['update_emp_ID'];
$query ="UPDATE emp_profile SET status='$status' WHERE empID='$emp_id'";
// now you can run your query and add some error checking
$result = mysql_query($query, $conn);
}

?>

Give it a try. If you are still struggling post again explaining the problems in DETAIL and include your database structure and a sample data dump so I can debug it against some data.

Good luck 😉

P.S. You should use "mysql_pconnect()" instead of "mysql_connect()"

jjjj

There are still some problems here.

After I click on the active and inactive radio button, it seems like doing some changes, but the page still remains the same.
can you tell me how to do the debuging.
I forward to you my database named emp_profile, and the table name is also named emp_profile.

CREATE TABLE emp_profile (
empID int(10) unsigned NOT NULL auto_increment,
empf_name varchar(60) NOT NULL,
empl_name varchar(60) NOT NULL,
emp_passwd varchar(32) NOT NULL,
emp_email varchar(200) NOT NULL unique,

emp_rank varchar(128) NOT NULL,
emp_department varchar(255) NOT NULL,
creatDate datetime,
status int(2),
PRIMARY KEY (empID)
) ;

############### insert into emp_profile ####################
insert into emp_profile (empID, empf_name, empl_name,
emp_passwd, emp_email, emp_rank, emp_department, creatDate, status) VALUES
(null, 'Chris', 'Tseng', 'cs160', 'sjsusjsu','instructor', 'cs,me',
now(), 1);

insert into emp_profile (empID, empf_name, empl_name,
emp_passwd, emp_email, emp_rank, emp_department, creatDate, status) VALUES
(null, 'Chris', 'Tseng', 'sjsusjsu', 'cs160', 'instructor', 'cs,me,oe',
now(), 1);

insert into emp_profile (empID, empf_name, empl_name,
emp_passwd, emp_email, emp_rank, emp_department, creatDate,status) VALUES
(NULL, 'John', 'Smith', '1234567', 'John@hotmail.com',
'student', 'cs,chem', now(), 0);

insert into emp_profile (empID, empf_name, empl_name,
emp_passwd, emp_email, emp_rank, emp_department, creatDate, status)
VALUES (null, 'Alice', 'Smith', '234567', 'alice@yahoo.com', 'stud',
'cs', now(), 1);

INSERT INTO emp_profile (empID, empf_name, empl_name, emp_passwd,
emp_email, emp_rank, emp_department, creatDate, status)
VALUES ('', 'Jing', 'Chang', 'pass', 'csa', 'admin', 'cs', now(), 1);

thanks for your very useful help

nnichols

The reason you are getting your results is because of a caching issue. I haven't figured out yet, but I'll post back as soon as I get it sorted.

😉

nnichols

I feel a right pratt now. Hopefully once I explain it we will both learn from my mistake.

if($_POST['status'])
{
$status = $_POST['status'];
$emp_id = $_POST['update_emp_ID'];
$query ="UPDATE emp_profile SET status='$status' WHERE empID='$emp_id'";
// now you can run your query and add some error checking
$result = mysql_query($query, $conn);
}

if ($_POST['status']) was never being run because it was being passed status '0' which was being read as status being false. STUPID, STUPID, ME!!!

This is exactly why we should always test a variables existance with "isset()"

The code below works fine now. I have included the update code at the top of the script instead of in an include. You should do the same - it is slightly more efficient (less HD I/O).

<?PHP

$dbName ="root"; 
$dbPass ="root"; 
$dbSite="test"; 

/*
* Make connection to database
* Select database to build into and from
*/

$conn = @mysql_pconnect("localhost",$dbName,$dbPass) 
or die("Could not connect to localhost"); 

mysql_select_db($dbSite, $conn) 
or die("Could not select database"); 

// do update if status has changed
if(isset($_POST['status']))
{
$status = $_POST['status'];
$emp_id = $_POST['update_emp_ID'];
$query ="UPDATE emp_profile SET status='$status' WHERE empID='$emp_id'";

// now you can run your query and add some error checking
$result = mysql_query($query, $conn);
if ($result)
	echo "Emp$emp_id has been successfully updated.";
else
	echo "Update failed: " . mysql_error();
}

// run main query
$query = "select * from emp_profile";

$query_result = mysql_query ($query);

# get number of rows returned by our query 
$num_of_rows = mysql_num_rows ($query_result); 

if($query_result)//ok query
{

echo "<table border='1' >";
echo "<tr><th>User Detail</th>
<th>empID</th>
<th>empf_name</th>
<th>empl_name</th>
<th>emp_passwd</th>
<th>emp_email</th>
<th>emp_rank</th>
<th>emp_department</th>
<th>creatDate</th>
<th>active</th>
<th>inactive</th></tr>";

while ($myrow = mysql_fetch_array($query_result)) {

$empID = $myrow["empID"]; 
$empf_name=$myrow["empf_name"]; 
$empl_name=$myrow["empl_name"]; 
$emp_passwd=$myrow["emp_passwd"]; 
$emp_email =$myrow["emp_email"]; 
$emp_rank=$myrow["emp_rank"]; 
$emp_department=$myrow["emp_department"]; 
$creatDate =$myrow["creatDate"]; 
$status=$myrow["status"]; 

//echo $query; 
echo "<form method=POST action=\"$PHP_SELF\">";
// add hidden form field for empID - so we know which emp to UPDATE
echo "<input type=\"hidden\" name=\"update_emp_ID\" value=\"$empID\">";

echo "<tr><td><a href=\"employerdetail.php\">User Details</a></td>
<td>$empID</td>
<td>$empf_name</td>
<td>$empl_name</td>
<td>$emp_passwd</td>
<td>$emp_email</td>
<td>$emp_rank</td>
<td>$emp_department</td>
<td>$creatDate</td>"; 

//<!-----Active & Inactive------>
$onclick = "onClick=\"this.form.submit();\"";
if($status==1) 
{ 
echo "<td><input type=\"radio\" name=\"status\" value=\"1\" checked></td> 
<td><input type=\"radio\" name=\"status\" value=\"0\" $onclick></td>"; 
} 
else 
{ 
echo "<td><input type=\"radio\" name=\"status\" value=\"1\" $onclick></td> 
<td><input type=\"radio\" name=\"status\" value=\"0\" checked></td>"; 
} 

echo "</tr>";

echo "</form>";

}//end while 
echo "</table>"; 


print "<font color=red>There are $num_of_rows registered HR manager(s) listed above.</font>"; 
}//end if

?>

I hope I'll learn from that, but I very much doubt it :rolleyes:
but maybe you will?

Good luck 😉

jjjj

I still had a problem when I was running the code.
After I clicked the radio button, it did the same as before(remain the same position), but when I clicked the back button on the top of the browser screen, the radio button position becomes what I really wanted. It seems like the page got updated twice - the correct one got overwritten by the original(wrong) one. But the code doesn't seem wrong to me. Can you explain what the problem is? Really appreciate your kind help!

nnichols

It sounds like a real caching issue this time, but try the following just to check -

Clear browser cache and close all open browser windows
Restart Apache
open the file in your browser
click on one of the radio buttons to force an update
[/list=1]

When the page reloads you should see an update success/failure message at the top. If you don't see this message it is definitely a caching issue.

If this is the case you could try one or all of the following headers (MUST be first output from your file) -
```
// examples from the php manual
header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");		// Date in the past
header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");	// always modified
header("Cache-Control: no-store, no-cache, must-revalidate");	// HTTP/1.1
header("Cache-Control: post-check=0, pre-check=0", false);
header("Pragma: no-cache");					// HTTP/1.0
```
Hope this helps 😉

P.S. If you are still having this same problem post again including your ENTIRE SCRIPT.

jjjj

It is a real caching issue here since I couldn't see any update checking message after I click the radio button.

The header isn't work, below is the error message:

Warning: Cannot add header information - headers already sent by (output started at c:\nusphere\apache\nsdocs\final\lichfinal\14administrationori\allemployers999.php:12) in c:\nusphere\apache\nsdocs\final\lichfinal\14administrationori\allemployers999.php on line 13

it's seems like the php can sent the hear infor automatically.
I'm really confused.

Here is my entire script

<Title>HR manager List Page</Title>

<font size=4>Registed Employers</font><hr width=100% align=left>

<?PHP
//header("Expires: Sat, 1 Jan 2000 00:00:00 GMT");
//header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
//header("Cache-Control: no-store, no-cache, must-revalidate");
//header("Cache-Control: post-check=0, pre-check=0", false);
//header("Pragma: no-cache");

$dbName ="";
$dbPass ="";
$dbSite="emp_profile";

/
Make connection to database
Select database to build into and from
/

$conn = @mysql_pconnect("localhost",$dbName,$dbPass)
or die("Could not connect to localhost");

mysql_select_db($dbSite, $conn)
or die("Could not select database");

// do update if status has changed
if(isset($POST['status']))
{
$status = $POST['status'];
$emp_id = $_POST['update_emp_ID'];
$query ="UPDATE emp_profile SET status='$status' WHERE empID='$emp_id'";

// now you can run your query and add some error checking
$result = mysql_query($query, $conn);
if ($result)
echo "Emp$emp_id has been successfully updated.";
else
echo "Update failed: " . mysql_error();
}

// run main query
$query = "select * from emp_profile";

$query_result = mysql_query ($query);

get number of rows returned by our query

$num_of_rows = mysql_num_rows ($query_result);

if($query_result)//ok query
{

echo "<table border='1' >";
echo "<tr><th>User Detail</th>
<th>empID</th>
<th>empf_name</th>
<th>empl_name</th>
<th>emp_passwd</th>
<th>emp_email</th>
<th>emp_rank</th>
<th>emp_department</th>
<th>creatDate</th>
<th>active</th>
<th>inactive</th></tr>";

while ($myrow = mysql_fetch_array($query_result)) {

$empID = $myrow["empID"];
$empf_name=$myrow["empf_name"];
$empl_name=$myrow["empl_name"];
$emp_passwd=$myrow["emp_passwd"];
$emp_email =$myrow["emp_email"];
$emp_rank=$myrow["emp_rank"];
$emp_department=$myrow["emp_department"];
$creatDate =$myrow["creatDate"];
$status=$myrow["status"];

//echo $query;
echo "<form method=POST action=\"$PHP_SELF\">";
// add hidden form field for empID - so we know which emp to UPDATE
echo "<input type=\"hidden\" name=\"update_emp_ID\" value=\"$empID\">";

echo "<tr><td><a href=\"employerdetail.php\">User Details</a></td>
<td>$empID</td>
<td>$empf_name</td>
<td>$empl_name</td>
<td>$emp_passwd</td>
<td>$emp_email</td>
<td>$emp_rank</td>
<td>$emp_department</td>
<td>$creatDate</td>";

//
$onclick = "onClick=\"this.form.submit();\"";
if($status==1)
{
echo "<td><input type=\"radio\" name=\"status\" value=\"1\" checked></td>
<td><input type=\"radio\" name=\"status\" value=\"0\" $onclick></td>";
}
else
{
echo "<td><input type=\"radio\" name=\"status\" value=\"1\" $onclick></td>
<td><input type=\"radio\" name=\"status\" value=\"0\" checked></td>";
}

echo "</tr>";

echo "</form>";

}//end while
echo "</table>";

print "<font color=red>There are $num_of_rows registered HR manager(s) listed above.</font>";
}//end if

</body>
</html>

nnichols

That is because you must put the headers before any other output to the browser. Right at the top! Nothing before it!
Just -

<?PHP
// headers must NOT be preceded by any other output
header("Expires: Sat, 1 Jan 2000 00:00:00 GMT");
header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
header("Cache-Control: no-store, no-cache, must-revalidate");
header("Cache-Control: post-check=0, pre-check=0", false);
header("Pragma: no-cache");

Good luck :rolleyes:

jjjj

Sorry for the question again.
I still couldn't get the page I was expecting when I put the header in the topest top of the script. no warning message, but still couldn't update the page.

Thanks for your very kind help

<?php

header("Expires: Sat, 1 Jan 2000 00:00:00 GMT");
header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
header("Cache-Control: no-store, no-cache, must-revalidate");
header("Cache-Control: post-check=0, pre-check=0", false);
header("Pragma: no-cache");

//<html>
//<head>

//<Title>HR manager List Page</Title>

//<body bgcolor="#f7f7e7" link="330000" alink="330000" vlink="000000" text="336699">

//<font size=4>Registed Employers</font><hr width=100% align=left>
print "Register Employers";

$dbName ="";
$dbPass ="";
$dbSite="emp_profile";
🙁

nnichols

I must admit I am at a loss on what the cause is now. I suggest you re-post with the complete code (new thread) with a title of "caching problem?" and see what comes back. Remember to post your code between [ PHP][/code], so it's easy to read.

I'm sorry I can't help you further on this one.

I hope you resolve this issue soon.

Good luck :rolleyes: