PartialAssetComposer::requireAssets() - Code Metrics - Inspection of "Initial push of changes to 2.0 for event based med..." - AsgardCms/Media - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — 2.0 (#49)

unknown

created 2016-07-15 07:35 UTC

PartialAssetComposer::requireAssets() A

↳ Parent: PartialAssetComposer

Complexity

Conditions	3
Paths	4

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
c	2
b	0
f	0
dl	0
loc	13
rs	9.4285
cc	3
eloc	7
nc	4
nop	0

<?php

namespace Modules\Media\Composers\Backend;

use Modules\Core\Foundation\Asset\Manager\AssetManager;
use Modules\Core\Foundation\Asset\Pipeline\AssetPipeline;
use Modules\Core\Foundation\Asset\Types\AssetTypeFactory;

class PartialAssetComposer
{
    /**
     * @var AssetManager
     */
    private $assetManager;
    /**
     * @var AssetPipeline
     */
    private $assetPipeline;
    /**
     * @var AssetTypeFactory
     */
    private $assetFactory;

    public function __construct()
    {
        $this->assetManager = app(AssetManager::class);
        $this->assetPipeline = app(AssetPipeline::class);
        $this->assetFactory = app(AssetTypeFactory::class);
    }

    public function compose()
    {
        $this->addAssets();
        $this->requireAssets();
    }

    /**
     * Add the assets from the config file on the asset manager
     */
    private function addAssets()
    {
        foreach (config('asgard.media.assets.media-partial-assets', []) as $assetName => $path) {
            $path = $this->assetFactory->make($path)->url();
            $this->assetManager->addAsset($assetName, $path);
        }
    }

    /**
     * Require assets from asset manager
     */
    private function requireAssets()
    {
        $css = config('asgard.media.assets.media-partial-required-assets.css');
        $js  = config('asgard.media.assets.media-partial-required-assets.js');

        if (!empty($css)) {
            $this->assetPipeline->requireCss($css);
        }

        if (!empty($js)) {
            $this->assetPipeline->requireJs($js);
        }
    }
}


1			<?php
2
3			namespace Modules\Media\Composers\Backend;
4
5			use Modules\Core\Foundation\Asset\Manager\AssetManager;
6			use Modules\Core\Foundation\Asset\Pipeline\AssetPipeline;
7			use Modules\Core\Foundation\Asset\Types\AssetTypeFactory;
8
9			class PartialAssetComposer
10			{
11			/**
12			* @var AssetManager
13			*/
14			private $assetManager;
15			/**
16			* @var AssetPipeline
17			*/
18			private $assetPipeline;
19			/**
20			* @var AssetTypeFactory
21			*/
22			private $assetFactory;
23
24			public function __construct()
25			{
26			$this->assetManager = app(AssetManager::class);
27			$this->assetPipeline = app(AssetPipeline::class);
28			$this->assetFactory = app(AssetTypeFactory::class);
29			}
30
31			public function compose()
32			{
33			$this->addAssets();
34			$this->requireAssets();
35			}
36
37			/**
38			* Add the assets from the config file on the asset manager
39			*/
40			private function addAssets()
41			{
42			foreach (config('asgard.media.assets.media-partial-assets', []) as $assetName => $path) {
43			$path = $this->assetFactory->make($path)->url();
44			$this->assetManager->addAsset($assetName, $path);
45			}
46			}
47
48			/**
49			* Require assets from asset manager
50			*/
51			private function requireAssets()
52			{
53			$css = config('asgard.media.assets.media-partial-required-assets.css');
54			$js = config('asgard.media.assets.media-partial-required-assets.js');
55
56			if (!empty($css)) {
57			$this->assetPipeline->requireCss($css);
58			}
59
60			if (!empty($js)) {
61			$this->assetPipeline->requireJs($js);
62			}
63			}
64			}
65

AsgardCms / Media

Pull Request — 2.0 (#49)

PartialAssetComposer::requireAssets() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like