API und Cronjobs mit RexStan bearbeitet

[christophboecker]

Sync.php:

(Ja, ist noch in Arbeit.)

• nur einmal nötige Abrufe ($this->getParam(...)) vor die Schleife gezogen. Als Zusatznutzen meckert RexStan nicht mehr, wenn weiter unten $status verwendet wird ("möglicherweise nicht initialisiert")
• Mit REVIEW:bzw. FIXME: markiert. Da stimmt was nicht. Bitte mal prüfen.
• Die .lang-Einträge sind noch unvollständig

Publish.php:

• Die zu '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:

• Die Elemente ...[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:

• RexStan tut sich mit sowas wie $category_id && $category = Category::get($category_id)schwer. Das hab ich aufgelöst.
• Es gibt zwei TODO:-Vermerke, weil es einen Parameter domain gibt, der aber tatsächlich nicht benutzt wird. Ich kann nicht entscheiden, ob der weg kann. Wenn er bleibt muss der Typ anggeben werden.

[alxndr-w]

Meintest du domain oder $lang?

[christophboecker]

In der Rss.php? VSCode markiert $domain als nicht benutzt:

[alxndr-w]

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.

[christophboecker]

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.

[alxndr-w]

Ich denke, int $domain_id wäre sinnvoller, da sich der Domainname ändern kann.