alphadevx / alpha

Alpha/Util/Logging/KPI.php

Doc Comments +1 added lines

@@ -182,6 +182,7 @@
      * not exist.
      *
      * @since 1.1
+     * @param string $stepName
      */
     public function logStep($stepName)
     {

Spacing +2 added lines, -2 removed lines

@@ -167,7 +167,7 @@ 
 
         $this->endTime = microtime(true);
 
-        $this->duration = $this->endTime - $this->startTime;
+        $this->duration = $this->endTime-$this->startTime;
 
         $logfile = new LogProviderFile();
         $logfile->setPath($config->get('app.file.store.dir').'logs/kpi-'.$this->name->getValue().'.csv');
@@ -189,7 +189,7 @@ 
 
         $this->endTime = microtime(true);
 
-        $this->duration = $this->endTime - $this->startTime;
+        $this->duration = $this->endTime-$this->startTime;
 
         $logfile = new LogProviderFile();
         $logfile->setPath($config->get('app.file.store.dir').'logs/kpi-'.$this->name->getValue().'.csv');

Alpha/Util/Logging/LogProviderFile.php

Doc Comments +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
     /**
      * Returns the size in megabytes of the log file on disc.
      *
-     * @return float
+     * @return integer
      *
      * @since 1.0
      */

Spacing +1 added lines, -1 removed lines

@@ -140,7 +140,7 @@
     {
         $size = filesize($this->path);
 
-        return ($size / 1024) / 1024;
+        return ($size/1024)/1024;
     }
 
     /**

Alpha/Util/Search/SearchProviderFactory.php

Doc Comments +2 added lines, -2 removed lines

@@ -63,11 +63,11 @@
      * A static method that attempts to return a SearchProviderInterface instance
      * based on the name of the provider class supplied.
      *
-     * @param $providerName The class name of the provider class, should be fully-qualified.
+     * @param string $providerName The class name of the provider class, should be fully-qualified.
      *
      * @throws Alpha\Exception\IllegalArguementException;
      *
-     * @return Alpha\Util\Search\SearchProviderInterface
+     * @return SearchProviderInterface|null
      *
      * @since 1.2.3
      */

Alpha/View/View.php

Doc Comments +1 added lines, -1 removed lines

@@ -1013,7 +1013,7 @@
     /**
      * Get the current view renderer provider.
      *
-     * @return Alpha\View\Renderer\RendererProviderInterface
+     * @return RendererProviderInterface
      *
      * @since 2.0
      */

Alpha/View/ViewState.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
      * Get the ViewState instance.  Loads from $_SESSION if its not already in memory, otherwise
      * a new instance will be returned with empty properties.
      *
-     * @return Alpha\View\ViewState
+     * @return string
      *
      * @since 1.0
      */

Alpha/View/Widget/Image.php

Doc Comments +1 added lines, -1 removed lines

@@ -153,7 +153,7 @@
     /**
      * The constructor.
      *
-     * @param $source
+     * @param string $source
      * @param $width
      * @param $height
      * @param $sourceType

Spacing +7 added lines, -7 removed lines

@@ -242,7 +242,7 @@ 
             }
         } else {
             // make a cache dir for the article
-            $cacheDir = $config->get('app.file.store.dir').'cache/images/article_'.mb_substr($this->source, mb_strpos($this->source, 'attachments/article_') + 20, 11);
+            $cacheDir = $config->get('app.file.store.dir').'cache/images/article_'.mb_substr($this->source, mb_strpos($this->source, 'attachments/article_')+20, 11);
             if (!file_exists($cacheDir)) {
                 $success = mkdir($cacheDir);
 
@@ -300,11 +300,11 @@ 
             $targetScreenY = $targetScreenResolution[1];
 
             // calculate the new units we will scale by
-            $xu = $targetScreenX / $originalScreenX;
-            $yu = $targetScreenY / $originalScreenY;
+            $xu = $targetScreenX/$originalScreenX;
+            $yu = $targetScreenY/$originalScreenY;
 
-            $this->width = new Integer(intval($this->width->getValue() * $xu));
-            $this->height = new Integer(intval($this->height->getValue() * $yu));
+            $this->width = new Integer(intval($this->width->getValue()*$xu));
+            $this->height = new Integer(intval($this->height->getValue()*$yu));
 
             // need to update the cache filename as the dimensions have changed
             $this->setFilename();
@@ -368,7 +368,7 @@ 
                 if ($this->sourceType->getValue() == 'png' && $config->get('cms.images.perserve.png')) {
                     imagepng($new_image);
                 } else {
-                    imagejpeg($new_image, null, 100 * $this->quality->getValue());
+                    imagejpeg($new_image, null, 100*$this->quality->getValue());
                 }
 
                 $this->cache($new_image);
@@ -392,7 +392,7 @@ 
         if ($this->sourceType->getValue() == 'png' && $config->get('cms.images.perserve.png')) {
             imagepng($image, $this->filename);
         } else {
-            imagejpeg($image, $this->filename, 100 * $this->quality->getValue());
+            imagejpeg($image, $this->filename, 100*$this->quality->getValue());
         }
     }
 

public/index.php

Unused Use Statements +9 added lines, -9 removed lines

@@ -2,15 +2,15 @@
 

 require_once __DIR__.'/../vendor/autoload.php';

 

-use Alpha\Controller\Front\FrontController;

-use Alpha\Util\Config\ConfigProvider;

-use Alpha\Util\Http\Filter\ClientBlacklistFilter;

-use Alpha\Util\Http\Filter\IPBlacklistFilter;

-use Alpha\Util\Http\Filter\ClientTempBlacklistFilter;

-use Alpha\Util\Http\Request;

-use Alpha\Util\Http\Response;

-use Alpha\Exception\ResourceNotFoundException;

-use Alpha\Exception\ResourceNotAllowedException;

+use Alpha\Controller\Front\FrontController;
+use Alpha\Util\Config\ConfigProvider;
+use Alpha\Util\Http\Filter\ClientBlacklistFilter;
+use Alpha\Util\Http\Filter\IPBlacklistFilter;
+use Alpha\Util\Http\Filter\ClientTempBlacklistFilter;
+use Alpha\Util\Http\Request;
+use Alpha\Util\Http\Response;
+use Alpha\Exception\ResourceNotFoundException;
+use Alpha\Exception\ResourceNotAllowedException;
 use Alpha\View\View;

 

 try {

Indentation +18 added lines, -18 removed lines

@@ -14,36 +14,36 @@
 use Alpha\View\View;

 

 try {

-	$config = ConfigProvider::getInstance();

+    $config = ConfigProvider::getInstance();

 

-	set_exception_handler('Alpha\Util\ErrorHandlers::catchException');

-	set_error_handler('Alpha\Util\ErrorHandlers::catchError', $config->get('php.error.log.level'));

+    set_exception_handler('Alpha\Util\ErrorHandlers::catchException');

+    set_error_handler('Alpha\Util\ErrorHandlers::catchError', $config->get('php.error.log.level'));

 

-	$front = new FrontController();

+    $front = new FrontController();

 	

-	if ($config->get('security.client.blacklist.filter.enabled')) {

-		$front->registerFilter(new ClientBlacklistFilter());

-	}

+    if ($config->get('security.client.blacklist.filter.enabled')) {

+        $front->registerFilter(new ClientBlacklistFilter());

+    }

 

-	if ($config->get('security.ip.blacklist.filter.enabled')) {

-		$front->registerFilter(new IPBlacklistFilter());

-	}

+    if ($config->get('security.ip.blacklist.filter.enabled')) {

+        $front->registerFilter(new IPBlacklistFilter());

+    }

 

-	if ($config->get('security.client.temp.blacklist.filter.enabled')) {

-		$front->registerFilter(new ClientTempBlacklistFilter());

-	}

+    if ($config->get('security.client.temp.blacklist.filter.enabled')) {

+        $front->registerFilter(new ClientTempBlacklistFilter());

+    }

 

-	$request = new Request();

-	$response = $front->process($request);

+    $request = new Request();

+    $response = $front->process($request);

 

 } catch (ResourceNotFoundException $rnfe) {

-	$response = new Response(404, View::renderErrorPage(404, $rnfe->getMessage(), array('Content-Type' => 'text/html')));

+    $response = new Response(404, View::renderErrorPage(404, $rnfe->getMessage(), array('Content-Type' => 'text/html')));

 } catch (ResourceNotAllowedException $rnae) {

-	$response = new Response(403, View::renderErrorPage(403, $rnae->getMessage(), array('Content-Type' => 'text/html')));

+    $response = new Response(403, View::renderErrorPage(403, $rnae->getMessage(), array('Content-Type' => 'text/html')));

 }

 

 if ($config->get('security.http.header.x.frame.options') != '')

-	$response->getHeader('X-Frame-Options', $config->get('security.http.header.x.frame.options'));

+    $response->getHeader('X-Frame-Options', $config->get('security.http.header.x.frame.options'));

 

 echo $response->send();

 

Braces +3 added lines, -2 removed lines

@@ -42,8 +42,9 @@
 	$response = new Response(403, View::renderErrorPage(403, $rnae->getMessage(), array('Content-Type' => 'text/html')));

 }

 

-if ($config->get('security.http.header.x.frame.options') != '')

-	$response->getHeader('X-Frame-Options', $config->get('security.http.header.x.frame.options'));

+if ($config->get('security.http.header.x.frame.options') != '') {
+	$response->getHeader('X-Frame-Options', $config->get('security.http.header.x.frame.options'));
+}
 

 echo $response->send();

 

phpunit.php

Indentation +11 added lines, -11 removed lines

@@ -9,19 +9,19 @@
 $config = ConfigProvider::getInstance();
 
 $dirs = array(
-	$config->get('app.file.store.dir').'logs',
-	$config->get('app.file.store.dir').'cache',
-	$config->get('app.file.store.dir').'cache/html',
-	$config->get('app.file.store.dir').'cache/images',
-	$config->get('app.file.store.dir').'cache/pdf',
-	$config->get('app.file.store.dir').'cache/xls',
-	$config->get('app.file.store.dir').'attachments',
-	);
+    $config->get('app.file.store.dir').'logs',
+    $config->get('app.file.store.dir').'cache',
+    $config->get('app.file.store.dir').'cache/html',
+    $config->get('app.file.store.dir').'cache/images',
+    $config->get('app.file.store.dir').'cache/pdf',
+    $config->get('app.file.store.dir').'cache/xls',
+    $config->get('app.file.store.dir').'attachments',
+    );
 
 foreach ($dirs as $dir) {
-	if (!file_exists($dir)) {
-		mkdir($dir, 0774);
-	}
+    if (!file_exists($dir)) {
+        mkdir($dir, 0774);
+    }
 }
 
 ?>
\ No newline at end of file

Alpha/View/Renderer/Html/RendererProviderHTML.php

Spacing +3 added lines, -3 removed lines

@@ -218,7 +218,7 @@ 
         // work out how many columns will be in the table
         $reflection = new ReflectionClass(get_class($this->BO));
         $properties = array_keys($reflection->getDefaultProperties());
-        $fields['colCount'] = 1 + count(array_diff($properties, $this->BO->getDefaultAttributes(), $this->BO->getTransientAttributes()));
+        $fields['colCount'] = 1+count(array_diff($properties, $this->BO->getDefaultAttributes(), $this->BO->getTransientAttributes()));
 
         // get the class attributes
         $properties = $reflection->getProperties();
@@ -239,7 +239,7 @@ 
                     $html .= '  <th>'.$this->BO->getDataLabel($propName).'</th>';
                 }
             } else {
-                $fields['colCount'] = $fields['colCount'] - 1;
+                $fields['colCount'] = $fields['colCount']-1;
             }
         }
         $html .= '</tr><tr>';
@@ -1009,7 +1009,7 @@ 
                 if ($viewState->get('markdownTextBoxRows') == '') {
                     $text = new TextBox($this->BO->getPropObject($name), $label, $name, 10);
                 } else {
-                    $text = new TextBox($this->BO->getPropObject($name), $label, $name, (integer) $viewState->get('markdownTextBoxRows'));
+                    $text = new TextBox($this->BO->getPropObject($name), $label, $name, (integer)$viewState->get('markdownTextBoxRows'));
                 }
 
                 $html .= $text->render();