BurningFlipside / CommonCode

Auth/LDAPCachableObject.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      *
      * @param string $fieldName The name of the field to retrieve
      *
-     * @return mixed string the value of the field
+     * @return string string the value of the field
      */
     protected function getFieldSingleValue($fieldName)
     {

Auth/User.php

Doc Comments +1 added lines, -1 removed lines

@@ -311,7 +311,7 @@
      *
      * @param stdClass $data The user's new data
      *
-     * @return boolean true if the user's data was changed, false otherwise
+     * @return boolean|null true if the user's data was changed, false otherwise
      */
     public function editUser($data)
     {

AuthProvider.php

Doc Comments +4 added lines, -4 removed lines

@@ -253,10 +253,10 @@
      * Get an array of Auth\PendingUser from a filtered set
      *
      * @param Data\Filter|boolean $filter The filter conditions or false to retreive all
-     * @param array|boolean $select The user fields to obtain or false to obtain all
-     * @param integer|boolean $top The number of users to obtain or false to obtain all
-     * @param integer|boolean $skip The number of users to skip or false to skip none
-     * @param array|boolean $orderby The field to sort by and the method to sort or false to not sort
+     * @param boolean $select The user fields to obtain or false to obtain all
+     * @param boolean $top The number of users to obtain or false to obtain all
+     * @param boolean $skip The number of users to skip or false to skip none
+     * @param boolean $orderby The field to sort by and the method to sort or false to not sort
      * @param string|boolean $methodName The AuthMethod if information is desired only from a particular Auth\Authenticator
      *
      * @return array|boolean An array of Auth\PendingUser objects or false if no pending users were found

Data/FilterClause.php

Doc Comments +1 added lines, -1 removed lines

@@ -176,7 +176,7 @@
      * Convert the right hand side of the filter clause into an Mongo array
      *
      * @param string $op The standard operator
-     * @param string $var The second variable in the operator
+     * @param string $var2 The second variable in the operator
      *
      * @return array An array of mongo operations
      */

Indentation +10 added lines, -10 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:

Data/SQLDataSet.php

Doc Comments +1 added lines, -1 removed lines

@@ -150,7 +150,7 @@
      * @param string $select The colums to read
      * @param boolean|string $count The number of rows to read
      * @param boolean|string $skip The number of rows to skip over
-     * @param boolean|array $sort The array to sort by or false to not sort
+     * @param boolean $sort The array to sort by or false to not sort
      *
      * @return false|array An array of all the returned records
      */

Indentation +1 added lines, -1 removed lines

@@ -248,7 +248,7 @@
         if($stmt === false)
         {
             if (php_sapi_name() !== "cli") {
-              error_log('DB query failed. '.print_r($this->pdo->errorInfo(), true));
+                error_log('DB query failed. '.print_r($this->pdo->errorInfo(), true));
             }
             return false;
         }

Http/FlipAdminPage.php

Doc Comments +3 added lines

@@ -11,6 +11,9 @@
         $this->setTemplateName('admin.html');
     }
 
+    /**
+     * @param string $adminGroup
+     */
     protected function userIsAdmin($adminGroup)
     {
         if($this->user === false || $this->user === null)

Indentation +4 added lines, -4 removed lines

@@ -49,7 +49,7 @@ 
         $this->addLink('Home', '..');
         if($this->user === false || $this->user === null)
         {
-          $this->content['body'] = '
+            $this->content['body'] = '
             <div id="content">
               <div class="row">
                 <div class="col-lg-12">
@@ -61,7 +61,7 @@ 
         }
         else if($this->isAdmin() === false)
         {
-          $this->content['body'] = '
+            $this->content['body'] = '
             <div id="content">
               <div class="row">
                 <div class="col-lg-12">
@@ -70,8 +70,8 @@ 
               </div>
             </div>
           ';
-          $this->content['header'] = array();
-          $this->content['cards'] = array();
+            $this->content['header'] = array();
+            $this->content['cards'] = array();
         }
         return parent::getContent();
     }

Http/Rest/DataTableAPI.php

Doc Comments +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     /**
      * Create the DataTableAPI for the given DataTable info
      *
-     * @param string $datSetName The name of the DataSet used in the Settings
+     * @param string $dataSetName The name of the DataSet used in the Settings
      * @param string $dataTableName The name of the table in the DataSet
      * @param string|false $primaryKeyName The table primary key. Must be specified for update/delete capable tables
      */

Indentation +2 added lines, -2 removed lines

@@ -108,7 +108,7 @@ 
         $odata = $request->getAttribute('odata', new \Flipside\ODataParams(array()));
         $params = $this->manipulateParameters($request, $odata);
         $areas = $dataTable->read($odata->filter, $odata->select, $odata->top,
-                                  $odata->skip, $odata->orderby, $params);
+                                    $odata->skip, $odata->orderby, $params);
         if($areas === false)
         {
             $areas = array();
@@ -159,7 +159,7 @@ 
         $odata = $request->getAttribute('odata', new \Flipside\ODataParams(array()));
         $filter = $this->getFilterForPrimaryKey($args['name']);
         $areas = $dataTable->read($filter, $odata->select, $odata->top,
-                                  $odata->skip, $odata->orderby);
+                                    $odata->skip, $odata->orderby);
         if(empty($areas))
         {
             return $response->withStatus(404);

Http/Rest/SerializationMiddleware.php

Doc Comments +3 added lines

@@ -24,6 +24,9 @@
         }
     }
 
+    /**
+     * @param string $param
+     */
     private function getParamFromArrayIfSet($array, $param, $default = null)
     {
         if(isset($array[$param]))

Indentation +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
             //The underlying API call gave us back a different content type. Just pass that on...
             return $response;
         }
-	$overrides = $request->getAttribute('serializeOverrides');
+    $overrides = $request->getAttribute('serializeOverrides');
         if($overrides->has($this->format))
         {
             return $this->reserializeBody($response, $overrides->get($this->format));

Http/WebPage.php

Doc Comments +3 added lines, -1 removed lines

@@ -78,6 +78,9 @@ 
         $this->loader->addPath($dir, $namespace);
     }
 
+    /**
+     * @param string $name
+     */
     public function setTemplateName($name)
     {
         $this->templateName = $name;
@@ -112,7 +115,6 @@ 
      * Add a JavaScript file from a set of files known to the framework
      *
      * @param string $jsFileID the ID of the JS file
-     * @param boolean $async Can the JS file be loaded asynchronously?
      */
     public function addWellKnownJS($jsFileID)
     {

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');