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

Image that is uploaded first, if opened first the picture is not displayed in the image media viewer in ocis backend #6289

Closed
SwikritiT opened this issue Jan 20, 2022 · 3 comments

Comments

@SwikritiT
Copy link
Contributor

SwikritiT commented Jan 20, 2022

The image that is uploaded first, if opened first the picture is not displayed in the image media viewer

Steps to reproduce

  1. docker-compose pull

  2. docker-compose up ocis selenium vnc

  3. Access ocis through vnc

  4. Log in to Einstein

  5. upload testavatar.jpg first and then upload testavatar.png

  6. Now open testavatar.jpg in a media viewer through the details panel. A blank white screen is shown instead of a picture.
    image1

  7. Go to the next picture testavatar.png through the media viewer. It displays.

image2

  1. Now go back to the previous picture. White screen again but the name at the bottom is testavatar.png

image3

  1. Delete both of the pictures and repeat step 4
  2. This time open testavatar.png first (i.e the image that was uploaded second) through details panel. It's displayed.

image4

  1. Switch to the next image. It is also displayed.

image5

  1. This issue does not exist for video formats or if only one picture is uploaded.

Expected behaviour

The image should be displayed no matter the order of upload.

Actual behaviour

As described above

@SwikritiT SwikritiT added the bug label Jan 20, 2022
@SwikritiT SwikritiT changed the title Image that is uploaded first, if opened first the picture is not displayed in the image media viewer Image that is uploaded first, if opened first the picture is not displayed in the image media viewer in ocis Jan 20, 2022
@SwikritiT SwikritiT changed the title Image that is uploaded first, if opened first the picture is not displayed in the image media viewer in ocis Image that is uploaded first, if opened first the picture is not displayed in the image media viewer in ocis backaend Jan 20, 2022
@SwikritiT SwikritiT changed the title Image that is uploaded first, if opened first the picture is not displayed in the image media viewer in ocis backaend Image that is uploaded first, if opened first the picture is not displayed in the image media viewer in ocis backend Jan 20, 2022
@individual-it
Copy link
Member

@SwikritiT please try to see if that works with an older version of web (as it works with the web that is compiled into ocis)
and with git bisect find the commit that caused it

@SwikritiT
Copy link
Contributor Author

I think the problem with the media viewer arose from this owncloud/ocis#2739 PR in ocis. I couldn't track it down to a single commit because a lot of ocis commits didn't build properly and I had to skip them. Upon git bisect, it showed that one of these might be the first faulty commits.

b63b322b6f0abfd2cb4e6a7f3359b6517e56f723
c36e99895bbfd25b923455feb6b1c16f0f52e576
5a29959a0bfe26fa468101207c0343cd283f737f
de7620af98a2ec3fdd36bc90a83cd0597a3fb89b
5ec72daaeb512efbc83c62e13fc098f4f49ed5a0 

Surprisingly I couldn't reproduce the problem in the current master ocis. It can be reproduced in commits from this PR owncloud/ocis#2739 to this owncloud/ocis#2989

but after this commit, it cannot (so maybe it fixed it?) owncloud/ocis@0bfc1f0

I also tried to reproduce it with the previous version of the web and yes it can be reproduced if the ocis is between owncloud/ocis#2739 to this owncloud/ocis#2989 commit

@phil-davis
Copy link
Contributor

Yes, the problem is not happening in CI any more when it points to the very latest oCIS commit id. So "something happened" in the last week or so that fixed it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants