BuddyPress (BuddyBoss)/WooCommerce conflict with Order History

nrherron92 commented 1 year ago

Reproduction Steps

HS-202788
Enable BuddyPress or BuddyBoss Platform with LifterLMS and WooCommerce (not the integration)
Enable BuddyPress/Buddyboss integration with LifterLMS
Go into BuddyPress/BuddyBoss member page for your account->courses->order history

Expected Behavior

with order history enabled in the integration options it should show up

Actual Behavior

BuddyPress/BuddyBoss's order history end point results in a 404 page.
if you turn WooCommerce off, it works as expected and does not 404
WooCommerce and LifterLMS Orders work just fine going through My-Account->Order History or Student Dashboard->Order History
It only happens with both are on together and only through BuddyPress/BuddyBoss's member dashboard

Error Messages / Logs

Include any relevant error messages or log files


<!-- Paste error logs / backtraces below this line -->


### System and Environment Information

<details>
<summary>System Report</summary>

<!-- Paste your System Report between the three backticks below this line -->

Wordpress

Home Url: https://natie.myliftersite.com Site Url: https://natie.myliftersite.com Login Url: https://natie.myliftersite.com/wp-login.php Version: 6.0.3 Debug Mode: No Debug Log: No Debug Display: Yes Locale: en_US Multisite: No Page For Posts: Not Set Page On Front: Not Set Permalink Structure: /%postname%/ Show On Front: posts Wp Cron: Yes

Settings

Version: 7.0.0 Db Version: 7.0.0 Course Catalog: Course Catalog (#120) [https://natie.myliftersite.com/courses/] Membership Catalog: Membership Catalog (#121) [https://natie.myliftersite.com/memberships/] Student Dashboard: Dashboard (#123) [https://natie.myliftersite.com/dashboard/] Checkout Page: Purchase (#122) [https://natie.myliftersite.com/purchase/] Course Catalog Per Page: 9 Course Catalog Sorting: menu_order Membership Catalog Per Page: 9 Membership Catalog Sorting: menu_order Site Membership: Not Set Courses Endpoint: my-courses Edit Endpoint: edit-account Lost Password Endpoint: lost-password Vouchers Endpoint: redeem-voucher Autogenerate Username: no Password Strength Meter: no Minimum Password Strength: Terms Required: no Terms Page: Not Set Checkout Names: Checkout Address: Checkout Phone: Checkout Email Confirmation: no Open Registration: no Registration Names: Registration Address: Registration Phone: Registration Voucher: Registration Email Confirmation: no Account Names: Account Address: Account Phone: Account Email Confirmation: no Confirmation Endpoint: confirm-payment Force Ssl Checkout: no Country: US Currency: USD Currency Position: left Thousand Separator: , Decimal Separator: . Decimals: 2 Trim Zero Decimals: no Recurring Payments: yes Email From Address: team+trial@lifterlms.com Email From Name: My LifterLMS Trial Site Email Footer Text: Email Header Image: Cert Bg Width: Cert Bg Height: Cert Legacy Compat:

Constants

LLMS_REMOVE_ALL_DATA: undefined LLMS_REST_DISABLE: undefined LLMS_SITE_FEATURE_RECURRING_PAYMENTS: undefined LLMS_SITE_IS_CLONE: undefined

Gateways

Manual: Disabled Manual Logging: Manual Order: 1

Server

Mysql Version: 5.7.39 Php Curl: Yes Php Default Timezone: UTC Php Fsockopen: Yes Php Max Input Vars: 5000 Php Max Upload Size: 512 MB Php Memory Limit: 256M Php Post Max Size: 1024M Php Soap: Yes Php Suhosin: No Php Time Limt: 30 Php Version: 8.0.24 Software: Apache/2.4.54 (Unix) OpenSSL/1.1.1 Wp Memory Limit: 256M

Browser

HTTP USER AGENT: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/106.0.0.0 Safari/537.36

Theme

Name: Twenty Nineteen Version: 2.3 Themeuri: https://wordpress.org/themes/twentynineteen/ Authoruri: https://wordpress.org/ Template: Child Theme: No Llms Support: No

Plugins

BuddyPress: 10.4.0 LifterLMS: 7.0.0 User Switching: 1.7.0 WooCommerce: 7.0.0 WP Mail Logging: 1.10.4

Integrations

BbPress: No BuddyPress: Yes

Template Overrides



</details>

This issue has be recreated:
+ [x] Locally
+ [x] On a staging site
+ [x] On a production website
+ [x] With only LifterLMS and a default theme

### Browser, Device, and Operating System Information

+ Browser name and version
+ Operating System name and version
+ Device name and version (if applicable)

eri-trabiccolo commented 1 year ago

We're kind aware of this. I think we should document it better. See: https://github.com/gocodebox/lifterlms/pull/1816#issuecomment-949832180 and thomas' reply: https://github.com/gocodebox/lifterlms/pull/1816#issuecomment-949875402

The To Do state has to be read as "To Document" here :D

nrherron92 commented 1 year ago

Gotcha! <3

eri-trabiccolo commented 1 year ago

So we've found that changing the LifterLMS' dashboard endpoints doesn't fix the issue so, we need to investigate and possibly fix this.

thomasplevy commented 1 year ago

@eri-trabiccolo and @nrherron92 Can we agree that this is a backloggable issue with the workaround of not using the orders endpoint on the BP profile screen?

Or do we need to issue a fix for this immediately as that workaround is not acceptable?

nrherron92 commented 1 year ago

I agree. As the order history can absolutely be reached through the normal LifterLMS dashboard at any time, this is not urgent.

thomasplevy commented 1 year ago

Cool thanks... if the user doesn't accept or like it, we'll take a closer look sooner.

eri-trabiccolo commented 1 year ago

yes agreed <3

gocodebox / lifterlms