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

Group Folder titles in breadcrumbs menu are incorrect #3068

Closed
nooblag opened this issue Jul 19, 2024 · 12 comments · Fixed by nextcloud/server#46694
Closed

Group Folder titles in breadcrumbs menu are incorrect #3068

nooblag opened this issue Jul 19, 2024 · 12 comments · Fixed by nextcloud/server#46694
Assignees
Labels
0. Needs triage Issues that need to be triaged bug

Comments

@nooblag
Copy link

nooblag commented Jul 19, 2024

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Update to Nextcloud 29.0.4
  2. Enter a group folder and click around the directory tree

Expected behaviour

Folder name displayed should be the name of the folder, not the name of the folder from the file system.

Actual behaviour

Name from file system is displayed.

Screenshot from 2024-07-19 15-35-26

Screenshot from 2024-07-19 15-44-08

Server configuration

Operating system: Ubuntu 22.04.4 LTS

Web server: nginx/1.18.0 (Ubuntu)

Database: mariadb Ver 15.1 Distrib 10.6.18-MariaDB, for debian-linux-gnu (x86_64) using EditLine wrapper

PHP version: PHP 8.2.21 (cli) (built: Jul 4 2024 16:26:28) (NTS)

Nextcloud version: (see Nextcloud admin page) 29.0.4.1

Group folders version: 17.0.1

Updated from an older Nextcloud/ownCloud or fresh install: Updated

Where did you install Nextcloud from: nextcloud.com

Are you using external storage, if yes which one: No

Are you using encryption: No

Are you using an external user-backend, if yes which one: No/none.

Client configuration

Browser: Firefox 128.0

Operating system: Linux

Logs

Browser log

Browser log
[DEBUG] files: Directory contents changed 
Object { app: "files", uid: "***redacted***", level: 0, view: {…}, folder: {…}, contents: [] }
​
app: "files"
​
contents: Array []
​
folder: Object { _data: Getter & Setter, _attributes: Getter & Setter, _knownDavService: Getter & Setter, … }
​
level: 0
​
uid: "***redacted***"
​
view: Object { _view: {…} }
​
<prototype>: Object { … }
core-common.js:1:10102
[DEBUG] uploader: Destination set 
Object { app: "uploader", uid: "***redacted***", level: 0, folder: {…} }
​
app: "uploader"
​
folder: Object { _data: Getter & Setter, _attributes: Getter & Setter, _knownDavService: Getter & Setter, … }
​
level: 0
​
uid: "***redacted***"
​
<prototype>: Object { … }
core-common.js:1:10102
[DEBUG] files: Fetched contents 
Object { app: "files", uid: "***redacted***", level: 0, dir: "/***redacted path***/Meeting Minutes", folder: {…}, contents: (3) […] }
​
app: "files"
​
contents: Array(3) [ {…}, {…}, {…} ]
​
dir: "/***redacted path***/Meeting Minutes"
​
folder: Object { _data: Getter & Setter, _attributes: Getter & Setter, _knownDavService: Getter & Setter, … }
​
level: 0
​
uid: "***redacted***"
​
<prototype>: Object { … }
core-common.js:1:10102
[DEBUG] files: Directory contents changed 
Object { app: "files", uid: "***redacted***", level: 0, view: {…}, folder: {…}, contents: (3) […] }
​
app: "files"
​
contents: Array(3) [ {…}, {…}, {…} ]
​
folder: Object { _data: Getter & Setter, _attributes: Getter & Setter, _knownDavService: Getter & Setter, … }
​​
__ob__: Object { shallow: false, mock: false, vmCount: 0, … }
​​
_attributes: 
​​
_children: 
​​
_data: 
​​
_knownDavService: 
​​
handler: 
​​
readonlyAttributes: 
​​
<get _attributes()>: function get()​​
<set _attributes()>: function set(t)​​
<get _children()>: function get()​​
<set _children()>: function set(t)​​
<get _data()>: function get()​​
<set _data()>: function set(t)​​
<get _knownDavService()>: function get()​​
<set _knownDavService()>: function set(t)​​
<get handler()>: function get()​​
<set handler()>: function set(t)​​
<get readonlyAttributes()>: function get()​​
<set readonlyAttributes()>: function set(t)​​
<prototype>: Object { … }
​
level: 0
​
uid: "***redacted***"
​
view: Object { _view: {…} }
​
<prototype>: Object { … }
​​
__defineGetter__: function __defineGetter__()
​​
__defineSetter__: function __defineSetter__()
​​
__lookupGetter__: function __lookupGetter__()
​​
__lookupSetter__: function __lookupSetter__()
​​
__proto__: 
​​
constructor: function Object()
​​
hasOwnProperty: function hasOwnProperty()
​​
isPrototypeOf: function isPrototypeOf()
​​
propertyIsEnumerable: function propertyIsEnumerable()
​​
toLocaleString: function toLocaleString()
​​
toString: function toString()
​​
valueOf: function valueOf()
​​
<get __proto__()>: function __proto__()
​​
<set __proto__()>: function __proto__()
@nooblag nooblag added 0. Needs triage Issues that need to be triaged bug labels Jul 19, 2024
@PW-RTR
Copy link

PW-RTR commented Jul 19, 2024

We have the same issue after updating our production system to Nextcloud 29.0.4 and Group Folders 17.0.1
Strangely, we don't have this issue on our Test system with same Nextcloud and Group Folders version.

@Dennis1993
Copy link

In our live system the same problem after the update to 29.0.4

image

@XueSheng-GIT
Copy link
Contributor

Having the same issue on three instances with NC 29.0.4. Could also be a files issue.
Reminds me of #2527.

@solracsf
Copy link
Member

Cc @susnux since this can be related to recent code changes in breadcrumbs handling.

@pablorgzzz
Copy link

pablorgzzz commented Jul 22, 2024

I have detected 3 issues

1 - Group folder titles in breadcrumbs menu are incorrect

2 - In nextcloud 29.0.3 there was an option in the menu that said "group folders", that option is no longer there

3 - If I put any number as the name of a group folder in front of a word that occupies more length than the visibility space, the number does not appear

13 16 57 - 04
13 01 59 - 01
13 23 18 - 05
13 15 47 - 03

@jmechnich
Copy link

The same issue affects NC 28.0.8 with group folders 16.0.7.

@susnux
Copy link
Contributor

susnux commented Jul 22, 2024

Partly fixed with nextcloud/server#46690

But there is a backend issue here:
Groupfolders return the internal mountpoint id as the displayname instead of the mount point name.

@pablorgzzz
Copy link

I have detected 3 issues

1 - Group folder titles in breadcrumbs menu are incorrect

2 - In nextcloud 29.0.3 there was an option in the menu that said "group folders", that option is no longer there

3 - If I put any number as the name of a group folder in front of a word that occupies more length than the visibility space, the number does not appear

13 16 57 - 04 13 01 59 - 01 13 23 18 - 05 13 15 47 - 03

The issues 2 and 3 are still there

@susnux
Copy link
Contributor

susnux commented Jul 24, 2024

The issues 2 and 3 are still there

nextcloud/server#46690

@ostasevych
Copy link

The same goes with 28.0.8

@susnux
Copy link
Contributor

susnux commented Jul 25, 2024

Will be fixed in 29.0.5 and 28.0.9

@solracsf
Copy link
Member

Confirmed that nextcloud/server#46694 fixes this issue. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Issues that need to be triaged bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants