notify the user when renaming to the same name #4347
Conversation
|
What about we toast a message saying "Your new name is same as old name. Rename isn't done". Or the best thing we can do is, we can disable save button if old and new names are same. Please let me know what you have in mind. |
|
I added a message to notify that no changes are made as new name matches the old name. Please let me know if you are thinking the same. |
VishnuSanal
changed the title
VishnuSanal
added
the
pr-awaiting-final-review
There was a problem hiding this comment.
As to not leave this stuck, I am approving. @VishnuSanal please only merge if you tested the code and works.
| HybridFile oldFile = new HybridFile(mode, oldPath); | ||
| HybridFile newFile; | ||
| if (Utils.isNullOrEmpty(newName)) { | ||
| newFile = new HybridFile(mode, newPath); | ||
| } else { | ||
| newFile = new HybridFile(mode, newPath, newName, isDirectory); | ||
| } | ||
| if (oldFile.getSimpleName().equals(newFile.getSimpleName())) { | ||
| Toast.makeText( |
There was a problem hiding this comment.
Toast should not be used anymore, in favor of Snackbar
There was a problem hiding this comment.
Okay, I am rescinding the approval. When "CREATE" is selected for a new folder, the popup closes. The way this should work is that CREATE should not be able to be selected and a text in red should appear under the text line saying "Cannot rename to already existing file name" or something similar.
See how naming a file works for reference:
Description
Added a condition to check if old name and new name of the file are same. Started renaming code only if the names are different.
Issue tracker
Fixes #4340
Automatic tests
Manual tests
Build tasks success
Successfully running following tasks on local:
./gradlew assembledebug./gradlew spotlessCheck