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

move (paste) with elFinderSupportVer1.js errors #1880

Closed
dup2 opened this issue Feb 1, 2017 · 3 comments
Closed

move (paste) with elFinderSupportVer1.js errors #1880

dup2 opened this issue Feb 1, 2017 · 3 comments
Assignees

Comments

@dup2
Copy link
Contributor

dup2 commented Feb 1, 2017

When moving an object (a jpeg file) to a folder, we get an error indicating that the file already exists at the new location - this is not true however the backend copies/moves the file correctly to the new location and returns a JSON fragment with the cwd, cdc and tree attributes.

How to reproduce

  • Drag a file to a folder (move)
  • The backend get's a "paste" request with a "dst" parameter and a target array with one element
  • The backend moves the file and returns a JSON with cwd, cdc and tree attributes and HTTP status 200
  • The frontend displays the error message
  • A subsequent reload displays a correct file listing in the source directory (file not there anymore) and the target directory (file is there - the move was performed)

Environment:

  • elFinder version 2.1.20
  • JQuery 1.11.4
  • Using transport proxy for version 1 (due to the backend API)

In this screenshot, the file "iMac-24.jpg" is being dragged and dropped onto the "Test" folder.

screen shot 2017-02-01 at 22 14 52

@nao-pon nao-pon self-assigned this Feb 3, 2017
@nao-pon nao-pon closed this as completed in 5c06757 Feb 3, 2017
@nao-pon
Copy link
Member

nao-pon commented Feb 3, 2017

@dup2 Thanks! 👍

@dup2
Copy link
Contributor Author

dup2 commented Feb 3, 2017

Thanks for this quick fix! I'll test it asap to confirm.

@dup2
Copy link
Contributor Author

dup2 commented Feb 3, 2017

Works with the master branch, Cheers!

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

2 participants