-
Notifications
You must be signed in to change notification settings - Fork 428
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
Changes for supporting ctx in payloadFactory #261
Conversation
@@ -132,7 +133,9 @@ private boolean mediate(MessageContext synCtx, String format) { | |||
} | |||
org.apache.axis2.context.MessageContext axis2MessageContext = ((Axis2MessageContext) synCtx).getAxis2MessageContext(); | |||
StringBuffer result = new StringBuffer(); | |||
regexTransform(result, synCtx, format); | |||
StringBuffer resultCTX = new StringBuffer(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't we use result (StringBuffer)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If same variable is used to do the replacing in both (ctx and args) it will append the both results in the payload.
Changes for supporting ctx in payloadFactory
|
||
private static final Log log = LogFactory.getLog(PayloadFactoryMediator.class); | ||
private static final Log log = LogFactory.getLog(PayloadFactoryMediator.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Formatting issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in the PR https://github.com/wso2/wso2-synapse/pull/363/files
No description provided.