ublaboo / datagrid

src/DataSource/DoctrineDataSource.php

Indentation +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 
 	/**
 	 * @return Doctrine\ORM\Query
-	*/
+	 */
 	public function getQuery()
 	{
 		return $this->data_source->getQuery();

Spacing +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
 		$or = [];
 
 		foreach ($condition as $column => $value) {
-			if($filter->isExactSearch()){
+			if ($filter->isExactSearch()) {
 				$this->data_source->where("$column = %s", $value);
 				continue;
 			}

src/Column/Action.php

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\Row;
 

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
 	 */

src/Column/ColumnLink.php

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\Row;
 

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(' ');

src/DataModel.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 
 namespace Ublaboo\DataGrid;
 
-use Nette;
 use DibiFluent;
 use DibiOdbcDriver;
 use DibiMsSqlDriver;

Spacing +4 added lines, -4 removed lines

@@ -148,17 +148,17 @@
 	/**
 	 * @param array
 	 */
-	public function aggregationRow(array $columns){
+	public function aggregationRow(array $columns) {
 		$hasAggregation = FALSE;
 		/** @var Column $column */
-		foreach($columns as $column){
-			if($column->hasAggregationFunction()){
+		foreach ($columns as $column) {
+			if ($column->hasAggregationFunction()) {
 				$hasAggregation = TRUE;
 				$aggregationFunction = $column->getAggregationFunction();
 				$this->data_source->addAggregationColumn($aggregationFunction->getAggregationType(), $aggregationFunction->getColumn());
 			}
 		}
-		if(!$hasAggregation){
+		if (!$hasAggregation) {
 			return FALSE;
 		}
 		$this->data_source->getAggregationData();

src/DataSource/DibiFluentDataSource.php

Unused Use Statements -1 removed lines

@@ -9,7 +9,6 @@
 namespace Ublaboo\DataGrid\DataSource;
 
 use DibiFluent;
-use Nette\Utils\Callback;
 use Nette\Utils\Strings;
 use Ublaboo\DataGrid\Filter;
 

Doc Comments +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 
 
 	/**
-	 * @param DibiFluent $data_source
+	 * @param Dibi\Fluent $data_source
 	 * @param string $primary_key
 	 */
 	public function __construct(\Dibi\Fluent $data_source, $primary_key)

Spacing +3 added lines, -3 removed lines

@@ -171,7 +171,7 @@ 
 				\dibi::IDENTIFIER
 			);
 
-			if ($filter->isExactSearch()){
+			if ($filter->isExactSearch()) {
 				$this->data_source->where("$column = %s", $value);
 				continue;
 			}
@@ -303,10 +303,10 @@ 
 
 	public function addAggregationColumn($aggregation_type, $column) {
 
-		$this->aggregation_data_source = $this->aggregation_data_source->select($aggregation_type .'(%n)', $column)->as($column);
+		$this->aggregation_data_source = $this->aggregation_data_source->select($aggregation_type.'(%n)', $column)->as($column);
 	}
 
-	public function getAggregationData(){
+	public function getAggregationData() {
 		$data = $this->aggregation_data_source->from($this->data_source)->as('data')->fetch();
 		return $data;
 	}

src/DataSource/FilterableDataSource.php

Unused Use Statements -2 removed lines

@@ -8,9 +8,7 @@
 
 namespace Ublaboo\DataGrid\DataSource;
 
-use DibiFluent;
 use Nette\Utils\Callback;
-use Nette\Utils\Strings;
 use Ublaboo\DataGrid\Filter;
 
 abstract class FilterableDataSource

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/DataSource/ArrayDataSource.php

Unused Use Statements -2 removed lines

@@ -10,9 +10,7 @@
 
 use Ublaboo\DataGrid\Filter\Filter;
 use Ublaboo\DataGrid\Filter\FilterDate;
-use Nette\Utils\Callback;
 use Nette\Utils\Strings;
-use Ublaboo\DataGrid\Exception\DataGridException;
 use Ublaboo\DataGrid\Exception\DataGridArrayDataSourceException;
 use Ublaboo\DataGrid\Utils\DateTimeHelper;
 use Ublaboo\DataGrid\Exception\DataGridDateTimeHelperException;

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)
 	{

Spacing +9 added lines, -9 removed lines

@@ -389,23 +389,23 @@ 
 	public function getAggregationData()
 	{
 		$result = [];
-		foreach($this->data as $row){
-			foreach($this->aggregations as $column => $aggregation_type){
-				switch($aggregation_type){
+		foreach ($this->data as $row) {
+			foreach ($this->aggregations as $column => $aggregation_type) {
+				switch ($aggregation_type) {
 					case ColumnAggregationFunction::$aggregation_type_sum:
 					case ColumnAggregationFunction::$aggregation_type_avg:
-						if(!isset($result[$column])){
+						if (!isset($result[$column])) {
 							$result[$column] = 0;
 						}
 						$result[$column] += $row[$column];
 						break;
 					case ColumnAggregationFunction::$aggregation_type_min:
-						if(!isset($result[$column]) || $row[$column] < $result[$column]){
+						if (!isset($result[$column]) || $row[$column] < $result[$column]) {
 							$result[$column] = $row[$column];
 						}
 						break;
 					case ColumnAggregationFunction::$aggregation_type_max:
-						if(!isset($result[$column]) || $row[$column] > $result[$column]){
+						if (!isset($result[$column]) || $row[$column] > $result[$column]) {
 							$result[$column] = $row[$column];
 						}
 						break;
@@ -413,9 +413,9 @@ 
 			}
 		}
 
-		foreach($this->aggregations as $column => $aggregation_type) {
-			if($aggregation_type == ColumnAggregationFunction::$aggregation_type_avg){
-				$result[$column] =  $result[$column] / count($this->data);
+		foreach ($this->aggregations as $column => $aggregation_type) {
+			if ($aggregation_type == ColumnAggregationFunction::$aggregation_type_avg) {
+				$result[$column] = $result[$column] / count($this->data);
 			}
 		}
 		return $result;

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)
 	{