MinifiablesHelper::getExtension()

Completed

Push — master ( 02b122...2a8558 )

by James Ekow Abaka

created 2017-06-26 20:28 UTC

↳ Parent: MinifiablesHelper

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
nc	1
dl	0
loc	1
ccs	0
cts	0
cp	0

<?php
namespace ntentan\honam\helpers;

use ntentan\honam\Helper;
use ntentan\honam\AssetsLoader;
use ntentan\honam\helpers\minifiables\Minifier;

abstract class MinifiablesHelper extends Helper
{
    private $minifiableScripts = array();
    private $otherScripts = array();
    private $context = 'default';
    private $combine = false;

    protected abstract function getExtension();

    protected abstract function getMinifier();

    protected abstract function getTag($url);

    
    private function makePublic($file)

    {
        return $this->getTag(AssetsLoader::load($this->getExtension() . "/" . basename($file), $file));        
    }
    
    private function writeTags($files)
    {
        $tags = '';
        foreach($files as $script)
        {
            $tags .= $this->makePublic($script);
        }
        return $tags;
    }
    
    public function __toString()
    {
        $minifiedScript = '';
        $tags = '';
        $filename = "/" . $this->getExtension() . "/combined_{$this->context}." . $this->getExtension();
        if($this->combine === true)
        {
            foreach($this->minifiableScripts as $script)
            {
                $minifiedScript .= file_get_contents($script);
            }
            file_put_contents(AssetsLoader::getDestinationDir() . $filename, Minifier::minify($minifiedScript, $this->getMinifier()));
            $tags = $this->getTag(AssetsLoader::getSiteUrl() . $filename);
        }
        else if($this->combine == false)

        {
            $tags .= $this->writeTags($this->minifiableScripts);
        }
        
        $tags .= $this->writeTags($this->otherScripts);
        
        $this->minifiableScripts = array();
        $this->otherScripts = array();
        
        return $tags;
    }

    public function help($arguments)
    {
        if(is_array($arguments))
        {
            foreach($arguments as $argument)
            {
                if($argument == '') continue;
                $this->otherScripts[]= $argument;
            }
        }
        else if($arguments != '')
        {
            $this->otherScripts[]= $arguments;
        }
        return $this;
    }

    public function add($script)
    {
        if($script != '' && is_string($script))
        { 
            $this->minifiableScripts[] = $script;
        }
        else if(is_array($script))
        {
            foreach($script as $scriptFile)
            {
                $this->minifiableScripts[] = $scriptFile;
            }
        }
        return $this;
    }
    
    public function setContext($context)
    {
        return $this->context($context);
    }
    
    public function setCombine($combine)
    {
        return $this->combine($combine);
    }
    
    public function context($context)
    {
        $this->context = $context;
        return $this;
    }
    
    public function combine($combine)
    {
        $this->combine = $combine;
        return $this;
    }    
}


1		<?php
2		namespace ntentan\honam\helpers;
3
4		use ntentan\honam\Helper;
5		use ntentan\honam\AssetsLoader;
6		use ntentan\honam\helpers\minifiables\Minifier;
7
8		abstract class MinifiablesHelper extends Helper
9		{
10		private $minifiableScripts = array();
11		private $otherScripts = array();
12		private $context = 'default';
13		private $combine = false;
14
15		protected abstract function getExtension();
		0 ignored issues – show Documentation introduced 2016-03-27 07:30 UTC by Report Bug Copy Issue Report For interfaces and abstract methods it is generally a good practice to add a `@return` annotation even if it is just `@return void` or `@return null`, so that implementors know what to do in the overridden method. For interface and abstract methods, it is impossible to infer the return type from the immediate code. In these cases, it is generally advisible to explicitly annotate these methods with a `@return` doc comment to communicate to implementors of these methods what they are expected to return. Loading history... Coding Style introduced 2016-03-27 07:30 UTC by Report Bug Copy Issue Report The abstract declaration must precede the visibility declaration Loading history...
16		protected abstract function getMinifier();
		0 ignored issues – show Documentation introduced 2016-03-27 07:30 UTC by Report Bug Copy Issue Report For interfaces and abstract methods it is generally a good practice to add a `@return` annotation even if it is just `@return void` or `@return null`, so that implementors know what to do in the overridden method. For interface and abstract methods, it is impossible to infer the return type from the immediate code. In these cases, it is generally advisible to explicitly annotate these methods with a `@return` doc comment to communicate to implementors of these methods what they are expected to return. Loading history... Coding Style introduced 2016-03-27 07:30 UTC by Report Bug Copy Issue Report The abstract declaration must precede the visibility declaration Loading history...
17		protected abstract function getTag($url);
		0 ignored issues – show Documentation introduced 2016-03-27 07:30 UTC by Report Bug Copy Issue Report For interfaces and abstract methods it is generally a good practice to add a `@return` annotation even if it is just `@return void` or `@return null`, so that implementors know what to do in the overridden method. For interface and abstract methods, it is impossible to infer the return type from the immediate code. In these cases, it is generally advisible to explicitly annotate these methods with a `@return` doc comment to communicate to implementors of these methods what they are expected to return. Loading history... Coding Style introduced 2016-03-27 07:30 UTC by Report Bug Copy Issue Report The abstract declaration must precede the visibility declaration Loading history...
18
19	7	private function makePublic($file)
		0 ignored issues – show Documentation introduced 2016-03-27 07:30 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
20		{
21	7	return $this->getTag(AssetsLoader::load($this->getExtension() . "/" . basename($file), $file));
22		}
23
24	11	private function writeTags($files)
25		{
26	11	$tags = '';
27	11	foreach($files as $script)
28		{
29	7	$tags .= $this->makePublic($script);
30		}
31	9	return $tags;
32		}
33
34	11	public function __toString()
35		{
36	11	$minifiedScript = '';
37	11	$tags = '';
38	11	$filename = "/" . $this->getExtension() . "/combined_{$this->context}." . $this->getExtension();
39	11	if($this->combine === true)
40		{
41	4	foreach($this->minifiableScripts as $script)
42		{
43	4	$minifiedScript .= file_get_contents($script);
44		}
45	4	file_put_contents(AssetsLoader::getDestinationDir() . $filename, Minifier::minify($minifiedScript, $this->getMinifier()));
46	4	$tags = $this->getTag(AssetsLoader::getSiteUrl() . $filename);
47		}
48	7	else if($this->combine == false)
		0 ignored issues – show Coding Style Best Practice introduced 2016-03-27 07:30 UTC by Report Bug Copy Issue Report It seems like you are loosely comparing two booleans. Considering using the strict comparison `===` instead. When comparing two booleans, it is generally considered safer to use the strict comparison operator. Loading history...
49		{
50	7	$tags .= $this->writeTags($this->minifiableScripts);
51		}
52
53	9	$tags .= $this->writeTags($this->otherScripts);
54
55	9	$this->minifiableScripts = array();
56	9	$this->otherScripts = array();
57
58	9	return $tags;
59		}
60
61	2	public function help($arguments)
62		{
63	2	if(is_array($arguments))
64		{
65	1	foreach($arguments as $argument)
66		{
67	1	if($argument == '') continue;
68	1	$this->otherScripts[]= $argument;
69		}
70		}
71	1	else if($arguments != '')
72		{
73	1	$this->otherScripts[]= $arguments;
74		}
75	2	return $this;
76		}
77
78	9	public function add($script)
79		{
80	9	if($script != '' && is_string($script))
81		{
82	7	$this->minifiableScripts[] = $script;
83		}
84	2	else if(is_array($script))
85		{
86	2	foreach($script as $scriptFile)
87		{
88	2	$this->minifiableScripts[] = $scriptFile;
89		}
90		}
91	9	return $this;
92		}
93
94		public function setContext($context)
95		{
96		return $this->context($context);
97		}
98
99		public function setCombine($combine)
100		{
101		return $this->combine($combine);
102		}
103
104	1	public function context($context)
105		{
106	1	$this->context = $context;
107	1	return $this;
108		}
109
110	4	public function combine($combine)
111		{
112	4	$this->combine = $combine;
113	4	return $this;
114		}
115		}
116

ntentan / honam

Push — master ( 02b122...2a8558 )

MinifiablesHelper::getExtension()

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like