BurningFlipside / CommonCode

Auth/LDAPAuthenticator.php

Doc Comments +2 added lines, -2 removed lines

@@ -233,7 +233,7 @@ 
      *
      * @param \stdClass	$data The AuthData from the session
      *
-     * @return null|\Auth\LDAPUser The LDAPUser represented by this data
+     * @return LDAPUser The LDAPUser represented by this data
      */
     public function getUser($data)
     {
@@ -245,7 +245,7 @@ 
      *
      * @param string $name The Group's name
      *
-     * @return null|\Auth\LDAPGroup The LDAPGroup represented by the name or null if not found
+     * @return null|LDAPGroup The LDAPGroup represented by the name or null if not found
      */
     public function getGroupByName($name)
     {

Auth/SQLAuthenticator.php

Doc Comments +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
     /**
      * Get the data table for Pending Users
      *
-     * @return boolean|\Data\DataTable The Pending User Data Table
+     * @return boolean The Pending User Data Table
      */
     private function getPendingUserDataTable()
     {

Email/DBEmail.php

Doc Comments +8 added lines

@@ -44,8 +44,16 @@
         return parent::getSubject();
     }
 
+    /**
+     * @return string
+     */
     public abstract function getSubstituteVars();
 
+    /**
+     * @param boolean $html
+     *
+     * @return string
+     */
     protected function getRawBodyText($html)
     {
         if(isset($this->dbData['body']))

Autoload.php

Spacing +5 added lines, -5 removed lines

@@ -9,17 +9,17 @@ 
 * @license http://www.apache.org/licenses/ Apache 2.0 License
 */
 
-spl_autoload_register(function ($class) {
+spl_autoload_register(function($class) {
 
     // project-specific namespace prefix
     $prefix = 'Flipside\\';
 
     // base directory for the namespace prefix
-    $base_dir = __DIR__ . '/';
+    $base_dir = __DIR__.'/';
 
     // does the class use the namespace prefix?
     $len = strlen($prefix);
-    if (strncmp($prefix, $class, $len) !== 0) {
+    if(strncmp($prefix, $class, $len) !== 0) {
         // no, move to the next registered autoloader
         return;
     }
@@ -30,10 +30,10 @@ 
     // replace the namespace prefix with the base directory, replace namespace
     // separators with directory separators in the relative class name, append
     // with .php
-    $file = $base_dir . str_replace('\\', '/', $relative_class) . '.php';
+    $file = $base_dir.str_replace('\\', '/', $relative_class).'.php';
 
     // if the file exists, require it
-    if (file_exists($file)) {
+    if(file_exists($file)) {
         require $file;
     }
 });

Braces +6 added lines, -3 removed lines

@@ -9,7 +9,8 @@ 
 * @license http://www.apache.org/licenses/ Apache 2.0 License
 */
 
-spl_autoload_register(function ($class) {
+spl_autoload_register(function ($class)
+{
 
     // project-specific namespace prefix
     $prefix = 'Flipside\\';
@@ -19,7 +20,8 @@ 
 
     // does the class use the namespace prefix?
     $len = strlen($prefix);
-    if (strncmp($prefix, $class, $len) !== 0) {
+    if (strncmp($prefix, $class, $len) !== 0)
+    {
         // no, move to the next registered autoloader
         return;
     }
@@ -33,7 +35,8 @@ 
     $file = $base_dir . str_replace('\\', '/', $relative_class) . '.php';
 
     // if the file exists, require it
-    if (file_exists($file)) {
+    if (file_exists($file))
+    {
         require $file;
     }
 });

Settings.php

Indentation +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
     public function getDataSetData($dataSetName)
     {
         if(!isset(\FlipsideSettings::$dataset) || !isset(\FlipsideSettings::$dataset[$dataSetName]))
-	{
+    {
             return false;
         }
         return \FlipsideSettings::$dataset[$dataSetName];

Braces +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
     public function getDataSetData($dataSetName)
     {
         if(!isset(\FlipsideSettings::$dataset) || !isset(\FlipsideSettings::$dataset[$dataSetName]))
-	{
+        {
             return false;
         }
         return \FlipsideSettings::$dataset[$dataSetName];

Log/PHPLog.php

Spacing +1 added lines, -1 removed lines

@@ -3,7 +3,7 @@
 
 class PHPLog extends LogService
 {
-    public function __construct($params=false)
+    public function __construct($params = false)
     {
         parent::__construct($params);
     }

Data/FilterClause.php

Indentation +14 added lines, -14 removed lines

@@ -140,9 +140,9 @@ 
             $case = true;
         }
         if($case)
-	{
+    {
             if(class_exists('MongoRegex'))
-	    {
+        {
                 return array($field=>array('$regex'=>new \MongoRegex('/'.$this->var2.'/i')));
             }
             else
@@ -236,15 +236,15 @@ 
         {
             case '=':
                 return array($this->var1=>$this->var2);
-	    case 'substringof':
-		if(class_exists('MongoRegex'))
-		{
-		    return array($this->var1=>array('$regex'=>new \MongoRegex('/'.$this->var2.'/i')));
-		}
-		else
-		{
+        case 'substringof':
+        if(class_exists('MongoRegex'))
+        {
+            return array($this->var1=>array('$regex'=>new \MongoRegex('/'.$this->var2.'/i')));
+        }
+        else
+        {
                     return array($this->var1=>array('$regex'=>new \MongoDB\BSON\Regex($this->var2, 'i')));
-		}
+        }
             case 'indexof':
                 return $this->getMongoIndexOfOperator();
             default:
@@ -254,15 +254,15 @@ 
 
     public function php_compare($value)
     {
-	if(is_array($value))
-	{
+    if(is_array($value))
+    {
             return $this->php_compare($value[$this->var1]);
-	}
+    }
         switch($this->op)
         {
             case '!=':
                 return $value != $this->var2;
-	    case '=':
+        case '=':
                 return $value == $this->var2;
             case '<':
                 return $value < $this->var2;

Braces +2 added lines, -2 removed lines

@@ -140,9 +140,9 @@
             $case = true;
         }
         if($case)
-	{
+        {
             if(class_exists('MongoRegex'))
-	    {
+            {
                 return array($field=>array('$regex'=>new \MongoRegex('/'.$this->var2.'/i')));
             }
             else

Http/WebPage.php

Indentation +9 added lines, -9 removed lines

@@ -22,10 +22,10 @@ 
 
         $twigSettings = array();
         if(\file_exists('/var/php_cache/twig'))
-    	{	
-        	$twigCache = $this->settings->getGlobalSetting('twig_cache', '/var/php_cache/twig');
-        	$twigSettings['cache'] = $twigCache;
-		    //$twigSettings = array('cache' => $twigCache, 'debug' => true);
+        {	
+            $twigCache = $this->settings->getGlobalSetting('twig_cache', '/var/php_cache/twig');
+            $twigSettings['cache'] = $twigCache;
+            //$twigSettings = array('cache' => $twigCache, 'debug' => true);
         }
         //$twigSettings['debug'] = true;
 
@@ -52,8 +52,8 @@ 
 
         $this->aboutUrl = $this->settings->getGlobalSetting('about_url', $this->wwwUrl.'/about');
         $this->content['header']['right']['About'] = array(
-          'url' => $this->aboutUrl,
-          'menu' => $this->settings->getGlobalSetting('about_menu', array(
+            'url' => $this->aboutUrl,
+            'menu' => $this->settings->getGlobalSetting('about_menu', array(
             'Burning Flipside' => $this->wwwUrl.'/about/event',
             'AAR, LLC' => $this->wwwUrl.'/organization/aar',
             'Privacy Policy' => $this->wwwUrl.'/about/privacy'
@@ -69,7 +69,7 @@ 
         $this->content['urls']['loginUrl'] = $this->loginUrl;
         $this->content['urls']['logoutUrl'] = $this->logoutUrl;
 
-	if($this->user === false || $this->user === null)
+    if($this->user === false || $this->user === null)
         {
             if(isset($_SERVER['REQUEST_URI']) && strstr($_SERVER['REQUEST_URI'], 'logout.php') === false)
             {
@@ -203,7 +203,7 @@ 
     {
         if(!isset($this->content['notifications']))
         {
-          $this->content['notifications'] = array();
+            $this->content['notifications'] = array();
         }
         array_push($this->content['notifications'], array('msg'=>$message, 'sev'=>$severity, 'dismissible'=>$dismissible));
     }
@@ -216,7 +216,7 @@ 
     {
         if(!isset($this->content['body']))
         {
-          $this->content['body'] = $this->body;
+            $this->content['body'] = $this->body;
         }
         //Add page JS just before rednering so it is after any added by the page explicitly
         // $this->addJS('js/'.basename($_SERVER['SCRIPT_NAME'], '.php').'.js');

Spacing +6 added lines, -6 removed lines

@@ -4,8 +4,8 @@ 
 use \Psr\Http\Message\ServerRequestInterface as Request;
 use \Psr\Http\Message\ResponseInterface as Response;
 
-require __DIR__ . '/../vendor/autoload.php';
-require_once(__DIR__ . '/../static.js_css.php');
+require __DIR__.'/../vendor/autoload.php';
+require_once(__DIR__.'/../static.js_css.php');
 
 class WebPage
 {
@@ -16,7 +16,7 @@ 
 
     public function __construct($title)
     {
-        \Sentry\init(['dsn' => 'https://8d76f6c4cb3b409bbe7ed4300e054afd@sentry.io/4283882' ]);
+        \Sentry\init(['dsn' => 'https://8d76f6c4cb3b409bbe7ed4300e054afd@sentry.io/4283882']);
         $this->settings = \Flipside\Settings::getInstance();
         $this->loader = new \Twig_Loader_Filesystem(dirname(__FILE__).'/../templates');
 
@@ -112,7 +112,7 @@ 
             $this->content['css'] = array($uri);
             return;
         }
-        array_push($this->content['css'],$uri);
+        array_push($this->content['css'], $uri);
     }
 
     /**
@@ -127,7 +127,7 @@ 
             $this->content['js'] = array($uri);
             return;
         }
-        array_push($this->content['js'],$uri);
+        array_push($this->content['js'], $uri);
     }
 
     /**
@@ -180,7 +180,7 @@ 
         {
             $data['menu'] = $submenu;
         }
-        $this->content['header']['right'] = array($name => $data)+$this->content['header']['right'];
+        $this->content['header']['right'] = array($name => $data) + $this->content['header']['right'];
     }
 
     /** Notification that is green for success */

Braces +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 
         $twigSettings = array();
         if(\file_exists('/var/php_cache/twig'))
-    	{	
+        {
         	$twigCache = $this->settings->getGlobalSetting('twig_cache', '/var/php_cache/twig');
         	$twigSettings['cache'] = $twigCache;
 		    //$twigSettings = array('cache' => $twigCache, 'debug' => true);
@@ -70,7 +70,7 @@ 
         $this->content['urls']['logoutUrl'] = $this->logoutUrl;
 
 	if($this->user === false || $this->user === null)
-        {
+	{
             if(isset($_SERVER['REQUEST_URI']) && strstr($_SERVER['REQUEST_URI'], 'logout.php') === false)
             {
                 $this->addLink('Login', $this->loginUrl);

Serialize/ExcelSerializer.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
         {
             return null;
         }
-	$data = $this->getArray($array);
+    $data = $this->getArray($array);
         $ssheat = new Spreadsheet();
         $sheat = $ssheat->getActiveSheet();
         $keys = array_shift($data);