Skip to content

Commit

Permalink
Only minify on environments not listed in 'ignore_environments'.
Browse files Browse the repository at this point in the history
  • Loading branch information
jorygeerts committed Apr 24, 2014
1 parent 1508f9f commit ff615d8
Showing 1 changed file with 11 additions and 3 deletions.
14 changes: 11 additions & 3 deletions src/CeesVanEgmond/Minify/Minify.php
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ class Minify
private $environment;

/**
* @var
* @var \CeesVanEgmond\Minify\Providers\JavaScript
*/
private $provider;

Expand Down Expand Up @@ -82,7 +82,7 @@ private function process($file)
{
$this->provider->add($file);

if($this->provider->make($this->buildPath))
if($this->minifyForCurrentEnvironment() && $this->provider->make($this->buildPath))
{
$this->provider->minify();
}
Expand All @@ -93,14 +93,22 @@ private function process($file)
*/
public function render()
{
if (in_array($this->environment, $this->config['ignore_environments']))
if (!$this->minifyForCurrentEnvironment())
{
return $this->provider->tags($this->attributes);
}

return $this->provider->tag($this->buildPath . $this->provider->getFilename(), $this->attributes);
}

/**
* @return bool
*/
protected function minifyForCurrentEnvironment()
{
return !in_array($this->environment, $this->config['ignore_environments']);
}

/**
* @return string
*/
Expand Down

0 comments on commit ff615d8

Please sign in to comment.