Nixyri/godot
1
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2024-11-10 06:03:09 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge pull request #44262 from akien-mga/fsdock-move-mention-conflicting-files

Browse Source 
FileSystemDock: List conflicting files on move
This commit is contained in:
Rémi Verschelde 2020-12-10 13:13:35 +01:00 committed by GitHub
commit 9aec0ad3b3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
editor/filesystem_dock.cpp
View File

@ -1505,7 +1505,8 @@ void FileSystemDock::_move_with_overwrite() {
_move_operation_confirm(to_move_path, true); _move_operation_confirm(to_move_path, true);
} }
bool FileSystemDock::_check_existing() { Vector<String> FileSystemDock::_check_existing() {
Vector<String> conflicting_items;
String &p_to_path = to_move_path; String &p_to_path = to_move_path;
for (int i = 0; i < to_move.size(); i++) { for (int i = 0; i < to_move.size(); i++) {
String ol_pth = to_move[i].path.ends_with("/") ? to_move[i].path.substr(0, to_move[i].path.length() - 1) : to_move[i].path; String ol_pth = to_move[i].path.ends_with("/") ? to_move[i].path.substr(0, to_move[i].path.length() - 1) : to_move[i].path;
@ -1515,21 +1516,24 @@ bool FileSystemDock::_check_existing() {
String old_path = (p_item.is_file || p_item.path.ends_with("/")) ? p_item.path : (p_item.path + "/"); String old_path = (p_item.is_file || p_item.path.ends_with("/")) ? p_item.path : (p_item.path + "/");
String new_path = (p_item.is_file || p_new_path.ends_with("/")) ? p_new_path : (p_new_path + "/"); String new_path = (p_item.is_file || p_new_path.ends_with("/")) ? p_new_path : (p_new_path + "/");
if (p_item.is_file && FileAccess::exists(new_path)) { if ((p_item.is_file && FileAccess::exists(new_path)) ||
return false; (!p_item.is_file && DirAccess::exists(new_path))) {
} else if (!p_item.is_file && DirAccess::exists(new_path)) { conflicting_items.push_back(old_path);
return false;
} }
} }
return true; return conflicting_items;
} }
void FileSystemDock::_move_operation_confirm(const String &p_to_path, bool p_overwrite) { void FileSystemDock::_move_operation_confirm(const String &p_to_path, bool p_overwrite) {
if (!p_overwrite) { if (!p_overwrite) {
to_move_path = p_to_path; to_move_path = p_to_path;
bool can_move = _check_existing(); Vector<String> conflicting_items = _check_existing();
if (!can_move) { if (!conflicting_items.empty()) {
// Ask to do something. // Ask to do something.
overwrite_dialog->set_text(vformat(
TTR("The following files or folders conflict with items in the target location '%s':\n\n%s\n\nDo you wish to overwrite them?"),
to_move_path,
String("\n").join(conflicting_items)));
overwrite_dialog->popup_centered(); overwrite_dialog->popup_centered();
return; return;
} }
@ -2849,7 +2853,6 @@ FileSystemDock::FileSystemDock(EditorNode *p_editor) {
rename_dialog->connect("confirmed", callable_mp(this, &FileSystemDock::_rename_operation_confirm)); rename_dialog->connect("confirmed", callable_mp(this, &FileSystemDock::_rename_operation_confirm));
overwrite_dialog = memnew(ConfirmationDialog); overwrite_dialog = memnew(ConfirmationDialog);
overwrite_dialog->set_text(TTR("There is already file or folder with the same name in this location."));
overwrite_dialog->get_ok()->set_text(TTR("Overwrite")); overwrite_dialog->get_ok()->set_text(TTR("Overwrite"));
add_child(overwrite_dialog); add_child(overwrite_dialog);
overwrite_dialog->connect("confirmed", callable_mp(this, &FileSystemDock::_move_with_overwrite)); overwrite_dialog->connect("confirmed", callable_mp(this, &FileSystemDock::_move_with_overwrite));

2
editor/filesystem_dock.h
View File

@ -236,7 +236,7 @@ private:
void _rename_operation_confirm(); void _rename_operation_confirm();
void _duplicate_operation_confirm(); void _duplicate_operation_confirm();
void _move_with_overwrite(); void _move_with_overwrite();
bool _check_existing(); Vector<String> _check_existing();
void _move_operation_confirm(const String &p_to_path, bool p_overwrite = false); void _move_operation_confirm(const String &p_to_path, bool p_overwrite = false);
void _tree_rmb_option(int p_option); void _tree_rmb_option(int p_option);