Photo Organizer

  • Status Closed
  • Percent Complete
  • Task Type Feature Request
  • Category User Interface
  • Assigned To No-one
  • Operating System All
  • Severity Low
  • Priority Very Low
  • Reported Version Devel
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Photo Organizer
Opened by WRXIzumi - 2006-11-26

FS#137 - Change default location in buld update to spool

Currently when you go to the bulk update page, the default is to update every single photo in the system (All Folders). This is a very dangerouus defualt as there is no way to back out of a bulk update without resotoring the database from a backup. (Don't ask me how I know this.) See attached diff that will set the "spool" as the default. In order to do this, it will display the "spool" option even when there are no photos in the spool. Worst case scenario is that you update the photos that you had already chosen for special consideration. (i.e. putting then in the spool)

Closed by  pizza
2006-11-26 23:05
Reason for closing:  Fixed
Additional comments about closing:  I'm satisfied with the checked-in solution, but if you don't like it, re-open this and we can work something out.

Somehow managed to delete one too many lines. Here is the correct diff.

pizza commented on 2006-11-26 22:54

I did this to myself about a week ago, only I needed to write some perl hackery to selectively restore a single column from my nightly backups.. I guess I never got around to fixing the default to be something more sane. :)

That said, I don't like your solution -- I think the user should explicitly select what they want, so there should be nothing selected by default. Firefox2 and IE7 both seem to handle this fine, so I'm committing that instead. (r872)

pizza commented on 2006-11-26 22:56

backported an equivalent fix to -stable (r873)

Fantastic, thanks for the change. One less thing for me to worry about.


Available keyboard shortcuts


Task Details

Task Editing