Fix register activation hook

redbitcz / simpleshop-wp-plugin

SimpleShop Wordpress Plugin

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

Other

2 stars 1 forks source link

Fix register activation hook #26

Closed jakubboucek closed 5 years ago

jakubboucek commented 5 years ago

Opravuje chybu registrace pluginu po instalaci. Nyní se do volání metody register_activation_hook() předávala chybná cesta, viz popsis ve #24.
Zároveň odstraňuje nežádoucí konstanty a opravení DI v rámci objektů pluginu.

Closes #24

vaclavgreif commented 5 years ago

@jakubboucek Tomuhle moc nerozumím. Jaký je důvod pro to mít možnost definovat jiný pluginmainfile? Proč ho pak posíláme do třídy Admin, když už jí předáváme Plugin, který ho zná? Proč jsou vlastně konstanty špatné?..:-)

jakubboucek commented 5 years ago

Ohledně otázky (ne)správnosti konstant doporučuji ke shlédnutí: https://www.youtube.com/watch?v=ODcfsRpQ0Pw - ostatně celé toto PR se kolem DI točí.

Jaký je důvod pro to mít možnost definovat jiný pluginmainfile?

Nejde o to "definovat jiný", ale spíš tam dostat jeho definici. Můžeme použít /../, ale kdybychom ten soubor někdy přejmenovali, tak vznikle chyba. Takhle si plugin sám poznačí, jaké jeho main-file a nemůže se splést.

Proč ho pak posíláme do třídy Admin, když už jí předáváme Plugin, který ho zná?

To je celkem správná připomínka, upravím.

jakubboucek commented 5 years ago

Proč ho pak posíláme do třídy Admin, když už jí předáváme Plugin, který ho zná?

Upraveno.

vaclavgreif commented 5 years ago

@jakubboucek Paráda, jen viz komentář v Gutenberg pull requestu, možná bych předával rovnou ten pluginDirUrl.