Query does not works with array of search criterias

[EthraZa]

Array of multiple search criteria is not working, just the first one works.

// Does not works, just the subject criteria is respected -(
$folder->query()->where([['subject' => 'foo'], ['UNSEEN']]);

// Works, all criterias are respected -)
$folder->query()->where(['subject' => 'foo'])->where(['UNSEEN']);

Looking the docs at Custom search criteria, since it uses an array of arrays, I came to the conclusion that multiple search criterias shoud work but, it may be not true. Yet, it would be cool if it works.

[EthraZa]

Loos like the where method just return the first instead to push it to the array.

So, as a quick solution I did my own function to increment wheres:

public function addWhere(object $query, array $search): object
    {
        foreach ($search as $k => $v) {
            if (is_array($v)) {
                $query = $this->addWhere($query, $v);
            } else if (is_numeric($k)) {
                $query->where($v);
            } else {
                $query->where([$k => $v]);
            }
            
        }

        return $query;
}

// Usage
$query = $this->addWhere($folder->query(), [['subject' => 'foo'], ['UNSEEN']]);

Maybe this logic can be imported to the where method itself in the future. ;)

[Webklex]

Hi @EthraZa ,
what do you think of replacing

php-imap/src/Query/WhereQuery.php

Lines 126 to 146 in 23c80eb

	public function where($criteria, $value = null) {
	if (is_array($criteria)) {
	foreach ($criteria as $key => $value) {
	if (is_numeric($key)) {
	return $this->where($value);
	}
	return $this->where($key, $value);
	}
	} else {
	$criteria = $this->validate_criteria($criteria);
	$value = $this->parse_value($value);
	if ($value === null || $value === '') {
	$this->query->push([$criteria]);
	} else {
	$this->query->push([$criteria, $value]);
	}
	}
	return $this;
	}

with this:

public function where($criteria, $value = null) {
    if (is_array($criteria)) {
        foreach ($criteria as $key => $value) {
            if (is_numeric($key)) {
                $this->where($value);
            }
            $this->where($key, $value);
        }
    } else {
        $criteria = $this->validate_criteria($criteria);
        $value = $this->parse_value($value);

        if ($value === null || $value === '') {
            $this->query->push([$criteria]);
        } else {
            $this->query->push([$criteria, $value]);
        }
    }

    return $this;
}

I believe this would work as well and would allow multiple search criteria as long as the provider accepts them:

$query = $folder->query()->where([['subject' => 'foo'], ['UNSEEN']]);

Best regards,

Update 2021-11-04:
I just pushed ae26ff5 which solved a stupid little problem in my suggestion from above and also works with all old formats. So everybody wins :) Thanks for your suggestion!

[Webklex]

Hi @EthraZa ,
this change is now available via the v3.0.0-alpha release.

Best regards,