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

Created folders cannot be moved to other folders - subfolders moved when renamed #9370

Closed
handkerchief333 opened this issue Feb 25, 2024 · 4 comments · Fixed by #10067
Closed
Assignees

Comments

@handkerchief333
Copy link

handkerchief333 commented Feb 25, 2024

Steps to reproduce

  1. Create a new folder in a mailbox
  2. Create a new subfolder in this folder
  3. Rename this subfolder
  4. The subfolder is renamed, but moved from the assigned top-level folder to the top level
  5. Try drag & drop the renamed folder back into another folder

Expected behavior

  1. When renaming folders that they still respect the hierarchy they are in.
  2. That you can nest already created folders in other folders.

Actual behavior

  1. Subfolders are automatically moved to the highest level when renamed
  2. You can no longer assign created folders to other folders.

Mail app version

3.5.6

Mailserver or service

No response

Operating system

FreeBSD 13.2

PHP engine version

PHP 8.2

Web server

Apache (supported)

Database

MariaDB

Additional info

Nextcloud v. 28.0.2

  • It also lacks the function to arrange folders in order of preference, which would also be enormously useful to keep a mailbox well organized.
  • It is not possible to rename a top-level folder that contains subfolders. This option is not available.
@ChristophWurst
Copy link
Member

When renaming folders that they still respect the hierarchy they are in.

Keep the browser console and network log open. Inspect the POST request sent when the mailbox is created. Tell us what is sent for name.

I can't reproduce.

@handkerchief333
Copy link
Author

Console log:
{"mailbox":{"databaseId":115,"id":"Q2xhbmBlbi9Db2xkL3Rlc2Y=","name":"TestA/TestB/test","accountId":17,"displayName":"test","attributes":[],"delimiter":"/","specialUse":[],"specialRole":0,"mailboxes":[],"syncInBackground":false,"unread":0,"myAcls":null,"shared":false,"envelopeLists":{},"path":"TestA/TestB"}}

Postmessage:
{"accountId":17,"name":"TestA/TestB/test"}

Ok I created folder "test" in folder "TestB". Interesting fact. In the folder list, folder "TestB" is at the top level and folder "TestA" is also without displayed subfolders. So there seems to be something wrong with the hierarchy display.

@lufer22
Copy link

lufer22 commented Apr 3, 2024

Hey,
I also faced this problem with renaming a subfolder.
To solve it, I made the following code:
File: src/components/NavigationMailbox.vue
Function: renameMailBox()

[...]
try {
let newName = this.mailboxName
if (this.mailbox.path) {
newName = this.mailbox.path + this.mailbox.delimiter + newName
}
await this.$store.dispatch('renameMailbox', {
account: this.account,
mailbox: this.mailbox,
newName,
})
this.renameLabel = true
this.renameInput = false
[...]

Resuming, I concatenated the whole path to folder's name.

Hope it helps
Thanks :D

@GretaD
Copy link
Contributor

GretaD commented Aug 28, 2024

The renaming issue is fixed here. the drag and drop for mailboxes was never an option on mail. But we do have drag and drop for envelopes, so maybe there was a confusion :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ☑️ Done
Development

Successfully merging a pull request may close this issue.

4 participants