mbirth / cops

lib/CustomColumnTypeText.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
         while ($post = $result->fetchObject())
         {
             $entryPContent = str_format(localize("bookword", $post->count), $post->count);
-            $entryPLinkArray = array(new LinkNavigation ($this->getUri($post->id)));
+            $entryPLinkArray = array(new LinkNavigation($this->getUri($post->id)));
 
             $entry = new Entry($post->name, $this->getEntryId($post->id), $entryPContent, $this->datatype, $entryPLinkArray, "", $post->count);
 

Braces +4 added lines, -3 removed lines

@@ -67,8 +67,7 @@ 
 
         $result = $this->getDb()->query($query);
         $entryArray = array();
-        while ($post = $result->fetchObject())
-        {
+        while ($post = $result->fetchObject()) {
             $entryPContent = str_format(localize("bookword", $post->count), $post->count);
             $entryPLinkArray = array(new LinkNavigation ($this->getUri($post->id)));
 
@@ -82,7 +81,9 @@ 
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === NULL || empty($desc)) {
+            $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        }
         return $desc;
     }
 

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
         if ($post = $result->fetchObject()) {
             return new CustomColumn($id, $post->name, $this);
         }
-        return NULL;
+        return null;
     }
 
     protected function getAllCustomValuesFromDatabase()
@@ -82,7 +82,7 @@ 
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === null || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
         return $desc;
     }
 
@@ -95,7 +95,7 @@ 
         if ($post = $result->fetchObject()) {
             return new CustomColumn($post->id, $post->name, $this);
         }
-        return new CustomColumn(NULL, "", $this);
+        return new CustomColumn(null, "", $this);
     }
 
     public function isSearchable()

lib/CustomColumnType.php

Spacing +11 added lines, -11 removed lines

@@ -11,18 +11,18 @@ 
  */
 abstract class CustomColumnType extends Base
 {
-    const ALL_CUSTOMS_ID       = "cops:custom";
+    const ALL_CUSTOMS_ID = "cops:custom";
 
-    const CUSTOM_TYPE_TEXT      = "text";        // type 1 + 2
-    const CUSTOM_TYPE_COMMENT   = "comments";    // type 3
-    const CUSTOM_TYPE_SERIES    = "series";      // type 4
+    const CUSTOM_TYPE_TEXT      = "text"; // type 1 + 2
+    const CUSTOM_TYPE_COMMENT   = "comments"; // type 3
+    const CUSTOM_TYPE_SERIES    = "series"; // type 4
     const CUSTOM_TYPE_ENUM      = "enumeration"; // type 5
-    const CUSTOM_TYPE_DATE      = "datetime";    // type 6
-    const CUSTOM_TYPE_FLOAT     = "float";       // type 7
-    const CUSTOM_TYPE_INT       = "int";         // type 8
-    const CUSTOM_TYPE_RATING    = "rating";      // type 9
-    const CUSTOM_TYPE_BOOL      = "bool";        // type 10
-    const CUSTOM_TYPE_COMPOSITE = "composite";   // type 11 + 12
+    const CUSTOM_TYPE_DATE      = "datetime"; // type 6
+    const CUSTOM_TYPE_FLOAT     = "float"; // type 7
+    const CUSTOM_TYPE_INT       = "int"; // type 8
+    const CUSTOM_TYPE_RATING    = "rating"; // type 9
+    const CUSTOM_TYPE_BOOL      = "bool"; // type 10
+    const CUSTOM_TYPE_COMPOSITE = "composite"; // type 11 + 12
 
     /** @var array[integer]CustomColumnType  */
     private static $customColumnCacheID = array();
@@ -110,7 +110,7 @@ 
         $result->execute(array($this->customId));
         if ($post = $result->fetchObject()) {
             $json = json_decode($post->display);
-            return (isset($json->description) && !empty($json->description)) ? $json->description : NULL;
+            return (isset($json->description) && !empty($json->description))?$json->description:NULL;
         }
         return NULL;
     }

Braces +9 added lines, -6 removed lines

@@ -181,8 +181,9 @@ 
     public static function createByCustomID($customId)
     {
         // Reuse already created CustomColumns for performance
-        if (array_key_exists($customId, self::$customColumnCacheID))
-            return self::$customColumnCacheID[$customId];
+        if (array_key_exists($customId, self::$customColumnCacheID)) {
+                    return self::$customColumnCacheID[$customId];
+        }
 
         $datatype = self::getDatatypeByCustomID($customId);
 
@@ -221,8 +222,9 @@ 
     public static function createByLookup($lookup)
     {
         // Reuse already created CustomColumns for performance
-        if (array_key_exists($lookup, self::$customColumnCacheLookup))
-            return self::$customColumnCacheLookup[$lookup];
+        if (array_key_exists($lookup, self::$customColumnCacheLookup)) {
+                    return self::$customColumnCacheLookup[$lookup];
+        }
 
         $result = parent::getDb()->prepare('SELECT id FROM custom_columns WHERE label = ?');
         $result->execute(array($lookup));
@@ -241,8 +243,9 @@ 
     public function getAllCustomValues()
     {
         // lazy loading
-        if ($this->customValues == NULL)
-            $this->customValues = $this->getAllCustomValuesFromDatabase();
+        if ($this->customValues == NULL) {
+                    $this->customValues = $this->getAllCustomValuesFromDatabase();
+        }
 
         return $this->customValues;
     }

Upper-Lower-Casing +8 added lines, -8 removed lines

@@ -37,14 +37,14 @@ 
     /** @var string the datatype of this column (one of the CUSTOM_TYPE_* constant values) */
     public $datatype;
     /** @var null|Entry[] */
-    private $customValues = NULL;
+    private $customValues = null;
 
     protected function __construct($pcustomId, $pdatatype)
     {
         $this->columnTitle = self::getTitleByCustomID($pcustomId);
         $this->customId = $pcustomId;
         $this->datatype = $pdatatype;
-        $this->customValues = NULL;
+        $this->customValues = null;
     }
 
     /**
@@ -110,9 +110,9 @@ 
         $result->execute(array($this->customId));
         if ($post = $result->fetchObject()) {
             $json = json_decode($post->display);
-            return (isset($json->description) && !empty($json->description)) ? $json->description : NULL;
+            return (isset($json->description) && !empty($json->description)) ? $json->description : null;
         }
-        return NULL;
+        return null;
     }
 
     /**
@@ -168,7 +168,7 @@ 
         if ($post = $result->fetchObject()) {
             return $post->datatype;
         }
-        return NULL;
+        return null;
     }
 
     /**
@@ -206,7 +206,7 @@ 
             case self::CUSTOM_TYPE_BOOL:
                 return self::$customColumnCacheID[$customId] = new CustomColumnTypeBool($customId);
             case self::CUSTOM_TYPE_COMPOSITE:
-                return NULL; //TODO Currently not supported
+                return null; //TODO Currently not supported
             default:
                 throw new Exception("Unkown column type: " . $datatype);
         }
@@ -229,7 +229,7 @@ 
         if ($post = $result->fetchObject()) {
             return self::$customColumnCacheLookup[$lookup] = self::createByCustomID($post->id);
         }
-        return self::$customColumnCacheLookup[$lookup] = NULL;
+        return self::$customColumnCacheLookup[$lookup] = null;
     }
 
     /**
@@ -241,7 +241,7 @@ 
     public function getAllCustomValues()
     {
         // lazy loading
-        if ($this->customValues == NULL)
+        if ($this->customValues == null)
             $this->customValues = $this->getAllCustomValuesFromDatabase();
 
         return $this->customValues;

lib/CustomColumnTypeBool.php

Spacing +3 added lines, -3 removed lines

@@ -11,8 +11,8 @@ 
     // PHP pre 5.6 does not support const arrays
     private $BOOLEAN_NAMES = array(
         -1 => "customcolumn.boolean.unknown", // localize("customcolumn.boolean.unknown")
-        00 => "customcolumn.boolean.no",      // localize("customcolumn.boolean.no")
-        +1 => "customcolumn.boolean.yes",     // localize("customcolumn.boolean.yes")
+        00 => "customcolumn.boolean.no", // localize("customcolumn.boolean.no")
+        +1 => "customcolumn.boolean.yes", // localize("customcolumn.boolean.yes")
     );
 
     protected function __construct($pcustomId)
@@ -60,7 +60,7 @@ 
         $entryArray = array();
         while ($post = $result->fetchObject()) {
             $entryPContent = str_format(localize("bookword", $post->count), $post->count);
-            $entryPLinkArray = array(new LinkNavigation ($this->getUri($post->id)));
+            $entryPLinkArray = array(new LinkNavigation($this->getUri($post->id)));
 
             $entry = new Entry(localize($this->BOOLEAN_NAMES[$post->id]), $this->getEntryId($post->id), $entryPContent, $this->datatype, $entryPLinkArray, "", $post->count);
 

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
             $query = str_format(Book::SQL_BOOKS_BY_CUSTOM_BOOL_TRUE, "{0}", "{1}", $this->getTableName());
             return array($query, array());
         } else {
-            return NULL;
+            return null;
         }
     }
 

lib/CustomColumnTypeDate.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
             $id = $date->format("Y-m-d");
 
             $entryPContent = str_format(localize("bookword", $post->count), $post->count);
-            $entryPLinkArray = array(new LinkNavigation ($this->getUri($id)));
+            $entryPLinkArray = array(new LinkNavigation($this->getUri($id)));
 
             $entry = new Entry($date->format(localize("customcolumn.date.format")), $this->getEntryId($id), $entryPContent, $this->datatype, $entryPLinkArray, "", $post->count);
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -62,7 +62,7 @@ 
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === null || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
         return $desc;
     }
 
@@ -77,7 +77,7 @@ 
 
             return new CustomColumn($date->format("Y-m-d"), $date->format(localize("customcolumn.date.format")), $this);
         }
-        return new CustomColumn(NULL, localize("customcolumn.date.unknown"), $this);
+        return new CustomColumn(null, localize("customcolumn.date.unknown"), $this);
     }
 
     public function isSearchable()

Braces +3 added lines, -1 removed lines

@@ -55,7 +55,9 @@
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === NULL || empty($desc)) {
+            $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        }
         return $desc;
     }
 

lib/CustomColumnTypeRating.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
 
     public function getCustom($id)
     {
-        return new CustomColumn ($id, str_format(localize("customcolumn.stars", $id / 2), $id / 2), $this);
+        return new CustomColumn($id, str_format(localize("customcolumn.stars", $id / 2), $id / 2), $this);
     }
 
     protected function getAllCustomValuesFromDatabase()

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -100,7 +100,7 @@
         if ($post = $result->fetchObject()) {
             return new CustomColumn($post->value, str_format(localize("customcolumn.stars", $post->value / 2), $post->value / 2), $this);
         }
-        return new CustomColumn(NULL, localize("customcolumn.rating.unknown"), $this);
+        return new CustomColumn(null, localize("customcolumn.rating.unknown"), $this);
     }
 
     public function isSearchable()

lib/CustomColumnTypeFloat.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === null || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
         return $desc;
     }
 
@@ -68,7 +68,7 @@ 
         if ($post = $result->fetchObject()) {
             return new CustomColumn($post->value, $post->value, $this);
         }
-        return new CustomColumn(NULL, localize("customcolumn.float.unknown"), $this);
+        return new CustomColumn(null, localize("customcolumn.float.unknown"), $this);
     }
 
     public function isSearchable()

Braces +3 added lines, -1 removed lines

@@ -55,7 +55,9 @@
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === NULL || empty($desc)) {
+            $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        }
         return $desc;
     }
 

lib/CustomColumnTypeComment.php

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -41,13 +41,13 @@ 
 
     protected function getAllCustomValuesFromDatabase()
     {
-        return NULL;
+        return null;
     }
 
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === null || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
         return $desc;
     }
 
@@ -60,7 +60,7 @@ 
         if ($post = $result->fetchObject()) {
             return new CustomColumn($post->id, $post->value, $this);
         }
-        return new CustomColumn(NULL, localize("customcolumn.float.unknown"), $this);
+        return new CustomColumn(null, localize("customcolumn.float.unknown"), $this);
     }
 
     public function isSearchable()

Braces +3 added lines, -1 removed lines

@@ -55,7 +55,9 @@
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === NULL || empty($desc)) {
+            $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        }
         return $desc;
     }
 

lib/CustomColumnTypeInteger.php

Braces +3 added lines, -1 removed lines

@@ -55,7 +55,9 @@
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === NULL || empty($desc)) {
+            $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        }
         return $desc;
     }
 

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -55,7 +55,7 @@ 
     public function getDescription()
     {
         $desc = $this->getDatabaseDescription();
-        if ($desc === NULL || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
+        if ($desc === null || empty($desc)) $desc = str_format(localize("customcolumn.description"), $this->getTitle());
         return $desc;
     }
 
@@ -68,7 +68,7 @@ 
         if ($post = $result->fetchObject()) {
             return new CustomColumn($post->value, $post->value, $this);
         }
-        return new CustomColumn(NULL, localize("customcolumn.int.unknown"), $this);
+        return new CustomColumn(null, localize("customcolumn.int.unknown"), $this);
     }
 
     public function isSearchable()

lib/CustomColumnTypeSeries.php

Upper-Lower-Casing +2 added lines, -2 removed lines

@@ -57,7 +57,7 @@ 
         if ($post = $result->fetchObject()) {
             return new CustomColumn($id, $post->name, $this);
         }
-        return NULL;
+        return null;
     }
 
     protected function getAllCustomValuesFromDatabase()
@@ -92,7 +92,7 @@ 
         if ($post = $result->fetchObject()) {
             return new CustomColumn($post->id, $post->name . " [" . $post->extra . "]", $this);
         }
-        return new CustomColumn(NULL, "", $this);
+        return new CustomColumn(null, "", $this);
     }
 
     public function isSearchable()