Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

BUG: Count Validator showing wrong error message #4034

Closed
1 task done
ghost opened this issue Feb 10, 2023 · 1 comment · Fixed by neos/neos-ui#3376
Closed
1 task done

BUG: Count Validator showing wrong error message #4034

ghost opened this issue Feb 10, 2023 · 1 comment · Fixed by neos/neos-ui#3376
Labels

Comments

@ghost
Copy link

ghost commented Feb 10, 2023

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

Currently when you have type: references and add a count validator the error message in backend is shown without the minimum and maximum integers that are in the yaml
The count must be between {minimum} and {maximum}.

Example from my Project:

products:
      type: references
      defaultValue: []
      ui:
        label: 'Products'
        reloadIfChanged: true
        inspector:
          group: 'products'
          hidden: 'ClientEval:node.properties.listType != "static"'
          editorOptions:
            nodeTypes:
              - 'Acme.Blog:Test.Test'
      validation:
        'Neos.Neos/Validation/CountValidator':
          minimum: '0'
          maximum: '3'

Expected Behavior

When you have type: references and add a count validator the error message in backend is shown with the minimum and maximum integers that are in the yaml
The count must be between 0 and 3.

Steps To Reproduce

  1. create property with type references
  2. Neos.Neos/Validation/CountValidator with minimum of 0 and maximum 3. (the numbers can of course be different)
  3. go into the backend and put 4 references in

Environment

- Flow: 8.1.2
- Neos: 8.1.2
- PHP: 8.1

Anything else?

No response

@ahaeslich
Copy link
Member

Thx for reporting. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant