ETS - Film systeem

Omdat ik beter wil leren scripten met classes ben ik bezig met een groep trainingsscripts. Deze scripts zak ik maken onder de naam ETS - Elekys training scripts (ja, ik ben inspiratieloos).
Waneer ze iets kunnen doen zet ik ze online, als open source.
Met andere woorden: iedereen mag ze gebruiken!

Dus om deze reeks te starten heb ik het V 1.0 van mijn, met classes gescripte, Muziek systeem!
Je kan er nummers van Youtube opzetten, die je kan bekijken.
Het systeem heb ik vanochtend snel in elkaar gezet, en kan dus nog niet heelveel, maar dit ga ik verder uitbreiden.
Maartoch, om er even in te komen: ETS - Film systeem 1.0 is uit, en klaar om te downloaden!

Oja, ga alsjeblieft niet lopen zeiken over bugs e.d: ik ga ze er toch niet uihalen!
Het is open source, wees blij dat je het hebt!
Demo: http://eleky.nl/film/index.php

Download 1.0: http://s000.tinyupload.com/?file_id=04935538479334472245 (10/04)
Download 1.1: http://s000.tinyupload.com/?file_id=95763343328209669052 (11/04)
Download 1.2: http://s000.tinyupload.com/?file_id=05359388204959838453 (12/04)
Download 1.3: http://s000.tinyupload.com/?file_id=33513721314323016160 (13/04)
Download 1.4: http://s000.tinyupload.com/?file_id=02707308743378197132 (15/04)
Download 1.5: http://s000.tinyupload.com/?file_id=00775177048147298662 (22/04)

Updates
1.0
Eerste versie, met de standaard mogelijkheden: bekijken, inlogsysteem, toevoegen indien ingelogd, zoveel keer bekeken, e.d. het begin zegmaar. Zit nog geen registratie bij!

1.1
Geen echte updates, code is gewoon verbetert met dank aan Rik!

1.2
Belangrijkste update is dat registreren is toegevoegd, dit geeft nu de mogelijkheid waarin iedereen zich kan registreren en nummers kan toevoegen aan de site.
Ook is er een stem gedeelte toegevoegd, dit kan alleen gebruikt worden door ingelogde leden!
Verder zit er nu een SQL code bij, en een Readme.txt voor de instalatie!

1.3
Er is een Admin paneel!
Eindelijk kan je zelf de videos bewerken, als je admin bent tenminste..
Een admin word nog niet uitzichzelf gemaakt, dus je moet zelf eerst registreren en dan via phpmyadmin je eigen status veranderen in Admin!
Ook is er nu een lijst toegevoegd met alle nummers die gesorteerd kan worden op artiest en nummer. Een de nummers starten nu automatisch en herhalen zich

1.4
Veel kleine aanpassingen!
Om te beginnen kunnen gebruikers (die zijn ingelogd) bepalen of ze willen dat een nummer autostart en/of herhaalt!
Ook zit in het menu nu een geheel automatisch genre systeem die uitzichzelf alle genres eruithaalt, hier hoef je dus nooit iets aan aan te passen!
Verder zijn er aan View 2 hokjes toegevoegd: Meer van en Gerelateerde Videos.

1.5
Na veel problemen met de iFrames is ook deze versie er: 1.5!
Met als belangrijkste update het comment systeem!
Ingelogde leden kunnen nu reageren op hun favorite videos, hier is alleen nog geen Admin paneel voor!
Verder kan er bij de lijst nu gezogd worden op nummer of artiest, en kan je aangeven als een nummer niet werkt + admin

Dan nu even heel iets anders
Omdat ik vrij veel positieve reacties krijg in de richting van dit script wil ik verder gaan met andere scripts, die ook Open Source worden. alleen gaat hier vrij veel tijd inzitten!
Daarom is het vanaf heden voor alle online staande versies van ETS verplicht om de volgende code in de footer te hebben staan:

Script by <a href="http://www.eleky.nl">Eleky</a>

Dit hoeft alleen op de paginas waar het script ook opstaat!
Indien u dit niet wilt kunt u een licentie aanschaffen voor een kleine bijdrage. Neem contact op voor meer info

Okay paar dingetjes:
- NEVER EVER meer echoen in je classes. ALTIJD dingen laten returnen. (op misschien een aantal classen na).
- Waarom een class aanmaken voor 1 functie
- De manier waarop je opmaak.php gebruikt is echt not done.
- Juist de dingen die je vaak gebruikt, komen niet terug in een class / functie. Wat net de bedoeling is van een class / functie. een voorbeeld daarvan is:

if(!isset($_SESSION[login])){
    header("Location: index.php");
    exit;
}

- $sql = mysql_fetch_object($sql); is ook echt een NONO, gebruik altijd fetch_assoc aangezien dat sneller is (tenzij je echt weet wat je doet).
-Let ook op je indenting (tabs en spaties) die is namelijk niet overal gelijk.

En nee ik loop niet te zeiken over bugs, ik geef je opbouwende kritiek zodat je het de volgende keer beter kan doen

Bedankt voor de reactieIk zal er proberen!
Die opmaak.php is bedoelt voor als je er een hele layout achter zou willen, dan leek me dit handiger!

Verder zijn er inderdaad classes voor 1 functie, op het moment, maar daar komen er nog meer bij.

Ik gebruik altijd fetch_object, vind ik makkelijker. maar indien het blijkbaar slechter is zou ik me proberen anders aan te wennen
EDIT: ik zie het niet als zeiken over bugs hoor, vind zulke dingen juist fijn om te horen!
Maar dingen als: hij crasht als ik dit doe hoef ik niet te weten!

Heey rik, ik heb ff een paar vraagjes, maar anders bumb ik dus moet maar ff via pb:

Rik schreef:

- NEVER EVER meer echoen in je classes. ALTIJD dingen laten returnen. (op misschien een aantal classen na).

-Let ook op je indenting (tabs en spaties) die is namelijk niet overal gelijk.

Warneer zou je WEL moeten echooen?
En waar loopt dat verkeerd?

EDIT: shit, verkeerde knop... moest een pb zijn, excuses voor (ongewilde) bumb

In principe zou je nooit moeten echo-en.
Tenminste, je template parser moet zou dat officieel moeten doen.
Dus probeer eens iets als Template Power of Template Lite.

Of in een errorhandler moet je ook wel echoen

Of gewoon die-en ofzo..

De opmerkingen van Rik zijn doorgevoerd!
Dank hiervoor!
Nieuwe link + demo staan in startpost!

Ik zoe dat je ook kunt stemmen maar daarvoor moet je inloggen? Miss handig als je ook kunt registreren?

Sorry voor dubbelpost hij bleef vasthangen

Ben nog bezig met Stem systeem, dus registreren doe ik bewust pas later, omdat anders iedereen gaat lopen focken met het niet werkende stem systeem!

Stem systeem werkt nu!
Ga morgen er ff een registreer ding inzetten zodat iedereen ff kan proberen!

Stem systeem werkt nu!
Ga morgen er ff een registreer ding inzetten zodat iedereen ff kan proberen!

EDIT: oeps, dubbelpost.....

Wou net ook even die v1.1 proberen maar er is een DB bij

OjaZou die bij de volgende versie doen!

Veel dubbelposting heaahah
ik ga het ff bekijken etc