[#ZF-6711] Zend_Filter_Input skips ALLOW_EMPTY metacommand processing if rule refers several fields. - Zend Framework Issue Tracker

Zend Framework

Zend_Filter_Input skips ALLOW_EMPTY metacommand processing if rule refers several fields.

Created: 18/May/09 07:34 AM Updated: 21/May/09 05:30 AM Resolved: 21/May/09 05:30 AM

Component/s:

Zend_Filter_Input

Affects Version/s:

None

Fix Version/s:

1.8.2

Time Tracking:

Not Specified

Issue Links:

Related

This issue is related to:

~~ZF-6725~~ [ZF-6711] fix produces an unit tests error

Language:

English

Description

« Hide

It uses:

if (count((array) $validatorRule[self::FIELDS]) > 1) {
    if (!$validatorRule[self::VALIDATOR_CHAIN]->isValid($data)) {
        $this->_invalidMessages[$validatorRule[self::RULE]] = $validatorRule[self::VALIDATOR_CHAIN]->getMessages();
        $this->_invalidErrors[$validatorRule[self::RULE]] = $validatorRule[self::VALIDATOR_CHAIN]->getErrors();
        return;
    }
} else {
...

construction.

if (count((array) $validatorRule[self::FIELDS]) == 1) {
...

actually should be used here.

But this code also skips ALLOW_EMPTY metacommand processing for this special case. So it should be removed at all.

Sort Order:

[ Permalink | « Hide ]

Alexander Veremyev added a comment - 18/May/09 07:55 AM

Fixed.

[ Permalink | « Hide ]

Alexander Veremyev added a comment - 21/May/09 02:18 AM

Issue is reopened.

It's correct to process several fields as a special case. But it still needs ALLOW_EMPTY metacommand processing.

[ Permalink | « Hide ]

Alexander Veremyev added a comment - 21/May/09 05:30 AM

Fixed.