Opened 5 years ago
Last modified 4 years ago
#229 testing defect
Saving workspace into a file renames a directory which should not be renamed
Reported by: | Patrik Svestka | Owned by: | Patrik Svestka |
---|---|---|---|
Priority: | major | Milestone: | |
Component: | default | Keywords: | |
Cc: | Also affects CVS HEAD (eXept version): | no |
Description (last modified by )
Steps to reproduce the issue:
1) create a directory e.g.
C:\prg\Smalltalk\StX\search_for_substring
2) create some code in workspace
3) select the path that is already there and copy the path from step
1)
into the Filename field and press enter
Now two things can happen
4a) If it is the first time you do this, you can end up with deleted directory
C:\prg\Smalltalk\StX\search_for_substring
and two filessearch_for_substring
andsearch_for_substring.bak
4b) You can get a renamed directorysearch_for_substring.bak
and file namedsearch_for_substring
(if you have already saved the file before)
Expected behavior:
If the file is same as the already created directory warn the user that he can't save the file due to the fact there is a directory with the same name.
Attachments (3)
Change History (9)
comment:1 Changed 5 years ago by
Description: | modified (diff) |
---|
comment:2 Changed 5 years ago by
Owner: | set to Patrik Svestka |
---|---|
Status: | new → assigned |
comment:3 Changed 5 years ago by
Changed 5 years ago by
Attachment: | WorkspaceApplicationTests-test_issue229.st added |
---|
Test proves that the behaviour of reported bug
Changed 5 years ago by
Attachment: | FileDialog-doAccept.st added |
---|
Changed 4 years ago by
Attachment: | libtool_fix_1_of_1_rev_73f73118f3b8_Issue__229__Saving_workspace_into_a_file_renames_a_directory_which_should_not_be_renamed.patch added |
---|
When directory only path is entered, it is correctly changed to it in the FileDialog?
comment:5 Changed 4 years ago by
Status: | assigned → accepted |
---|
comment:6 Changed 4 years ago by
Status: | accepted → testing |
---|
Adding to expecting behaviour:
Perpahs it would be even better to change to a directory when user enters a directory instead of saving it. That is matter of view and approach.
Anyways I'm attaching a test case to show the bug: