Allow developer to external access to plugin instance

redbitcz / simpleshop-wp-plugin

SimpleShop Wordpress Plugin

https://www.simpleshop.cz/category/wordpress-plugin/

Other

2 stars 1 forks source link

Allow developer to external access to plugin instance #25

Closed jakubboucek closed 5 years ago

jakubboucek commented 5 years ago

Instead create new instance by new Loader() use new instance container Simpleshop::getInstance().
It makes container to prevent re-create it twice.
Developer still can force create new instance via argument $instanceId (Simpleshop::getInstance('test')).
Added back compatibility for case developer already using new Loader() access.

Closes #20

vaclavgreif commented 5 years ago

@jakubboucek Jaký je důvod pro to, aby někdo inicializoval novou instanci pluginu? Tím by se podle mě duplicitně zavolaly všechny hooky, které máme v konstruktorech jednotlivých metod, např. třídy Admin... Nebo to tak není?

jakubboucek commented 5 years ago

Jaký je důvod pro to, aby někdo inicializoval novou instanci pluginu?

Hm, původně jsem tomu pluginu dával větší izolaci a chtěl jsem to tam nechat, abych si nezavřel cestu pro případné testy. Ale pak jsem to zjednodušil a už to není třeba. Upravím.

jakubboucek commented 5 years ago

Jaký je důvod pro to, aby někdo inicializoval novou instanci pluginu?

Upraveno, už se tam neřeší multi-instancovost. V případě potřeby ji lze vždy vyrobit přes new Plugin().

vaclavgreif commented 5 years ago

@jakubboucek Určitě může být, je to podle mě to samé jako jsem měl já funkcí ssc(), jen obalené ve třídě, jestli to je tak lepší, jsem pro..:-)