QA CopyRoleRights

Zitat von »stevieswebsite«

"Hab mal nen altes Plugin von mir ausgekramt.
Dieses erlaubt es dem User, die Rollen-Rechte einer Rolle auf eine andere zu übertragen, also zu kopieren.
Stand: 100% Fertig."

Repository: http://code.contentlion.de/Plugin+CopyRoleRights/

Kommentar von JuKu: "Leider hat dieses Plugin noch einiges, was dir Code-mäßig nicht gefällt."
- Also darf ich kritisch sein:

Klasse Plugin_CopyRoleRights_RoleRights
- Wenn __construct keinen Inhalt ist, kannst du die komplett rausnehmen
- Einrückungen passen nicht (immer unterschiedlich)
- was passiert wenn $from_id oder $to_id nicht numerisch sind? (SQL-Fehler)
- das \r\n macht nicht wirklich sind, es verlangsamt nur das Query - Kannst am besten rausnehmen
- Das mit dem $i kannst du leichter haben. Schau dir mal implode an.
- Was passiert, wenn keine Einträge in der Tabelle role_rights zu der from_id gefunden werden? (SQL-Fehler)
- Auch bei Events müssen die Namespaces eingehalten werden
- Die Fukntion copyRights könnte man auch statisch machen

Klasse Plugin_CopyRoleRights_RoleRightsPage
- Wofür brauchst du die Eigenschaft $guestbook?
- Einrückungen passen nicht
- PHPDoc wäre wünschenswert
- Urls mit der UrlRewriting-Klasse machen
- Was ist, wenn HTML-Code in einer Variable steht? (htmlentities)

Sonstiges
- Statt Bindestrichen Unterrstriche in Sprachvariablen nutzen
- Keine feste Menü-ID in der activate.php (kann abweichen, am besten mit select schauen)
- Wenn du schon Sprachdateien hast, nutze sie auch im Installer