Bug 247

Description Tod Creasey 2001-10-10 21:38:52 EDT

If you copy over text using the Compare Browser when doing a Compare with 
Stream Contents
you do not get prompted to save changed when you close the browser.

STEPS
	1) Create a file foo.txt with content.
	2) Release it to the stream
	3) Edit foo.txt
	4) Save and compare with stream
	5) Copy over stream contents
	6) Close the compare browser - no prompt for changes.


NOTES:

JEM (9/12/2001 11:38:47 AM)
	Possibly related to
		1GFMRMH: ITPVCM:ALL - Project compare: edited files aren't 
marked

JEM (9/12/2001 2:36:54 PM)
	Impossible to do properly given the current compare framework. If we 
get rid of our implementation
	of VCMCompareEditorInput.isSaveNeeded(), then the when we close the 
editor we get the following behaviour:

	1. Editor says changes exist, do you want to save? (y/n/cancel)
	2. User says "yes"
	3. CompareEditor.doSave() first sends setInput(null) to each pane, then 
sends save() to the CompareEditorInput.
	BUT
	setInput(null) sent to each pane may cause additional prompting 
(changes exist, do you want to save? (y/n/cancel))
	because, in ContentMergeViewer.inputChanged(), if fConfirmSave is true, 
the pane prompts for saving changes.

	We have no control over fConfirmSave, as it is a global setting. 
(Anyway, even if we did, we wouldn't turn it off
	because it is needed for the case that the user clicks on a different 
item in the tree).

	Why does this not happen in the normal (non-VCM) compare case? Because 
we enable the save actions in
	our editors, and in ContentMergeViewer.inputChanged(), the prompting 
only happens if the save actions are
	enabled.

	This code path needs to be fixed before we can prompt in this way.

JEM (9/12/2001 3:31:02 PM)
	Waiting on Compare PR:

	1GJYGAX: ITPJUI:WIN2000 - Compare: Save code path problematic