Looking the code, I think using provided variables does not pass integrity check

Comments

fago created an issue. See original summary.

fago’s picture

fago
Primary language German
Location Vienna
commented
Related issues: +#2677026: [META] Make refining context and checking integrity work
fago’s picture

fago
Primary language German
Location Vienna
commented
Parent issue: » #2677026: [META] Make refining context and checking integrity work
Related issues: -#2677026: [META] Make refining context and checking integrity work
fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Active » Needs review

ok, it actually works :) Looks like it only misses context-refining (and does not use the prepared metadata state?)
Context-refining is dealt with in #2677028: Variables are not provided with refined context

Added test-coverage to prove it: https://github.com/fago/rules/pull/423

  • fago committed 660a505 on 8.x-3.x 
    Issue #2677042 by fago: Test that using provided variables passes...
fago’s picture

fago
Primary language German
Location Vienna
commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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