Repository navigation
Closed
Labels
Description
bug description
trying to add files from within nextcloud to deck card, but list of files /directories only shows grey lines
To Reproduce
Steps to reproduce the behavior:
- Go to 'Deck'
- Click on existing deck card to see the details
- Click on 'Attachments'
- Click on 'Share from files'
Expected behavior
A list of files / directories should be presented, but instead there are only grey lines
Client details:
- OS: docker latest
- Browser: chrome, Firefox
- Version Nextcloud: 29.0.0 deck: 1.13.0
- Device: desktop
Logs
Browser log
Insert your browser log here, this could for example include:
a) The javascript console log
main.js:56 TypeError: Cannot read properties of undefined (reading 'authType')
at H (request.js:71:17)
at index.mjs:405:117
at index.js:2:55017
at R.execute (index.js:2:54338)
at R.patchInline (index.js:2:55925)
at X (index.js:2:59820)
at index.js:2:59436
at index.js:2:58252
at index.js:2:86641
at n (index.js:2:86363)
l.default.config.errorHandler @ main.js:56
mn @ vue.runtime.esm.js:3047
un @ vue.runtime.esm.js:3024
(anonymous) @ vue.runtime.esm.js:3035
Promise.catch (async)
cn @ vue.runtime.esm.js:3035
qn @ vue.runtime.esm.js:4048
insert @ vue.runtime.esm.js:4443
E @ vue.runtime.esm.js:6966
xi.nodeOps @ vue.runtime.esm.js:7180
e._update @ vue.runtime.esm.js:3785
r @ vue.runtime.esm.js:3891
e.get @ vue.runtime.esm.js:3462
e.run @ vue.runtime.esm.js:3538
rr @ vue.runtime.esm.js:4141
(anonymous) @ vue.runtime.esm.js:3159
An @ vue.runtime.esm.js:3081
Promise.then (async)
fn @ vue.runtime.esm.js:3106
Tn @ vue.runtime.esm.js:3171
ar @ vue.runtime.esm.js:4227
e.update @ vue.runtime.esm.js:3529
e.$forceUpdate @ vue.runtime.esm.js:3810
c @ vue.runtime.esm.js:2777
(anonymous) @ vue.runtime.esm.js:2797
(anonymous) @ vue.runtime.esm.js:312
Promise.then (async)
m @ vue.runtime.esm.js:2821
mr @ vue.runtime.esm.js:4494
(anonymous) @ vue.runtime.esm.js:2943
sn @ vue.runtime.esm.js:2882
e.$createElement @ vue.runtime.esm.js:2656
render @ legacy.mjs:36
e._render @ vue.runtime.esm.js:2700
r @ vue.runtime.esm.js:3891
e.get @ vue.runtime.esm.js:3462
e @ vue.runtime.esm.js:3452
e @ vue.runtime.esm.js:3908
jr.$mount @ vue.runtime.esm.js:8797
e._init @ vue.runtime.esm.js:5724
jr @ vue.runtime.esm.js:5785
c @ legacy.mjs:33
r @ legacy.mjs:100
(anonymous) @ index-CRno7JSe.mjs:17
pick @ index-CRno7JSe.mjs:10
await in pick (async)
shareFromFiles @ AttachmentList.vue:228
click @ AttachmentList.vue:1
cn @ vue.runtime.esm.js:3033
n @ vue.runtime.esm.js:1832
cn @ vue.runtime.esm.js:3033
e.$emit @ vue.runtime.esm.js:3732
click @ NcButton.mjs:199
cn @ vue.runtime.esm.js:3033
n @ vue.runtime.esm.js:1832
i._wrapper @ vue.runtime.esm.js:7500
Show 49 more frames
Show less
</details>
Reacted by Alex Grädel, Nils K, László Gyaraki, Samuel Philipp and anneelinh

Activity
CompassResearch commentedon May 7, 2024
Same problem. We have analyzed logs and error msgs and tried various fixes, but didn't help.
Here's a Chat GPT analysis based on our error messages:
The error messages you've provided from your Nextcloud 29.0.0 instance, specifically concerning the DECK app, highlight a PHP coding issue in the DECK app's ConfigService.php. Here’s a breakdown of the key components of the error and what they might indicate about the underlying problem:
Error Details
Both error messages point to the same issue within a method in the ConfigService.php file at line 234. The error message:
"Optional parameter $userId declared before required parameter $path is implicitly treated as a required parameter"
This error is typical of a function or method declaration in PHP where an optional parameter (one with a default value) is placed before a required parameter (one without a default value) in the function's signature. This is problematic in PHP because it forces all parameters defined after an optional parameter to also be treated as optional, which can lead to unexpected behavior or errors.
Context of the Error
The errors are triggered during GET requests to the Nextcloud server:
URL /ocs/v2.php/apps/user_status/api/v1/user_status?format=json
URL /ocs/v2.php/apps/notifications/api/v2/notifications?format=json
These URLs suggest that the requests are related to fetching user status and notifications, respectively.
Analyzing the Impact
Although the URLs involved in the errors pertain to user statuses and notifications, the error originates from the deck app's codebase, suggesting that the DECK app might be extending or interacting with these areas of Nextcloud.
This might indicate that there's a broader integration issue within the DECK app that could be affecting its ability to properly manage or access file structures as you’ve been experiencing.
Steps to Resolve
Review and Modify Code: The immediate resolution would be to check the ConfigService.php file around line 234 and adjust the parameter order. The optional parameter $userId should be placed after all required parameters, or it should be made required by removing its default value if that contextually makes sense.
Update and Test: After making code changes, thoroughly test the DECK app in a development environment to ensure that the change does not negatively affect other functionalities.
Check for Updates: Verify if there's an update available for the DECK app that addresses this issue. Sometimes, such problems are quickly fixed once identified, and an update might already be available.
Fallback Plan: If the issue is critical and a fix is not readily available, consider rolling back to an earlier version of the DECK app or Nextcloud where this issue was not present, if such a rollback would not cause other issues.
We tried that, but it didn't help either.
Client details:
OS: Linux
Browser: Chrome, Firefox
Version Nextcloud: 29.0.0 deck: 1.13.0
Device: desktop
Keeper-of-the-Keys commentedon May 9, 2024
This is probably related to the new File Picker dialog
nextcloud/server#39792
araspitzu commentedon May 14, 2024
I'm also affected by this issue, when trying to select an attachment for a card the file browser is not working and shows up but remains in a disabled state. The file browser works correctly from other apps.
Client details:
luistiktok commentedon May 14, 2024
Same happens when trying to achieve the same but in the calendar app
leclou69 commentedon May 16, 2024
Yeah, indeed. Also still persists in the calendar app after upgrading it today to latest 4.7.4.
So perhaps sth the main nc branch should have an eye on.
In Mail i.e. it works as it should.
leclou69 commentedon Jun 5, 2024
In calendar it seems to be fixed with te actual version 4.7.6
leowenlu commentedon Jun 17, 2024
got the exactly same error, any one has solution for this?
dock: 1.13
nextCloud: 29.0.1
Spheren commentedon Jun 26, 2024
Same here.
dock: 1.13
NC: 29.0.3
leclou69 commentedon Jun 28, 2024
Today Deck 1.13.1 was released.
The problem is solved