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.
Zet hem anders even opnieuw in de lib.
Ik vraag in eerste instantie om op-en-aanmerkingen.
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