Community Addon -> YCom

[TobiasKrais]

YCom Anpassungen

[tbaddade]

Danke für den PR. Wenn das Methoden oder Funktionen sind, wäre es gut das die Regex genauer wäre. Ansonsten passiert es, dass community auch im Fließtext mit ycom ersetzt wird.

[TobiasKrais]

• checkperm ist eine Methode.
• community lediglich der Addon Name. Die Ersetzung finde ich auch potentiell riskant.
• rex_com_auth ist der Klassenname.

[TobiasKrais]

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]

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]

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

[tbaddade]

Ja, perfekt. Das is es.

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

[TobiasKrais]

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

[gharlan]

' wird nicht richtig escaped.. #26