Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Renaming folder crashes Godot or message appears that the folder already exists (Godot v4.3 dev 6) #91511

Closed
MatthiasBae opened this issue May 3, 2024 · 5 comments

Comments

@MatthiasBae
Copy link

Tested versions

  • Tested in 4.3 dev 6

System information

Godot v4.3.dev6.mono - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4070 Ti (NVIDIA; 31.0.15.5222) - 13th Gen Intel(R) Core(TM) i9-13900KF (32 Threads)

Issue description

Renaming a folder the first time crashes Godot completely. It just closes. After opening and renaming folders again it says that (no matter what name you choose) that the folder already exists. But it still renames the folder

Steps to reproduce

  1. Create a project
  2. Create a folder
  3. Rename the folder

Minimal reproduction project (MRP)

foldertesting.zip

@MatthiasBae
Copy link
Author

image

@AThousandShips
Copy link
Member

AThousandShips commented May 3, 2024

@spiralhalo
Copy link

spiralhalo commented May 12, 2024

Supplementing the issue with my own experience:

  • I got "A file or folder with this name already exists." with both folder and file renaming in 4.3 dev6. Doesn't happen in 4.3 dev5.

  • The message only shows up if I confirm with ENTER. If I press ESC, the message doesn't show up even though the renaming still took place successfully.

EDIT: Just noticed in 4.2.1 pressing ESC actually cancels the renaming, so the fact that the rename took place anyway might be another regression.

EDIT2: Opened another issue for the above: #91878

@AThousandShips
Copy link
Member

This should be solved on latest master after:

@akien-mga
Copy link
Member

Yes this should be fixed for Windows by #91112.

Still reproducible on Linux however, but I'll open a separate issue.

@akien-mga akien-mga added this to the 4.3 milestone May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants