Social Media
Foren
QA CopyRoleRights
|
QA CopyRoleRights
"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
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
"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
Habe jetzt alles gleich behoben.
@guestbook:
Gar nicht, hatte damals nur die Klasse daraus kopiert und abgeändert.
@Sonstiges:
Habe ich schon in dem anderem Thread genannt. (--> changePassword)
@Feste ID:
Ok, habs gemacht.
|
- Die Url, die du oben mit @link angibst, kommt auf ne 404: http://contentlion.pf-control.de/copyrolerights.html
- "Allowes you to copy role rights." => "Allows you to copy the roles rights"
- "Allowes you to copy role rights." => "Allows you to copy the roles rights"
Ähnliche Themen
-
- QR Code Generator »
- QR-Code Generator in neuer Version (27. Januar 2013, 11:55)
-
- Projekt und Plugin Vorstellung »
- Advent, Advent ein Plugin Brennt (1. Dezember 2012, 19:46)