Last modified: 2005-06-14 17:20:00 UTC
If someone uploads an image which already exists, the old image is overwritten without any logentry or history change if the user doesn't click the ReUpload button. Reason: The new image is saved with the same filename as the old image and the reupload button restores it to the old version. Possible Solutions: 1.) Save the image with a temporary name and rename it on confirm. (Would require a maintainance script which removes old images) 2.) Restore the image on the next action of the user (Same problem as now if the user does nothing after the warning) 3.) Let the user reupload the image when he accepts to overwrite the old image
Created attachment 46 [details] slightly tested patch which implements version 1 (diff against REL1_3)
Can't reproduce this any more. Has this been fixed in the meantime?
Bug confirmed still present in REL1_3 and HEAD.
I've made a somewhat more extensive reworking based on Benjamin's patch. All 'warning'ed uploads are held in the temp dir until confirmed or canceled. Also I fixed a logic error in the check for protected images, moved some of the checks around, and generally did some clean-up. While the situation with files getting left there if the user takes no action is unsatisfying, it's actually the same as what the previous code did, only now the garbage files are in a separate tree and no longer pollute the main image file area. Future improvements are welcome, of course. Fixed in HEAD, REL1_3, and live on Wiki*edia.
