Opened 7 years ago

Closed 6 years ago

#1660 closed bug (fixed)

WSOD after uploading pictures

Reported by: planetcruiser Owned by: mahouni
Priority: blocker Milestone: 0.8
Component: BW Gallery Keywords:
Cc: mahouni

Description

Issue:

Clues:

  • The image appears in the gallery just fine

Change History (8)

comment:1 Changed 6 years ago by mahouni

  • Cc mahouni added
  • Owner set to mahouni
  • Status changed from new to assigned

pushed to develop, ready for testing on alpha...
https://gitorious.org/bewelcome/rox/commit/8c2a05f74855bab3f8b7f2cf198fa97905c84bbe

At the moment it will be redirected always to members gallery, even when the upload failed.

It seems that

$this->_model->uploadProcess($vars)

returns "false" even when the upload was successful.. Open a new ticket for that , but at the moment I won't fix that.

comment:2 Changed 6 years ago by globetrotter_tt

Is this already on alpha for testing?

comment:3 Changed 6 years ago by mahouni

Yes, ready for testing on alpha.

PS: you can check the rev version at the bottom of alpha.bewelcome.org and compare it with the commit log on gitorious. Alpha is at 8c2a05f at the moment. So the changes from this ticket are already there.

You still can see the warnings when browsing e.g. the forums. But these notices should go away when alpha is updated with jsfan's next commit.

comment:4 Changed 6 years ago by sitarane

For a photo of 1.9M (according to nautilus), I get a connection timout on the url http://alpha.bewelcome.org/gallery/uploaded

I'm on a superhighway type upload speed.

Reducing size to a more manageable 500k and everything works smoothly.

Ubuntu 12.04 / firefox 15

Last edited 6 years ago by sitarane (previous) (diff)

comment:5 Changed 6 years ago by toub

Cool, works fine.

comment:6 Changed 6 years ago by crumbking

Okay tested on alpha. It redirects to gallery/show/user/crumbking/images

Seems to work.

@mahouni

TODO: should callback to gallery/upload and display an error message

I guess let's open another ticket for that?

comment:7 Changed 6 years ago by mahouni

Yep, I forgot to add the link here: the new ticket is here:

http://trac.bewelcome.org/ticket/1676

comment:8 Changed 6 years ago by crumbking

  • Resolution set to fixed
  • Status changed from assigned to closed
Note: See TracTickets for help on using tickets.