Inclue RHS factResult after condition evaluation

Offen

#279 erstellt am 16.9.2021 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 2/5
Geschätzter Aufwand: 1-3 Stunden
Anfängerfreundlichkeit: 25/100

Issue-Typ: Feature
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: javascript
Bereich: backend

Rechercherichtung

Beginne mit src/rule.js bei den Zeilen 207-212 und vergleiche sein Verhalten mit examples/08-fact-comparison.js. Verfolge, wie die Auswertung der Bedingung das Regelergebnis aufbaut, und überprüfe anschließend, dass ein rechtsseitiges factResult zusätzlich zum bestehenden linksseitigen Ergebnis enthalten ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

We are using the functionality to allow the LHS attribute to be compared to a RHS attribute (demonstrated in this example) but noticed that in the rule result, only the LHS factResult is included. Would it be possible to also include the RHS factResult? I am not too familiar with the code but it seems like the logic is contained below

https://github.com/CacheControl/json-rules-engine/blob/90272d61ed7bea8b7d4000305511e932490d2e2e/src/rule.js#L207-L212

and a logic statement along the lines of
if (condition.value is an object) condition.value.factResult = evaluationResult.rightHandSideValue
could be added?

If there are no issues with this I am happy to open a PR

Vorherrschende Sprache: JavaScript
Sterne: 3.1k
Forks: 507
PR-Merge-Kennzahlen: Keine gemergten PRs in 30 T.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus CacheControl/json-rules-engine

Add evaluated fact(s) to onSuccess handler Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

CacheControl/json-rules-engine#427 · 1 Reaktion · Erstellt am 22.1.2026
Missing Changelog for Version 7.x - Unofficial Community Changelog Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 35/100

CacheControl/json-rules-engine#425 · Erstellt am 19.12.2025
fact params is not working when I am using fact in evaluationDetails > currentValue Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 48/100

CacheControl/json-rules-engine#424 · 1 Kommentar · Erstellt am 05.12.2025
How to handle array of objects? Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 25/100

CacheControl/json-rules-engine#421 · 1 Kommentar · Erstellt am 17.6.2025
jsonpath-plus expects browser to use ESM Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 38/100

CacheControl/json-rules-engine#417 · 1 Reaktion · Erstellt am 22.3.2025

Alle Issues in CacheControl/json-rules-engine