The schema for rules_expression is wrong (found in rules.expression.schema.yml) - it is defined as a type: mapping but there is no mapping given. Because of this, all the other schemas that inherit from the rules_expression type end up defining their own id: and uuid: elements - this should not be necessary because all the rules_expression.* schemas should inherit these elements from the parent rules_expression.

Patch fixes this. Manually tested this using the config_inspector module and all configured Rules on my site still pass schema validation after the patch.

NOTE: This patch is a prelude to fixing #2648334: Implement action/condition reordering with drag & drop because we can't add a 'weight' element that is inherited by all expressions unless the schemas are properly defined.

CommentFileSizeAuthor
rules_expression_schema.patch1.95 KBtr

Comments

TR created an issue. See original summary.

tr’s picture

Issue summary: View changes

  • TR committed 4249742 on 8.x-3.x
    Issue #3099653 by TR: Fix rules_expression.* configuration schema
    
tr’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.