Implement "remove" for photos

Bug #1367258 reported by Fabio Zaramella
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Foto
In Progress
Critical
Erasmo Marín

Bug Description

At the moment the code doesn't exist, and the remove button does nothing.

Changed in foto:
importance: Undecided → High
milestone: none → foto-1.0
Changed in foto:
status: New → Confirmed
Revision history for this message
Erasmo Marín (erasmo-marin) wrote :

Should it ask to delete from disk or do this by default and ask only confirmation?

Other option is to implement a recycle bin like other software does.

Changed in foto:
assignee: nobody → Erasmo Marín (erasmo-marin)
Revision history for this message
Fabio Zaramella (fabiozaramella) wrote :

By default, at least at the moment, it should just remove the photos from the library, in future there should be an option in a preference window to remove it from disk, in future, not now, it's not necessary.

Revision history for this message
Fabio Zaramella (fabiozaramella) wrote :

Are you working on this?

Revision history for this message
Erasmo Marín (erasmo-marin) wrote :

Yep. It's already implemented, but deleting a picture collection from the database is slow if you do it with multiples queries, so, I have to write a function for "massive deletes". When ready, I will make a merge proposal, that way you can review it too.

Changed in foto:
status: Confirmed → In Progress
Changed in foto:
importance: High → Critical
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.