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

tagged photos issue #777

Closed
Lionel-cl opened this issue May 27, 2021 · 5 comments · Fixed by #783
Closed

tagged photos issue #777

Lionel-cl opened this issue May 27, 2021 · 5 comments · Fixed by #783
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working

Comments

@Lionel-cl
Copy link

Hello,

I allow myself to relaunch this ticket: #204
I have the same problem: an error appears as soon as you click on "tagged photos" menu.

The same problem appears on with the nextcloud demo itself :
https://demo2.nextcloud.com/index.php/apps/photos/tags

Is it due to my base? It doesn't seem so to me.
Thank you for your help,

Screenshots
Capture d’écran 2021-05-27 à 10 34 58

  • OS: MacOS / Webversion
  • Browser : Chrome
  • Version : NC21.0.2 (issue was on NC21.0.0 too)
@Lionel-cl Lionel-cl added 0. Needs triage Pending approval or rejection. This issue is pending approval. bug Something isn't working labels May 27, 2021
@skjnldsv
Copy link
Member

On the breakpoint, can you tell me the value of the key?
Maybe even pull out the two fileInfo objects and paste them here?

@skjnldsv skjnldsv added the needs info Not enough information provided label May 27, 2021
@Lionel-cl
Copy link
Author

@skjnldsv ,
Is-this ok for you ?
this comes from the breakpoint on systemtags.js:40
because i don't know why but I can't stop the execution on the breakpoints in fileUtils.js:96
Take a look where the program is running and where the variables are displayed...
breakpoints in fileUtils.js do not work.
Thank you very much

Capture d’écran 2021-05-27 à 13 36 55
Capture d’écran 2021-05-27 à 13 29 12

  • fileInfo1:
    *
    basename: 12
    canAssign: true
    displayName: 2010
    etag: null
    filename: "/systemtags/12"
    id: 12
    lastmod: null
    mime: ""
    size: 0
    type: "file"
    userAssignable: false
    userVisible: true
    proto: Object

    fileInfo2:
    *
    basename: "systemtags"
    canAssign: ""
    displayName: ""
    etag: null
    filename: "/systemtags"
    id: ""
    lastmod: null
    mime: ""
    size: 0
    type: "file"
    userAssignable: ""
    userVisible: ""
    proto: Object

    key: "displayName"
    o: undefined
    r: undefined
    this: undefined

@skjnldsv skjnldsv added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending approval or rejection. This issue is pending approval. needs info Not enough information provided labels May 27, 2021
@skjnldsv
Copy link
Member

Perfect!! Thank you! 🎉

@skjnldsv
Copy link
Member

It happens when we have one basename that is a number and the other that is a string

@Lionel-cl
Copy link
Author

problem of managing strings in JS. easy to correct? Everything should be a string, no?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants