woocommerce/src
Ron Rennick cbfe4bd595
Merge pull request #26166 from Chouby/reserve_stock_filter
Add filter for the query for reserved stock
2020-08-10 16:42:04 -03:00
..
Admin Added handling to prevent namespace conflicts with included packages 2020-04-28 14:04:00 -07:00
Blocks Added handling to prevent namespace conflicts with included packages 2020-04-28 14:04:00 -07:00
Checkout/Helpers Merge pull request #26166 from Chouby/reserve_stock_filter 2020-08-10 16:42:04 -03:00
Internal Removed package tag from src 2020-08-05 14:23:50 -03:00
Proxies Removed package tag from src 2020-08-05 14:23:50 -03:00
Autoloader.php Removed package tag from src 2020-08-05 14:23:50 -03:00
Container.php Removed package tag from src 2020-08-05 14:23:50 -03:00
Packages.php Merge pull request #27239 from woocommerce/fix/package-tag 2020-08-07 13:37:24 -03:00
README.md Add documentation about the container and good coding practices. 2020-07-24 09:25:12 +02:00

README.md

WooCommerce src files

Table of contents

This directory is home to new WooCommerce class files under the Automattic\WooCommerce namespace using PSR-4 file naming. This is to take full advantage of autoloading.

Ideally, all the new code for WooCommerce should consist of classes following the PSR-4 naming and living in this directory, and the code in the includes directory should receive the minimum amount of changes required for bug fixing. This will not always be possible but that should be the rule of thumb.

A PSR-11 container is in place for registering and resolving the classes in this directory by using the dependency injection pattern. There are tools in place to interact with legacy code (and code outside the srcdirectory in general) in a way that makes it easy to write unit tests.

Installing Composer

Composer is used to generate autoload class-maps for the files here. The stable release of WooCommerce comes with the autoloader, however, if you're running a development version you'll need to use Composer.

If you don't have Composer installed, go and check how to install Composer and then continue here.

Updating the autoloader class maps

If you add a class to WooCommerce you need to run the following to ensure it's included in the autoloader class-maps:

composer dump-autoload

Installing packages

To install the packages WooCommerce requires, from the main directory run:

composer install

To update packages run:

composer update

The container

WooCommerce uses a PSR-11 compatible container for registering and resolving all the classes in this directory by using the dependency injection pattern. More specifically, we use the container from The PHP League; this is relevant when registering classes, but not when resolving them. The full class name of the container used is Automattic\WooCommerce\Container (it uses the PHP League's container under the hood).

Resolving a class means asking the container to provide an instance of the class (or interface). Registering a class means telling the container how the class should be resolved.

In principle, the container should be used to register and resolve all the classes in the src directory. The exception might be data-only classes that could be created the old way (using a plain new statement); but as a rule of thumb, the container should always be used.

There are two ways to resolve registered classes, depending on from where they are resolved:

  • Classes in the src directory specify their dependencies as constructor arguments, which are automatically supplied by the container when the class is resolved (this is called constructor injection).
  • For code in the includes directory there's a wc_get_container function that will return the container, then its get method can be used to resolve any class.

Resolving classes

There are two ways to resolve registered classes, depending on from where they need to be resolved:

1. Other classes in the src directory

When a class in the src directory depends on other one classes from the same directory, it should use constructor injection. This means specifying these dependencies as constructor arguments with appropriate type hints, and storing these in private variables, ready to be used when needed:

use TheService1Namespace\Service1;
use TheService2Namespace\Service2;

class TheClassWithDependencies {
    private $service1;

    private $service2;

    public function __construct( Service1Class $service1, Service2Class $service2 ) {
        $this->$service1 = $service1;
        $this->$service2 = $service2;
    }

    public function method_that_needs_service_1() {
        $this->service1->do_something();
    }
}

Whenever the container is about to resolve TheClassWithDependencies it will also resolve Service1Class and Service2Class and pass them as constructor arguments to the requested class. If these service classes have constructor arguments too then those will also be appropriately resolved recursively.

A "lazy" approach is also possible if needed: you can specify the container itself as a constructor argument (using \Psr\Container\ContainerInterface as type hint), and use its get method to obtain the required instance at the appropriate time:

use TheService1Namespace\Service1;

class TheClassWithDependencies {
    private $container;

    public function __construct( \Psr\Container\ContainerInterface $container ) {
        $this->$container = $container;
    }

    public function method_that_needs_service_1() {
        $this->container->get( Service1::class )->do_something();
    }
}

In general, however, constructor injection is preferred and the lazy approach should be used only when really necessary.

2. Code in the includes directory

When you need to use classes defined in the src directory from within legacy code in includes, use the wc_get_container function to get the instance of the container, then resolve the required class with get:

use TheService1Namespace\Service1;

function wc_function_that_needs_service_1() {
    $service = wc_get_container()->get( Service1::class );
    $service->do_something();
}

This is also the recommended approach when moving code from includes to src while keeping the existing entry points for the old code in place for compatibility.

Worth noting: the container will throw a ContainerException when receiving a request for resolving a class that hasn't been registered. All classes need to have been registered prior to being resolved.

Registering classes

For a class to be resolvable using the container, it needs to have been previously registered in the same container.

The Container class is "read-only", in that it has a get method to resolve classes but it doesn't have any method to register classes. Instead, class registration is done by using service providers. That's how the whole process would go when creating a new class:

First, create the class in the appropriate namespace (and thus in the matching folder), remember that the base namespace for the classes in the src directory is Atuomattic\WooCommerce. If the class depends on other classes from src, specify these dependencies as constructor arguments in detailed above.

Example of such a class:

namespace Automattic\WooCommerce\TheClassNamespace;
use Automattic\WooCommerce\TheDependencyNamespace\TheDependencyClass;


class TheClass {
    private $the_dependency;
    
    public function __construct( TheDependencyClass $dependency ) {
        $this->the_dependency = $dependency;
    }
            
}

Then, create a <class name>ServiceProvider class in the src/Internal/DependencyManagement/ServiceProviders folder (and thus in the appropriate namespace) as follows:

namespace Automattic\WooCommerce\Internal\DependencyManagement\ServiceProviders;

use Automattic\WooCommerce\Internal\DependencyManagement\AbstractServiceProvider;
use Automattic\WooCommerce\TheClassNamespace\TheClass;
use Automattic\WooCommerce\TheDependencyNamespace\TheDependencyClass;

class TheClassServiceProvider extends AbstractServiceProvider {

	protected $provides = array(
		TheClass::class
	);

	public function register() {
		$this->add( TheClass::class )->addArgument( TheDependencyClass::class );
	}
}

Last (but certainly not least, don't forget this step!), add the class name of the service provider to the $service_providers property in the Container class.

Worth noting:

  • In the example the service provider is used to register only one class, but service providers can be used to register a group of related classes. The $provides property must contain all the names of the classes that the provider can register.
  • The container will invoke the provider register method the first time any of the classes in $provides is resolved.
  • If you look at the service provider documentation you will see that classes are registered using this->getContainer()->add. WooCommerce's AbstractServiceProvider adds a utility add method itself that serves the same purpose.
  • You can use share instead of add to register single-instance classes (the class is instantiated only once and cached, so the same instance is returned every time the class is resolved).

If the class being registered has constructor arguments then the add (or share) method must be followed by as many addArguments calls as needed. WooCommerce's AbstractServiceProvider adds a utility add_with_auto_arguments method (and a sibling share_with_auto_arguments method) that uses reflection to figure out and register all the constructor arguments (which need to have type hints). Please have in mind the possible performance penalty incurred by the usage of reflection when using this helper method.

An alternative version of the service provider, which is used to register both the class and its dependency, and which takes advantage of add_with_auto_arguments, could be as follows:

namespace Automattic\WooCommerce\Internal\DependencyManagement\ServiceProviders;

use Automattic\WooCommerce\Internal\DependencyManagement\AbstractServiceProvider;
use Automattic\WooCommerce\TheClassNamespace\TheClass;
use Automattic\WooCommerce\TheDependencyNamespace\TheDependencyClass;

class TheClassServiceProvider extends AbstractServiceProvider {

	protected $provides = array(
		TheClass::class,
        TheDependencyClass::class
	);

	public function register() {
        $this->share( TheDependencyClass::class );
		$this->share_with_auto_arguments( ActionsProxy::class );
	}
}

Using concretes

By default, the add and share methods instruct the container to resolve the registered class by using new to create a new instance of the class. But these methods accept an optional $concrete argument that can be used to tell the container to resolve the class in a different way. $concrete may be one of the following:

  • A class name

The supplied class name will be instantiated when the registered class name is resolved. This is especially useful to register interfaces, example:

$this->add( TheInterface::class, TheClassImplementingTheInterface::class );
  • An object

The supplied object will be returned then the registerd class name is resolved. Example:

$instance = new TheClass();
$this->add( TheClass::class, $instance );
  • A closure

The closure will be executed and the result value will be returned when the registerd class name is resolved. Example:

$factory = function( TheDependencyClass $dependency ) {
    return new TheClass( $dependency );
};

$this->add( TheClass::class, $factory );

Note that if the closure is defined as a function with arguments, the supplied parameters will be resolved too.

A note on legacy classes

The container is intended for registering only classes in the src folder. There is a check in place to prevent classes outside the root Automattic\Woocommerce namespace from being registered.

This implies that classes outside src can't be constructor-injected, and thus must not be used as type hints in constructor arguments. There are mechanisms in place to interact with "outside" code (including code from the includes folder and third-party code) in a way that makes it easy to write unit tests.

The Internal namespace

While it's up to the developer to choose the appropriate namespaces for any newly created classes, and those namespaces should make sense from a semantic point of view, there's one namespace that has a special meaning: Automattic\WooCommerce\Internal.

Classes in Automattic\WooCommerce\Internal are meant to be WooCommerce infrastructure code that might change in future releases. In other words, for code inside that namespace, backwards compatibility of the public surface is not guaranteed: future releases might include breaking changes including renaming or renaming classes, renaming or removing public methods, or changing the signature of public methods. The code in this namespace is considered "internal", whereas all the other code in src is considered "public".

What this implies for you as developer depends on what type of contribution are you making:

  • If you are woking on WooCommerce core: When you need to add a new class please think carefully if the class could be useful for plugins. If you really think so, add it to the appropriate namespace rooted at Automattic\WooCommerce. If not, add it to the appropriate namespace but rooted at Automattic\WooCommerce\Internal.

    • When in doubt, always make the code internal. If an internal class is later deemed to be worth being made public, the change can be made easily (by just changing the class namespace) and nothing will break. Turning a public class into an internal class, on the other hand, is impossible since it could break existing plugins.
  • If you are a plugin developer: You should never use code from the Automattic\WooCommerce\Internal namespace in your plugins. Doing so might cause your plugin to break in future versions of WooCommerce.

Interacting with legacy code

Here by "legacy code" we refer mainly to the old WooCommerce code in the includes directory, but the mechanisms described in this section are useful for dealing with any code outside the src directory.

The code in the src directory can for sure interact directly with legacy code. A function needs to be called? Call it. You need an instance of an object? Instantiate it. The problem is that this makes the code difficult to test: it's not easy to mock functions (unless you use hacks, or objects that are instantiated directly with new or whose instance is retrieved via a TheClass::instance() method).

But we want the WooCommerce code base (and especially the code in src) to be well covered by unit tests, and so there are mechanisms in place to interact with legacy code while keeping the code testable.

The LegacyProxy class

LegacyProxy is a class that contains three public methods intended to allow interaction with legacy code:

  • get_instance_of: Retrieves an instance of a legacy (non-src) class.
  • call_function: Calls a standalone function.
  • call_static: Calls a static method in a class.

Whenever a src class needs to get an instance of a legacy class, or call a function, or call a static method from another class, and that would make the code difficult to test, it should use the LegacyProxy methods instead.

But how does using LegacyProxy help in making the code testable? The trick is that when tests run what is registered instead of LegacyProxy is an instance of MockableLegacyProxy, a class with the same public surface but with additional methods that allow to easily mock legacy classes, functions and static methods.

Using the legacy proxy

LegacyProxy is a class that is registered in the container as any other class, so an instance can be obtained by using constructor injection:

use Automattic\WooCommerce\Proxies\LegacyProxy;

class TheClass {
    private $legacy_proxy;

    public function __construct( LegacyProxy $legacy_proxy ) {
        $this->legacy_proxy = $legacy_proxy;            
    }

    public function do_something_using_some_function() {
        $this->legacy_proxy->call_function( 'the_function_name', 'param1', 'param2' );
    }
}

However, the recommended way (especially when no other dependencies need to be constructor-injected) is to use the equivalent methods in the WooCommerce class via the WC() helper, like this:

class TheClass {
    public function do_something_using_some_function() {
        WC()->call_function( 'the_function_name', 'param1', 'param2' );
    }
}

Both ways are completely equivalent since the helper methods are just doing wc_get_container()->get( LegacyProxy::class )->... under the hood.

Using the mockable proxy in tests

When unit tests run the container will return an instance of MockableLegacyProxy when LegacyProxy is resolved. This class has the same public methods as LegacyProxy but also the following ones:

  • register_class_mocks: defines mocks for classes that are retrieved via get_instance_of.
  • register_function_mocks: defines mocks for functions that are invoked via call_function.
  • register_static_mocks: defines mocks for functions that are invoked via call_static.

These methods could be accessed via wc_get_container()->get( LegacyProxy::class )->register... directly from the tests, but the preferred way is to use the equivalent helper methods offered by the WC_Unit_Test_Case class,: register_legacy_proxy_class_mocks, register_legacy_proxy_function_mocks and register_legacy_proxy_static_mocks.

Here's an example of how function mocks are defined:

// In this context '$this' is a class that extends WC_Unit_Test_Case

$this->register_legacy_proxy_function_mocks(
	array(
		'the_function_name' => function( $param1, $param2 ) {
			return "I'm the mock of the_function_name and I was invoked with $param1 and $param2.";
		},
	)
);

Of course, for the cases where no mocks are defined MockableLegacyProxy works the same way as LegacyProxy.

Please see the code of the MockableLegacyProxy class and its unit tests for more detailed usage instructions and examples.

But how does get_instance_of work?

We use a container to resolve instances of classes in the src directory, but how does the legacy proxy's get_instance_of know how to resolve legacy classes?

This is a mostly ad-hoc process. When a class has a special way to be instantiated or retrieved (e.g. a static instance method), then that is used; otherwise the method fallbacks to simply creating a new instance of the class using new.

This means that the get_instance_of method will most likely need to evolve over time to cover additional special cases. Take a look at the method code in LegacyProxy for details on how to properly make changes to the method.

Creating specialized proxies

While helpful to make the code testable, using the legacy proxy can make the code somewhat more difficult to read or maintain, so it should be used judiciously and only when really needed to make the code properly testable.

That said, an alternative middle ground would be to create more specialized cases for frequently used pieces of legacy code, for example:

class ActionsProxy {
	public function did_action( $tag ) {
		return did_action( $tag );
	}

	public function apply_filters( $tag, $value, ...$parameters ) {
		return apply_filters( $tag, $value, ...$parameters );
	}
}

Note however that such a class would have to be explicitly constructor-injected (unless additional helper methods are defined in the WooCommerce class), and that you would need to create a pairing mock class (e.g. MockableActionsProxy) and replace the original registration using wc_get_container()->replace( ActionsProxy::class, MockableActionsProxy::class ).

Defining new actions and filters

WordPress' hooks (actions and filters) are a very powerful extensibility mechanism and it's the core tool that allows WooCommerce extensions to be developer. However it has been often (ab)used in the WooCommerce core codebase to drive internal logic, e.g. an action is triggered from within one class or function with the assumption that somewhere there's some other class or function that will handle it and continue whatever processing is supposed to happen.

In order to keep the code as easy as reasonably possible to read and maintain, hooks shouldn't be used to drive WooCommerce's internal logic and processes. If you need the services of a given class or function, please call these directly (by using constructor injection or the legacy proxy as appropriate to get access to the desired service). New hooks should be introduced only if they provide a valuable extension point for plugins.

As usual, there might be reasonable exceptions to this; but please keep this rule in mind whenever you consider creating a new hook.

Writing unit tests

Unit tests are a fundamental tool to keep the code reliable and reasonably safe from regression errors. To that end, any new code added to the WooCommerce codebase, but especially to the src directory, should be reasonably covered by such tests.

If you are a WooCommerce core team member or a contributor from other team at Automattic: Please write unit tests to cover any code addition or modification that you make to the src directory (and ideally the same for the includes directory, by the way). There are always reasonable exceptions, but the rule of thumb is that all code should be covered by tests.

If you are an external contributor: When adding or changing code on the WooCommerce codebase, and especially in the src directory, adding unit tests is recommended but not mandatory: no contributions will be rejected solely for lacking unit tests. However, please try to at least make the code easily testable by honoring the container and constructor injection mechanism, and by using the legacy proxy to interact with legacy code when needed. If you do so, the WooCommerce team or other contributors will be able to add the missing tests.

Mocking dependencies

Since all the dependencies for classes in this directory are constructor-injected or retrieved lazily by directly accessing the container, it's easy to mock them by either manually creating a mock class with the same public surface or by using PHPUnit's test doubles:

$dependency_mock = somehow_create_mock();
$sut = new TheClassToTest( $dependency_mock ); //sut = System Under Test
$result = $sut->do_something();
$this->assertEquals( $result, 'the expected result' );

However, while this works well for simple scenarios, in the real world dependencies will often have other dependencies in turn, so instantiating all the required intermediate objects will be complex. To make things easier, while tests run the Container class is replaced with an ExtendedContainer class that has a couple of additional methods:

  • replace: allows defining a new replacement concrete for a given class registration.
  • reset_all_resolved: discards all the cached resolutions. You may need when mocking classes that have been defined as shared.

It's worth noting that at unit testing session bootstrap time reset_all_resolved is called once to reset any cached resolutions made during WC install, and replace is used to swap the LegacyProxy with a MockableLegacyProxy.

The same example using replace:

$dependency_mock = somehow_create_mock();
$container = wc_get_container();
$container->reset_all_resolved(); //if either the SUT or the dependency are shared
$container->replace( TheDependencyClass::class, $dependency_mock );
$sut = $container->get( TheClassToTest::class );
$result = $sut->do_something();
$this->assertEquals( $result, 'the expected result' );

Note: of course all of this applies to dependencies from the src directory, for mocking legacy dependencies the MockableLegacyProxy should be used instead.