Gopal50 / tortoisesvn

Automatically exported from code.google.com/p/tortoisesvn
0 stars 0 forks source link

Restore after commit - Unchecked files are not restored upon commit #640

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Reported on the mailing list:
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=30776
55

Using TortoiseSVN 1.8.6, I noticed the following problem with the restore after 
commit feature: 

If I complete the commit dialog, only those files which were marked for commit 
are restored. Unchecked files are not restored. In contrast, when I canceling 
the commit dialog, all files regardless of their checkbox status are restored. 
This is especially bad since files may become "unchecked" unintentionally when 
all changes are undone after selecting "restore after commit" and then the 
commit dialog is refreshed:

Steps to reproduce:
1. Modify two files in a working copy ("FileA" and "FileB")
2. In the commit dialog, select "Restore after commit" for FileA
3. Launch the diff editor for FileA and undo all changes
4. Refresh commit dialog (FileA vanishes from file list)
5. (Enter commit message and) Commit Changes 

Expected Behaviour:
After commiting, FileA should be restored, i.e. it should contain the changes 
which were done in step 1.

Observed Behaviour:
After commiting, FileA is unchanged, the changes from step 1 are lost.

I think that completing the commit or aborting it should show the same 
behavior, i.e. either unchecked files should always be restored in both cases 
or they should not be restored in both cases. If they are not restored, some 
confirmation dialog would be desirable. Personally, I would prefer to have them 
restored always.

Original issue reported on code.google.com by tortoisesvn on 7 May 2014 at 6:47

GoogleCodeExporter commented 8 years ago

Original comment by tortoisesvn on 7 May 2014 at 7:03

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r25479.

Original comment by tortoisesvn on 7 May 2014 at 7:22

GoogleCodeExporter commented 8 years ago
Great, thanks for this prompt fix!

Original comment by michael8...@googlemail.com on 7 May 2014 at 8:29

GoogleCodeExporter commented 8 years ago
This issue was closed by revision r25493.

Original comment by tortoisesvn on 12 May 2014 at 8:25