Email is only send once PHP / MySQL

Question

Good afternoon everyone,

I am currently working on an email system with distance calculation but it is only working half. The email is being send (it is also calculating the distance, only send email if same or lower then 100 kilometers).

But the email is only being send once, only the first MySQL row is being displayed in my email. Shouldn't I receive an email of every company in my database instead of only the first one? Currently I have my own email set as the $to variable, after I receive all the emails I want the $to to be a variable for all the other companies as a while setup.

Here is my code, hope someone can help me with it.

add_action('gform_after_submission_1', 'send_to_kitchen_companies', 10, 2);
function send_to_kitchen_companies($entry, $form) {

    $servername7 = 'localhost';
    $username7 = 'username';
    $password7 = 'password';
    $dbname7 = 'databasename';
    $conn7 = mysql_connect($servername7, $username7, $password7);
    mysql_select_db('databasename');
    $sqlAZ = 'SELECT * FROM creat_companies_zip_lat_long';
    $resultpieAZ = mysql_query ($sqlAZ);

    while($blsrowAZ = mysql_fetch_array($resultpieAZ, MYSQL_ASSOC)) {               

        function strafter($string, $substring) {
            $pos = strpos($string, $substring);
                if ($pos === false)
            return $string;
                else  
            return(substr($string, $pos+strlen($substring)));
        }

        function strbefore($string, $substring) {
            $pos = strpos($string, $substring);
                if ($pos === false)
            return $string;
                else  
            return(substr($string, 0, $pos));
        }   
        $mylatlong = $entry['94'];
        $mylat0 = strafter($mylatlong,'"');
        $mylat1 = strafter($mylat0,'"');
        $mylat2 = strafter($mylat1,'"');
        $latitude1 = strbefore($mylat2,'"');

        $mylong0 = strafter($mylat2,'"');
        $mylong1 = strafter($mylong0,'"');
        $mylong2 = strafter($mylong1,'"');
        $mylong3 = strafter($mylong2,'"');
        $longitude1 = strbefore($mylong3,'"');

        $latitudecomp = $blsrowAZ['latitude'];
        $longitudecomp = $blsrowAZ['longitude'];

        $latitudeFrom = $latitude1;
        $longitudeFrom = $longitude1;

        $latitudeTo = $latitudecomp;
        $longitudeTo = $longitudeFrom;

        //Calculate distance from latitude and longitude
        $theta = $longitudeFrom - $longitudeTo;
        $dist = sin(deg2rad($latitudeFrom)) * sin(deg2rad($latitudeTo)) +  cos(deg2rad($latitudeFrom)) * cos(deg2rad($latitudeTo)) * cos(deg2rad($theta));
        $dist = acos($dist);
        $dist = rad2deg($dist);
        $miles = $dist * 60 * 1.1515;

        $distance = ($miles * 1.609344);
        if ($distance <= 100) {

$to = "[email protected]";
$subject = "Een test email in HTML";

$message = "
            <html>
            <head>
            <title>Test HTML email</title>
            </head>
            <body>
            <p>Geachte " . $blsrowAZ['field_company_name_value'] . " De afstand is minder dan 100 kilometer en dus geschikt voor email</p>
            <table>
            <tr>
            <th>Latitude Aanvrager</th>
            <th>Longitude Aanvrager</th>
            </tr>
            <tr>
            <td>". $latitude1 ."</td>
            <td>". $longitude1 ."</td>
            </tr>
            <tr>
            <td>Latitude Bedrijf</td>
            <td>Longitude Bedrijf</td>
            </tr>
            <tr>
            <td>". $latitudecomp ."</td>
            <td>". $longitudecomp ."</td>
            </tr>
            <tr>
            <td>Totale Afstand</td>
            </tr>
            <tr>
            <td>". $distance ."</td>
            </tr>
            </table>
            </body>
            </html>
            ";

// Always set content-type when sending HTML email
$headers = "MIME-Version: 1.0" . "\r\n".
       "Content-type:text/html;charset=UTF-8" . "\r\n".
       'From: [email protected]' . "\r\n".
       'Reply-To: [email protected]' . "\r\n" .
       'X-Mailer: PHP/' . phpversion();

mail($to,$subject,$message,$headers);
        }
    }
}
mysql_close($conn7);

If you're working on new code please stop using the mysql_* functions. They were deprecated in PHP 5.5, which is so old it no longer even receives security updates, and completely removed in PHP 7. Instead, use PDO or mysqli_*. See stackoverflow.com/q/12859942/354577 for details. — Chris
– Chris, Commented Nov 23, 2016 at 13:20
so many things wrong here the most serious of which is perhaps the use of mysql_* functions. Then you are looping through the whole table and doing the distance calculation in PHP. it's far more efficient to do that in the database. Specially using the mysql geospatial data types and functions. Thirdly you have defined a function inside the loop. While this is syntactically correct, it's unwieldy and not readble. — e4c5
– e4c5, Commented Nov 23, 2016 at 13:20
Side note: don't use mail() except for extremely simple internal stuff, it's an awful function that does nothing to ensure your email is properly formatted as per standards. Use a third-party library like Swift Mailer or PhpMailer instead. — Álvaro González
– Álvaro González, Commented Nov 23, 2016 at 13:23
Your MAIN issue is that you are declaring functions inside a loop, the second time it tries to declare the functions it will throw an error that the functions already exists. Move you functions declaration outside the loop — phobia82
– phobia82, Commented Nov 23, 2016 at 13:36

phobia82 · Accepted Answer · 2016-11-23 13:37:48Z

1

Your MAIN issue is that you are declaring functions inside a loop, the second time it tries to declare the functions it will throw an error that the functions already exists. Move you functions declaration outside the loop

http://php.net/manual/en/functions.user-defined.php

answered Nov 23, 2016 at 13:37

phobia82

1,2578 silver badges10 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Henk Rensenbrink Over a year ago

Thanks, I know it has a couple of more problems but this indeed solved my issue. Thanks alot.

Álvaro González Over a year ago

@HenkRensenbrink If you didn't see that, your PHP development box is configured to hide error messages and other useful information. Please take care of that!

Collectives™ on Stack Overflow

Email is only send once PHP / MySQL

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related