Witam,
nie wiedziałem w jakim dziale to umieścić także w razie czego proszę o przeniesienie.
Potrzebuje z pewnymi kanałami rss być na bieżąco dlatego też posiadam skrypt, który dany kanał parsuje i wstawia do bazy danych po czym wysyła maila z daną informacją na adres e-mail. Po kilku minutach sprawdza kanał i jeżeli jest nowy wpis (czyli nie ma go w bazie) wysyła maila i dodaje do bazy by nie wysyłać po raz kolejny tego samego.

Kłopot polega na tym, że na niektórych kanałach hula to fajnie, a na niektórych pewne wpisy nie są dodawane do bazy i cały czas są wysyłane.

Np. dzisiaj próbowałem kanał rss bash.org.pl i tylko jedna "news" nie został dodany do bazy i wysyłany jest non stop.

Co może być przyczyną i jak naprawić ?
Z góry dzięki.


Wykorzystywany parser to Simplepie.

Kod skryptu:

[PHP] pobierz, plaintext 
<?php
 
    header('Content-Type: text/html');
	require_once(dirname(__FILE__).'/config.php');
	require_once(dirname(__FILE__).'/simplepie.php');
 
	$connect = mysql_connect($dbhost,$dbuser,$dbpass) or die("Cannot Connect Database");
 
 
	if(!(mysql_select_db($dbbase))){
		echo "CANNOT SELECT DATABASE";
		die();
	}
    // Call SimplePie
	$feed = new SimplePie();
 
	$feed->set_feed_url($urls);
 
	$feed->enable_cache('false');
	$feed->set_cache_location($cachedir);
	$cachetime = (intval($cachetime) / 60); //convert from seconds to minutes
	$feed->set_cache_duration($cachetime);
 
	// Init feed
	$feed->init();
	// Make sure the page is being served with the UTF-8 headers.
	$feed->handle_content_type();
	$items = $feed->get_items();
 
	foreach($items as $item){
 
		$title = $item->get_title();
		$guid = md5($item->get_id());
		$desc = $item->get_description();
		$link = $item->get_link();
 
		// Check Row
		$query = mysql_query("SELECT * FROM rssgoemail WHERE guid='$guid'");
		$row = mysql_num_rows($query);
 
		// If row empty send email and happy blogging
		if( $row < 1){
 
			$mail = $desc."<br /><br /><a href=\"".$link."\" rel=\"nofollow\">Read More</a>";
 
	        $send = mail($email, $title, $mail, "From: {$title}");	
 
			echo "Send ".$title."<br />";
 
			if($send){
				mysql_query("INSERT INTO rssgoemail(title,guid,description) VALUES ('$title','$guid','$desc')");
			}
		}else{
			continue;
		}
 
	}?>
[PHP] pobierz, plaintext 

Skoro wpisy są wysyłanie, ale nie są dodawane do bazy to przyczyna prawdopodobnie leży tu:

[PHP] pobierz, plaintext 
 $send = mail($email, $title, $mail, "From: {$title}");	
 //...
if($send){
mysql_query("INSERT INTO rssgoemail(title,guid,description) VALUES ('$title','$guid','$desc')");
}
[PHP] pobierz, plaintext 

Przynajmniej ja bym od tego zaczął. Jesteś pewny, że do funkcji mail przesyłasz wszystkie potrzebne argumenty odpowiednio?

Chyba tak. Jak wspominalem wcześniej np. z kanału rss strony bash.org.pl wszystko zostało zrobione oprócz jednego news'a, który nie wiem czemu nie został dodany i był wysyłany ciągle.

Być może to był błąd po ich stronie i np. news miał taką samą treść, ale inne id, co by prowadziło do generowania różnych skrótów MD5. Dodaj tam sobie jakiś mini system logów i następnym razem, w takim przypadku będziesz wiedział co poszło nie tak.

To chyba nie błąd tej samej treści a innego id. Podejrzewam, że to przez niedozwolone znaki w tytule newsów. Czy to możliwe, że przez znaki typu ' lub / rzeczy nie są dodawane do bazy? Jeżeli tak to jak to ominąć ?

Przepraszam jeżeli pytania są zbyt banalne lecz na php znam się jak prawiczek na sexie.

No właśnie kolego. Problemem jest to, jak tworzysz zapytanie do bazy:

[PHP] pobierz, plaintext 
"INSERT INTO rssgoemail(title,guid,description) VALUES ('$title','$guid','$desc')"
[PHP] pobierz, plaintext 

Zauważ, że jeśli w tytule/opisie będziesz miał znak ' to zapytanie się rozwali. Musisz się przed tym zabezpieczyć.
Uniwersalnym rozwiązaniem będzie zastosowanie mysql_real_escape_string, który wszystkie niebezpieczne znaki odpowiednio "przerobi".

W takim razie o co trzeba uzupełnić skrypt by wysłać prawidłowe zapytanie do bazy i przy kolejny sprawdzeniu było widać, że taki wpis już istnieje i nie trzeba wysyłać maila?

Ahh.. ok.

[PHP] pobierz, plaintext 
<?php
 
    header('Content-Type: text/html');
	require_once(dirname(__FILE__).'/config.php');
	require_once(dirname(__FILE__).'/simplepie.php');
 
	$connect = mysql_connect($dbhost,$dbuser,$dbpass) or die("Cannot Connect Database");
 
 
	if(!(mysql_select_db($dbbase))){
		echo "CANNOT SELECT DATABASE";
		die();
	}
    // Call SimplePie
	$feed = new SimplePie();
 
	$feed->set_feed_url($urls);
 
	$feed->enable_cache('false');
	$feed->set_cache_location($cachedir);
	$cachetime = (intval($cachetime) / 60); //convert from seconds to minutes
	$feed->set_cache_duration($cachetime);
 
	// Init feed
	$feed->init();
	// Make sure the page is being served with the UTF-8 headers.
	$feed->handle_content_type();
	$items = $feed->get_items();
 
	foreach($items as $item){
 
		$title = $item->get_title();
		$guid = md5($item->get_id());
		$desc = $item->get_description();
		$link = $item->get_link();
 
		// Check Row
		$query = mysql_query("SELECT * FROM rssgoemail WHERE guid='$guid'");
		$row = mysql_num_rows($query);
 
		// If row empty send email and happy blogging
		if( $row < 1){
 
			$mail = $desc."<br /><br /><a href=\"".$link."\" rel=\"nofollow\">Read More</a>";
 
	        $send = mail($email, $title, $mail, "From: {$title}");	
 
			echo "Send ".$title."<br />";
 
			if($send){
				$title = mysql_real_escape_string( $title );
				$guid = mysql_real_escape_string( $guid );
				$desc = mysql_real_escape_string( $desc );
 
				mysql_query("INSERT INTO rssgoemail(title,guid,description) VALUES ('$title','$guid','$desc')");
			}
		}else{
			continue;
		}
 
	}?>
[PHP] pobierz, plaintext 

Sprawdzanie, czy został wpis istnieje już jest. Po prostu, nie działało, gdyż jest oparte na bazie. Skoro wpisów Ci pewnych nie dodawał...

Smiga aż miło. Dziękuję bardzo za pomoc. Pozdrawiam