Merge pull request #100 from woocommerce/PCP-72-error-on-plugin-activ…

…ation-with-i

Update to 0.2 of `dhii\module-interface` dep

composer.json

@@ -4,7 +4,7 @@
     "description": "PayPal Commerce Platform for WooCommerce",
     "license": "GPL-2.0",
     "require": {
-        "dhii/module-interface": "0.1",
+        "dhii/module-interface": "^0.2",
         "psr/container": "1.0.0",
         "container-interop/service-provider": "^0.4.0",
         "dhii/containers": "v0.1.0-alpha1",

modules/ppcp-admin-notices/src/class-adminnotices.php

@@ -36,7 +36,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface $container The container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 		add_action(
 			'admin_notices',
 			function() use ( $container ) {

modules/ppcp-api-client/src/class-apimodule.php

@@ -37,7 +37,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface $container The container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 	}
 
 	/**

modules/ppcp-button/src/class-buttonmodule.php

@@ -43,7 +43,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface|null $container The Container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 
 		add_action(
 			'wp',

modules/ppcp-onboarding/src/class-onboardingmodule.php

@@ -40,7 +40,8 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface|null $container The container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
+
 		$asset_loader = $container->get( 'onboarding.assets' );
 		/**
 		 * The OnboardingAssets.

modules/ppcp-session/src/class-sessionmodule.php

@@ -37,7 +37,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface|null $container The container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 		add_action(
 			'woocommerce_init',
 			function () use ( $container ) {

modules/ppcp-subscription/src/class-subscriptionmodule.php

@@ -44,7 +44,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface|null $container The container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 		add_action(
 			'woocommerce_scheduled_subscription_payment_' . PayPalGateway::ID,
 			function ( $amount, $order ) use ( $container ) {

modules/ppcp-wc-gateway/src/class-wcgatewaymodule.php

@@ -53,7 +53,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface|null $container The container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 		$this->register_payment_gateways( $container );
 		$this->register_order_functionality( $container );
 		$this->register_columns( $container );

modules/ppcp-webhooks/src/class-webhookmodule.php

@@ -36,7 +36,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface|null $container The Container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 		add_action(
 			'rest_api_init',
 			static function () use ( $container ) {

modules/woocommerce-logging/src/class-woocommerceloggingmodule.php

@@ -36,7 +36,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface $container The container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 	}
 
 

src/class-pluginmodule.php

@@ -31,7 +31,7 @@ public function setup(): ServiceProviderInterface {
 	 *
 	 * @param ContainerInterface|null $container The Container.
 	 */
-	public function run( ContainerInterface $container = null ) {
+	public function run( ContainerInterface $container ) {
 	}
 
 	/**