laminas-api-tools / api-tools-admin

Laminas API Tools Admin module
https://api-tools.getlaminas.org/documentation
BSD 3-Clause "New" or "Revised" License
13 stars 21 forks source link

InArray Validator meta data not complete #37

Open michalbundyra opened 4 years ago

michalbundyra commented 4 years ago

The metadata for the InArray validator is not complete. It is missing the haystack key. However, this might have been intentional as there is no way to set an array to the haystack via the admin. But then you could argue why even have this validator in Apigility admin at all.

I solved this in a project by extending the InArray validator and adding a haystack and haystack_delimiter key to the validator metadata which allows me to pass a comma separated string to the setHaystack() method, exploding it to an array based on the haystack_delimiter key. Not sure if this is the best way to do it, but it is working for my needs.


namespace Validator;

use Zend\Validator\InArray as ZendInArray;

/**
 * Class InArray
 * @package Validator
 */
class InArray extends ZendInArray {

    /**
     * The delimiter to be used to explode the string to an array
     * @var string
     */
    protected $haystackDelimiter = ',';

    /**
     * @return string
     */
    public function getHaystackDelimiter()
    {
        return $this->haystackDelimiter;
    }

    /**
     * @param string $haystackDelimiter
     * @return InArray
     */
    public function setHaystackDelimiter($haystackDelimiter)
    {
        $this->haystackDelimiter = $haystackDelimiter;
        return $this;
    }

    /**
     * @param array|string $haystack
     * @return ZendInArray
     */
    public function setHaystack($haystack)
    {
        if( is_string($haystack) ) {
            if( ! strstr($haystack, $this->getHaystackDelimiter()) ) {
                $haystack = array($haystack);
            } else {
                $haystack = explode($this->getHaystackDelimiter(), $haystack);
            }
        }

        return parent::setHaystack($haystack);
    }
} 

And the config of:

return array(
    /**
     * Validators
     */
    'validators'=>array(
        'invokables'=>array(
            'Validator\InArray'            =>'Validator\InArray',
        ),
    ),

    /**
     * Validator MetaData
     */
    'validator_metadata' => array(
        'Validator\InArray'=> array(
            'strict' => 'bool',
            'recursive' => 'bool',
            'haystack'=>'string',
            'haystack_delimiter'=>'string',
        ),
    ),
);

Originally posted by @spectravp at https://github.com/zfcampus/zf-apigility-admin/issues/242