[grinditwp]

Ursprungligen postat av BjörnJ

Det behöver inte vara en loop eftersom det är LIMIT 1. Man kan köra med if istället.

PHP-kod:

if($row = mysql_fetch_assoc($result))
  behandla existerande rad
else
  insert 


Som crazzy skrev skulle jag använda UNIQUE, men det är som sagt inget svar på varför det inte fungerar som det är nu.

Varför kör du med stripslashes() på det du läser från DB? Det ska normalt inte behövas.

Jag antar att innehållet i $url inte kan påverkas från klientsidan, eller att det är kontrollerat med regexp eller liknande. Om inte måste du köra mysql_real_escape_string() på den också, både vid SELECT och INSERT.

stripslashes har inte med $url att göra, $title dock, skrivs in i db med real_escape_string, men det är inte den som är problemet.

[BjörnJ]

Ursprungligen postat av grinditwp

stripslashes har inte med $url att göra, $title dock, skrivs in i db med real_escape_string, men det är inte den som är problemet.

Jag menade inte att stripslashes() har något med $url att göra. Det var olika kommentarer.

Man behöver inte använda stripslashes() när man hämtar från DB bara för att man använde mysql_real_escape_string() när man skrev till DB.

Däremot bör man i vissa fall använda htmlspecialchars() innan man skriver ut information som man hade lagrat i DB.

[grinditwp]

Ursprungligen postat av BjörnJ

Jag menade inte att stripslashes() har något med $url att göra. Det var olika kommentarer.

Man behöver inte använda stripslashes() när man hämtar från DB bara för att man använde mysql_real_escape_string() när man skrev till DB.

Däremot bör man i vissa fall använda htmlspecialchars() innan man skriver ut information som man hade lagrat i DB.

varför finns stripslashes då?

[BjörnJ]

Ursprungligen postat av grinditwp

varför finns stripslashes då?

För att ta bort de extra slasharna om man vill använda datat till något annat än att stoppa in i en databas. Det är väl egentligen bara aktuellt om magic quotes är aktiverat på servern, vilket jag tror var default för gamla PHP-versioner, men inte nu längre.