When the suggested patch for bug 30120 has been applied, it becomes apparent that the logic controllers are missing implementations of the "clear" method. Because when a new logic controller is added, for example "IF controller", and there already exists an "If controller" in the test plan with an expression set, the newly added "If controller" has the same expression initially. The newly added "If controller" should have a blank expression initially.
Created attachment 20115 [details] Suggested patch This patch adds implementation of the "clear" method for most of the logic controllers. It does not implement "clear" for the IncludeController and ModuleController. The patch also adds // $NON-NLS-1$ comments, let me know if that is only confusing, and I can make a patch with only the clear method. The patch also adds a resource string for the label for the dropdown box of ModuleController.
Thanks - added to SVN in r535147 and r535151. As to the NLS-only and resource fixes, I added them as separate commits. It's OK to add NLS markers to files that already are being updated, but in future please use a separate patch file for changes to other files that are unrelated to the current bug report. This makes it easier to follow the history later.
Created attachment 20130 [details] Implement clear for IncludeController and ModuleController This patch adds "clear" method to the two remaining logic controllers, IncludeController and ModuleController
Thanks - patch 20130 added to SVN in r535284
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/1932