Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Upload button does not work #200

Closed
Sanookmakmak opened this issue Feb 4, 2017 · 17 comments
Closed

Upload button does not work #200

Sanookmakmak opened this issue Feb 4, 2017 · 17 comments
Labels
bug Something isn't working high High priority regression Regression of a previous working feature

Comments

@Sanookmakmak
Copy link

Sanookmakmak commented Feb 4, 2017

Steps to reproduce

  1. Open the gallery
  2. Click on the + sign
  3. Try to upload a picture

Expected behaviour

Picture should be uploaded

Actual behaviour

Nothing happens

More information

https://help.nextcloud.com/t/nc-11-gallery-16-0-0-no-picture-upload-possible/

This issue is already filed as bug but posted in the wrong category.

nextcloud/server#3229

Can an admin move it to here please?
Thank you.

@oparoz
Copy link
Member

oparoz commented Feb 4, 2017

We can't move an issue across projects, so just copy/paste the OP from server to replace the OP here. Thanks!

@Sanookmakmak
Copy link
Author

done @oparoz

@oparoz
Copy link
Member

oparoz commented Feb 10, 2017

Drag and drop is also broken.

@rullzer Any idea what has changed in the server in 11 regarding uploads?

@JaredBoone
Copy link

When this is fixed please consider backporting to the stable11 branch.

@oparoz oparoz added the backport-request Pending backport by the backport-bot label Feb 13, 2017
@oparoz
Copy link
Member

oparoz commented Feb 13, 2017

The first step is for people to post logs so that we can get a hint of what's happening.

@JaredBoone
Copy link

I have reproduced this in Windows 7 with Chrome 56.0.2924.87 and Firefox 51.0.1. I have also reproduce it in OS X Yosemite 10.10.5 with Chrome 56.0.2924.87 and Firefox 51.0.1.

Here are the steps to reproduce:

  • Open Gallery app
  • Click on upload "+" icon
  • Click on the "Upload" menu item
  • Select an image file to upload from my local file system
  • Click the "Open" button
  • Observe that the file selector dialog closes, but nothing else happens and no file is uploaded. In the browser web developer tools, no network requests are sent to the server, no javascript console log or error message appears.

How can I help?

@oparoz
Copy link
Member

oparoz commented Feb 13, 2017

PLease add 'debug' => true,` to your config.php to see if it helps generate browser logs.

@JaredBoone
Copy link

I set 'debug' to true in config.php but I don't see any additional output in the browser logs.

I also modified apps/files/js/file-upload.js - I set "_trace" to true, but I see no console output from the "log" function in that file.

@JaredBoone
Copy link

Uploading this same file works fine when I use the "Files" app. Also, when using the Files app, I see multiple lines of browser console log output from file-upload.js (after setting _trace set to true).

@oparoz
Copy link
Member

oparoz commented Feb 14, 2017

Basically, this commit broke everything:
nextcloud/server@59c5be1

So someone needs to match the changes or we need to remove the functionality.

@Sanookmakmak
Copy link
Author

we need to remove the functionality.

Why does the files app can upload the file without any problems and the gallery app can't?

@oparoz
Copy link
Member

oparoz commented Feb 16, 2017

Why does the files app can upload the file without any problems and the gallery app can't?

Because there isn't a shared component apps can use to make it possible to perform classic files operations such as upload, move, copy. Each app has to implement it. In order to not have to write too much code which is unrelated to Files, I re-used the base libraries, but the way it works has changed last year, so Gallery needs to get the same changes.

@Sanookmakmak
Copy link
Author

Because there isn't a shared component apps can use to make it possible to perform classic files operations such as upload, move, copy. Each app has to implement it.

So every developer has to invent the wheel again. Wouldn't that be a nice improvement for a new Nextcloud version?
;-)

@jasonbayton
Copy link
Member

@Sanookmakmak interestingly a comment a user Stuart on the forum keeps bringing up. Shared intents and whatnot.

It's a grand idea.

@hudriwu
Copy link

hudriwu commented Mar 2, 2017

any news? does an update to 11.0.2 solve the problem?

@oparoz
Copy link
Member

oparoz commented Mar 2, 2017

No, this will be in NC 12 and backported to 11.0.3

@oparoz oparoz added high High priority regression Regression of a previous working feature labels Apr 16, 2017
@oparoz
Copy link
Member

oparoz commented Apr 16, 2017

Fix is in #238

@MorrisJobke MorrisJobke removed the backport-request Pending backport by the backport-bot label Jan 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working high High priority regression Regression of a previous working feature
Projects
None yet
Development

No branches or pull requests

6 participants