[PHP-MySQL] $_GET['categorie'] maar ook degene met spaties!

[Goochem]

Ik heb het volgende stukje code:

PHP:

<?php

mysql_connect() or die (mysql_error());
mysql_select_db('database') or die (mysql_error());

$subs = mysql_query("SELECT DISTINCT sub FROM products ORDER BY sub ASC") or die (mysql_error());

while ($subrow = mysql_fetch_array($subs)) {
?>
<a href=<?=$_SERVER['PHP_SELF']?>?sub=<? print $subrow['sub']; ?>><?=$subrow['sub']?></a>
<?="<BR>"; }?>

?>

Output van dit geintje is een hele lijst met alle sub-categorieën in de database, bijvoorbeeld Monitoren, Harddisks, Mainboards, etc.

Deze sub-categorieën zijn per stuk allemaal een link zoals bijvoorbeeld pagina.php?sub=HARDDISKS.

Het probleem is nu dat er categorieën bij zitten die bestaan uit meerdere woorden, bijvoorbeeld SOFTWARE BACKUP of MAINBOARDS AMD, daarvan wordt alleen het eerste woord (bijvoorbeeld SOFTWARE) in de link weergegeven (pagina.php?sub=SOFTWARE ipv pagina.php?sub=SOFTWARE%20BACKUP.. (de spatie pakt ie dus niet mee, %20 hoort daar te komen, niet?)....

Iemand een idee hoe dit aan te pakken?

[[ti]]

php.net/urlencode

[dennistd]

Kun je de database niet aanpassen dat je bijvoorbeeld SOFTWARE_BACKUP krijgt?

[BtM909]

http://nl3.php.net/manual/en/function.urlencode.php

Should do the trick

[momania]

Je gebruikt hier presentatie waardes ook weer als input voor de rest van je programmatuur. Daar zit je fout

Sla dus in je products tabel apart een presentatie naam op en een naam om verder mee te werken in je programmatuur. Heb je ook nooit last van spaties

[vriesdude]

momania schreef op donderdag 04 augustus 2005 @ 10:21:
Je gebruikt hier presentatie waardes ook weer als input voor de rest van je programmatuur. Daar zit je fout

Sla dus in je products tabel apart een presentatie naam op en een naam om verder mee te werken in je programmatuur. Heb je ook nooit last van spaties

Bij voorkeur zelf een Unique/Primary Key veld (int)

[Bosmonster]

vriesdude schreef op donderdag 04 augustus 2005 @ 10:28:
[...]

Bij voorkeur zelf een Unique/Primary Key veld (int)

Bij voorkeur een string ivm met zoekmachines en vriendelijke url's.

Dat je intern een int als PK gebruikt lijkt me logisch.

[Janoz]

Iemand een idee hoe dit aan te pakken?

Leer fatsoenlijk html. Het is gebruikelijk om aanhalingstekens om attributen heen te zetten.

[crisp]

Janoz schreef op donderdag 04 augustus 2005 @ 10:33:
[...]


Leer fatsoenlijk html. Het is gebruikelijk om aanhalingstekens om attributen heen te zetten.

Dat niet alleen; HTML-attributen moeten ook HTML-encoded zijn. Daarnaast moeten query-vars URL-encoded zijn. Om het nog leuker te maken: PHP_SELF kan ism multiviews ook rare tekens bevatten, en bij het gebruik van magic_quotes zijn quotes ook nog eens voorzien van slashes

Helemaal correct zou dit dus zijn:

PHP:

$url = (get_magic_quotes_gpc() ? stripslashes($_SERVER['PHP_SELF']) : $_SERVER['PHP_SELF']) . '?sub=' . urlencode($subrow['sub']);
echo '<a href="' . htmlspecialchars($url) . '">' . htmlspecialchars($subrow['sub']) . '</a><br>';

[supakeen]

crisp schreef op donderdag 04 augustus 2005 @ 10:42:
[...]

Dat niet alleen; HTML-attributen moeten ook HTML-encoded zijn. Daarnaast moeten query-vars URL-encoded zijn. Om het nog leuker te maken: PHP_SELF kan ism multiviews ook rare tekens bevatten, en bij het gebruik van magic_quotes zijn quotes ook nog eens voorzien van slashes

Helemaal correct zou dit dus zijn:

PHP:

$url = (get_magic_quotes_gpc() ? stripslashes($_SERVER['PHP_SELF']) : $_SERVER['PHP_SELF']) . '?sub=' . urlencode($subrow['sub']);
echo '<a href="' . htmlspecialchars($url) . '">' . htmlspecialchars($subrow['sub']) . '</a><br>';

Gebruik jij je if/else constructies altijd met die ? en :

[ Voor 11% gewijzigd door supakeen op 04-08-2005 10:46 ]

[NMe]

zmn schreef op donderdag 04 augustus 2005 @ 10:46:
Gebruik jij je if/else constructies altijd met die ? en :

offtopic:
Boeit dat? Wanneer het overzichtelijk kan blijven is het met die ternary operator een stuk korter om te tikken.

Maar nu graag weer ontopic.

[mklerx]

crisp schreef op donderdag 04 augustus 2005 @ 10:42:
[...]

Dat niet alleen; HTML-attributen moeten ook HTML-encoded zijn. Daarnaast moeten query-vars URL-encoded zijn. Om het nog leuker te maken: PHP_SELF kan ism multiviews ook rare tekens bevatten, en bij het gebruik van magic_quotes zijn quotes ook nog eens voorzien van slashes

Helemaal correct zou dit dus zijn:

PHP:

$url = (get_magic_quotes_gpc() ? stripslashes($_SERVER['PHP_SELF']) : $_SERVER['PHP_SELF']) . '?sub=' . urlencode($subrow['sub']);
echo '<a href="' . htmlspecialchars($url) . '">' . htmlspecialchars($subrow['sub']) . '</a><br>';

Volgens mij is die stripslashes niet nodig, omdat als magic_quotes_gpc aanstaat alleen de variabelen in de GET, POST en COOKIES (GPC) arrays ge'addslashed' zijn en niet de variabelen in $_SERVER en lokale variabelen.
Toch?

[crisp]

mklerx schreef op donderdag 04 augustus 2005 @ 20:59:
[...]


Volgens mij is die stripslashes niet nodig, omdat als magic_quotes_gpc aanstaat alleen de variabelen in de GET, POST en COOKIES (GPC) arrays ge'addslashed' zijn en niet de variabelen in $_SERVER en lokale variabelen.
Toch?

Nee, toch niet; dingen als $_SERVER['PHP_SELF'] en $_SERVER['HTTP_USER_AGENT'] enzo zijn ook voorzien van 'magic' quotes...

[Verwijderd]

Goochem schreef op donderdag 04 augustus 2005 @ 10:09:

Het probleem is nu dat er categorieën bij zitten die bestaan uit meerdere woorden, bijvoorbeeld SOFTWARE BACKUP of MAINBOARDS AMD, daarvan wordt alleen het eerste woord (bijvoorbeeld SOFTWARE) in de link weergegeven (pagina.php?sub=SOFTWARE ipv pagina.php?sub=SOFTWARE%20BACKUP.. (de spatie pakt ie dus niet mee, %20 hoort daar te komen, niet?)....

Iemand een idee hoe dit aan te pakken?

Deze code kan je helpen;

PHP:

if($tmp = explode(" ", $value)) {
    echo $tmp[0];
} else {
    echo $value;
}

Wees echter zelf zo verstandig om eens te kijken hoe je nette code kunt schrijven, in zowel HTML als PHP. Je switched nu bijvoorbeeld constant tussen HTML en PHP, totaal overbodig. Daarna kun je gaan kijken wat de mogelijkheden van PHP zijn. Succes!

[drm]

crisp:
Nee, toch niet; dingen als $_SERVER['PHP_SELF'] en $_SERVER['HTTP_USER_AGENT'] enzo zijn ook voorzien van 'magic' quotes...

En daarom:

PHP:

<? 
if ( get_magic_quotes_gpc () ) {
   die ( 'What twat has turned this option on? '
         . 'Find him, poke him with a stick and turn off this option' );
}
?>

magic quotes

[SchizoDuckie]

drm schreef op vrijdag 05 augustus 2005 @ 09:01:
[...]
En daarom:

PHP:

<? 
if ( get_magic_quotes_gpc () ) {
   die ( 'What twat has turned this option on? '
         . 'Find him, poke him with a stick and turn off this option' );
}
?>

magic quotes

vergeet niet even te melden om wélke optie het gaat in je foutmelding

[Bosmonster]

Waarom dan niet gewoon standaard in je script:

PHP:

set_magic_quotes_runtime (0);

Ben je ook overal vanaf.

[crisp]

Bosmonster schreef op vrijdag 05 augustus 2005 @ 14:06:
Waarom dan niet gewoon standaard in je script:

PHP:

set_magic_quotes_runtime (0);

Ben je ook overal vanaf.

nee, magic_quotes_runtime != magic_quotes_gpc

[drm]

SchizoDuckie:
vergeet niet even te melden om wélke optie het gaat in je foutmelding

you got the point

[NMe]

drm schreef op vrijdag 05 augustus 2005 @ 09:01:
En daarom:

PHP:

<? 
if ( get_magic_quotes_gpc () ) {
   die ( 'What twat has turned this option on? '
         . 'Find him, poke him with a stick and turn off this option' );
}
?>

magic quotes

offtopic:
Hmm, ik herinner me vaag iets dat een slim persoon ooit heeft gezegd over die()...wie was dat ook alweer?

edit:
Bah, drm is een vieze editert.

[ Voor 5% gewijzigd door NMe op 05-08-2005 22:28 ]

[drm]

-NMe-:

offtopic:
Hmm, ik herinner me vaag iets dat een slim persoon ooit heeft gezegd over die()...wie was dat ook alweer?

Hoezo