search

yakamara / yconverter

👨🏻‍🍳 YConverter - REDAXO Addon
25 stars 7 forks source link

Weitere Ersetzungen hinzugefügt #22

Closed TobiasKrais closed 7 years ago

TobiasKrais commented 7 years ago

Folgende Änderungen:

alxndr-w commented 7 years ago

Wie schaut's mit diesem PR aus?

tbaddade commented 7 years ago

Wie schaut's mit diesem PR aus?

Würden wir gern aufnehmen, nur müsste @TobiasKrais die OO Ersetzungen anpassen.

TobiasKrais commented 7 years ago

Welchen Teil mit den OO Ersetzungen meinst du? Den hier? ['OOArticle' => 'rex_article'], ['OOCategory' => 'rex_category'], ['OOMedia' => 'rex_media'], ['OOMediaCategory' => 'rex_media_category'],

Das hat so seine Richtigkeit. Nachdem nämlich alle Ersetzungen durchgelaufen sind, bleiben noch einige OO's übrig, z.B. so eine: $variable instance of OOArticle. Mit den obigen Ersetzungen wird auch damit aufgeräumt, aber erst nachdem alle anderen OO's schon verarbeitet sind. Funktioniert 1a.

tbaddade commented 7 years ago

Das hat so seine Richtigkeit. Nachdem nämlich alle Ersetzungen durchgelaufen sind, bleiben noch einige OO's übrig, z.B. so eine: $variable instance of OOArticle

Dann gehst du davon aus, dass ggf. andere Methoden oder Prüfungen auch in R5 vorhanden sind bzw. funktionieren. Ich würde das lieber spezieller halten, also nicht zu sehr verallgemeinern. Viele Regex hätte ich auch zusammenfassen können, was aber die Übersichtlichkeit erschwert hätte. Daher würde ich eher sowas notieren (.*?)\s*instanceof\s*OOArticle => $1 und deine anderen OOs zusätzlich aufnehmen. Welche weiteren OOs blieben bei dir noch übrig, außerinstanceof?

TobiasKrais commented 7 years ago

Das stimmt, außer instanceof ist mir in der Praxis nichts anderes unter gekommen. Den Code hab ich mal angepasst. Passt der neue Code?

tbaddade commented 7 years ago

Danke … Ich werde es später nur noch in der Liste verschieben.

TobiasKrais commented 7 years ago

Danke!