Shoutbox

PHP help? // Never thought I'd ask. =p - Printable Version

-Shoutbox (https://shoutbox.menthix.net)
+-- Forum: MsgHelp Archive (/forumdisplay.php?fid=58)
+--- Forum: Skype & Technology (/forumdisplay.php?fid=9)
+---- Forum: Tech Talk (/forumdisplay.php?fid=17)
+----- Thread: PHP help? // Never thought I'd ask. =p (/showthread.php?tid=51518)

PHP help? // Never thought I'd ask. =p by brian on 10-07-2005 at 11:56 PM

Anyways, I'm maybe kind of stuck on this crap; what this foreach loops supposed to do is check the DB if $a[2] exists; if it does; then it ignores it, if it doesn't, then it adds it, and it's kind of ignoring them all. :X

code:
if($_REQUEST[act] == 'add')
{
    // check mandatory fields
    if(!$_REQUEST[lBulk]) { echo "No emails imported (No values)."; exit; }

    $tmp = $_REQUEST[lBulk];
    $atmp = split("\r\n", $tmp);

    foreach ($atmp as $asdf) {
        $a = split(",", $asdf);
        $qry = mysql_query("SELECT lEmail FROM ar_lead WHERE lEmail='$a[2]'", $id);
        if(mysql_num_rows($qry)) { $erg = 1; }
        if ($erg == 1) {
                echo "Ignored " . $a[0] . " " . $a[1] . " (" . $a[2] . ")<br>";
        } else {
            $aqry = mysql_query("INSERT INTO ar_lead(lName, lLast, lEmail, lDateEntry, lDateOut, lMOut) VALUES('$a[0]', '$a[1]', '$a[2]', current_date, current_date, '-1')", $id);
            // start mail system
            $ms = new MailSystem($aConfig);

            if($ms->setLead(mysql_insert_id($id)) && $ms->setMsg(0)) {
                $ms->mailLead();
                echo "Imported " . $a[0] . " " . $a[1] . " (" . $a[2] . ") Successfully<br>";
            }
        }
    }
    echo "<br><b>Finished Importing Successfully!</b>";
}
?>

RE: PHP help? // Never thought I'd ask. =p by J-Thread on 10-08-2005 at 07:26 AM

WOW that code really hurts:|. Here's my version:

code:
<?php
if($_REQUEST['act'] == 'add')
{
   // check mandatory fields
   if(empty($_REQUEST['lBulk'])) {
      echo "No emails imported (No values).";
      exit;
   }
   
   $tmp = $_REQUEST['lBulk'];
   $atmp = split("\r\n", $tmp);
   
   foreach ($atmp as $asdf) {
      $a = split(",", $asdf);
      $qry = mysql_query("SELECT lEmail FROM ar_lead WHERE lEmail='" . $a[2] . "'", $id);
      if(mysql_num_rows($qry) < 1) {
         $erg = true;
      }
      if ($erg) {
         echo "Ignored " . $a[0] . " " . $a[1] . " (" . $a[2] . ")<br>";
      }
      else {
         $aqry = mysql_query("INSERT INTO ar_lead(lName, lLast, lEmail, lDateEntry, lDateOut, lMOut) VALUES('" . $a[0] . "', '" . $a[1] . "', '" . $a[2] . "', current_date, current_date, '-1')", $id);
         // start mail system
         $ms = new MailSystem($aConfig);
         
         if($ms->setLead(mysql_insert_id($id)) && $ms->setMsg(0)) {
            $ms->mailLead();
            echo "Imported " . $a[0] . " " . $a[1] . " (" . $a[2] . ") Successfully<br>";
         }
      }
   }
   echo "<br><b>Finished Importing Successfully!</b>";
}
?>

Change $_REQUEST to either $_GET or $_POST, choose the one that you're using. Don't use $_REQUEST!

This code should work....