Can anyone help?

stualk

Hi Folks,

I posted on here a few weeks ago about checking a database before entering the contents of an online form into that database. The advise I got was really good but I've gone about it in slightly the wrong way and got myself really stuck! Is anyone around to give me a bit of support by email? The code is in place, it just needs adjusting slightly to make it work. I'm desperate!!

Email me on webmaster@1226.net if you are able to help me fix this and I'll send you the code I've used.

Many thanks
Stu

bubblenut

The general method here is to solve the problems on the forum for a couple of reasons.
a) Someone else is (or will be in the future) probably having exactly the same problem and the dialogue may be helpfull to them
b) It makes the person helping you a private consultant and therefore should really be paid.

If you post your code up there'll be lots of people ready and willing to help you, me for one 🙂

Bubble

stualk

Ok, take a look at this code below and tell me what I'm doing wrong because I've tried just about everything I can think of to make it work and it just won't!

============================================
if ($action =="addscript" ) {

$start_date_from_form = strtotime("$start_year-$start_month-$start_number");
print("
 
");
$end_date_from_form = strtotime("$end_year-$end_month-$end_number");

$numresults=mysql_query("select id from courses_main where instructor = '$instructor' order by id");
$numrows=mysql_num_rows($numresults);
$result = mysql_query("select * from courses_main where instructor = '$instructor' order by id") or die("Cant connect to the database. Please try later");
for ($i=1; $i<=$numrows; $i++) {
if ($row = mysql_fetch_array($result)) {
$id = $row["id"];
$start_day = $row["start_day_db"];
$start_number = $row["start_number_db"];
$start_month = $row["start_month_db"];
$start_year = $row["start_year_db"];
$end_day = $row["end_day_db"];
$end_number = $row["end_number_db"];
$end_month = $row["end_month_db"];
$end_year = $row["end_year_db"];
$instructor = $row["instructor_db"];

$end_date = strtotime("$end_year_db-$end_month_db-$end_number_db");
$start_date = strtotime("$start_year_db-$start_month_db-$start_number_db");
$difference = $end_date - $start_date;

$result_one = $start_date_from_form - $start_date;
$result_two = $end_date - $start_date;

}}}

if ($result_one <= $result_two)
{
print("
INSTRUCTOR NOT AVAILABLE ON THAT DATE. PLEASE GO BACK AND SELECT ANOTHER DATE. 
");
}
elseif ($result_one > $result_two)
{
print("
THANK YOU. COURSE IS NOW BEING ADDED...
");

//mysql_query ("insert into courses_main (course_title, location, instructor, start_day, start_number, start_month, start_year, duration, end_day, end_number, end_month, end_year, course_capacity, course_notes, added_by, course_type)
// values ('$course_title', '$location', '$instructor', '$start_day', '$start_number', '$start_month', '$start_year', '$duration', '$end_day', '$end_number', '$end_month', '$end_year', '$course_capacity', '$course_notes', '$name', '$course_type')");

//if ($multiple == "yes"){print("<meta http-equiv=\"refresh\" content=\"1;URL='?action=add&&name=$name'\"> adding new course details..."); }

//else {print("<meta http-equiv=\"refresh\" content=\"1;URL='?action=default&&name=$name'\"> adding new course details..."); }

What this code basically does is to avoid a member of staff being assigned to a course on a date which they already have a course. If they are already booked it rejects it, stops you adding the data and displays an error message. If the instructor is free on that date, it just adds the details to the database without problem.

What do you think? What am I doing wrong?

stualk

Have you got any advice on my problem? It's a pretty urgent matter and I need to think up a solution quick!!

bubblenut

What's going wrong with it?
What errors are it throwing?

bubblenut

Here's a couple of light notes

if ($action =="addscript" ) {

$start_date_from_form = strtotime("$start_year-$start_month-$start_number");
print("
<br>
");
$end_date_from_form = strtotime("$end_year-$end_month-$end_number");

//no need to do a full query just to get the number of rows. You can get this from the
//result on the second query you're doing (you don't nede to though as you'll see in a bit)
$numresults=mysql_query("select id from courses_main where instructor = '$instructor' order by id");
$numrows=mysql_num_rows($numresults);
$result = mysql_query("select * from courses_main where instructor = '$instructor' order by id") or die("Cant connect to the database. Please try later");
//You don't need to use a for as the if on the second line implies, mysql_fetch_array
//will return false at the end of the result set so you can just use a while loop.
for ($i=1; $i<=$numrows; $i++) {
if ($row = mysql_fetch_array($result)) {
$id = $row["id"];
$start_day = $row["start_day_db"];
$start_number = $row["start_number_db"];
$start_month = $row["start_month_db"];
$start_year = $row["start_year_db"];
$end_day = $row["end_day_db"];
$end_number = $row["end_number_db"];
$end_month = $row["end_month_db"];
$end_year = $row["end_year_db"];
$instructor = $row["instructor_db"];

//Might be easier to use mktime here
$end_date = strtotime("$end_year_db-$end_month_db-$end_number_db");
$start_date = strtotime("$start_year_db-$start_month_db-$start_number_db");
$difference = $end_date - $start_date;

$result_one = $start_date_from_form - $start_date;
$result_two = $end_date - $start_date;

}}}

if ($result_one <= $result_two)
{
print("
<font face=\"Arial, Helvetica, sans-serif\" size=\"2\">INSTRUCTOR NOT AVAILABLE ON THAT DATE. PLEASE GO BACK AND SELECT ANOTHER DATE.<br></font>
");
}
elseif ($result_one > $result_two)
{
print("
<font face=\"Arial, Helvetica, sans-serif\" size=\"2\">THANK YOU. COURSE IS NOW BEING ADDED...</font>
");


//mysql_query ("insert into courses_main (course_title, location, instructor, start_day, start_number, start_month, start_year, duration, end_day, end_number
// values ('$course_title', '$location', '$instructor', '$start_day', '$start_number', '$start_month', '$start_year', '$duration', '$end_day', '$end_number',

//if ($multiple == "yes"){print("<meta http-equiv=\"refresh\" content=\"1;URL='?action=add&&name=$name'\"> adding new course details..."); }
//else {print("<meta http-equiv=\"refresh\" content=\"1;URL='?action=default&&name

HTH
Bubble