Opened 10 years ago

Closed 10 years ago

Last modified 7 years ago

#2653 closed enhancement (fixed)

RFE: Re-Use of screenshot IDs

Reported by: izzy Owned by: Radek Bartoň
Priority: normal Component: ScreenshotsPlugin
Severity: minor Keywords:
Cc: Trac Release: 0.10


If some image was deleted, the ID is lost. The idea behind this seems to be clear to me (re-using the ID could result in old references to this ID getting re-activated with the wrong target). Still there should be a possibility to re-use the ID, e.g. if a screenshot was accidentally deleted.

Idea of implementation:

  • in the matrix view, next to the "Add" button an input could be placed (for the ID to replace), together with an additional checkbox (confirm replace) which is by default always unchecked. If used, on submit there should be an additional warning (Javascript popup or additional page in between)
  • selecting "screenshots/1" (where "1" is a deleted ID) either by typing in the URL or by following some broken link, a user with SCREENSHOTS_ADMIN should not only receive the error message, but the possibility to upload a new screenshot.

Well, Radek - looks like I'm keeping you busy, huh? ;)

Attachments (0)

Change History (5)

comment:1 Changed 10 years ago by Radek Bartoň

Status: newassigned

I'm affraid that inserting screenshot with missing ID won't be possible since IDs are assigned by DB backend and I'm not aware that there is a way to request certain ID. At least not a common for sqlite, PostgreSQL and MySQL database.

I'll consult DB documentation about this.

Generating own IDs to accomplish this is IMHO too intrusive change for such minor feature.

Nevertheless, replacing existing screenshot could be done.

Well, Radek - looks like I'm keeping you busy, huh? ;)

No worry, but be aware that I'm working on Trac's plugins only in spare time and when I have a taste for it.

comment:2 Changed 10 years ago by Radek Bartoň

Resolution: fixed
Status: assignedclosed

First bullet is obsoleted by possibility of screenshot replacement in r4314. Second bullet is implemented there too.

comment:3 Changed 10 years ago by izzy

I'm really impressed - will checkout ASAP to have it available! :) Guess you now have goot reason to bump the version number to 0.6 <g> Thank you!

comment:4 Changed 10 years ago by Radek Bartoň

I'll shift version to 0.6 with DB schema update needed for #2656 which I'm currently working on. I'll backport all these new features to 0.10 at once when all opened RFE tickets will be implemented. Thanks for your patience.

comment:5 Changed 10 years ago by izzy

That's great news. I will look forward to that. Finally my fault - why did I open so many tickets ;)

Thank you again for the great work you are doing! I love ScreenshotsPlugin ;)

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain Radek Bartoň.
The resolution will be deleted.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.