Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

$value is an uncountable object error in V::size validator when calling filterBy with between #3382

Closed
texnixe opened this issue May 28, 2021 · 1 comment · Fixed by #3384
Closed

$value is an uncountable object error in V::size validator when calling filterBy with between #3382

texnixe opened this issue May 28, 2021 · 1 comment · Fixed by #3384
Assignees
@afbora
Milestone
3.5.7

Comments

@texnixe
Copy link
Member

texnixe commented May 28, 2021 •
edited by afbora
Loading

Describe the bug

Example code that causes the error $value is an uncountable object in kirby/src/Toolkit/V.php on line 478.

<?php dump($page->children()->filterBy('height', 'between', [2, 7])); ?>

The error occurs because $value is a field object in this case.

Kirby Version
Tested with 3.5.5 and 3.5.6-RC-1
@afbora afbora self-assigned this May 29, 2021
@afbora afbora added this to the 3.6.x milestone May 29, 2021
afbora added a commit that referenced this issue May 29, 2021
@afbora
Supports countable field object #3382
6d72037
@afbora afbora mentioned this issue May 29, 2021
Merged
5 tasks
@afbora afbora linked a pull request May 29, 2021 that will close this issue
Supports countable field objects #3384
Merged
5 tasks
@lukasbestle lukasbestle modified the milestones: 3.6.x, 3.6.0 May 29, 2021
distantnative pushed a commit that referenced this issue Jun 8, 2021
@afbora
Supports countable field object #3382
c393806
@distantnative distantnative modified the milestones: 3.6.0, 3.5.7 Jun 8, 2021
@afbora
Copy link
Member

afbora commented Jun 8, 2021

✅

@afbora afbora closed this as completed Jun 8, 2021
afbora added a commit that referenced this issue Jun 21, 2021
@afbora
Supports countable field object #3382
2e18815
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@afbora afbora

Labels
None yet
Milestone
3.5.7
Development

Successfully merging a pull request may close this issue.

Supports countable field objects getkirby/kirby
4 participants
@lukasbestle @afbora @distantnative @texnixe