Correct exception type

[delacry]

Hi, shouldn't \Ublaboo\DataGrid\Exception\DataGridException extend \LogicException instead of \Exception?

Because I don't want to propagate nor catch those exceptions, they appear to be in methods like setSortable or setRenderer and my IDE (PhpStorm) is always reporting there is exception that is not handled, but those kind of exceptions are logic exceptions, only if there is a serious bug caused by code, the exceptions should be thrown, and there is no reason to catch them, only avoid them.

[radimvaculik]

I'm in. cc @paveljanda @f3l1x

[paveljanda]

It is a good idea. But sadly, not all those exception are Logic. Some could be Runtime.

[delacry]

@paveljanda that's cool, because runtime exception is also ignored by default in phpstorm

[f3l1x]

I think we should introduce Runtime and Logic exception for datagrid.

Logical for dev errors .
Runtime for runtime errors.