[PLUGINS-1775] Added mqsiprofile script field to broker steps : create, delete, star…#36
[PLUGINS-1775] Added mqsiprofile script field to broker steps : create, delete, star…#36Jyoti-Prakash01 wants to merge 2 commits intoairfrom
Conversation
jrbarto
left a comment
jrbarto
left a comment
There was a problem hiding this comment.
Hey JP. I don't get what you're doing here. All you did was create a bunch of properties in the plugin.xml, and they're never used in the scripts.
Also, jarPath isn't a necessary field. If you need help with this please ask for it.
|
The properties those I have created are used by mqsi_profile_wrapper.groovy. Which is being passed for broker steps in plugins.xml. Let me know if I am wrong. |
|
Also, along with jarPath and other properties that I have added are being used in mqsi_profile_wrapper.groovy. |
|
Oh okay, I see you're using the wrapper now instead of directly using the script. I think that's a good call. However, the jarPath shouldn't be a property for mqsi scripts. That property is only necessary when using the Java IBM Integration API. Otherwise it's useless and just makes the step more complicated. Note that if you straight up remove the property right now it would break the plugin. In the wrapper script there is a split statement that splits the contents of the jarPath. You can remove the jarPath property from your new steps if you modify the wrapper script to first check if jarPath exists before splitting its contents. |
2 participants
…t, stop.