Completed
Push — 4.3 ( 28cdbc )
by David
21:20 queued 01:32
created
src/Mouf/Database/TDBM/Utils/ObjectBeanPropertyDescriptor.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
     {
66 66
         // First, are there many column or only one?
67 67
         // If one column, we name the setter after it. Otherwise, we name the setter after the table name
68
-        if (count($this->foreignKey->getLocalColumns()) > 1) {
68
+        if (count($this->foreignKey->getLocalColumns())>1) {
69 69
             $name = TDBMDaoGenerator::toSingular(TDBMDaoGenerator::toCamelCase($this->foreignKey->getForeignTableName()));
70 70
             if ($this->alternativeName) {
71 71
                 $camelizedColumns = array_map(['Mouf\\Database\\TDBM\\Utils\\TDBMDaoGenerator', 'toCamelCase'], $this->foreignKey->getLocalColumns());
@@ -78,8 +78,8 @@  discard block
 block discarded – undo
78 78
             if (strpos(strtolower($column), 'id_') === 0) {
79 79
                 $column = substr($column, 3);
80 80
             }
81
-            if (strrpos(strtolower($column), '_id') === strlen($column) - 3) {
82
-                $column = substr($column, 0, strlen($column) - 3);
81
+            if (strrpos(strtolower($column), '_id') === strlen($column)-3) {
82
+                $column = substr($column, 0, strlen($column)-3);
83 83
             }
84 84
             $name = TDBMDaoGenerator::toCamelCase($column);
85 85
             if ($this->alternativeName) {
Please login to merge, or discard this patch.
src/Mouf/Database/TDBM/InnerResultArray.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 
84 84
     private function toIndex($offset)
85 85
     {
86
-        if ($offset < 0 || filter_var($offset, FILTER_VALIDATE_INT) === false) {
86
+        if ($offset<0 || filter_var($offset, FILTER_VALIDATE_INT) === false) {
87 87
             throw new TDBMInvalidOffsetException('Trying to access result set using offset "'.$offset.'". An offset must be a positive integer.');
88 88
         }
89 89
         if ($this->statement === null) {
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
     public function next()
101 101
     {
102 102
         // Let's overload the next() method to store the result.
103
-        if (isset($this->results[$this->key + 1])) {
103
+        if (isset($this->results[$this->key+1])) {
104 104
             ++$this->key;
105 105
             $this->current = $this->results[$this->key];
106 106
         } else {
Please login to merge, or discard this patch.
src/Mouf/Database/TDBM/PageIterator.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
      */
113 113
     public function getCurrentPage()
114 114
     {
115
-        return floor($this->offset / $this->limit) + 1;
115
+        return floor($this->offset/$this->limit)+1;
116 116
     }
117 117
 
118 118
     /**
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
      */
252 252
     public function jsonSerialize()
253 253
     {
254
-        return array_map(function (AbstractTDBMObject $item) {
254
+        return array_map(function(AbstractTDBMObject $item) {
255 255
             return $item->jsonSerialize();
256 256
         }, $this->toArray());
257 257
     }
Please login to merge, or discard this patch.
src/Mouf/Database/TDBM/Controllers/TdbmInstallController.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -181,6 +181,9 @@
 block discarded – undo
181 181
 
182 182
     protected $errorMsg;
183 183
 
184
+    /**
185
+     * @param string $msg
186
+     */
184 187
     private function displayErrorMsg($msg)
185 188
     {
186 189
         $this->errorMsg = $msg;
Please login to merge, or discard this patch.
src/Mouf/Database/TDBM/Controllers/TdbmController.php 1 patch
Doc Comments   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
      * @Action
87 87
      *
88 88
      * @param string $name
89
-     * @param bool   $selfedit
89
+     * @param string|boolean   $selfedit
90 90
      */
91 91
     public function generate($name, $daonamespace, $beannamespace, $daofactoryclassname, $daofactoryinstancename, $storeInUtc = 0, $selfedit = 'false', $useCustomComposer = false, $composerFile = '')
92 92
     {
@@ -109,6 +109,8 @@  discard block
 block discarded – undo
109 109
      * @param string      $daofactoryinstancename
110 110
      * @param string      $selfedit
111 111
      * @param bool        $storeInUtc
112
+     * @param boolean $useCustomComposer
113
+     * @param string $composerFile
112 114
      *
113 115
      * @throws \Mouf\MoufException
114 116
      */
Please login to merge, or discard this patch.
src/Mouf/Database/TDBM/ResultIterator.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -245,7 +245,7 @@
 block discarded – undo
245 245
      */
246 246
     public function jsonSerialize($stopRecursion = false)
247 247
     {
248
-        return array_map(function (AbstractTDBMObject $item) use ($stopRecursion) {
248
+        return array_map(function(AbstractTDBMObject $item) use ($stopRecursion) {
249 249
             return $item->jsonSerialize($stopRecursion);
250 250
         }, $this->toArray());
251 251
     }
Please login to merge, or discard this patch.
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -62,6 +62,10 @@  discard block
 block discarded – undo
62 62
 
63 63
     private $logger;
64 64
 
65
+    /**
66
+     * @param null|string $className
67
+     * @param integer $mode
68
+     */
65 69
     public function __construct(QueryFactory $queryFactory, array $parameters, $objectStorage, $className, TDBMService $tdbmService, MagicQuery $magicQuery, $mode, LoggerInterface $logger)
66 70
     {
67 71
         if ($mode !== null && $mode !== TDBMService::MODE_CURSOR && $mode !== TDBMService::MODE_ARRAY) {
@@ -300,7 +304,6 @@  discard block
 block discarded – undo
300 304
      *
301 305
      *  $resultSet = $resultSet->withParameters('label ASC, status DESC');
302 306
      *
303
-     * @param string|UncheckedOrderBy|null $orderBy
304 307
      *
305 308
      * @return ResultIterator
306 309
      */
Please login to merge, or discard this patch.
src/Mouf/Database/TDBM/AlterableResultIterator.php 1 patch
Doc Comments   +4 added lines, -3 removed lines patch added patch discarded remove patch
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
     /**
69 69
      * Adds an additional object to the result set (if not already available).
70 70
      *
71
-     * @param $object
71
+     * @param AbstractTDBMObject $object
72 72
      */
73 73
     public function add($object)
74 74
     {
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
     /**
86 86
      * Removes an object from the result set.
87 87
      *
88
-     * @param $object
88
+     * @param AbstractTDBMObject $object
89 89
      */
90 90
     public function remove($object)
91 91
     {
@@ -205,6 +205,7 @@  discard block
 block discarded – undo
205 205
 
206 206
     /**
207 207
      * @param int $offset
208
+     * @param integer $limit
208 209
      *
209 210
      * @return \Porpaginas\Page
210 211
      */
@@ -227,7 +228,7 @@  discard block
 block discarded – undo
227 228
     /**
228 229
      * Return an iterator over all results of the paginatable.
229 230
      *
230
-     * @return Iterator
231
+     * @return \Iterator
231 232
      */
232 233
     public function getIterator()
233 234
     {
Please login to merge, or discard this patch.
src/Mouf/Database/TDBM/Utils/TDBMDaoGenerator.php 2 patches
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -83,11 +83,11 @@  discard block
 block discarded – undo
83 83
 
84 84
         // Remove all beans and daos from junction tables
85 85
         $junctionTables = $this->schemaAnalyzer->detectJunctionTables(true);
86
-        $junctionTableNames = array_map(function (Table $table) {
86
+        $junctionTableNames = array_map(function(Table $table) {
87 87
             return $table->getName();
88 88
         }, $junctionTables);
89 89
 
90
-        $tableList = array_filter($tableList, function (Table $table) use ($junctionTableNames) {
90
+        $tableList = array_filter($tableList, function(Table $table) use ($junctionTableNames) {
91 91
             return !in_array($table->getName(), $junctionTableNames);
92 92
         });
93 93
 
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
         // Ok, let's return the list of all tables.
101 101
         // These will be used by the calling script to create Mouf instances.
102 102
 
103
-        return array_map(function (Table $table) {
103
+        return array_map(function(Table $table) {
104 104
             return $table->getName();
105 105
         }, $tableList);
106 106
     }
@@ -294,7 +294,7 @@  discard block
 block discarded – undo
294 294
         $usedBeans[] = $beanClassName;
295 295
         // Let's suppress duplicates in used beans (if any)
296 296
         $usedBeans = array_flip(array_flip($usedBeans));
297
-        $useStatements = array_map(function ($usedBean) {
297
+        $useStatements = array_map(function($usedBean) {
298 298
             return "use $usedBean;\n";
299 299
         }, $usedBeans);
300 300
 
@@ -644,7 +644,7 @@  discard block
 block discarded – undo
644 644
                 $pos = strpos($str, ' ');
645 645
             }
646 646
             $before = substr($str, 0, $pos);
647
-            $after = substr($str, $pos + 1);
647
+            $after = substr($str, $pos+1);
648 648
             $str = $before.strtoupper(substr($after, 0, 1)).substr($after, 1);
649 649
         }
650 650
 
Please login to merge, or discard this patch.
Doc Comments   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -156,7 +156,7 @@  discard block
 block discarded – undo
156 156
     /**
157 157
      * Returns the name of the base bean class from the table name.
158 158
      *
159
-     * @param $tableName
159
+     * @param string $tableName
160 160
      *
161 161
      * @return string
162 162
      */
@@ -168,7 +168,7 @@  discard block
 block discarded – undo
168 168
     /**
169 169
      * Returns the name of the base DAO class from the table name.
170 170
      *
171
-     * @param $tableName
171
+     * @param string $tableName
172 172
      *
173 173
      * @return string
174 174
      */
@@ -186,6 +186,7 @@  discard block
 block discarded – undo
186 186
      * @param Table           $table           The table
187 187
      * @param string          $beannamespace   The namespace of the bean
188 188
      * @param ClassNameMapper $classNameMapper
189
+     * @param boolean $storeInUtc
189 190
      *
190 191
      * @throws TDBMException
191 192
      */
@@ -548,6 +549,8 @@  discard block
 block discarded – undo
548 549
      * Generates the factory bean.
549 550
      *
550 551
      * @param Table[] $tableList
552
+     * @param string $daoFactoryClassName
553
+     * @param string $daoNamespace
551 554
      */
552 555
     private function generateFactory(array $tableList, $daoFactoryClassName, $daoNamespace, ClassNameMapper $classNameMapper)
553 556
     {
@@ -656,7 +659,7 @@  discard block
 block discarded – undo
656 659
      * Tries to put string to the singular form (if it is plural).
657 660
      * We assume the table names are in english.
658 661
      *
659
-     * @param $str string
662
+     * @param string $str string
660 663
      *
661 664
      * @return string
662 665
      */
Please login to merge, or discard this patch.
src/Mouf/Database/TDBM/TDBMSchemaAnalyzer.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@
 block discarded – undo
124 124
     public function getIncomingForeignKeys($tableName)
125 125
     {
126 126
         $junctionTables = $this->schemaAnalyzer->detectJunctionTables(true);
127
-        $junctionTableNames = array_map(function (Table $table) {
127
+        $junctionTableNames = array_map(function(Table $table) {
128 128
             return $table->getName();
129 129
         }, $junctionTables);
130 130
         $childrenRelationships = $this->schemaAnalyzer->getChildrenRelationships($tableName);
Please login to merge, or discard this patch.