Grids icon indicating copy to clipboard operation
Grids copied to clipboard

Added ability to use SQL "table.field" notation as FieldConfig name

Open nachtsheim opened this issue 8 years ago • 3 comments

Example: If i use field name "u.id" now first will be tried to read "id" property and then "u". If something went wrong, now exceptions are thrown.

nachtsheim avatar Jul 26 '16 06:07 nachtsheim

Hi! Thanks for contribution. Very interesting improvement. It's definently needful.

Now I afraid that it can broke backward compatibility & this package has no tests in opposition to https://github.com/view-components/grids so merging will be not easy.

Can you modify pull request or create another for merging it into 'dev' branch?

Regarding this part:

$res = is_object($res) ? $res->{$part} : $res[$part];

I think it requires isset check.

todo: check that it works with DbalDataProvider

Nayjest avatar Jul 26 '16 12:07 Nayjest

One more question: why you don't use view-components/grids ? Contributing there would be much easier.

Nayjest avatar Jul 26 '16 12:07 Nayjest

It also should work when filtering (currently: ambiguous column)

amenk avatar Jan 17 '17 17:01 amenk