Verfasst: 26.08.2004 00:03
Ich nehme dir gar nichts übel...., da der MODja nicht von mir stammt.Acid hat geschrieben:Ansonsten gute Arbeit..
- der Referer könnte noch verlinkt werden
- in der Übersicht der Logs solltest du eine Pagination einbauen, da die Ladezeit grösser wird, umso mehr Einträge vorhanden sind (v.a. dann bei Gästen)
- das in der "IP: Logs"-Ansicht z.B. bei den Gästen (wo ja alle Gäste unter "Anonymous" angezeigt werden) die zuletzt geloggte IP angezeigt wird, könnte etwas verwirrend sein (eventuell eine Notiz anbringen)
- "Anonymous" sollte nicht mit´m Profil verlinkt sein
- lange Referer bzw. Page Views sollten gekürzt werden (zumindest die Ansicht, Link nicht), da es sonst unnötigerweise Scrollbalken ergibt.
- eventuell noch gethostbyaddr() benutzen, auch wenn es nicht immer funktioniert (ebenso HTTP_USER_AGENT)
- teilweise werden die Schriften nicht formatiert (fehlende Angaben zu Schriftfarbe, -grösse oder -art (bzw. Styleklassen) merkt man erst, wenn ein anderes Template als subSilver benutzt wird)
- die letzte Änderung ($view_page) solltest du nochmal überprüfen, da dadurch nur noch der Dateiname angezeigt wird (nicht mehr der Ordner, was eventuell einen Unterschied macht)
- (später dann auch) Sprach-/Templateunterstützung/Löschen von bestimmten Einträgen
(Ich hoffe, du nimmst mir diese "Kritikpunkte" nicht übel.)
Zunächst einmal bin ich froh, dass ich mit meinen "relativ geringen" PHP
Kenntnissen, die aber immer besser werden, den MOD überhaupt so zum
Laufen gebracht habe.
Deine Anregungen sind alle GUT und RICHTIG....
Danke dafür, werde ich ALLES umsetzten.....dass ÜBT ungemein.
Leider muss ich morgen früh in´s Krankenhaus, Nierensteine, OP.
sobald ich wieder unter den Lebenden weile, werde ich mich an´s Werk
machen.
Also noch ein wenig Geduld.
Ach ja nochwas, wenn ich fragen darf:
Den Hinweis Verstehe ich nicht ganz, da es bei mir funktioniert..?die letzte Änderung ($view_page) solltest du nochmal überprüfen, da dadurch nur noch der Dateiname angezeigt wird (nicht mehr der Ordner, was eventuell einen Unterschied macht)
Peter