If a function received argument of type List( String ) [1], some of whose elements were in fact Z9/References or some other unresolved type (e.g. [2]), those elements would not be resolved, causing validation to fail (among other defects). This should be rectified.
[1]
{
Z1K1: 'Z7',
Z7K1: 'Z881',
Z881K1: 'Z6'
}[2]
[ 'Z6', 'Z100041' ]
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Fully evaluate arguments before executing functions. | mediawiki/services/function-orchestrator | master | +251 -12 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Declined | None | T321827 Switch to a More Eager Evaluation Strategy | |||
| Resolved | cmassaro | T317230 Resolve Z18s, Z9s, and Z7s Which Are Not at the Top Level of an Argument |
We are going to hold off on solving this one explicitly. If the validator overhaul works correctly, this will no longer be an issue.
Note that I think this issue is not depending on the validation strategy (on the contrary, this issue would even be a blocker for a working validation strategy). We can fix and should fix it independently.
Change 861933 had a related patch set uploaded (by Cory Massaro; author: Cory Massaro):
[mediawiki/services/function-orchestrator@master] Fully evaluate arguments before executing functions.
Change 861933 merged by jenkins-bot:
[mediawiki/services/function-orchestrator@master] Fully evaluate arguments before executing functions.