Skip to:
Content

BuddyPress.org

Opened 8 months ago

Closed 4 months ago

#8308 closed enhancement (fixed)

2.0 - BP CLI

Reported by: espellcaste Owned by: espellcaste
Milestone: 7.0.0 Priority: normal
Severity: normal Version:
Component: Core Keywords: has-patch commit
Cc:

Description

The 2.0 version of the WP BP CLI is almost 100%. https://github.com/buddypress/wp-cli-buddypress

This update was on my list of TO-DO for quite some time. It comes with several changes: https://github.com/buddypress/wp-cli-buddypress/blob/master/CHANGELOG.md

I'm creating this ticket to add a few changes to BuddyPress core to better work together with the package.

First: we need to update BuddyPress trunk with the master version of the WP BP CLI package. This will allow people using the BP Beta Tester plugin to use the latest version of the CLI. When we decide to actually launch we change for the 2.0 tag release.

Second: I'd like to add a conditional check in the plugin to make sure the package from the plugin doesn't conflict with the one installed with wp package install buddypress.

This is for people that installs the package via the cli. Right now, installing via the package fetches from master and is always the latest. But due to the class conflict, one can't use the latest if desired.

Patch incoming. :)

Attachments (2)

8308.diff (1.2 KB) - added by espellcaste 6 months ago.
8308-2.diff (1.2 KB) - added by espellcaste 6 months ago.

Download all attachments as: .zip

Change History (10)

#1 @espellcaste
8 months ago

  • Owner set to espellcaste
  • Status changed from assigned to accepted

#2 @espellcaste
8 months ago

  • Status changed from accepted to assigned

@espellcaste
6 months ago

#3 @espellcaste
6 months ago

  • Keywords has-patch added; needs-patch removed

#4 @imath
6 months ago

Looks good! Thanks for your work on the patch and on the BP CLI commands.

I'd suggest to add a space before the last ) in:
! class_exists( 'Buddypress\CLI\Command\BuddypressCommand')

@espellcaste
6 months ago

#5 @espellcaste
6 months ago

@imath I'd love your review of the new patch.

#6 @imath
6 months ago

  • Keywords commit added

Let's have it in!

This ticket was mentioned in Slack in #buddypress by imath. View the logs.


6 months ago

#8 @espellcaste
4 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 12737:

Update the BP CLI package to the 2.0 version (https://github.com/buddypress/wp-cli-buddypress/releases/tag/2.0).

Also, add a conditional to avoid conflict with the BuddyPress plugin version. This makes sure
there is no conflict when the package is installed via the wp package install buddypress command.

Props imath

Fixes #8308

Note: See TracTickets for help on using tickets.