Skip to content
This repository has been archived by the owner on May 3, 2021. It is now read-only.

error when moving photo to album #220

Closed
rnwgnr opened this issue Feb 27, 2019 · 1 comment
Closed

error when moving photo to album #220

rnwgnr opened this issue Feb 27, 2019 · 1 comment
Labels
bug Something isn't working JS - Lychee-Front JavaScript related see https://github.com/LycheeOrg/Lychee-front
Milestone

Comments

@rnwgnr
Copy link
Contributor

rnwgnr commented Feb 27, 2019

Detailed description of the problem

When viewing a single photo, it is not possible to move the photo to an album using the folder icon in the context menu.
Moving a single or even multiple photos from the album view works well.

Steps to reproduce the issue

  1. Open a photo in the photo view.
  2. Click the folder icon top right and select an album.
  3. Nothing happens.

Console shows an error:
TypeError: callback is not a function main.js:1764:12

I did a bit of debugging and found, that when using the folder icon in photo view, the "callback" parameter of contextmenu.move is null, although it is set in the caller. As i'm not that familiar with JS, i have no clue where to look.

Browser and system

Manjaro Linux and Firefox 65.0.1
Windows 8.1 and IE11

ildyria referenced this issue in LycheeOrg/Lychee-front Feb 28, 2019
ildyria added a commit that referenced this issue Feb 28, 2019
@ildyria ildyria added this to the v3.2.14 milestone Feb 28, 2019
@ildyria
Copy link
Member

ildyria commented Feb 28, 2019

Nicely spotted. 👍

@ildyria ildyria added bug Something isn't working JS - Lychee-Front JavaScript related see https://github.com/LycheeOrg/Lychee-front labels Mar 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working JS - Lychee-Front JavaScript related see https://github.com/LycheeOrg/Lychee-front
Projects
None yet
Development

No branches or pull requests

2 participants