@@ -121,51 +121,64 @@ |
||
| 121 | 121 | |
| 122 | 122 | foreach($table->columns() as $column_name => $column) |
| 123 | 123 | { |
| 124 | - if(isset($filters[$column_name]) && is_string($filters[$column_name])) |
|
| 125 | - $Query->aw_eq($column_name, $filters[$column_name]); |
|
| 124 | + if(isset($filters[$column_name]) && is_string($filters[$column_name])) { |
|
| 125 | + $Query->aw_eq($column_name, $filters[$column_name]); |
|
| 126 | + } |
|
| 126 | 127 | } |
| 127 | 128 | |
| 128 | 129 | if(is_subclass_of($event = new $class(), '\HexMakina\kadro\Models\Interfaces\EventInterface')) |
| 129 | 130 | { |
| 130 | - if(!empty($filters['date_start'])) |
|
| 131 | - $Query->aw_gte($event->event_field(), $filters['date_start'], $Query->table_label(), ':filter_date_start'); |
|
| 131 | + if(!empty($filters['date_start'])) { |
|
| 132 | + $Query->aw_gte($event->event_field(), $filters['date_start'], $Query->table_label(), ':filter_date_start'); |
|
| 133 | + } |
|
| 132 | 134 | |
| 133 | - if(!empty($filters['date_stop'])) |
|
| 134 | - $Query->aw_lte($event->event_field(), $filters['date_stop'], $Query->table_label(), ':filter_date_stop'); |
|
| 135 | + if(!empty($filters['date_stop'])) { |
|
| 136 | + $Query->aw_lte($event->event_field(), $filters['date_stop'], $Query->table_label(), ':filter_date_stop'); |
|
| 137 | + } |
|
| 135 | 138 | |
| 136 | - if(empty($options['order_by'])) |
|
| 137 | - $Query->order_by([$event->event_field(), 'DESC']); |
|
| 139 | + if(empty($options['order_by'])) { |
|
| 140 | + $Query->order_by([$event->event_field(), 'DESC']); |
|
| 141 | + } |
|
| 138 | 142 | } |
| 139 | 143 | |
| 140 | - if(isset($filters['content'])) $Query->aw_filter_content($filters['content']); |
|
| 144 | + if(isset($filters['content'])) { |
|
| 145 | + $Query->aw_filter_content($filters['content']); |
|
| 146 | + } |
|
| 141 | 147 | |
| 142 | 148 | if(isset($filters['ids'])) |
| 143 | 149 | { |
| 144 | - if(empty($filters['ids'])) |
|
| 145 | - $Query->and_where('1=0'); // TODO: this is a new low.. find another way to cancel query |
|
| 146 | - else $Query->aw_numeric_in('id', $filters['ids'], $Query->table_label()); |
|
| 150 | + if(empty($filters['ids'])) { |
|
| 151 | + $Query->and_where('1=0'); |
|
| 152 | + } |
|
| 153 | + // TODO: this is a new low.. find another way to cancel query |
|
| 154 | + else { |
|
| 155 | + $Query->aw_numeric_in('id', $filters['ids'], $Query->table_label()); |
|
| 156 | + } |
|
| 147 | 157 | } |
| 148 | 158 | |
| 149 | - if(isset($options['order_by'])) // TODO commenting required about the array situation |
|
| 159 | + if(isset($options['order_by'])) { |
|
| 160 | + // TODO commenting required about the array situation |
|
| 150 | 161 | { |
| 151 | 162 | $order_by = $options['order_by']; |
| 163 | + } |
|
| 152 | 164 | |
| 153 | - if(is_string($order_by)) |
|
| 154 | - $Query->order_by($order_by); |
|
| 155 | - |
|
| 156 | - elseif(is_array($order_by)) |
|
| 157 | - foreach($options['order_by'] as $order_by) |
|
| 165 | + if(is_string($order_by)) { |
|
| 166 | + $Query->order_by($order_by); |
|
| 167 | + } elseif(is_array($order_by)) { |
|
| 168 | + foreach($options['order_by'] as $order_by) |
|
| 158 | 169 | { |
| 159 | 170 | if(!isset($order_by[2])) |
| 160 | 171 | array_unshift($order_by, ''); |
| 172 | + } |
|
| 161 | 173 | |
| 162 | 174 | list($order_table, $order_field, $order_direction) = $order_by; |
| 163 | 175 | $Query->order_by([$order_table ?? '', $order_field, $order_direction]); |
| 164 | 176 | } |
| 165 | 177 | } |
| 166 | 178 | |
| 167 | - if(isset($options['limit']) && is_array($options['limit'])) |
|
| 168 | - $Query->limit($options['limit'][1], $options['limit'][0]); |
|
| 179 | + if(isset($options['limit']) && is_array($options['limit'])) { |
|
| 180 | + $Query->limit($options['limit'][1], $options['limit'][0]); |
|
| 181 | + } |
|
| 169 | 182 | |
| 170 | 183 | return $Query; |
| 171 | 184 | } |