dragan Posted February 1, 2019 Share Posted February 1, 2019 Today I came across a strange warning message (orange): I created a repeater, assigned a few fields. One field is a checkbox. Below that are a bunch of fields (text, image type). The fields below the checkbox were set to required only if checkbox=1. Which I have used a million times in one combination or another. But after saving the repeater field, I got an orange warning banner at the top: "required if conditions are not possible within repeaters". (translated from German, from the top of my head). Funny thing is: It all works, as expected. I searched the open issues on Github but found nothing. Anyone else seen something like this? Perhaps it's a legacy warning message that should have been removed by now? PW version: one of the last 1-2 (dev) Link to comment Share on other sites More sharing options...
Robin S Posted February 2, 2019 Share Posted February 2, 2019 I just tested and couldn't find any problem with required if conditions inside a repeater - like you said, they seem to work fine. And I tried it some while ago and couldn't find a problem then either: So maybe there are some specific types of required if conditions that don't work, or maybe it was a limitation that was later fixed. Do you want to file a GitHub issue so Ryan can take a look and either clarify the types of condition that don't work or else remove the notice? Edit: scratch all that - the required if conditions don't work as expected inside a repeater. At first it seems that they do work, because when you fulfill the required if condition the asterisk appears and the field is indeed required. But the problem is that field is always required, so even if you don't fulfill the required if condition and the asterisk is removed the field is still required. Link to comment Share on other sites More sharing options...
Robin S Posted February 3, 2019 Share Posted February 3, 2019 I did some testing to see what is needed to get required-if conditions working within repeaters and opened a request here: https://github.com/processwire/processwire-requests/issues/262 2 Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now