Opened 10 years ago

Closed 9 years ago

#113 closed bug (fixed)

Gallery: Delete button is not working - part 2

Reported by: globetrotter_tt Owned by: micha
Priority: critical Milestone: 0.1.2-outreach-bugfixing
Component: BW General Keywords: gallery hot
Cc:

Description

In the gallery it is not possible to delete a picture after uploading.
There is no function for this so far

files:

*https://www.bevolunteer.org/trac/browser/trunk/templates/apps/gallery/image.php

*https://www.bevolunteer.org/trac/browser/trunk/build/gallery/gallery.ctrl.php

Change History (20)

comment:1 Changed 10 years ago by public

  • Owner set to salgado

comment:2 Changed 10 years ago by micha

Any progress on this? I would like to help here if help is needed.

comment:3 Changed 10 years ago by philipp

what's the status here? any chance to get it done by this week to get it into the release? would be very great!

comment:4 Changed 10 years ago by philipp

no reply yet, so I take it out of this release. we can add it as soon afterwards

comment:5 Changed 10 years ago by philipp

  • Milestone changed from 0.1-outreach-release to unassigned

comment:7 Changed 10 years ago by micha

Wow, looks like our first real bogus entry in here ;)

comment:8 Changed 10 years ago by public

  • Owner changed from salgado to public

comment:9 Changed 10 years ago by public

  • Owner public deleted

comment:10 Changed 9 years ago by micha

  • follow_up set to none
  • Milestone changed from unassigned to 0.1.1-outreach-bugfixing

comment:11 Changed 9 years ago by micha

  • Owner set to micha
  • Status changed from new to assigned

I already found a way to delete entries to the pictures in the database, so they won't show up anymore in our gallery. Unfortunately this doesn't cover the deletion of the file. Unwanted waste of webspace would be the result. On the other hand, maybe we don't want to delete all the uploaded pictures forever. In case of abuse, we might still need them afterwards?!


I dont want to decide that on my own but talk about it here instead.. any opinions? Maybe we should move the files to a seperate folder instead of deleting them and after a certain period of time, they get deleted automatically by a script?! The database could be handled in a similar way..

comment:12 Changed 9 years ago by micha

  • follow_up changed from none to test

made it work now, both mysql and file deletion. Please check!

comment:13 Changed 9 years ago by micha

the commit is rev.3520

comment:14 Changed 9 years ago by midsch

  • follow_up changed from test to move to alpha

I tested at http://test.bewelcome.org/: delete of an image works at least for the user, i can't see if the image is still in the database (tested image number was 27, name was couch.jpg)

But see also:

  • Error when editing image name -> ticket #270
  • The whole gallery area are is not very userfriendly -> ticket #267

comment:15 follow-up: Changed 9 years ago by matrixpoint

  • follow_up changed from move to alpha to test on alpha

Moved to alpha in r3546.

comment:16 in reply to: ↑ 15 Changed 9 years ago by micha

tested again on alpha and seems to work but still need to check that with a sysadmin. Otherwise there is no real proof for it..

comment:17 Changed 9 years ago by matrixpoint

I tested this on local host to see if the file and database entries are removed and they are. Still needs to be verified by a sysadmin on alpha.

It would be good if tickets #270, #276 could be brought along with this to release.

comment:18 Changed 9 years ago by micha

  • follow_up changed from test on alpha to release

ok, me and JeanYves? tested this on alpha now. Works and can go to production.

comment:19 Changed 9 years ago by matrixpoint

Moved to production in r3587.

comment:20 Changed 9 years ago by matrixpoint

  • follow_up changed from release to test on alpha
  • Summary changed from Gallery: Delete button is not working to Gallery: Delete button is not working - part 2

r3578, just moved to alpha, contains further gallery upgrades associated with this ticket.
Some changes have been made to the image delete operation.
Words have been added throughout the gallery app.

What needs to be tested?

comment:21 Changed 9 years ago by matrixpoint

  • follow_up changed from test on alpha to none
  • Resolution set to fixed
  • Status changed from assigned to closed

Released to production today, and the delete function appears to work on www.bewelcome.org.
Closing this ticket unless new bug reports come in.

Note: See TracTickets for help on using tickets.