Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix register activation hook #26

Merged
merged 2 commits into from
Oct 14, 2019
Merged

Fix register activation hook #26

merged 2 commits into from
Oct 14, 2019

Conversation

jakubboucek
Copy link
Member

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

Closes #24

@jakubboucek jakubboucek self-assigned this Oct 13, 2019
@jakubboucek jakubboucek mentioned this pull request Oct 13, 2019
Closed
@jakubboucek jakubboucek force-pushed the fix-activation-hook branch 2 times, most recently from c6871eb to 479d063 Compare October 13, 2019 16:02
jakubboucek added a commit that referenced this pull request Oct 13, 2019
@jakubboucek
Fix plugin constant based on #26
8536add
@vaclavgreif
Copy link
Collaborator

@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
Copy link
Member Author

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 jakubboucek force-pushed the plugin-public-interface branch from 9417b4f to 610988f Compare October 13, 2019 19:42
jakubboucek added a commit that referenced this pull request Oct 13, 2019
@jakubboucek
Fix plugin constant based on #26
7754542
@jakubboucek
Copy link
Member Author

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

Upraveno.

@jakubboucek jakubboucek force-pushed the plugin-public-interface branch from 610988f to 2e7631c Compare October 14, 2019 03:46
jakubboucek added a commit that referenced this pull request Oct 14, 2019
@jakubboucek
Fix plugin constant based on #26
cc72971
@jakubboucek jakubboucek added this to the v2.1.0 milestone Oct 14, 2019
@vaclavgreif
Copy link
Collaborator

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

@jakubboucek jakubboucek changed the base branch from plugin-public-interface to master October 14, 2019 17:26
@jakubboucek jakubboucek merged commit f64dd6f into master Oct 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vaclavgreif vaclavgreif vaclavgreif approved these changes

Assignees

@jakubboucek jakubboucek

Labels
None yet
Milestone
v2.1.0
Development

Successfully merging this pull request may close these issues.

Nefukční register activation hook
2 participants
@jakubboucek @vaclavgreif