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-21285 Buildkit Installs failing for WordPress #11102

Closed
wants to merge 1 commit into from

Conversation

kcristiano
Copy link
Member

@kcristiano kcristiano commented Oct 10, 2017

#11031 Updated the way we define frontend and backend urls for WP. Testing was done with manual installs before merge. Buildkit installs are failing for WP.

Currently manual installs are succedding from nightly tarballs. Manual upgrades are successful. Buildkit installs fail.

On Buildkit the backed url is defined as http://example.orgwp-admin instead of http://example.org/wp-admin

https://github.com/civicrm/civicrm-core/pull/11031/files#diff-1544096e8e60bfe9fe73eeca2844804eR66 is the likely culprit.


@kcristiano kcristiano changed the title CRM-21285 iCRMCRM-21285 Buildkit Installs failing for WordPress CRM-21285 CRMCRM-21285 Buildkit Installs failing for WordPress Oct 10, 2017
@kcristiano kcristiano changed the title CRM-21285 CRMCRM-21285 Buildkit Installs failing for WordPress CRM-21285 Buildkit Installs failing for WordPress Oct 10, 2017
@alifrumin
Copy link
Contributor

alifrumin commented Oct 10, 2017

I ran civibuild create wpcv47m --type wp-demo --patch https://github.com/civicrm/civicrm-core/pull/11102 --admin-pass admin --url http://wpcv47m.test --web-root ~/buildkit/build/wpcv47m with the most recent version of buildkit and was able to successfully create a wordpress site loaded with civicrm data.

testing urls

backend url http://wpcv47m.test/wp-admin/admin.php?page=CiviCRM works as it should but missing / between http://wpcv47m.test and civicrm on frontend pages ex: http://wpcv47m.testcivicrm/?page=CiviCRM&q=civicrm/profile/create&gid=2&reset=1

…UF_BASEURL may or may not have a trailing slash
@seamuslee001
Copy link
Contributor

Note this should get fixed by #11111

@kcristiano
Copy link
Member Author

closing in favor of the PR merge from 4.7.26-rc from @seamuslee001

@kcristiano kcristiano closed this Oct 11, 2017
@kcristiano
Copy link
Member Author

#11112 for branch 4.7.26-rc improves this

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.

4 participants