| Conditions | 20 |
| Paths | 372 |
| Total Lines | 119 |
| Code Lines | 73 |
| Lines | 0 |
| Ratio | 0 % |
| Changes | 0 | ||
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
| 1 | <?php |
||
| 85 | protected function getColumnsList(string $mainTable, array $additionalTablesFetch = array(), $orderBy = null, bool $canAddAdditionalTablesFetch = false): array |
||
| 86 | { |
||
| 87 | // From the table name and the additional tables we want to fetch, let's build a list of all tables |
||
| 88 | // that must be part of the select columns. |
||
| 89 | |||
| 90 | $connection = $this->tdbmService->getConnection(); |
||
| 91 | |||
| 92 | $tableGroups = []; |
||
| 93 | $allFetchedTables = $this->tdbmService->_getRelatedTablesByInheritance($mainTable); |
||
| 94 | $tableGroupName = $this->getTableGroupName($allFetchedTables); |
||
| 95 | foreach ($allFetchedTables as $table) { |
||
| 96 | $tableGroups[$table] = $tableGroupName; |
||
| 97 | } |
||
| 98 | |||
| 99 | $columnsList = []; |
||
| 100 | $columnDescList = []; |
||
| 101 | $sortColumn = 0; |
||
| 102 | $reconstructedOrderBy = null; |
||
| 103 | |||
| 104 | if (is_string($orderBy)) { |
||
| 105 | $orderBy = trim($orderBy); |
||
| 106 | if ($orderBy === '') { |
||
| 107 | $orderBy = null; |
||
| 108 | } |
||
| 109 | } |
||
| 110 | |||
| 111 | // Now, let's deal with "order by columns" |
||
| 112 | if ($orderBy !== null) { |
||
| 113 | $securedOrderBy = true; |
||
| 114 | $reconstructedOrderBys = []; |
||
| 115 | if ($orderBy instanceof UncheckedOrderBy) { |
||
| 116 | $securedOrderBy = false; |
||
| 117 | $orderBy = $orderBy->getOrderBy(); |
||
| 118 | $reconstructedOrderBy = $orderBy; |
||
| 119 | } |
||
| 120 | $orderByColumns = $this->orderByAnalyzer->analyzeOrderBy($orderBy); |
||
| 121 | |||
| 122 | // If we sort by a column, there is a high chance we will fetch the bean containing this column. |
||
| 123 | // Hence, we should add the table to the $additionalTablesFetch |
||
| 124 | foreach ($orderByColumns as $orderByColumn) { |
||
| 125 | if ($orderByColumn['type'] === 'colref') { |
||
| 126 | if ($orderByColumn['table'] !== null) { |
||
| 127 | if ($canAddAdditionalTablesFetch) { |
||
| 128 | $additionalTablesFetch[] = $orderByColumn['table']; |
||
| 129 | } else { |
||
| 130 | $sortColumnName = 'sort_column_'.$sortColumn; |
||
| 131 | $mysqlPlatform = new MySqlPlatform(); |
||
| 132 | $columnsList[] = $mysqlPlatform->quoteIdentifier($orderByColumn['table']).'.'.$mysqlPlatform->quoteIdentifier($orderByColumn['column']).' as '.$sortColumnName; |
||
| 133 | $columnDescList[$sortColumnName] = [ |
||
| 134 | 'tableGroup' => null, |
||
| 135 | ]; |
||
| 136 | ++$sortColumn; |
||
| 137 | } |
||
| 138 | } |
||
| 139 | if ($securedOrderBy) { |
||
| 140 | // Let's protect via MySQL since we go through MagicJoin |
||
| 141 | $mysqlPlatform = new MySqlPlatform(); |
||
| 142 | $reconstructedOrderBys[] = ($orderByColumn['table'] !== null ? $mysqlPlatform->quoteIdentifier($orderByColumn['table']).'.' : '').$mysqlPlatform->quoteIdentifier($orderByColumn['column']).' '.$orderByColumn['direction']; |
||
| 143 | } |
||
| 144 | } elseif ($orderByColumn['type'] === 'expr') { |
||
| 145 | $sortColumnName = 'sort_column_'.$sortColumn; |
||
| 146 | $columnsList[] = $orderByColumn['expr'].' as '.$connection->quoteIdentifier($sortColumnName); |
||
| 147 | $columnDescList[$sortColumnName] = [ |
||
| 148 | 'tableGroup' => null, |
||
| 149 | ]; |
||
| 150 | ++$sortColumn; |
||
| 151 | |||
| 152 | if ($securedOrderBy) { |
||
| 153 | throw new TDBMInvalidArgumentException('Invalid ORDER BY column: "'.$orderByColumn['expr'].'". If you want to use expression in your ORDER BY clause, you must wrap them in a UncheckedOrderBy object. For instance: new UncheckedOrderBy("col1 + col2 DESC")'); |
||
| 154 | } |
||
| 155 | } |
||
| 156 | } |
||
| 157 | |||
| 158 | if ($reconstructedOrderBy === null) { |
||
| 159 | $reconstructedOrderBy = implode(', ', $reconstructedOrderBys); |
||
| 160 | } |
||
| 161 | } |
||
| 162 | |||
| 163 | foreach ($additionalTablesFetch as $additionalTable) { |
||
| 164 | if (in_array($additionalTable, $allFetchedTables, true)) { |
||
| 165 | continue; |
||
| 166 | } |
||
| 167 | |||
| 168 | $relatedTables = $this->tdbmService->_getRelatedTablesByInheritance($additionalTable); |
||
| 169 | $tableGroupName = $this->getTableGroupName($relatedTables); |
||
| 170 | foreach ($relatedTables as $table) { |
||
| 171 | $tableGroups[$table] = $tableGroupName; |
||
| 172 | } |
||
| 173 | $allFetchedTables = array_merge($allFetchedTables, $relatedTables); |
||
| 174 | } |
||
| 175 | |||
| 176 | // Let's remove any duplicate |
||
| 177 | $allFetchedTables = array_unique($allFetchedTables); |
||
| 178 | |||
| 179 | // We quote in MySQL because MagicJoin requires MySQL style quotes |
||
| 180 | $mysqlPlatform = new MySqlPlatform(); |
||
| 181 | |||
| 182 | // Now, let's build the column list |
||
| 183 | foreach ($allFetchedTables as $table) { |
||
| 184 | foreach ($this->schema->getTable($table)->getColumns() as $column) { |
||
| 185 | $columnName = $column->getName(); |
||
| 186 | $alias = self::getColumnAlias($table, $columnName); |
||
| 187 | $columnDescList[$alias] = [ |
||
| 188 | 'as' => $alias, |
||
| 189 | 'table' => $table, |
||
| 190 | 'column' => $columnName, |
||
| 191 | 'type' => $column->getType(), |
||
| 192 | 'tableGroup' => $tableGroups[$table], |
||
| 193 | ]; |
||
| 194 | $columnsList[] = sprintf( |
||
| 195 | '%s.%s as %s', |
||
| 196 | $mysqlPlatform->quoteIdentifier($table), |
||
| 197 | $mysqlPlatform->quoteIdentifier($columnName), |
||
| 198 | $connection->quoteIdentifier($alias) |
||
| 199 | ); |
||
| 200 | } |
||
| 201 | } |
||
| 202 | |||
| 203 | return [$columnDescList, $columnsList, $reconstructedOrderBy]; |
||
| 204 | } |
||
| 310 |