Closed christophboecker closed 3 weeks ago
Meintest du domain
oder $lang
?
In der Rss.php? VSCode markiert $domain als nicht benutzt:
Ah, dort!
Ich denke, hier hatte ich die Variable als 2. Parameter vorgesehen, um die Einträge pro Domain ausgeben zu können. Schließlich ist Neues Multidomain-fähig, News können einzelnen Domains zugeordnet werden.
Der 2. Parameter ist also dafür reserviert, das auch in die Logik noch einzubauen.
OK, prima. Ich habe in der Schnittstelle der Methoden "string" als Datentyp für $domain gesetzt, ein fehlendes "rex_yform_manager_collection" nachgetragen und die TODOs entfernt.
Sync.php:
(Ja, ist noch in Arbeit.)
$this->getParam(...)
) vor die Schleife gezogen. Als Zusatznutzen meckert RexStan nicht mehr, wenn weiter unten$status
verwendet wird ("möglicherweise nicht initialisiert")REVIEW:
bzw.FIXME:
markiert. Da stimmt was nicht. Bitte mal prüfen.Publish.php:
'neues_entry_publish_error'
und'neues_entry_publish_success'
gehörenden .lang-Einträge werden als Template in sprintf genutzt. Der vorkommende Platzhalter%s
ist für Strings, abr tatsächlich wird eine Zahl eingefügt; Alle .lang-Dateien geändert auf%d
.Restful.php:
...[get][fields] haben ja den jeweiligen Klassennamen inkl. Namespace in ausgeschriebener Form. Daher habe ich die Texte in die ::class-Schreibweise geändert (
Entry::class` usw.)Rss.php:
$category_id && $category = Category::get($category_id)
schwer. Das hab ich aufgelöst.TODO:
-Vermerke, weil es einen Parameterdomain
gibt, der aber tatsächlich nicht benutzt wird. Ich kann nicht entscheiden, ob der weg kann. Wenn er bleibt muss der Typ anggeben werden.