[php] Message pop-up doet het niet meer in sendmail scriptje

[Bombelman]

Hi allemaal

Ik heb een pagina (link onderaan) die niet meer doet wat het wel deed een poosje geleden.
Het is een sendmail script die foto's uit een 4images gallery naar een ingevoerd e-mail
adres moet sturen, en na het versturen moet er een pop-up komen met de tekst zoals die
op regel 65... Helaas werkt deze pop-up niet meer.

Het script werkt verder wel, de foto wordt verzonden.

Heeft iemand enig idee waar dit aan kan liggen ?

mijn dank is groot !!

Regel 65: $returntext = "Hey ! Your requested picture was just mailed to you, check your e-mail promptly !";

PHP:

$main_template = 'sendemail';
define('GET_CACHES', 1);
define('ROOT_PATH', './');
include_once(ROOT_PATH.'global.php');
require(ROOT_PATH.'includes/sessions.php');
$user_access = get_permission();
include_once(ROOT_PATH.'includes/page_header.php');

$returntext = "";

if (!empty($_POST["txtname"]))
{ 
    include_once("../class/class.phpmailer.php");
    $returntext = "";
    
    //select the image data
    $sql = "SELECT i.image_id, i.image_name, i.cat_id, i.image_media_file, c.cat_name
        FROM (".IMAGES_TABLE." i,  ".CATEGORIES_TABLE." c)
        WHERE i.image_id = $image_id AND c.cat_id = i.cat_id";
    $image = $site_db->query_firstrow($sql);
    
    if ($image['image_id'])
    { 
        //check if the user already pass the 3 images
        $sql = "SELECT COUNT(*) AS c FROM 4images_sendemail WHERE email = '".$_POST["txtemail"]."' AND catid = ".$image['cat_id'];
        $useremail = $site_db->query_firstrow($sql);
        //$useremail = 1;
        if ($useremail['c'] < 6)
        //if ($useremail < 6)
        {
            $path = "/home/partypix/public_html/photos/data/media/".$image['cat_id']."/digicel/".$image['image_media_file'];
            if (file_exists($path))
            {
                $mail = new PHPMailer();
                
                $mail->From      = "[e-mail]";
                $mail->FromName  = "partypix Suriname";
                $mail->Mailer    = "smtp";
            
                $mail->IsHTML(true);
                
                // HTML body
                $myFile = "partypixsuriname.html";
                $fh = fopen($myFile, 'r');
                $body = fread($fh, filesize($myFile));
                fclose($fh);
                
                $imgpath = "http://www.partypixsuriname.com/photos";
                $body = str_replace("[path]", $imgpath, $body);
                $body = str_replace("[name]", $_POST["txtname"], $body);
                $body = str_replace("[pic]", $image['image_name'], $body);
                $body = str_replace("[cat]", $image['cat_name'], $body);
                //echo $body;
                
                $mail->Subject = "Your picture from Partypix";
                $mail->Body    = $body;
                $mail->AddAddress($_POST["txtemail"], $_POST["txtname"]);
                
                $mail->AddAttachment($path, "YourPicture.jpg", "base64", "application/image"); 
                
                if(!$mail->Send())
                    $returntext = "There has been a mail error while sending to " . $_POST["txtname"];              
                else
                {
                    $returntext = "Hey ! Your requested picture was just mailed to you, check your e-mail promptly !";
                    //update download counter
                    $sql = "UPDATE ".IMAGES_TABLE." SET image_downloads=image_downloads+1 WHERE image_id=".$image['image_id'];
                    //echo $sql,"<br />";
                    $site_db->query($sql);
                    //keep the download history
                    $sql = "INSERT INTO 4images_sendemail (imageid, catid, name, email, phone, provider, date) 
                            VALUES (".$image['image_id'].", ".$image['cat_id'].", '".$_POST["txtname"]."', '".$_POST["txtemail"]."', '".$_POST["txtphone"]."', '".$_POST["provider"]."', NOW())";
                    //echo $sql,"<br />";
                    
                    $site_db->query($sql);
                }
                
                // Clear all addresses and attachments for next loop
                $mail->ClearAddresses();
                $mail->ClearAttachments();
            } else $returntext = "Sorry, Image not found";  
        }else $returntext = "Sorry, you already have 3 pictures sent from this category.";
    } else $returntext = "Sorry, No Image record anymore";
}

link naar site: *snip*

[ Voor 0% gewijzigd door Creepy op 28-07-2009 10:27 ]

[Vinnienerd]

Je doet ook niks met de variabele $returntekst...

[Anoniem: 311111]

PHP:

if($returntext){
 echo "<script language=\"JavaScript\">\n";
 echo "alert('".$returntext."');\n";
 echo "</script>\n";
}

Onderaan je script plaatsen

[ Voor 7% gewijzigd door Anoniem: 311111 op 24-07-2009 21:47 ]

[japaveh]

* japaveh ziet het zoveelste topic voorbij komen.

Kijk dit script ook eens goed na op SQL injection...

[Bombelman]

Thanks voor de reactie ! Het draait nog een oud script, dat ge-update moet worden, maar de site is zodanig aangepast, dat het updaten meer problemen met zich mee zou brengen dan het zou moeten verhelpen... In hoevere kun je me hiermee helpen ? Uiteraard, niet onvergoed

[Bombelman]

Overigens Dystized, het heeft gewerkt !!

Ik heb dan nog een laatste vraag hieraan gerelateerd:
Hoe kan ik na het verzenden laten verwezen weg van de sendemail pagina,
terug naar de foto-pagina zelf, dus :
*snip*
(of *snip* kan ook !)

Mijn dank is groot !!

[ Voor 22% gewijzigd door Creepy op 28-07-2009 10:30 ]

[Bombelman]

Vinnienerd schreef op vrijdag 24 juli 2009 @ 21:29:
Je doet ook niks met de variabele $returntekst...

Het script is veeeeeeeeeel langer. Hierboven lijk het door de php-tags dat dit alles is, maar het zou misschien te lang worden en het leek erop alsof daar niet veel relevants in stond voor dit topic.. Welliswaar kan ik wel de gehele php-pagina posten (?).

[Anoniem: 311111]

Bombelman schreef op zondag 26 juli 2009 @ 15:35:
Overigens Dystized, het heeft gewerkt !!

Ik heb dan nog een laatste vraag hieraan gerelateerd:
Hoe kan ik na het verzenden laten verwezen weg van de sendemail pagina,
terug naar de foto-pagina zelf, dus :
*snip*
(of *snip* kan ook !)

Mijn dank is groot !!

Ik zou wel eens willen proberen je script te updaten voor je...


Maar even on-topic:

Helemaal bovenaan je script (ook boven de HTML-tags!):

PHP:

ob_start();

Dan op de plek waarop je verwijzen wilt:

PHP:

Header('Location: /photos/details.php?id='.$id);
ob_end_flush();

Geen dank.

[ Voor 15% gewijzigd door Creepy op 28-07-2009 10:30 ]

[Bombelman]

Helaas, dit laatste wil niet werken...

Maar kan ook omdat ik niet precies begrijp waar de plek is "waarop ik wil verwijzen"

Ik heb diverse plekken in het document geprobeerd maar het hielp niet.
Als ik de pagina opende, verweest het al gelijk verkeerd door naar een andere pagina...

Thanks !

[Creepy]

Succes met debuggen zou ik zeggen We verwachten van je dat je dat in eerste instantie zelf doet en dat je ook aangeeft wat je dan al hebt geprobeerd.

Zomaar je code dumpen en alleen aangeven wat het probleem is, is hiet niet de bedoeling. Het is hier geen afhaalbalie voor oplossing of een helpdesk. Je wilt zelfs voor een vergoeding iemand de boel laten fixen en dat is werving. Werving is op het forum hier niet toegestaan.

Dit staat allemaal netjes uitgelegd in Het algemeen beleid. Dus als je dat eens kan doorlezen, zoveel tekst is het nu ook weer niet

[ Voor 15% gewijzigd door Creepy op 28-07-2009 10:29 ]