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

Remaining issues with CC + Woo activation logic #176

Open
3 tasks
nylen opened this issue Jan 13, 2020 · 2 comments
Open
3 tasks

Remaining issues with CC + Woo activation logic #176

nylen opened this issue Jan 13, 2020 · 2 comments
Labels
Discussion enhancement New feature or request Needs PR Issue to be fixed. Needs a PR.

Comments

@nylen
Copy link
Contributor

nylen commented Jan 13, 2020

After #173 and #175 Classic Commerce should be able to prevent a user from causing fatal errors by trying to activate both CC and Woo at the same time from within the normal UI, on a single-site installation. There are some special cases that are not yet covered:

  • If a user is activating and deactivating plugins some other way (e.g. using WP-CLI or a management plugin), they will still be able to cause fatal errors by trying to activate both CC and Woo at the same time.
  • I have not tested this activation logic on multisite installations but I suspect users can also cause fatal errors there. Remember that on multisite a plugin can be network activated or only activated for a single site, with different code paths for each case, which means there are more possible cases to test.
  • There may still be other ways to break this.

I am not sure if these issues are a high enough priority to fix in the code (I am thinking probably not for version 1 unless we find other things that still don't work correctly). However these caveats do at least need to be documented for users, and this documentation needs to be included in release posts or linked from there so that people who want to try out CC will see it.

@timbocode
Copy link
Contributor

You would probably break it by manually tweaking the database and adding entries for both woocommerce/woocommerce.php and classic-commerce/classic-commerce.php in active_plugins in the cp_options table but only an idiot would do that.

But you're right. There are bound to be some other cases where it will break but hopefully these will be discovered during alpha and beta testing.

When we announce the release of alpha3, we should specifically ask for testers.

@timbocode timbocode added this to the 1.0.0-beta1 milestone Jan 17, 2020
@bahiirwa
Copy link
Collaborator

bahiirwa commented May 4, 2020

I have done some edits to @nylen's first comment for us to track the issue since it is 1.0.0-beta1

@timbocode timbocode removed this from the 1.0.0-beta1 milestone May 16, 2020
@bahiirwa bahiirwa added Discussion enhancement New feature or request Needs PR Issue to be fixed. Needs a PR. labels Oct 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion enhancement New feature or request Needs PR Issue to be fixed. Needs a PR.
Projects
None yet
Development

No branches or pull requests

3 participants