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

CRM-21812 - don't specify installation type post install #125

Merged
merged 1 commit into from
Jul 19, 2018

Conversation

MegaphoneJon
Copy link
Contributor

@MegaphoneJon MegaphoneJon commented Feb 28, 2018

The wp-cli-login-server plugin checks for the presence of elements in the $_GET array, and fails if any are found. This conflicts with CiviCRM, which places:

$_GET['civicrm_install_type'] = 'wordpress';

into every page load.

I confirmed that this value is only used for install, and placed a conditional around it to only insert pre-install. I successfully installed CiviCRM and engaged in normal use with the conditional applied, and confirmed that this fixed the WP plugin.

This is intended to close aaemnnosttv/wp-cli-login-command#24.


@christianwach
Copy link
Member

FWIW, this is also fixed in #123 as can be seen here.

@kcristiano
Copy link
Member

I prefer #123 in favor of this PR. They solve the same issue, but #123 has other benefits as well.

@kcristiano
Copy link
Member

@christianwach as #123 needs more testing I think we should merge this now and rebase #123 Any objections?

@christianwach
Copy link
Member

@kcristiano No objections here

@MegaphoneJon
Copy link
Contributor Author

@eileenmcnaughton This has been reviewed by @christianwach and @kcristiano, but I don't think any of us have merge rights. Are you able to merge this for us? Thanks!

@eileenmcnaughton
Copy link
Contributor

Yes

@eileenmcnaughton eileenmcnaughton merged commit fd6ba3e into civicrm:master Jul 19, 2018
@MegaphoneJon MegaphoneJon deleted the CRM-21812 branch August 23, 2018 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Plugin conflict with CiviCRM
5 participants