DO NOT REPLY [Bug 25441] - TestPlan changes sometimes detected incorrectly (isDirty)

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=25441>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=25441





------- Additional Comments From alf@i100.no  2007-05-07 02:27 -------
Created an attachment (id=20135)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=20135&action=view)
Suggested patch

This patch handles merging of test plans, by forcing the "dirty" flag to true
when a test plan is merged into another test plan.

This patch also solves the problem where a test plan is incorrectly marked as
"dirty" when it is not dirty. This happens when properties are set on the
testplan node, as described in bug 41779. The fix is to initialize the tree
model with the node loaded from the test plan, instead of having the default
testplan node in the treemodel.

The patch also makes the "Load" action close the current testplan only when the
user selects a file to load. This means that if a user now has a currently open
test plan, then uses the "Load" menu option, but clicks "Cancel" in the file
dialog, the test plan he had open is still open. Previously, the current test
plan was closed before the file dialog appear, which is annoying.

This patch also fixes bug 41779, as mentioned above.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org