thecodingmachine / database.tdbm

src/Mouf/Database/TDBM/Controllers/TdbmInstallController.php

Doc Comments +3 added lines, -2 removed lines

@@ -144,9 +144,7 @@ 
      *
      * @param string $daonamespace
      * @param string $beannamespace
-     * @param int    $keepSupport
      * @param int    $storeInUtc
-     * @param int    $castDatesToDateTime
      * @param string $selfedit
      *
      * @throws \Mouf\MoufException
@@ -183,6 +181,9 @@ 
 
     protected $errorMsg;
 
+    /**
+     * @param string $msg
+     */
     private function displayErrorMsg($msg)
     {
         $this->errorMsg = $msg;

src/Mouf/Database/TDBM/PageIterator.php

Doc Comments +4 added lines, -1 removed lines

@@ -60,6 +60,9 @@ 
 
     private $innerResultIterator;
 
+    /**
+     * @param integer $offset
+     */
     public function __construct(ResultIterator $parentResult, $magicSql, array $parameters, $limit, $offset, array $columnDescriptors, $objectStorage, $className, TDBMService $tdbmService, MagicQuery $magicQuery, $mode)
     {
         $this->parentResult = $parentResult;
@@ -108,7 +111,7 @@ 
     }
 
     /**
-     * @return int
+     * @return double
      */
     public function getCurrentPage()
     {

Spacing +2 added lines, -2 removed lines

@@ -112,7 +112,7 @@ 
      */
     public function getCurrentPage()
     {
-        return floor($this->offset / $this->limit) + 1;
+        return floor($this->offset/$this->limit)+1;
     }
 
     /**
@@ -251,7 +251,7 @@ 
      */
     public function jsonSerialize()
     {
-        return array_map(function (AbstractTDBMObject $item) {
+        return array_map(function(AbstractTDBMObject $item) {
             return $item->jsonSerialize();
         }, $this->toArray());
     }

src/Mouf/Database/TDBM/ResultIterator.php

Doc Comments +8 added lines

@@ -66,6 +66,13 @@ 
 
     private $mode;
 
+    /**
+     * @param string $magicSql
+     * @param string $magicSqlCount
+     * @param WeakrefObjectStorage $objectStorage
+     * @param string|null $className
+     * @param integer $mode
+     */
     public function __construct($magicSql, $magicSqlCount, array $parameters, array $columnDescriptors, $objectStorage, $className, TDBMService $tdbmService, MagicQuery $magicQuery, $mode)
     {
         $this->magicSql = $magicSql;
@@ -147,6 +154,7 @@ 
 
     /**
      * @param int $offset
+     * @param integer $limit
      *
      * @return PageIterator
      */

Spacing +1 added lines, -1 removed lines

@@ -241,7 +241,7 @@
      */
     public function jsonSerialize()
     {
-        return array_map(function (AbstractTDBMObject $item) {
+        return array_map(function(AbstractTDBMObject $item) {
             return $item->jsonSerialize();
         }, $this->toArray());
     }

src/Mouf/Database/TDBM/TDBMService.php

Doc Comments +3 added lines, -3 removed lines

@@ -777,7 +777,7 @@ 
      * This is used internally by TDBM to add an object to the list of objects that have been
      * created/updated but not saved yet.
      *
-     * @param AbstractTDBMObject $myObject
+     * @param DbRow $myObject
      */
     public function _addToToSaveObjectList(DbRow $myObject)
     {
@@ -1441,7 +1441,7 @@ 
     }
 
     /**
-     * @param $table
+     * @param string $table
      * @param array  $primaryKeys
      * @param array  $additionalTablesFetch
      * @param bool   $lazy                  Whether to perform lazy loading on this object or not.
@@ -1616,7 +1616,7 @@ 
      * @param $pivotTableName
      * @param AbstractTDBMObject $bean
      *
-     * @return AbstractTDBMObject[]
+     * @return ResultIterator
      */
     public function _getRelatedBeans($pivotTableName, AbstractTDBMObject $bean)
     {

Spacing +5 added lines, -5 removed lines

@@ -1201,7 +1201,7 @@ 
         sort($tables);
 
         return $this->fromCache($this->cachePrefix.'_linkbetweeninheritedtables_'.implode('__split__', $tables),
-            function () use ($tables) {
+            function() use ($tables) {
                 return $this->_getLinkBetweenInheritedTablesWithoutCache($tables);
             });
     }
@@ -1249,7 +1249,7 @@ 
      */
     public function _getRelatedTablesByInheritance($table)
     {
-        return $this->fromCache($this->cachePrefix.'_relatedtables_'.$table, function () use ($table) {
+        return $this->fromCache($this->cachePrefix.'_relatedtables_'.$table, function() use ($table) {
             return $this->_getRelatedTablesByInheritanceWithoutCache($table);
         });
     }
@@ -1509,7 +1509,7 @@ 
         $objects = $this->findObjects($mainTable, $filterString, $parameters, null, $additionalTablesFetch, self::MODE_ARRAY, $className);
         $page = $objects->take(0, 2);
         $count = $page->count();
-        if ($count > 1) {
+        if ($count>1) {
             throw new DuplicateRowException("Error while querying an object for table '$mainTable': More than 1 row have been returned, but we should have received at most one.");
         } elseif ($count === 0) {
             return;
@@ -1626,7 +1626,7 @@ 
         $remoteTable = $remoteFk->getForeignTableName();
 
         $primaryKeys = $this->getPrimaryKeyValues($bean);
-        $columnNames = array_map(function ($name) use ($pivotTableName) { return $pivotTableName.'.'.$name; }, $localFk->getLocalColumns());
+        $columnNames = array_map(function($name) use ($pivotTableName) { return $pivotTableName.'.'.$name; }, $localFk->getLocalColumns());
 
         $filter = array_combine($columnNames, $primaryKeys);
 
@@ -1647,7 +1647,7 @@ 
         $table1 = $fks[0]->getForeignTableName();
         $table2 = $fks[1]->getForeignTableName();
 
-        $beanTables = array_map(function (DbRow $dbRow) { return $dbRow->_getDbTableName(); }, $bean->_getDbRows());
+        $beanTables = array_map(function(DbRow $dbRow) { return $dbRow->_getDbTableName(); }, $bean->_getDbRows());
 
         if (in_array($table1, $beanTables)) {
             return [$fks[0], $fks[1]];

src/Mouf/Database/TDBM/Utils/TDBMDaoGenerator.php

Doc Comments +10 added lines, -5 removed lines

@@ -43,7 +43,6 @@ 
     /**
      * Constructor.
      *
-     * @param Connection $dbConnection The connection to the database.
      */
     public function __construct(SchemaAnalyzer $schemaAnalyzer, Schema $schema, TDBMSchemaAnalyzer $tdbmSchemaAnalyzer)
     {
@@ -100,6 +99,9 @@ 
      * Generates in one method call the daos and the beans for one table.
      *
      * @param $tableName
+     * @param string $daonamespace
+     * @param string $beannamespace
+     * @param boolean $storeInUtc
      */
     public function generateDaoAndBean(Table $table, $daonamespace, $beannamespace, ClassNameMapper $classNameMapper, $storeInUtc)
     {
@@ -140,7 +142,7 @@ 
     /**
      * Returns the name of the base bean class from the table name.
      *
-     * @param $tableName
+     * @param string $tableName
      *
      * @return string
      */
@@ -152,7 +154,7 @@ 
     /**
      * Returns the name of the base DAO class from the table name.
      *
-     * @param $tableName
+     * @param string $tableName
      *
      * @return string
      */
@@ -223,9 +225,10 @@ 
     /**
      * Writes the PHP bean DAO with simple functions to create/get/save objects.
      *
-     * @param string $fileName  The file that will be written (without the directory)
      * @param string $className The name of the class
-     * @param string $tableName The name of the table
+     * @param string $table The name of the table
+     * @param string $baseClassName
+     * @param string $beanClassName
      */
     public function generateDao($className, $baseClassName, $beanClassName, Table $table, $daonamespace, $beannamespace, ClassNameMapper $classNameMapper)
     {
@@ -468,6 +471,8 @@ 
      * Generates the factory bean.
      *
      * @param Table[] $tableList
+     * @param string $daoFactoryClassName
+     * @param string $daoNamespace
      */
     private function generateFactory(array $tableList, $daoFactoryClassName, $daoNamespace, ClassNameMapper $classNameMapper)
     {

Spacing +4 added lines, -4 removed lines

@@ -76,11 +76,11 @@ 
 
         // Remove all beans and daos from junction tables
         $junctionTables = $this->schemaAnalyzer->detectJunctionTables();
-        $junctionTableNames = array_map(function (Table $table) {
+        $junctionTableNames = array_map(function(Table $table) {
             return $table->getName();
         }, $junctionTables);
 
-        $tableList = array_filter($tableList, function (Table $table) use ($junctionTableNames) {
+        $tableList = array_filter($tableList, function(Table $table) use ($junctionTableNames) {
             return !in_array($table->getName(), $junctionTableNames);
         });
 
@@ -93,7 +93,7 @@ 
         // Ok, let's return the list of all tables.
         // These will be used by the calling script to create Mouf instances.
 
-        return array_map(function (Table $table) { return $table->getName(); }, $tableList);
+        return array_map(function(Table $table) { return $table->getName(); }, $tableList);
     }
 
     /**
@@ -558,7 +558,7 @@ 
                 $pos = strpos($str, ' ');
             }
             $before = substr($str, 0, $pos);
-            $after = substr($str, $pos + 1);
+            $after = substr($str, $pos+1);
             $str = $before.strtoupper(substr($after, 0, 1)).substr($after, 1);
         }
 

src/Mouf/Database/TDBM/Controllers/TdbmController.php

Spacing +4 added lines, -4 removed lines

@@ -126,14 +126,14 @@
         if (strpos($daonamespace, '\\') === 0) {
             $daonamespace = substr($daonamespace, 1);
         }
-        if (strpos($daonamespace, '\\') === strlen($daonamespace) - 1) {
-            $daonamespace = substr($daonamespace, 0, strlen($daonamespace) - 1);
+        if (strpos($daonamespace, '\\') === strlen($daonamespace)-1) {
+            $daonamespace = substr($daonamespace, 0, strlen($daonamespace)-1);
         }
         if (strpos($beannamespace, '\\') === 0) {
             $beannamespace = substr($beannamespace, 1);
         }
-        if (strpos($beannamespace, '\\') === strlen($beannamespace) - 1) {
-            $beannamespace = substr($beannamespace, 0, strlen($beannamespace) - 1);
+        if (strpos($beannamespace, '\\') === strlen($beannamespace)-1) {
+            $beannamespace = substr($beannamespace, 0, strlen($beannamespace)-1);
         }
 
         $tdbmService = new InstanceProxy($name);

src/Mouf/Database/TDBM/TDBMSchemaAnalyzer.php

Spacing +1 added lines, -1 removed lines

@@ -124,7 +124,7 @@
     public function getIncomingForeignKeys($tableName)
     {
         $junctionTables = $this->schemaAnalyzer->detectJunctionTables();
-        $junctionTableNames = array_map(function (Table $table) { return $table->getName(); }, $junctionTables);
+        $junctionTableNames = array_map(function(Table $table) { return $table->getName(); }, $junctionTables);
         $childrenRelationships = $this->schemaAnalyzer->getChildrenRelationships($tableName);
 
         $fks = [];

src/Mouf/Database/TDBM/AmbiguityException.php

Spacing +2 added lines, -2 removed lines

@@ -41,7 +41,7 @@ 
             echo 'Then, pass the $hint variable to your getObjects function.';
             echo '</div>';
 
-            $global_height_px += $height_px + 10;
+            $global_height_px += $height_px+10;
             ++$i;
         }
     }
@@ -61,7 +61,7 @@ 
         $possible_sub_paths = self::getAllPossiblePathsRec($sub_table_paths);
         $return_table_paths = array();
         foreach ($table_path['paths'] as $path) {
-            if (count($possible_sub_paths) > 0) {
+            if (count($possible_sub_paths)>0) {
                 foreach ($possible_sub_paths as $possible_sub_path) {
                     $return_table_paths[] = array_merge(array(array('paths' => array($path))), $possible_sub_path);
                 }

src/Mouf/Database/TDBM/Utils/BeanDescriptor.php

Spacing +4 added lines, -4 removed lines

@@ -91,7 +91,7 @@ 
      */
     public function getConstructorProperties()
     {
-        $constructorProperties = array_filter($this->beanPropertyDescriptors, function (AbstractBeanPropertyDescriptor $property) {
+        $constructorProperties = array_filter($this->beanPropertyDescriptors, function(AbstractBeanPropertyDescriptor $property) {
            return $property->isCompulsory();
         });
 
@@ -105,7 +105,7 @@ 
      */
     public function getExposedProperties()
     {
-        $exposedProperties = array_filter($this->beanPropertyDescriptors, function (AbstractBeanPropertyDescriptor $property) {
+        $exposedProperties = array_filter($this->beanPropertyDescriptors, function(AbstractBeanPropertyDescriptor $property) {
             return $property->getTable()->getName() == $this->table->getName();
         });
 
@@ -271,7 +271,7 @@ 
         $fksByMethodName = [];
 
         foreach ($fksByTable as $tableName => $fksForTable) {
-            if (count($fksForTable) > 1) {
+            if (count($fksForTable)>1) {
                 foreach ($fksForTable as $fk) {
                     $methodName = 'get'.TDBMDaoGenerator::toCamelCase($fk->getLocalTableName()).'By';
 
@@ -385,7 +385,7 @@ 
 
         $finalDescs = [];
         foreach ($descs as $descArray) {
-            if (count($descArray) > 1) {
+            if (count($descArray)>1) {
                 foreach ($descArray as $desc) {
                     $desc['name'] = TDBMDaoGenerator::toCamelCase($desc['remoteFK']->getForeignTableName()).'By'.TDBMDaoGenerator::toCamelCase($desc['table']->getName());
                     $finalDescs[] = $desc;