Kategoriesystem überprüft nur existens der Kategorie

Hal

Mir ist bei der erstellung eines Plugins gerade aufgefallen, dass das Kategoriesystem mittels getCategory aus der abstrakten Klasse AbstractDecoratedCategory nur prüft ob eine Kategorie existiert, nicht aber ob sie zum Plugin gehört.

z.B. beim Artikel hinzufügen:

$category = ArticleCategory::getCategory($this->categoryID);
if ($category === null) {
   throw new UserInputException('categoryID', 'invalid');
}

Wenn ich den Inhalt Manipuliere mit einer ID die nicht existiert, so erhalte ich die Ausgabe wcf.acp.article.category.error.invalid. Hier fehlt übrigens die sprachvariable.

Wenn ich nun aber eine ID aus einer Kategorie aus einem Plugin oder sonst wo nehme die existiert, so wird der Eintrag dann in die Datenbank geschrieben mit der "ungültigen" ID.

Gruss

Oliver

Das ja. Jedoch geht es mir eher darum, dass man durch einfache Manipulation einen falschen Wert in die Datenbank bekommt. WoltLab möchte ja auch das die Pluginentwickler prüfen, dass keine falsche ID in die Datenbank kommt. Dies finde ich auch sehr sinnvoll. Ich finde da müsste man auch schauen das unteranderem bei Artikeln oder anderen Orten nicht eine categorieID eingespeichert werden kann welche nicht zum Object der Artikel gehört.

Quote from Morik

Dafür bist doch am ende du verantwortlich, vor dem Speichern musst du eh überprüfen ob die Kategory zugänglich isAccessible() ist in der du den Artikel erstellen möchtest und genau dort wird auch der object type geprüft ?

Da sag ich doch auch nichts zu. Aber gut.

Dann muss ich es umformulieren das halt z.B. beim Artikel nicht überprüft wird ob die Kategorie für die Artikel existiert.

Ich habe ja nicht gesagt das es schlimm ist. Aber es ist durchaus etwas was man in einer weiteren Version korrigieren könnte.

Genau. Sorry. Ich gehe mal davon aus das dies nicht nur bei den Artikeln ist. Die Funktion getcategory wird ja mehrfach für die Validierung verwendet.

Das stimmt. Wie gesagt soll ja nur ein Hinweis sein was man ggf mal verbessern könnte. Es kann auch gerne zu den Vorschlägen verschoben werden.