Re: [windup-dev] RuleModel et al.
Forgot to mention the context of my thoughts.
I am not sure what are the plans for integration of Eclipse, Windup,
Migration site and Customer Portal; but it's obvious that the "Rule"
will be present everywhere. So I think we will need some common class
with basic Rule info, which will be part of the API.
Not sure if it all should be the same class, maybe not. But at least for
Eclipse module, having a RuleModel would be beneficial, since we will
have some automatic converter for the Eclipse <--> Windup API, and we
might want to show more about the rule in Eclipse than just a cryptic ID.
In Windup itself, we might look up the rule in some Map, but what's the
advantage against simply adding the reference to anything added by a
rule? This could be even automatized later, simply by intercepting the
calls to services somehow.
Ondra
On 22.7.2014 02:59, Ondrej Zizka wrote:
So far, an ID and a reference to the Ruleset. The ruleset then would
probably have further info, like, version etc.
https://github.com/OndraZizka/windup/blob/3940b146f811ab6e5fff1cb6c6def71...
Anyway, even if it was just an ID, OOP principles suggest to encapsulate
that ID to a type. My experience agrees. I may be wrong though.
Ondra
On 22.7.2014 02:40, Jess Sightler wrote:
> I'm not opposed to this idea... except that I don't know what a
> "RuleModel" would actually contain, other than the ID.
>
> What are you proposing it to contain?
>
> On 07/21/2014 07:03 PM, Ondrej Zizka wrote:
>> We should have $subj:
>>
>> We need to refer to the rules in the report.
>> We agreed to store all information in the graph.
>> Current ID is not guaranteed to be the same over runs.
>> Current ID has no namespaces.
>>
>> my2c.
>> Ondra
>>
>> _______________________________________________
>> windup-dev mailing list
>> windup-dev(a)lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/windup-dev
> _______________________________________________
> windup-dev mailing list
> windup-dev(a)lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/windup-dev
_______________________________________________
windup-dev mailing list
windup-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/windup-dev