refs #976: fixed Datatable name validation

stwe / DatatablesBundle

This Bundle integrates the jQuery DataTables plugin into your Symfony application.

355 stars 236 forks source link

refs #976: fixed Datatable name validation #977

Closed Eloar closed 2 years ago

Eloar commented 2 years ago

added test for invalid name validation
changed thrown exception from \Exception to more specific \LogicException
updated documentation

As validation worked only if datatable name was composed of only invalid characters exception type change might not be considered backward incompatible.

Adresses #976 issue

Eloar commented 2 years ago

@stephanvierkant would you like to review and merge that fix?

stephanvierkant commented 2 years ago

Looks good to me!