Conversation
nielsm5
left a comment
There was a problem hiding this comment.
Je laat nu de parameter het al vertalen naar een formaat, maar je zou ook kunnen stellen dat de XsltSender en DataSonnetPipe een Map object moeten kunnen omzetten? Dan ga je wel mogelijk rare messages krijgen, dus zeer waarschijnlijk een enorm slecht idee dit...
Het lijkt me dat het erg "messy" gaat worden als we een |
evandongen
left a comment
There was a problem hiding this comment.
1 opmerking, verder akkoord!
| } else { | ||
| Message sourceMsg = Message.asMessage(sourceObject); | ||
| if (StringUtils.isNotEmpty(getContextKey())) { | ||
| if ("*".equals(getContextKey())) { |
There was a problem hiding this comment.
Kan je hier een constant van maken die beschrijft wat het betekent?
| @@ -490,9 +496,74 @@ public Object getValue(ParameterValueList alreadyResolvedParameters, Message mes | |||
| return result; | |||
There was a problem hiding this comment.
Pff, wat een method is dit zeg.
There was a problem hiding this comment.
Ja, vreselijk. Ik probeer af en toe om er langzamerhand iets beters van te maken, maar dat is lastig. Misschien in een volgende change, wanneer we value-methods gaan aanpakken (#1242)
|
Closes #9755