Skip to content

Adding BackdropCommerceEntityController via hook_autoload_info()#41

Merged
elisseck merged 1 commit into
backdrop-contrib:1.x-1.xfrom
aheathgr:issues/40
Aug 9, 2025
Merged

Adding BackdropCommerceEntityController via hook_autoload_info()#41
elisseck merged 1 commit into
backdrop-contrib:1.x-1.xfrom
aheathgr:issues/40

Conversation

@aheathgr

@aheathgr aheathgr commented Jul 19, 2025

Copy link
Copy Markdown
Contributor

Fixes #40

This fixes an error we were getting when we enabled commerce_customer, Class "BackdropCommerceEntityController" not found in require_once() (line 12 of /var/www/backdrop.local/modules/commerce/modules/customer/includes/commerce_customer_profile.controller.inc).

We just needed to make sure we were including that class in hook_autoload_info() in the commerce_customer module.

Fixes backdrop-contrib#40

This fixes an error we were getting when we enabled commerce_customer,
Class "BackdropCommerceEntityController" not found in require_once()

We just needed to make sure we were including that class in
hook_autoload_info() in the commerce_customer module.
@elisseck elisseck merged commit 5132d3e into backdrop-contrib:1.x-1.x Aug 9, 2025
1 check failed
@argiepiano

Copy link
Copy Markdown
Contributor

This was merged despite the multiple test failures?

@elisseck

elisseck commented Aug 9, 2025

Copy link
Copy Markdown
Collaborator

@argiepiano couple of things going on here:

  1. Github actions is still spitting out wildly different results than my local env (php8.2, ubuntu 24.04, backdrop 1.31.0) so i'm not relying on that GHA report at all yet.
  2. Using my local as a baseline, the latest changes are running more tests than before and the failures are on newly running ones... so I actually think we've resolved some previous test exceptions and uncovered a new issue that we just couldn't see in the test runs before.

@argiepiano

Copy link
Copy Markdown
Contributor

@elisseck, regarding your comment about local tests passing, and the ones on GHA not: have you tried running your local tests with the Drupal compatibility layer off? Some of the failures may be related to deprecated Drupal functions, as GHA may be running the tests with the D layer off. Just spitballing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enabling commerce_customer triggers class error

3 participants