Automatisch Fotoalbum V3

Door Arjan Kapteijn op 15-11-2008 16:14

1.769 views

Aangezien je tegenwoordig blijkbaar aangeklaagd, of in ieder geval uitgescholden kan word omdat je een 2 en een half jaar oud script niet update... Hierbij een versie 3, vanmiddag in elkaar gezet 'from scratch' en dat koste helaas meer tijd dan ik eraan wou besteden.

http://test.i-cey.net/voorbeelden/i-cey.net/?pagina=fotoalbum

Mensen nog op-en-aanmerkingen? Iemand die even de moeite wilt doen om de veiligheid te checken? Is het bijv. nog wenselijk om meer if/elsjes te gebruiken en dus bijv. dingen te gaan checken als het aanmaken van thumbs.

Loran DP

15-11-2008 16:50

Zet hem anders even opnieuw in de lib.

A Arjan Kapteijn

15-11-2008 16:57

Ik vraag in eerste instantie om op-en-aanmerkingen.

K Klaasjan Boven

15-11-2008 17:56

Mischien nog file_exists toevoegen hier
if(isset($_GET['afbeelding']) && !empty($_GET['afbeelding']))
en dat geldt ook voor
if(isset($_GET['album']) && !empty($_GET['album']))

Ik heb hem niet helemaal doorgenomen maar het ziet er zeer goed uit

http://test.i-cey.net/voorbeelden/i-cey.net/?pagina=fotoalbum&album=fotoalbum/blaat

wekt namelijk ook nu

D Danny K

15-11-2008 20:16

Zo even snel:
- $map een constante maken zodat $map in het script niet meer gewijzigd kan worden.
Beginners tegen zichzelf beschermen en gewoon duidelijkheid.
Je gebruikt hem ook in de functie thumbnail, het kan voor de beginners totaal onduidelijk zijn hoe het met de scope zit.

- Array met toegestane extensies (mime types) maken.

- Je chmod alles naar 777, is dit werkelijk nodig?

- Checken of extensie / mime type toegestaan kan dmv array wat ik hierboven zeg, dan hoef je niet een hele lange if te maken.

- Zoals je zelf al zegt; sommige dingen zou je beter met een extra ifje kunnen doen om een onverwachte aktie te voorkomen.

- Duidelijke var namen: $tmp -> $tmp2 of $map -> $map2 (persoonlijke voorkeur ;))

- Het gebruik van de functie getimagesize() kan je misschien beter vervangen door andere functies. getimagesize() kan onverwachte foutmeldingen veroorzaken.

Verder lijkt het redelijk veilig.

Reageren

Inloggen om te reageren