search

yakamara / yconverter

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

Community Addon -> YCom #25

Closed TobiasKrais closed 6 years ago

TobiasKrais commented 6 years ago

YCom Anpassungen

TobiasKrais commented 6 years ago

Noch zu der Ersetzung von 'community'. Hier ein Beispiel der alten Verwendung: OOAddon::isAvailable('community'). Das sollte zu rex_addon::get('ycom')->isAvailable() werden. Hast du eine gute Idee, wie man das absichern kann?

tbaddade commented 6 years ago

Hast du eine gute Idee, wie man das absichern kann?

Da wird es bereits spezieller. Ersetzt wird das bereits an dieser Stelle, aber eben noch mit dem alten Namen. https://github.com/yakamara/yconverter/blob/master/lib/YConverter/Converter.php#L96-L98

Teste mal das hier bitte:

['rex_addon::get\((["']{1})community$1\)->isActivated\(\)' => 'rex_addon::get(\'ycom\')->isActivated()'],
['rex_addon::get\((["']{1})community$1\)->isAvailable\(\)' => 'rex_addon::get(\'ycom\')->isAvailable()'],
['rex_addon::get\((["']{1})community$1\)->isInstalled\(\)' => 'rex_addon::get(\'ycom\')->isInstalled()'],

Ggf. müsste man eine Rename für AddOns noch mitbringen.

TobiasKrais commented 6 years ago

Ja, perfekt. Das is es. Danke für den Tipp!

tbaddade commented 6 years ago

Ja, perfekt. Das is es.

D.h. du kannst bestätigen, dass die obigen Zeilen korrekt ersetzt werden?

TobiasKrais commented 6 years ago

Ja, das wollte ich damit ausdrücken :-).