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.

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

Abstract single plugin activation to seperate method #375

Merged
merged 2 commits into from
Apr 28, 2015

Conversation

jrfnl
Copy link
Contributor

@jrfnl jrfnl commented Apr 28, 2015

No description provided.

@jrfnl jrfnl added this to the 2.5.0 milestone Apr 28, 2015

} else {
echo '<p>', esc_html( $this->strings['plugin_activated'] ), '</p>';
if( $this->activate_single_plugin( $plugin_activate, $slug, true ) === true ) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yoda conditions please.

- reverse function true/false logic - now more logical from a function point of view, but completely incomprehensible on the receiving end of the function.
- darn yoda (Seriously, if you use strict checking - as I do -, why the heck would I need it ? makes code so much less readable...)
@jrfnl jrfnl force-pushed the feature/abstract-single-plugin-activation branch from 84df8f9 to 9d3f66a Compare April 28, 2015 18:59
@jrfnl
Copy link
Contributor Author

jrfnl commented Apr 28, 2015

Updated & rebased

GaryJones added a commit that referenced this pull request Apr 28, 2015
…ugin-activation

Abstract single plugin activation to seperate method
@GaryJones GaryJones merged commit 4e76cdd into develop Apr 28, 2015
@GaryJones GaryJones deleted the feature/abstract-single-plugin-activation branch April 28, 2015 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants