haystack in Zend\Validator\Inarray removed after remove other field

laminas-api-tools / api-tools-admin-ui

Laminas API Tools Admin UI module

https://api-tools.getlaminas.org/documentation

BSD 3-Clause "New" or "Revised" License

6 stars 11 forks source link

haystack in Zend\Validator\Inarray removed after remove other field #5

Open weierophinney opened 4 years ago

weierophinney commented 4 years ago

steps to reproduce:

define fields: foo and bar
bar field has Zend\Validator\Inarray, as we can't set haystack via GUI, then set manually at module.config.php
remove foo field via GUI

actual result: "haystack" option in bar validator definitions removed. expected result: "haystack" option in bar validator definitions should not be removed.

Originally posted by @samsonasik at https://github.com/zfcampus/zf-apigility-admin-ui/issues/138

weierophinney commented 4 years ago

@samsonasik I was looking on it, and it seems to be a bit more complex issue. It needs changes also in zf-apigility-admin. Basically we need add option haystack in validator_metadata config: https://github.com/zfcampus/zf-apigility-admin/blob/master/config/module.config.php#L1400-L1403 but then also in admin-ui we need to add support for array option values (now it works only with bool/int/string)...

Originally posted by @michalbundyra at https://github.com/zfcampus/zf-apigility-admin-ui/issues/138#issuecomment-282259503