ublaboo / datagrid

src/Column/Action.php

Doc Comments +3 added lines, -2 removed lines

@@ -185,7 +185,7 @@ 
 	/**
 	 * Set icon
 	 * @param string|callable $icon
-	 * @return static|callable
+	 * @return Action
 	 * @throws DataGridException
 	 */
 	public function setIcon($icon)
@@ -270,7 +270,8 @@ 
 
 	/**
 	 * Check whether given property is string or callable
-	 * @param  mixed $property
+	 * @param  callable $property
+	 * @param string $name
 	 * @return void
 	 * @throws DataGridException
 	 */

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 use Nette\Utils\Html;
 use Ublaboo\DataGrid\DataGrid;
-use Ublaboo\DataGrid\Exception\DataGridHasToBeAttachedToPresenterComponentException;
 use Ublaboo\DataGrid\Exception\DataGridException;
 use Ublaboo\DataGrid\Exception\DataGridColumnRendererException;
 use Ublaboo\DataGrid\Row;

src/Column/ColumnStatus.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
 		$this->key = $key;
 
-		$this->setTemplate(__DIR__ . '/../templates/column_status.latte');
+		$this->setTemplate(__DIR__.'/../templates/column_status.latte');
 	}
 
 

src/Row.php

Doc Comments +1 added lines, -1 removed lines

@@ -219,7 +219,7 @@
 	/**
 	 * @param  string        $key
 	 * @param  Column\Column $column
-	 * @return void
+	 * @return Column\Column
 	 */
 	public function applyColumnCallback($key, Column\Column $column)
 	{

src/GroupAction/GroupSelectAction.php

Unused Use Statements -2 removed lines

@@ -8,8 +8,6 @@
 
 namespace Ublaboo\DataGrid\GroupAction;
 
-use Nette;
-
 /**
  * @method void onSelect()
  */

src/DataSource/ApiDataSource.php

Spacing +1 added lines, -1 removed lines

@@ -192,7 +192,7 @@
 		 */
 		foreach ($sorting->getSort() as $column => $order) {
 			$this->sort_column = $column;
-			$this->order_column  = $order;
+			$this->order_column = $order;
 		}
 
 		return $this;

src/DataSource/ArrayDataSource.php

Doc Comments +4 added lines, -4 removed lines

@@ -169,7 +169,7 @@ 
 	 * Filter by multi select value
 	 * @param  mixed  $row
 	 * @param  FilterMultiSelect $filter
-	 * @return void
+	 * @return boolean
 	 */
 	public function applyFilterMultiSelect($row, FilterMultiSelect $filter)
 	{
@@ -183,7 +183,7 @@ 
 	/**
 	 * @param  mixed  $row
 	 * @param  FilterRange $filter
-	 * @return void
+	 * @return boolean
 	 */
 	public function applyFilterRange($row, FilterRange $filter)
 	{
@@ -209,7 +209,7 @@ 
 	/**
 	 * @param  mixed  $row
 	 * @param  FilterDateRange $filter
-	 * @return void
+	 * @return boolean
 	 */
 	public function applyFilterDateRange($row, FilterDateRange $filter)
 	{
@@ -270,7 +270,7 @@ 
 	 * Apply fitler date and tell whether row value matches or not
 	 * @param  mixed  $row
 	 * @param  Filter $filter
-	 * @return mixed
+	 * @return boolean|null
 	 */
 	protected function applyFilterDate($row, FilterDate $filter)
 	{

Unused Use Statements -2 removed lines

@@ -14,9 +14,7 @@
 use Ublaboo\DataGrid\Filter\FilterRange;
 use Ublaboo\DataGrid\Filter\FilterDateRange;
 use Ublaboo\DataGrid\Filter\FilterText;
-use Nette\Utils\Callback;
 use Nette\Utils\Strings;
-use Ublaboo\DataGrid\Exception\DataGridException;
 use Ublaboo\DataGrid\Exception\DataGridArrayDataSourceException;
 use Ublaboo\DataGrid\Exception\DataGridDateTimeHelperException;
 use Ublaboo\DataGrid\Utils\DateTimeHelper;

src/DataGrid.php

Spacing +3 added lines, -3 removed lines

@@ -396,7 +396,7 @@ 
 		foreach ($items as $item) {
 			$rows[] = $row = new Row($this, $item, $this->getPrimaryKey());
 
-			if (!$hasGroupActionOnRows && $row->hasGroupAction()){
+			if (!$hasGroupActionOnRows && $row->hasGroupAction()) {
 				$hasGroupActionOnRows = TRUE;
 			}
 			
@@ -413,7 +413,7 @@ 
 			}
 		}
 
-		if ($hasGroupActionOnRows){
+		if ($hasGroupActionOnRows) {
 			$hasGroupActionOnRows = $this->hasGroupActions();
 		}
 
@@ -2762,7 +2762,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."
 			);
 		}
 

Unused Use Statements -1 removed lines

@@ -18,7 +18,6 @@
 use Ublaboo\DataGrid\Exception\DataGridFilterNotFoundException;
 use Ublaboo\DataGrid\Exception\DataGridHasToBeAttachedToPresenterComponentException;
 use Ublaboo\DataGrid\Filter\IFilterDate;
-use Ublaboo\DataGrid\Utils\DateTimeHelper;
 use Ublaboo\DataGrid\Utils\Sorting;
 use Ublaboo\DataGrid\InlineEdit\InlineEdit;
 use Ublaboo\DataGrid\ColumnsSummary;

Doc Comments +3 added lines, -4 removed lines

@@ -1428,7 +1428,6 @@ 
 
 	/**
 	 * If we want to sent some initial filter
-	 * @param array $filter
 	 * @param bool  $use_on_reset
 	 * @return static
 	 */
@@ -2499,7 +2498,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)
@@ -2818,7 +2817,7 @@ 
 
 	/**
 	 * Get items detail parameters
-	 * @return array
+	 * @return Column\ItemDetail
 	 */
 	public function getItemsDetail()
 	{
@@ -2892,7 +2891,7 @@ 
 
 
 	/**
-	 * @return Nette\Forms\Container|NULL
+	 * @return Utils\ItemDetailForm|null
 	 */
 	public function getItemDetailForm()
 	{

src/Column/ColumnLink.php

Spacing +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
 		$element = $a;
 
 		if ($this->icon) {
-			$a->addHtml(Html::el('span')->class(DataGrid::$icon_prefix . $this->icon));
+			$a->addHtml(Html::el('span')->class(DataGrid::$icon_prefix.$this->icon));
 
 			if (strlen($value)) {
 				$a->addHtml(' ');

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 use Nette\Utils\Html;
 use Ublaboo\DataGrid\DataGrid;
 use Ublaboo\DataGrid\Row;
-use Ublaboo\DataGrid\Exception\DataGridHasToBeAttachedToPresenterComponentException;
 use Ublaboo\DataGrid\Exception\DataGridColumnRendererException;
 
 class ColumnLink extends Column

src/DataSource/NetteDatabaseTableDataSource.php

Spacing +7 added lines, -7 removed lines

@@ -60,7 +60,7 @@ 
 		}
 
 		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)
 		);
 	}
 
@@ -171,8 +171,8 @@ 
 			$like = '(';
 			$args = [];
 
-			if($filter->isExactSearch()){
-				$like .=  "$column = ? OR ";
+			if ($filter->isExactSearch()) {
+				$like .= "$column = ? OR ";
 				$args[] = "$value";
 			} else {
 				if ($filter->hasSplitWordsSearch() === FALSE) {
@@ -185,7 +185,7 @@ 
 					$args[] = "%$word%";
 				}
 			}
-			$like = substr($like, 0, strlen($like) - 4) . ')';
+			$like = substr($like, 0, strlen($like) - 4).')';
 
 			$or[] = $like;
 			$big_or .= "$like OR ";
@@ -223,9 +223,9 @@ 
 		
 			foreach ($values as $value) {
 				if ($i == $length) {
-					$or .= $filter->getColumn() . ' = ?)';
+					$or .= $filter->getColumn().' = ?)';
 				} else {
-					$or .= $filter->getColumn() . ' = ? OR ';
+					$or .= $filter->getColumn().' = ? OR ';
 				}
 
 				$i++;
@@ -235,7 +235,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));
 		}
 	}