Completed
Branch EDTR/routing (63c8e0)
by
unknown
52:00 queued 43:18
created
core/domain/entities/routing/handlers/admin/WordPressPluginsPage.php 2 patches
Indentation   +47 added lines, -47 removed lines patch added patch discarded remove patch
@@ -17,55 +17,55 @@
 block discarded – undo
17 17
 class WordPressPluginsPage extends Route
18 18
 {
19 19
 
20
-    /**
21
-     * returns true if the current request matches this route
22
-     *
23
-     * @return bool
24
-     * @since   $VID:$
25
-     */
26
-    public function matchesCurrentRequest()
27
-    {
28
-        return $this->request->isAdmin()
29
-               && strpos($this->request->requestUri(), 'wp-admin/plugins.php') !== false;
30
-    }
20
+	/**
21
+	 * returns true if the current request matches this route
22
+	 *
23
+	 * @return bool
24
+	 * @since   $VID:$
25
+	 */
26
+	public function matchesCurrentRequest()
27
+	{
28
+		return $this->request->isAdmin()
29
+			   && strpos($this->request->requestUri(), 'wp-admin/plugins.php') !== false;
30
+	}
31 31
 
32 32
 
33
-    /**
34
-     * @since $VID:$
35
-     */
36
-    protected function registerDependencies() {
37
-        $this->dependency_map->registerDependencies(
38
-            'EventEspresso\core\domain\services\assets\WordpressPluginsPageAssetManager',
39
-            [
40
-                'EventEspresso\core\domain\Domain'                   => EE_Dependency_Map::load_from_cache,
41
-                'EventEspresso\core\services\assets\AssetCollection' => EE_Dependency_Map::load_from_cache,
42
-                'EventEspresso\core\services\assets\Registry'        => EE_Dependency_Map::load_from_cache,
43
-                'EventEspresso\core\domain\services\admin\ExitModal' => EE_Dependency_Map::load_from_cache,
44
-            ]
45
-        );
46
-        $this->dependency_map->registerDependencies(
47
-            'EventEspresso\core\domain\services\admin\ExitModal',
48
-            [ 'EventEspresso\core\services\assets\Registry' => EE_Dependency_Map::load_from_cache ]
49
-        );
50
-        $this->dependency_map->registerDependencies(
51
-            'EventEspresso\core\domain\services\admin\PluginUpsells',
52
-            [ 'EventEspresso\core\domain\Domain' => EE_Dependency_Map::load_from_cache ]
53
-        );
54
-    }
33
+	/**
34
+	 * @since $VID:$
35
+	 */
36
+	protected function registerDependencies() {
37
+		$this->dependency_map->registerDependencies(
38
+			'EventEspresso\core\domain\services\assets\WordpressPluginsPageAssetManager',
39
+			[
40
+				'EventEspresso\core\domain\Domain'                   => EE_Dependency_Map::load_from_cache,
41
+				'EventEspresso\core\services\assets\AssetCollection' => EE_Dependency_Map::load_from_cache,
42
+				'EventEspresso\core\services\assets\Registry'        => EE_Dependency_Map::load_from_cache,
43
+				'EventEspresso\core\domain\services\admin\ExitModal' => EE_Dependency_Map::load_from_cache,
44
+			]
45
+		);
46
+		$this->dependency_map->registerDependencies(
47
+			'EventEspresso\core\domain\services\admin\ExitModal',
48
+			[ 'EventEspresso\core\services\assets\Registry' => EE_Dependency_Map::load_from_cache ]
49
+		);
50
+		$this->dependency_map->registerDependencies(
51
+			'EventEspresso\core\domain\services\admin\PluginUpsells',
52
+			[ 'EventEspresso\core\domain\Domain' => EE_Dependency_Map::load_from_cache ]
53
+		);
54
+	}
55 55
 
56 56
 
57
-    /**
58
-     * implements logic required to run during request
59
-     *
60
-     * @return bool
61
-     * @since   $VID:$
62
-     */
63
-    protected function requestHandler()
64
-    {
65
-        $this->loader->getShared('EventEspresso\core\domain\services\assets\WordpressPluginsPageAssetManager');
66
-        /** @var PluginUpsells $plugin_upsells */
67
-        $plugin_upsells = $this->loader->getShared('EventEspresso\core\domain\services\admin\PluginUpsells');
68
-        $plugin_upsells->decafUpsells();
69
-        return true;
70
-    }
57
+	/**
58
+	 * implements logic required to run during request
59
+	 *
60
+	 * @return bool
61
+	 * @since   $VID:$
62
+	 */
63
+	protected function requestHandler()
64
+	{
65
+		$this->loader->getShared('EventEspresso\core\domain\services\assets\WordpressPluginsPageAssetManager');
66
+		/** @var PluginUpsells $plugin_upsells */
67
+		$plugin_upsells = $this->loader->getShared('EventEspresso\core\domain\services\admin\PluginUpsells');
68
+		$plugin_upsells->decafUpsells();
69
+		return true;
70
+	}
71 71
 }
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -45,11 +45,11 @@
 block discarded – undo
45 45
         );
46 46
         $this->dependency_map->registerDependencies(
47 47
             'EventEspresso\core\domain\services\admin\ExitModal',
48
-            [ 'EventEspresso\core\services\assets\Registry' => EE_Dependency_Map::load_from_cache ]
48
+            ['EventEspresso\core\services\assets\Registry' => EE_Dependency_Map::load_from_cache]
49 49
         );
50 50
         $this->dependency_map->registerDependencies(
51 51
             'EventEspresso\core\domain\services\admin\PluginUpsells',
52
-            [ 'EventEspresso\core\domain\Domain' => EE_Dependency_Map::load_from_cache ]
52
+            ['EventEspresso\core\domain\Domain' => EE_Dependency_Map::load_from_cache]
53 53
         );
54 54
     }
55 55
 
Please login to merge, or discard this patch.
core/domain/entities/routing/handlers/RouteHandlersDependencyMap.php 2 patches
Indentation   +51 added lines, -51 removed lines patch added patch discarded remove patch
@@ -15,57 +15,57 @@
 block discarded – undo
15 15
  */
16 16
 class RouteHandlersDependencyMap extends DependencyMapRegistrar
17 17
 {
18
-    /**
19
-     * @return bool
20
-     * @see   EE_Dependency_Map::registerDependencies()
21
-     * @since $VID:$
22
-     */
23
-    public function registerDependencies()
24
-    {
25
-        $default_dependencies = [
26
-            'EE_Dependency_Map'                           => EE_Dependency_Map::load_from_cache,
27
-            'EventEspresso\core\services\loaders\Loader'  => EE_Dependency_Map::load_from_cache,
28
-            'EventEspresso\core\services\request\Request' => EE_Dependency_Map::load_from_cache,
29
-        ];
30
-        $frontend_dependencies = [
31
-            'EE_Dependency_Map'                           => EE_Dependency_Map::load_from_cache,
32
-            'EE_Maintenance_Mode'                         => EE_Dependency_Map::load_from_cache,
33
-            'EventEspresso\core\services\loaders\Loader'  => EE_Dependency_Map::load_from_cache,
34
-            'EventEspresso\core\services\request\Request' => EE_Dependency_Map::load_from_cache,
35
-        ];
36
-        $routes = [
37
-            'EventEspresso\core\domain\entities\routing\handlers\admin\EspressoEventEditor',
38
-            'EventEspresso\core\domain\entities\routing\handlers\admin\EspressoLegacyAdmin',
39
-            'EventEspresso\core\domain\entities\routing\handlers\admin\PueRequests',
40
-            'EventEspresso\core\domain\entities\routing\handlers\admin\WordPressPluginsPage',
41
-            'EventEspresso\core\domain\entities\routing\handlers\frontend\ShortcodeRequests',
42
-            'EventEspresso\core\domain\entities\routing\handlers\shared\AssetRequests',
43
-            'EventEspresso\core\domain\entities\routing\handlers\shared\GQLRequests',
44
-            'EventEspresso\core\domain\entities\routing\handlers\shared\RestApiRequests',
45
-            'EventEspresso\core\domain\entities\routing\handlers\shared\SessionRequests',
46
-            'EventEspresso\core\domain\entities\routing\handlers\shared\WordPressHeartbeat',
47
-        ];
48
-        // set initially to true but toggled permanently to false if anything fails
49
-        $registered = true;
50
-        foreach ($routes as $route) {
51
-            $registered = $this->dependency_map->registerDependencies($route, $default_dependencies )
52
-                ? $registered // keep previous value whether it is true or false
53
-                : false; // toggle value to false
54
-        }
55
-        $registered = $this->dependency_map->registerDependencies(
56
-            'EventEspresso\core\domain\entities\routing\handlers\admin\PersonalDataRequests',
57
-            $frontend_dependencies
58
-        )
59
-            ? $registered // keep previous value whether it is true or false
60
-            : false; // toggle value to false
18
+	/**
19
+	 * @return bool
20
+	 * @see   EE_Dependency_Map::registerDependencies()
21
+	 * @since $VID:$
22
+	 */
23
+	public function registerDependencies()
24
+	{
25
+		$default_dependencies = [
26
+			'EE_Dependency_Map'                           => EE_Dependency_Map::load_from_cache,
27
+			'EventEspresso\core\services\loaders\Loader'  => EE_Dependency_Map::load_from_cache,
28
+			'EventEspresso\core\services\request\Request' => EE_Dependency_Map::load_from_cache,
29
+		];
30
+		$frontend_dependencies = [
31
+			'EE_Dependency_Map'                           => EE_Dependency_Map::load_from_cache,
32
+			'EE_Maintenance_Mode'                         => EE_Dependency_Map::load_from_cache,
33
+			'EventEspresso\core\services\loaders\Loader'  => EE_Dependency_Map::load_from_cache,
34
+			'EventEspresso\core\services\request\Request' => EE_Dependency_Map::load_from_cache,
35
+		];
36
+		$routes = [
37
+			'EventEspresso\core\domain\entities\routing\handlers\admin\EspressoEventEditor',
38
+			'EventEspresso\core\domain\entities\routing\handlers\admin\EspressoLegacyAdmin',
39
+			'EventEspresso\core\domain\entities\routing\handlers\admin\PueRequests',
40
+			'EventEspresso\core\domain\entities\routing\handlers\admin\WordPressPluginsPage',
41
+			'EventEspresso\core\domain\entities\routing\handlers\frontend\ShortcodeRequests',
42
+			'EventEspresso\core\domain\entities\routing\handlers\shared\AssetRequests',
43
+			'EventEspresso\core\domain\entities\routing\handlers\shared\GQLRequests',
44
+			'EventEspresso\core\domain\entities\routing\handlers\shared\RestApiRequests',
45
+			'EventEspresso\core\domain\entities\routing\handlers\shared\SessionRequests',
46
+			'EventEspresso\core\domain\entities\routing\handlers\shared\WordPressHeartbeat',
47
+		];
48
+		// set initially to true but toggled permanently to false if anything fails
49
+		$registered = true;
50
+		foreach ($routes as $route) {
51
+			$registered = $this->dependency_map->registerDependencies($route, $default_dependencies )
52
+				? $registered // keep previous value whether it is true or false
53
+				: false; // toggle value to false
54
+		}
55
+		$registered = $this->dependency_map->registerDependencies(
56
+			'EventEspresso\core\domain\entities\routing\handlers\admin\PersonalDataRequests',
57
+			$frontend_dependencies
58
+		)
59
+			? $registered // keep previous value whether it is true or false
60
+			: false; // toggle value to false
61 61
 
62
-        $registered = $this->dependency_map->registerDependencies(
63
-            'EventEspresso\core\domain\entities\routing\handlers\frontend\FrontendRequests',
64
-            $frontend_dependencies
65
-        )
66
-            ? $registered // keep previous value whether it is true or false
67
-            : false; // toggle value to false
62
+		$registered = $this->dependency_map->registerDependencies(
63
+			'EventEspresso\core\domain\entities\routing\handlers\frontend\FrontendRequests',
64
+			$frontend_dependencies
65
+		)
66
+			? $registered // keep previous value whether it is true or false
67
+			: false; // toggle value to false
68 68
 
69
-        return $registered;
70
-    }
69
+		return $registered;
70
+	}
71 71
 }
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
         // set initially to true but toggled permanently to false if anything fails
49 49
         $registered = true;
50 50
         foreach ($routes as $route) {
51
-            $registered = $this->dependency_map->registerDependencies($route, $default_dependencies )
51
+            $registered = $this->dependency_map->registerDependencies($route, $default_dependencies)
52 52
                 ? $registered // keep previous value whether it is true or false
53 53
                 : false; // toggle value to false
54 54
         }
Please login to merge, or discard this patch.