ublaboo / datagrid

src/DataGrid.php

Doc Comments +1 added lines, -2 removed lines

@@ -1532,7 +1532,6 @@ 
 	/**
 	 * If we want to sent some initial filter
 	 *
-	 * @param array $filter
 	 * @param bool  $use_on_reset
 	 * @return static
 	 */
@@ -2766,7 +2765,7 @@ 
 /**
  * Set options of select "items_per_page"
  *
- * @param array $items_per_page_list
+ * @param integer[] $items_per_page_list
  * @return static
  */
 	public function setItemsPerPageList(array $items_per_page_list, $include_all = true)

Spacing +4 added lines, -4 removed lines

@@ -673,7 +673,7 @@ 
 	 */
 	public function getOriginalTemplateFile(): string
 	{
-		return __DIR__ . '/templates/datagrid.latte';
+		return __DIR__.'/templates/datagrid.latte';
 	}
 
 
@@ -929,7 +929,7 @@ 
 		 * Set tree view template file
 		 */
 		if (!$this->template_file) {
-			$this->setTemplateFile(__DIR__ . '/templates/datagrid_tree.latte');
+			$this->setTemplateFile(__DIR__.'/templates/datagrid_tree.latte');
 		}
 
 		return $this;
@@ -3006,7 +3006,7 @@ 
  */
 	public function getSessionSectionName(): string
 	{
-		return $this->getPresenter()->getName() . ':' . $this->getUniqueId();
+		return $this->getPresenter()->getName().':'.$this->getUniqueId();
 	}
 
 
@@ -3596,7 +3596,7 @@ 
 
 		if (!($parent instanceof PresenterComponent)) {
 			throw new DataGridHasToBeAttachedToPresenterComponentException(
-				"DataGrid is attached to: '" . get_class($parent) . "', but instance of PresenterComponent is needed."
+				"DataGrid is attached to: '".get_class($parent)."', but instance of PresenterComponent is needed."
 			);
 		}
 

src/Filter/Filter.php

Doc Comments +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
 	/**
 	 * Get filter key
 	 *
-	 * @return mixed
+	 * @return string
 	 */
 	public function getKey()
 	{

src/Traits/TButtonRenderer.php

Doc Comments +1 added lines, -1 removed lines

@@ -117,7 +117,7 @@
 	 * Set column replacements
 	 *
 	 * @param  array $replacements
-	 * @return Column
+	 * @return TButtonRenderer
 	 */
 	public function setReplacement(array $replacements): Column
 	{

src/Traits/TLink.php

Doc Comments +1 added lines

@@ -79,6 +79,7 @@
 
 	/**
 	 * @throws DataGridLinkCreationException
+	 * @param string $href
 	 */
 	private function throwHierarchyLookupException(DataGrid $grid, $href, $params): void
 	{

Spacing +5 added lines, -5 removed lines

@@ -82,14 +82,14 @@
 	 */
 	private function throwHierarchyLookupException(DataGrid $grid, $href, $params): void
 	{
-		$desiredHandler = get_class($grid->getParent()) . '::handle' . ucfirst($href) . '()';
+		$desiredHandler = get_class($grid->getParent()).'::handle'.ucfirst($href).'()';
 
 		throw new DataGridLinkCreationException(
 			'DataGrid could not create link "'
-			. $href . '" - did not find any signal handler in componenet hierarchy from '
-			. get_class($grid->getParent()) . ' up to the '
-			. get_class($grid->getPresenter()) . '. '
-			. 'Try adding handler ' . $desiredHandler
+			. $href.'" - did not find any signal handler in componenet hierarchy from '
+			. get_class($grid->getParent()).' up to the '
+			. get_class($grid->getPresenter()).'. '
+			. 'Try adding handler '.$desiredHandler
 		);
 	}
 

src/DataSource/NextrasDataSource.php

Spacing +7 added lines, -7 removed lines

@@ -105,8 +105,8 @@ 
 			$date_end = clone $date;
 
 			$this->data_source = $this->data_source->findBy([
-				$this->prepareColumn($column) . '>=' => $date->setTime(0, 0, 0),
-				$this->prepareColumn($column) . '<=' => $date_end->setTime(23, 59, 59),
+				$this->prepareColumn($column).'>=' => $date->setTime(0, 0, 0),
+				$this->prepareColumn($column).'<=' => $date_end->setTime(23, 59, 59),
 			]);
 		}
 
@@ -130,12 +130,12 @@ 
 		$dataCondition = [];
 		if ($value_from) {
 			$date_from = DateTimeHelper::tryConvertToDateTime($value_from, [$filter->getPhpFormat()]);
-			$dataCondition[$this->prepareColumn($filter->getColumn()) . '>='] = $date_from->setTime(0, 0, 0);
+			$dataCondition[$this->prepareColumn($filter->getColumn()).'>='] = $date_from->setTime(0, 0, 0);
 		}
 
 		if ($value_to) {
 			$date_to = DateTimeHelper::tryConvertToDateTime($value_to, [$filter->getPhpFormat()]);
-			$dataCondition[$this->prepareColumn($filter->getColumn()) . '<='] = $date_to->setTime(23, 59, 59);
+			$dataCondition[$this->prepareColumn($filter->getColumn()).'<='] = $date_to->setTime(23, 59, 59);
 		}
 
 		if (!empty($dataCondition)) {
@@ -160,11 +160,11 @@ 
 		$dataCondition = [];
 
 		if ($value_from) {
-			$dataCondition[$this->prepareColumn($filter->getColumn()) . '>='] = $value_from;
+			$dataCondition[$this->prepareColumn($filter->getColumn()).'>='] = $value_from;
 		}
 
 		if ($value_to) {
-			$dataCondition[$this->prepareColumn($filter->getColumn()) . '<='] = $value_to;
+			$dataCondition[$this->prepareColumn($filter->getColumn()).'<='] = $value_to;
 		}
 
 		if (!empty($dataCondition)) {
@@ -315,7 +315,7 @@ 
 	private function prepareColumn(string $column): string
 	{
 		if (Strings::contains($column, '.')) {
-			return 'this->' . str_replace('.', '->', $column);
+			return 'this->'.str_replace('.', '->', $column);
 		}
 		return $column;
 	}

src/DataSource/DoctrineDataSource.php

Spacing +3 added lines, -3 removed lines

@@ -87,7 +87,7 @@ 
 			$this->root_alias = current($this->root_alias);
 		}
 
-		return $this->root_alias . '.' . $column;
+		return $this->root_alias.'.'.$column;
 	}
 
 
@@ -288,7 +288,7 @@ 
 		$p = $this->getPlaceholder();
 
 		$values = $filter->getCondition()[$filter->getColumn()];
-		$expr = $this->data_source->expr()->in($c, ':' . $p);
+		$expr = $this->data_source->expr()->in($c, ':'.$p);
 
 		$this->data_source->andWhere($expr)->setParameter($p, $values);
 	}
@@ -371,7 +371,7 @@ 
 	 */
 	public function getPlaceholder(): int
 	{
-		return 'param' . ($this->placeholder++);
+		return 'param'.($this->placeholder++);
 	}
 
 

src/DataSource/NetteDatabaseTableDataSource.php

Spacing +8 added lines, -8 removed lines

@@ -62,7 +62,7 @@ 
 		} catch (\LogicException $e) {
 			if ($data_source_sql_builder->getGroup() !== '') {
 				return $this->data_source->count(
-					'DISTINCT ' . Strings::replace($data_source_sql_builder->getGroup(), '~ (DESC|ASC)~')
+					'DISTINCT '.Strings::replace($data_source_sql_builder->getGroup(), '~ (DESC|ASC)~')
 				);
 			}
 
@@ -71,11 +71,11 @@ 
 
 		if ($data_source_sql_builder->getGroup() !== '') {
 			return $this->data_source->count(
-				'DISTINCT ' . Strings::replace($data_source_sql_builder->getGroup(), '~ (DESC|ASC)~')
+				'DISTINCT '.Strings::replace($data_source_sql_builder->getGroup(), '~ (DESC|ASC)~')
 			);
 		} else {
 			return $this->data_source->count(
-				$this->data_source->getName() . '.' . (is_array($primary) ? reset($primary) : $primary)
+				$this->data_source->getName().'.'.(is_array($primary) ? reset($primary) : $primary)
 			);
 		}
 	}
@@ -206,7 +206,7 @@ 
 					$args[] = "%$word%";
 				}
 			}
-			$like = substr($like, 0, strlen($like) - 4) . ')';
+			$like = substr($like, 0, strlen($like) - 4).')';
 
 			$or[] = $like;
 			$big_or .= "$like OR ";
@@ -214,7 +214,7 @@ 
 		}
 
 		if (sizeof($or) > 1) {
-			$big_or = substr($big_or, 0, strlen($big_or) - 4) . ')';
+			$big_or = substr($big_or, 0, strlen($big_or) - 4).')';
 
 			$query = array_merge([$big_or], $big_or_args);
 
@@ -245,9 +245,9 @@ 
 
 			foreach ($values as $value) {
 				if ($i === $length) {
-					$or .= $filter->getColumn() . ' = ?)';
+					$or .= $filter->getColumn().' = ?)';
 				} else {
-					$or .= $filter->getColumn() . ' = ? OR ';
+					$or .= $filter->getColumn().' = ? OR ';
 				}
 
 				$i++;
@@ -257,7 +257,7 @@ 
 
 			call_user_func_array([$this->data_source, 'where'], $values);
 		} else {
-			$this->data_source->where($filter->getColumn() . ' = ?', reset($values));
+			$this->data_source->where($filter->getColumn().' = ?', reset($values));
 		}
 	}
 

src/AggregationFunction/FunctionSum.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
 		if ($dataSource instanceof QueryBuilder) {
 			$column = Strings::contains($this->column, '.')
 				? $this->column
-				: current($dataSource->getRootAliases()) . '.' . $this->column;
+				: current($dataSource->getRootAliases()).'.'.$this->column;
 
 			$this->result = $dataSource
 				->select(sprintf('SUM(%s)', $column))
@@ -75,7 +75,7 @@ 
 		}
 
 		if ($dataSource instanceof Collection) {
-			$dataSource->forAll(function ($key, $value) {
+			$dataSource->forAll(function($key, $value) {
 				$this->result += PropertyAccessHelper::getValue($value, $this->column);
 			});
 		}

src/AggregationFunction/TDataGridAggregationFunction.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
 		}
 
 		if (isset($this->aggregationFunctions[$key])) {
-			throw new DataGridException('There is already a AggregationFunction defined on column ' . $key);
+			throw new DataGridException('There is already a AggregationFunction defined on column '.$key);
 		}
 
 		if ($this->multipleAggregationFunction instanceof IMultipleAggregationFunction) {