Jump to content

[SOLVED] Recently working validation now not taking place

mikebyrne

By mikebyrne
in PHP Coding Help

 Share

Recommended Posts

mikebyrne Advanced Member

mikebyrne

Posted
Posted

My form validation is now suddenly decided to stop working and I cant see why??

 

If i leavd every field blank I get no error reporting at all

 

My code is:

 

<?PHP

require_once("adminconnect.php");
$con=mysql_connect("localhost","root","");
mysql_select_db("test",$con);

$ProductName = mysql_real_escape_string(trim($_POST['ProductName']));
$Description = mysql_real_escape_string(trim($_POST['Description']));
$Price = mysql_real_escape_string(trim($_POST['Price']));
$Stockamount = mysql_real_escape_string(trim($_POST['Stockamount']));
$Type = $_POST['Type'];
$Display = strtoupper($_POST['display']);
$Image = $_POST['Image'];
$Image2 = $_POST['Image2'];


if($_POST["submit"]=='Upload File')
{

while(list($key,$value) = each($_FILES['uploadedfile']['name']))
           {
         if(!empty($value))
         {
            $filename = $value;
            $add = "Pictures/".$filename;   
            //echo $add;                                 
            copy($_FILES['uploadedfile']['tmp_name'][$key], $add);
            chmod("$add",0776);
         }
         
      }
      $fileName = $_FILES['uploadedfile']['name'];
      //$a[] = $fileName;
      
         
      $img= implode(",",$fileName);
      $imag=explode(",",$img);
      //echo $imag[0];   
         //print_r($image[0]);
//$Image1= $image[0];
//echo $image[0];
//$Image2= $image[1];
// Where the file is going to be placed
//$target_path = "Pictures/".$_FILES['uploadedfile']['name'];
//echo $target_path;
//
///* Add the original filename to our target path.
//Result is "uploads/filename.extension" */
//$_FILES['uploadedfile']['tmp_name'];
//
//
//if(move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target_path)) {
//    echo "The file ".  basename( $_FILES['uploadedfile']['name']).
//    " has been uploaded";
//} else{
//    echo "There was an error uploading the file, please try again!";
//}

if($_POST["action"] == "Add"){

       $valid=1;
       if($_FILES['uploadedfile']['name'][0]=="" && $_FILES['uploadedfile']['name'][1]=="")
      {
       $valid=0;
         $style_name = "background-color:#FF5959";
         $error_image = "Please UPLOAD A FILE";
      }
         
                        
      if ($_FILES['uploadedfile1']['error'] != 0) {
                $valid=0;
                $style_name = "background-color:#FF5959";
                $error_image = "Please UPLOAD A FILE";
                        }      
                        
      if ($_POST['Type'] =="") {
                $valid=0;
                $style_name = "background-color:#FF5959";
                $error_type = "Please enter the product type";
                        }   
                        
      if ($_POST['ProductName']=="") {
                $valid=0;
                $style_name = "background-color:#FF5959";
                $error_name = "The Product Name seems to be mising?";
                                      }
      
      if ($_POST['Description']=="") {
                $valid=0;
                $style_name = "background-color:#FF5959";
                $error_description = "Your description seems to be mising?";
                              }
      if ($_POST['Stockamount']=="") {
                $valid=0;
                $style_name = "background-color:#FF5959";
                $error_amount = "Please enter the ammount of stock recivied";
                              }
                     
      if(empty($_POST['Price']))
                              {
      $valid=0;
      $style_name = "background-color:#FF5959";
      $error_price = "Please enter the price";
                              }
       if(preg_match("/\A[0-9]{1,16}(\.[0-9]{2}){0,1}$/",$_POST['Price']) != 1)
                              {
      $valid = 0;
      $style_name = "background-color:#FF5959";
      $error_price = 'Price is invalid. Numbers only (eg: 123.05)';
                              }
      if(empty($_POST['Stockamount']))
                              {
      $valid=0;
      $style_name = "background-color:#FF5959";
      $error_amount = "Please enter a stock ammount";
                              }
       if(preg_match("/\A[0-9]{1,16}(\.[0-9]{2}){0,1}$/",$_POST['Stockamount']) != 1)
                              {
      $valid = 0;
      $style_name = "background-color:#FF5959";
      $error_amount = 'Ammonut is invalid. Numbers only (eg: 1,2,3)';
                              }                     
if ($valid==1) {       
                $sql="INSERT INTO $tbl_name (Producttype, ProductName,Stockamount,Display, Description,Price, Image, Image2) VALUES ('$Type','$ProductName','$Stockamount','$Display' ,'$Description', '$Price', '$imag[0]', '$imag[1]')";

echo $_POST['Type'];
echo $_POST['ProductName'];
echo $_POST['Stockamount'];
echo $_POST['Display'];
echo $_POST['Description'];
echo $_POST['Price'];
echo $_POST['Image'];
echo $_POST['Image2'];
echo $sql;

$result=mysql_query($sql)or die(mysql_error()."<p>With Query
$sql");
}
}

}
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>e-commerce admin page sample |item</title>
<link href="../Admin_files/admin.css" rel="stylesheet" type="text/css" />
<script type="text/JavaScript" src="../includes/wrapper.js"></script>
</head>
<body>
<div id="Box">
  <div id="logoBox"><img src="../Admin_files/logo.gif" width="51" height="125" border="0" /></div>
  <div id="contentBox">
<form action="add.php" onsubmit="return valid()"  enctype="multipart/form-data"method="post">

      <!-- logo start -->
      <div id="container">
        <div class="padTop28">
          <!-- -->
        </div>
        <div class="clr">
          <!-- -->
        </div>
      </div>

      <div class="clr">
        <!-- -->
      </div>
      <!-- logo finish -->
      <!-- menu start -->
      <div id="container">
            <div id="menu1">
         <div id="menu_off"><!-- --></div>
         <div id="menu_text_off"><a href="../admin_files/start.php" class="black">Order</a></div>
         <div id="menu_space1"><!-- --></div>
         <div id="menu_off"><!-- --></div>
            <div id="menu_text_off"><a href="../admin_files/sales.php" class="black">Sales</a></div>
         <div id="menu_space1"><!-- --></div>
         <div id="menu_off"><!-- --></div>
         <div id="menu_text_off"><a href="../admin_files/list.php" class="black">Edit item</a></div>
         <div id="menu_space1"><!-- --></div>
         <div id="menu_off"><!-- --></div>
         <div id="menu_text_off"><a href="../admin_files/postage.php" class="black">Postage fee</a></div>
            <div id="menu_space1"><!-- --></div>
         <div id="menu_off"><!-- --></div>
            <div id="menu_text_off"><a href="../admin_files/adminreg.php" class="black">Add User</a></div>
            <div id="menu_space1"><!-- --></div>
         <div id="menu_on"><!-- --></div>
            <div id="menu_text_on"><a href="../admin_files/add.php" class="black_on">Add Product</a></div>
         <div id="menu_space1"><!-- --></div>
      </div>
          <!-- -->
         <!-- Double Space undernath CD -->
          <div class="clr"><!-- --></div>      <div class="clr"><!-- --></div>
      <div class="padTop5"><!-- --></div>
      <div class="clr"><!-- --></div>
        <div class="clr"><!-- --></div>      <div class="clr"><!-- --></div>
      <div class="padTop5"><!-- --></div>
      <div class="clr"><!-- --></div>
      <!-- menu finish -->
      <!-- top start -->
      <div id="container">
        <div id="line">
          <!-- -->

        </div>
      </div>
      <div class="clr">
        <!-- -->
      </div>
      <div id="containerBg1">
        <div class="padTop15">
          <!-- -->
        </div>

        <div class="clr">
          <!-- -->
        </div>
        <div id="titleText">EDIT ITEM</div>
        <div class="clr">
          <!-- -->
        </div>
        <div class="padTop15">

          <!-- -->
        </div>
        <div class="clr">
          <!-- -->
        </div>
      </div>
      <div class="clr">
        <!-- -->
      </div>

      <!-- top finish -->
      <!-- data title start -->
      <div id="containerBg2">
        <div class="padTop2">
          <!-- -->
        </div>
        <div class="clr">
          <!-- -->
        </div>

        <div class="titleBox">
          <table width="850" border="0" cellspacing="0" cellpadding="0">
            <tr align="left">
              <td width="33"> </td>
              <td width="290" align="left">IMAGE</td>
              <td width="527">DETAIL</td>
            </tr>
          </table>

        </div>
        <div class="clr">
          <!-- -->
        </div>
      </div>
      <div class="clr">
        <!-- -->
      </div>
      <!-- data title finish -->

      <!-- 1px space start -->
      <div id="containerBg1">
        <div class="padTop1">
          <!-- -->
        </div>
        <div class="clr">
          <!-- -->
        </div>
      </div>

      <div class="clr">
        <!-- -->
      </div>
      <!-- 1px space finish -->
      <!-- data top start -->
      <div id="containerBg3">
        <div class="padTop1">
          <!-- -->
        </div>

        <div class="clr">
          <!-- -->
        </div>
      </div>
      <div class="clr">
        <!-- -->
      </div>
      <!-- data top finish -->
      <!-- data content start -->



      <div id="containerBg4">
      <div class="padTop15">
        <!-- -->
      </div>
      <div class="clr">
        <!-- -->
      </div>
      <div class="padTop1">
        <!-- data Left start -->

        <div class="editLeft">
          <!-- picture1 start -->
          <div class="editLeftBox">PICTURE 1:</div>
          <div class="clr">
            <!-- -->
          </div>
          <div class="padTop5">
            <!-- -->

          </div>
          <div class="clr">
            <!-- -->
          </div>
          <div class="editLeftBox"><img src="Pictures/No_image.png" alt="pic1" width="150" height="200" /></div>
          <div class="clr">
            <!-- -->
          </div>
          <div class="padTop5">

            <!-- -->
          </div>
          <div class="clr">
            <!-- -->
          </div>
         
      
            <input type="hidden" name="MAX_FILE_SIZE" value="100000"  />
Choose a file to upload: <input name="uploadedfile[]"  id="uploadedfile[]" type="file" />

<!--<input type="submit" name="submit" value="Upload File" />--><?php echo $error_image; ?>
        <div class="editLeftBox">  </div>
          <div class="clr">

            <!-- -->
          </div>
          <!-- picture1 finish -->
          <div class="padTop50">
            <!-- -->
          </div>
          <div class="clr">
            <!-- -->
          </div>

          <!-- picture2 start -->
          <div class="editLeftBox">PICTURE 2:</div>
          <div class="clr">
            <!-- -->
          </div>
          <div class="padTop5">
            <!-- -->
          </div>

          <div class="clr">
            <!-- -->
          </div>
          <div class="editLeftBox"><img src="Pictures/No_image.png" alt="pic1" width="150" height="200" /></div>
          <div class="clr">
            <!-- -->
          </div>
          <div class="padTop5">
            <!-- -->

          </div>
          <div class="clr">
            <!-- -->
          </div>
          <div class="editLeftBox">
            <input type="hidden" name="MAX_FILE_SIZE" value="100000" />
Choose a file to upload: <input name="uploadedfile[]" id="uploadedfile[]" type="file" />

<input type="submit"name="submit" value="Upload File" /><?php echo $error_image; ?>
          </div>
          <div class="clr">
            <!-- -->

          </div>
          <!-- picture2 finish -->
        </div>
        <!-- data Left finish -->
        <!-- data Right start -->
        <div class="editRight">
          <!-- dat start -->
          <div class="editRightBox">
            <div class="editRightText">Product Type:</div>

            <div class="editRightForm">
           
            <select name="Type" class="000">
   <option value="">------</option> returns $_POST['Type'] = ""
   <option value="Game" >GAME</option> returns $_POST['Type'] = "Game"
   <option value="DVD" >DVD</option> returns $_POST['Type'] = "DVD"
   <option value="CD" >CD</option> returns $_POST['Type'] = "CD"
          </select>
            <td width="269"><font color="#FF0000" style="font-size: 8pt"><?php echo $error_type; ?></font></td>
            </div>
            <div class="clr">
              <!-- -->
            </div>
          </div>
          <div class="clr">
            <!-- -->

          </div>
          <!-- data finish -->
          <div class="padTop3">
            <!-- -->
          </div>
          <div class="clr">
            <!-- -->
          </div>
          <!-- dat start -->

          <div class="editRightBox">
            <div class="editRightText">Product Name:</div>
            <div class="editRightForm">
              <input type="text" class="itemEditForm02" name="ProductName" value="<?php $row['ProductName'];?>" />
              <td width="269"><font color="#FF0000" style="font-size: 8pt"><?php echo $error_name; ?></font></td>
            </div>
            <div class="clr">
              <!-- -->
            </div>

          </div>
          <div class="clr">
            <!-- -->
          </div>
          <!-- data finish -->
          <div class="padTop3">
            <!-- -->
          </div>
          <div class="clr">

            <!-- -->
          </div>
          <!-- dat start -->
          <div class="editRightBox">
            <div class="editRightText">Description:</div>
            <div class="editRightForm">
              <textarea type="text" class="itemEditForm03" name="Description" ><?php echo $row['Description'];?></textarea>
     <font color="#FF0000" style="font-size: 8pt"><?php echo $error_description; ?></font>
            <div class="clr">
              <!-- -->
            </div>
          </div>
          <div class="clr">
            <!-- -->
          </div>
          <!-- data finish -->
          <div class="padTop3">

            <!-- -->
          </div>
          <div class="clr">
            <!-- -->
          </div>
          <!-- dat start -->
          <div class="editRightBox">
            <div class="editRightText">Stock Rec'd:</div>

            <div class="editRightForm">
              <input type="text" class="itemEditForm02" name="Stockamount" value="<?php $row['Stockamount'];?>" />
              <td width="269"><font color="#FF0000" style="font-size: 8pt"><?php echo $error_amount; ?></font></td>
            </div>
            <div class="clr">
              <!-- -->
            </div>
          </div>
          <div class="clr">
            <!-- -->

          </div>
          <!-- data finish -->
          <div class="padTop3">
            <!-- -->
          </div>
          <div class="clr">
            <!-- -->
          </div>
          <!-- dat start -->

          <div class="editRightBox">
            <div class="editRightText">Price:</div>
            <div class="editRightForm">
              <div class="editItemPrice1">€</div>
              <div class="editItemPrice2">
                <input type="text" class="itemEditForm04" name="Price" value="<?php $row['Price'];?>" /> <td width="269"><font color="#FF0000" style="font-size: 8pt"><?php echo $error_price; ?></font></td>
              </div>
              <div class="clr">

                <!-- -->
              </div>
              <div class="clr">
                <!-- -->
              </div>
            </div>
            <div class="clr">
              <!-- -->
            </div>


            <!-- data finish -->
            <div class="padTop3">
              <!-- -->
            </div>
            <div class="clr">
              <!-- -->
            </div>
            <!-- dat start -->
            <div class="editRightBox">

             
                <div class="clr">
                  <!-- -->

                </div>
              </div>
              <div class="clr">
                <!-- -->
              </div>
            </div>
            <div class="clr">
              <!-- -->
            </div>

            <!-- data finish -->
            <div class="padTop3">

              <!-- -->
            </div>
            <div class="clr">
              <!-- -->
            </div>
            <!-- dat start -->
            <div class="editRightBox">

              <div class="editRightText">Display:</div>
              <div class="editRightForm">
                <div class="editRadio1">
                  <input name="display" type="radio" value="yes" checked="checked" />
                </div>
                <div class="editRadio2">show</div>
                <div class="editRadio3">
                  <input name="display" type="radio" value="no" />

                </div>
                <div class="editRadio4">hide</div>
              </div>
              <div class="clr">
                <!-- -->
              </div>
            </div>
            <div class="clr">

              <!-- -->
            </div>
            <!-- data finish -->
            <div class="padTop75">
              <!-- -->
            </div>
            <div class="clr">
              <!-- -->
            </div>

         

          </div>
          <!-- data Right finish -->
        </div>
        <div class="clr">
          <!-- -->
        </div>
        <div class="padTop11">
          <!-- -->
        </div>

        <div class="clr">
          <!-- -->
        </div>
        <div id="dottedIn">
          <!-- -->
        </div>
        <div class="clr">
          <!-- -->
        </div>

        <!-- data finish -->
     <!-- btn start -->
        <div class="clr">
          <!-- -->
        </div>
        <div class="padTop16">
          <!-- -->
        </div>
        <div class="clr">

          <!-- -->
        </div>
        <div id="btn">
          <input type=hidden name=action value="Add">
          <input type="submit" name="Submit" value=""  style="border:0;background:url(../Admin_files/btn_update.gif) no-repeat;width:73px;height:23px;">
          <div id="btnSpace">
            <!-- -->
          </div>
   <input type="reset" value="" style="border:0;background:url(../Admin_files/btn_clear.gif) no-repeat;width:73px;height:23px;" onclick="return confirm('Are you sure you want to clear all data?')">
          <div class="clr">

            <!-- -->
          </div>
        </div>
        <div class="clr">
          <!-- -->
        </div>
        <div class="padTop16">
          <!-- -->
        </div>

        <div class="clr">
          <!-- -->
        </div>
        <!-- btn finish -->
      </div>
      <div class="clr">
        <!-- -->
      </div>

      <!-- data content finish -->

      <!-- data btm start -->
      <div id="containerBg3">
        <div class="padTop1">
          <!-- -->
        </div>
        <div class="clr">
          <!-- -->
        </div>
      </div>

      <div class="clr">
        <!-- -->
      </div>
      <!-- data btm finish -->
     
    </form>
  </div>
  <!-- btm start -->
  <div id="containerBg1">
    <div class="padTop15">

      <!-- -->
    </div>
    <div class="clr">
      <!-- -->
    </div>
  </div>
  <div class="clr">
    <!-- -->
  </div>

  <div id="container">
    <div id="line">
      <!-- -->
    </div>
  </div>
  <div class="clr">
    <!-- -->
  </div>
  <div class="padTop16">

    <!-- -->
  </div>
  <div class="clr">
    <!-- -->
  </div>
  <!-- btm finish -->
</div>
</body>
</html>

Link to comment
Share on other sites
mikebyrne Advanced Member

mikebyrne

Posted
  • Author
Posted

Ah think im onto the problem now. The validation works when the user hits the Upload button under image2 but not with my desired button

located

 

<div id="btn">
          <input type=hidden name=action value="Add">
          <input type="submit" name="Submit" value=""  style="border:0;background:url(../Admin_files/btn_update.gif) no-repeat;width:73px;height:23px;">
          <div id="btnSpace">
            <!-- -->
          </div>

 

 

 

Link to comment
Share on other sites
mikebyrne Advanced Member

mikebyrne

Posted
  • Author
Posted

Would I be better off getting rid of the "Submit" button and putting the validation

 

if($_POST["submit"]=='Upload File')
{

while(list($key,$value) = each($_FILES['uploadedfile']['name']))
           {
         if(!empty($value))
         {
            $filename = $value;
            $add = "Pictures/".$filename;   
            //echo $add;                                 
            copy($_FILES['uploadedfile']['tmp_name'][$key], $add);
            chmod("$add",0776);
         }
         
      }
      $fileName = $_FILES['uploadedfile']['name'];
      //$a[] = $fileName;
      
         
      $img= implode(",",$fileName);
      $imag=explode(",",$img);
      //echo $imag[0];   
         //print_r($image[0]);
//$Image1= $image[0];
//echo $image[0];
//$Image2= $image[1];
// Where the file is going to be placed
//$target_path = "Pictures/".$_FILES['uploadedfile']['name'];
//echo $target_path;
//
///* Add the original filename to our target path.
//Result is "uploads/filename.extension" */
//$_FILES['uploadedfile']['tmp_name'];
//
//
//if(move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target_path)) {
//    echo "The file ".  basename( $_FILES['uploadedfile']['name']).
//    " has been uploaded";
//} else{
//    echo "There was an error uploading the file, please try again!";
//}

 

Under the

 

if($_POST["action"] == "Add"){

 

 

Link to comment
Share on other sites
GingerRobot Regular Member

GingerRobot

Posted
Posted

I've not fully read this, but i do notice one thing. In your PHP you have this line;

 

if($_POST["submit"]=='Upload File')

 

However, your submit button has the name 'Submit' - with a capital S. Array keys are case sensitive, so you need to change that.

 

Also, you appear not to have given your submit button a value, in which case the above will never be true. You might want to replace the above line with:

 

if(isset($_POST["submit"]))

Link to comment
Share on other sites
mikebyrne Advanced Member

mikebyrne

Posted
  • Author
Posted

I'll probably get rid of the submit button (dont like the look of it) and code around the below button

 

<div id="btn">
          <input type=hidden name=action value="Add">
          <input type="submit" name="Submit" value=""  style="border:0;background:url(../Admin_files/btn_update.gif) no-repeat;width:73px;height:23px;">
          <div id="btnSpace">
            <!-- -->
          </div>

 

How would I adjust the validation to allow for this??

 

Link to comment
Share on other sites
zenag Newbie

zenag

Posted
Posted

then remove that upload file button ...

and include query..

 

 

while(list($key,$value) = each($_FILES['uploadedfile']['name']))

          {

        if(!empty($value))

        {

            $filename = $value;

            $add = "Pictures/".$filename; 

            //echo $add;                               

            copy($_FILES['uploadedfile']['tmp_name'][$key], $add);

            chmod("$add",0776);

        }

       

      }

      $fileName = $_FILES['uploadedfile']['name'];

      //$a[] = $fileName;

     

       

      $img= implode(",",$fileName);

      $imag=explode(",",$img);

 

 

inside......  if($_POST["action"] == "Add"){    ...

 

Link to comment
Share on other sites
mikebyrne Advanced Member

mikebyrne

Posted
  • Author
Posted

I'll be near my laptop in an hour or so, thanks for your help.

 

A couple of quick questions:

 

1) Is the chmod("$add",0776); needed as im using windows?

 

2) What does the below statement do?

 

$img= implode(",",$fileName);

      $imag=explode(",",$img);

 

 

Link to comment
Share on other sites
zenag Newbie

zenag

Posted
Posted

$img= implode(",",$fileName);

echo $img;

 

it will seperate the array of filenames using comma....

 

      $imag=explode(",",$img);

 

comma seperated values are splited into two variables.. as $imag[0],$imag[1]..

so that $imag[0] gets inserted into one field & $imag[1] into another field...

 

just echo $img; & echo $imag; you can understand that...

Link to comment
Share on other sites
This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
 Share
Go to topic listing
×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.