Bug report: checking in a properties change to a directory also checks in the files, even if unselected

In the 'check for modifications' dialogue box an entry is shown for a directory
if some property (for example, svn:ignore) of that directory has changed. An
entry is also shown for a file inside that directory if the contents of the file
has changed. You can select one or the other independently and right-click to
commit. In particular you can select just the directory and leave the file
unselected, then right-click on the line for the directory and commit. I
expected this would commit only the property change and not the contents of the
file, since I had unselected the file for commit. But it did both.

To reproduce:

- Create a directory 'test' in your working copy, add it to svn, and check it in.

- Create a file 'hello' inside the new 'test' directory with content 'Hello
there'. Add 'hello' to svn and check it in.

- Create a file 'ignoreme' inside 'test' but do not add it to svn.

- Edit the existing file 'hello' and change its content to 'Goodbye'.

- Right-click on the directory 'test' in Windows Explorer and do TortoiseSVN ->
Check for modifications. Make sure 'Show unversioned files' is ticked. Two
files are shown, 'hello' and 'ignoreme'.

- Right-click on 'ignoreme' in the list of modifications and Add to ignore list
-> ignoreme. The list changes to show two items: the 'hello' file as before and
the 'test' directory.

- Right-click on 'test' in the list of modifications. Make sure that just
'test' is selected and 'hello' is not selected. Do 'Compare with base'. The
window that appears shows a change to svn:ignore but no other changes.

- Then again make sure just 'test' is selected and 'hello' is deselected,
right-click and 'Commit'.

Expected result: a commit dialogue box appears for a change to just 'test' and
not for 'hello'.

Actual result: by default both things are included in the commit and if you
don't look carefully at the list of changes in the bottom half of the commit
dialogue you will commit the change to 'test' by accident, even though it was
not part of the set you selected.

-- 
Ed Avis <eda_at_waniasset.com>
---------------------------------------------------------------------
To unsubscribe, e-mail: users-unsubscribe_at_tortoisesvn.tigris.org
For additional commands, e-mail: users-help_at_tortoisesvn.tigris.org