Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ForeachSpec, RuleOutcome, Builders, flere rulesets #80

Merged
merged 11 commits into from
Feb 17, 2022

Conversation

jolarsen
Copy link
Collaborator

@jolarsen jolarsen commented Sep 12, 2021

Opprinnelig var tanken å produsere output i specification og sende videre til neste regel. Dessuten parmetrisere RuleReasonRef<T,C> for outcome og beregnet og som konsekvens også Evaluation. Det ville blitt mye omskriving.

Denne PRen støtter muteringspraksis og legger opp til å transportere ut utfall og beregnet resultat via RuleOutcome

  • RuleReasonRef bruk RuleOutcome eller lokal implementasjon av interface for å frakte ut resultat
  • ForeachSpecification som sammen med Sequence kan erstatte DynamicRuleService + medScope
  • Buildere for Sequence + ComputationalIf - tilsvarende som for ConditionalOr
  • ComputationalIf uten else-regel (gir ja())
  • Rette feil i SequenceEvalutation - bør plukke fra siste, ikke nr 2 i liste + EvalSummary der rotnode=leaf
  • Fjernet Specification.visit (finnes ingen impl av SpecificationVisitor),

Refleksjon: Uthenting av sluttresultat - trenger vi både muterende input og evaluationProperties ? Flere prosjekt setter masse evaluationProperties - men leser aldri disse. Trengs properties som annet en dokumentasjon i regelsporing output-json?

@palfi palfi removed their request for review November 18, 2021 09:03
@jolarsen jolarsen force-pushed the bugfix/alle-properties-sequence branch from f86ada2 to 02709b3 Compare January 30, 2022 14:47
@jolarsen jolarsen changed the title Diskusjonsutkast forbedring av Sequence ForeachSpec, RuleOutcome, flere rulesets Feb 16, 2022
@jolarsen jolarsen changed the title ForeachSpec, RuleOutcome, flere rulesets ForeachSpec, RuleOutcome, Builders, flere rulesets Feb 17, 2022
@ekun ekun self-requested a review February 17, 2022 18:43
@jolarsen jolarsen merged commit f855e29 into master Feb 17, 2022
@jolarsen jolarsen deleted the bugfix/alle-properties-sequence branch February 17, 2022 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants