[pp-1] Update use of Range constraint with min 0 to use PositiveOrZero instead

Created on 30 June 2025, 7 days ago

Problem/Motivation

In 🐛 Implement constraints for Pager config Active we enabled the symfony PositiveOrZero constraint because it is more expressive than using Range with a minimum of zero.
When that is in we should update existing uses of Range with a minimum of zero to use the more expressive PositiveOrZero - this is that issue.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

📌 Task
Status

Postponed

Version

11.0 🔥

Component

configuration system

Created by

🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @larowlan
  • 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10

    Blocker is in

  • 🇧🇪Belgium borisson_ Mechelen, 🇧🇪

    Tagging this issue as novice, we already have test coverage so all that really needs to change is in the config schemas for these changes.

  • First commit to issue fork.
  • Pipeline finished with Success
    4 days ago
    Total: 611s
    #537818
  • 🇮🇳India ajinkya45 Pune

    MR ready for review

  • 🇧🇪Belgium borisson_ Mechelen, 🇧🇪

    This is missing instances in:

    1. core/modules/search/config/schema/search.schema.yml
    2. user.flood in core/modules/user/config/schema/user.schema.yml
    3. core/modules/dblog/config/schema/dblog.schema.yml
    4. core/modules/navigation/config/schema/navigation.schema.yml
    5. core/modules/announcements_feed/config/schema/announcements_feed.schema.yml
    6. core/modules/automated_cron/config/schema/automated_cron.schema.yml
    7. core/modules/system/tests/modules/icon_test/config/schema/icon_test.schema.yml

    I used a very inaccurate grep to find these: rg --multiline --multiline-dotall 'Range:(.*)min: 0', but I manually checked these and those should be fixed as well. Please make sure to check all schema files.

  • Pipeline finished with Success
    4 days ago
    Total: 589s
    #538012
  • 🇮🇳India ajinkya45 Pune

    Thanks for the detailed list! I've gone through all the .schema.yml files and updated the `Range(min: 0)` instances to PositiveOrZero wherever applicable. Let me know if I missed anything!

Production build 0.71.5 2024