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

[2.1.1] [Linux] Folder removal confirmation dialog contains wrong folder name #4577

Closed
maxxer opened this issue Mar 17, 2016 · 4 comments
Closed
Assignees
Milestone

Comments

@maxxer
Copy link

maxxer commented Mar 17, 2016

client version 2.1.1, Ubuntu Linux.

  1. setup oC with a single account
  2. add a second oC account with a new local folder
  3. on the second account press the three dots on the folder and then Remove folder from sync

The popup which appears contains the name ownCloud1, while the locally synced folder name is oc3j (in my case)

folder_name_delete

@guruz guruz changed the title Folder removal confirmation dialog contains wrong folder name [2.1.1] [Linux] Folder removal confirmation dialog contains wrong folder name Mar 17, 2016
@guruz guruz added the type:bug label Mar 17, 2016
@guruz guruz assigned guruz and unassigned danimo Apr 6, 2016
@guruz guruz added the ReadyToTest QA, please validate the fix/enhancement label Apr 14, 2016
@guruz guruz added this to the 2.2.0-current milestone Apr 14, 2016
@guruz guruz added sev3-medium and removed ReadyToTest QA, please validate the fix/enhancement labels Apr 19, 2016
guruz added a commit that referenced this issue Apr 20, 2016
UI: Remove alias from more places #4577 #4695
@guruz guruz added the ReadyToTest QA, please validate the fix/enhancement label Apr 20, 2016
@mcastroSG
Copy link

Hi hi:

Steps executed

  1. Install ownCloud Client setting and account.
  2. Add a second account setting another shared folder.
  3. On second account shared folder click at tree dots button and choose remove folder sync connection.

Results

Client shows at popup : "Do you really want to stop syncing the folder ownCloud Testpilot Edition?" . Does not matter which shared folder we want to remove, always shows same text.

Client Version: Version 2.2.0nightly2016421 (build 1557)
Client OS: Ubuntu 14.04 LTE

@guruz
Copy link
Contributor

guruz commented Apr 21, 2016

QString Folder::aliasGui() const

You have two accounts configured or two folders at the same account?
So for both accounts it shows "Do you really want to stop syncing the folder ownCloud Testpilot Edition?"?

Looks like I have not considered this in 6152ce4

@guruz guruz removed the ReadyToTest QA, please validate the fix/enhancement label Apr 21, 2016
@mcastroSG
Copy link

@guruz two different accounts and each one ,with one different shared sync folder. ;)

guruz added a commit that referenced this issue Apr 26, 2016
UI: Fix account name/alias display oddities #4577
@guruz guruz added the ReadyToTest QA, please validate the fix/enhancement label Apr 26, 2016
@mcastroSG
Copy link

Hi hi:

Steps executed

  1. Install ownCloud Client setting and account.
  2. Add a second account setting another shared folder.
  3. On second account shared folder click at tree dots button and choose remove folder sync connection.

Results

Client shows at popup : "Do you really want to stop syncing the folder XXXX ?" .
Client Version: Version 2.2.0nightly2016427 (build 1570)
Client OS: Ubuntu 14.04 LTE

@maxxer may you test again with later nightly ?

@mcastroSG mcastroSG added Tested and removed ReadyToTest QA, please validate the fix/enhancement Tested labels Apr 27, 2016
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