Ticket #840 (closed defect: wontfix)

Opened 7 years ago

Last modified 2 months ago

pressing delete link in ticket comments ticks the wrong comment

Reported by: moo Assigned to: rjollos
Priority: high Component: TicketDeletePlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

when there's a ticket comment deleted in the middle, say. 1 2 3 4, and 2 is deleted already (i guess) if i press "delete" on 3, it ticks the wrong comment in the admin page.

ticking the "change at ***" also won't ticks the fields

Attachments

Change History

12/10/10 12:07:15 changed by rjollos

  • owner changed from coderanger to rjollos.
  • status changed from new to assigned.

I think this may be related to #6535. I'll take a look.

12/16/10 09:16:55 changed by rjollos

  • priority changed from normal to high.

02/09/12 18:48:18 changed by rjollos

  • status changed from assigned to new.

(follow-up: ↓ 5 ) 07/26/12 22:29:36 changed by rjollos

  • status changed from new to closed.
  • resolution set to wontfix.

0.10 is no longer supported.

(in reply to: ↑ 4 ; follow-up: ↓ 6 ) 08/09/12 00:36:46 changed by ChrisNelson

Replying to rjollos:

0.10 is no longer supported.

OK. But I still see this in 0.11.6 (using plugin version 2.0). Was there a similar ticket against 0.11? Is this fixed in 3.0?

(in reply to: ↑ 5 ; follow-up: ↓ 7 ) 08/09/12 03:30:04 changed by rjollos

Replying to ChrisNelson:

OK. But I still see this in 0.11.6 (using plugin version 2.0). Was there a similar ticket against 0.11? Is this fixed in 3.0?

I had thought the issue was fixed in [6924]. I haven't been able to reproduce. There were several revs of version 2.0, which is why I started tagging the egg builds with the SVN revision. Do you know what SVN revision you are currently running?

If you can test with 3.0dev and reproduce the issue, I'll get it fixed.

(in reply to: ↑ 6 ) 08/09/12 14:56:44 changed by ChrisNelson

Replying to rjollos:

Replying to ChrisNelson:

OK. But I still see this in 0.11.6 (using plugin version 2.0). Was there a similar ticket against 0.11? Is this fixed in 3.0?

I had thought the issue was fixed in [6924].

That diff doesn't seem to relate to this issue. The problem is that Trac doesn't number at least attachments and maybe edits of the description and the deletion is invoked with "&cnum=n" but instead of looking for 'n' in the oldvalue field where comments are numbered, it looks for the nth group of ticket changes so unnumbered changes (like attachments) throw the delete plugin off.

I haven't been able to reproduce.

  • Create a ticket
  • Add a comment
  • Edit the description
  • Attach a file
  • Add a comment
  • Respond to one of the comments
  • Delete one of the comments

So far, so good.

  • Try to delete on of the other comments

The wrong one is checked on the delete preview. (It may matter which one you comment on and which one you delete first. I'll verify when I test this.)

There were several revs of version 2.0, which is why I started tagging the egg builds with the SVN revision. Do you know what SVN revision you are currently running?

No, sorry.

If you can test with 3.0dev and reproduce the issue, I'll get it fixed.

I'll try to do this today or tomorrow.

08/09/12 15:07:41 changed by rjollos

  • status changed from closed to reopened.
  • release changed from 0.10 to 0.11.
  • resolution deleted.

08/09/12 15:09:49 changed by rjollos

  • status changed from reopened to new.

(follow-ups: ↓ 12 ↓ 14 ) 08/09/12 16:59:41 changed by ChrisNelson

Downloaded the .zip from the plugin page and installed. It says it's "3.0.0dev". For me, this behaves worse than 2.0. I don't like the reverse chronological sort of changes. But apropos of this ticket, the wrong change is checked. Let me know if the sequence above isn't enough to reproduce it.

(in reply to: ↑ 11 ) 08/09/12 20:06:47 changed by rjollos

Replying to ChrisNelson:

Downloaded the .zip from the plugin page and installed. It says it's "3.0.0dev". For me, this behaves worse than 2.0. I don't like the reverse chronological sort of changes.

There were so pretty major changes in [11802], which dropped all of the JavaScript and implemented everything in Python via Genshi. This should have resolved several possible plugin conflicts, see comment:35:ticket:1749. However, I'm not surprised that there might be some behavior changes that I didn't notice when modifying and applying the patch.

I'm happy to fix any specific behavior change you don't like if you open a ticket for the issue.

But apropos of this ticket, the wrong change is checked. Let me know if the sequence above isn't enough to reproduce it.

I'll focus on fixing this issue first.

09/25/12 10:24:29 changed by rjollos

  • keywords set to regression.

(in reply to: ↑ 11 ; follow-up: ↓ 15 ) 10/02/12 22:26:22 changed by rjollos

Replying to ChrisNelson:

It says it's "3.0.0dev". For me, this behaves worse than 2.0. I don't like the reverse chronological sort of changes.

Do you want the most recent change at the bottom of the page, or at the top of the page?

(in reply to: ↑ 14 ; follow-up: ↓ 16 ) 10/02/12 22:35:31 changed by ChrisNelson

Replying to rjollos:

Replying to ChrisNelson:

It says it's "3.0.0dev". For me, this behaves worse than 2.0. I don't like the reverse chronological sort of changes.

Do you want the most recent change at the bottom of the page, or at the top of the page?

I want the most recent change at the bottom, same order as in the ticket. (Unless Trac 1.0 implemented a last at the top ordering or something, then it gets ugly.)

(in reply to: ↑ 15 ) 10/02/12 22:38:04 changed by rjollos

Replying to ChrisNelson:

I want the most recent change at the bottom, same order as in the ticket. (Unless Trac 1.0 implemented a last at the top ordering or something, then it gets ugly.)

In Trac 1.0 you can reverse the sorting with a radiobutton selection, but this plugin is not needed with Trac 0.12 and later, so it won't be an issue for us.

10/02/12 22:44:51 changed by rjollos

(In [12098]) Refs #840, #10263:

  • Added check of Trac version. The plugin won't install in Trac 0.12 and later. In Trac 0.12 and later, enable the equivalent functionality by setting tracopt.ticket.deleter = enabled.
  • When a "change" checkbox is selected, all of the child "field" checkboxes will be selected.
  • Improved warning that is issued when an invalid ticket ID is entered. The left # characters in case the user enters #100 for a ticket ID, rather than 100.
  • Fixed some regressions introduced in #840, when migrating from the javascript version of the plugin, to the genshi version.
    • Alert confirmation dialog has been restored.
    • Changes sort chronlogically.

Please test and report back, if you can.

10/02/12 23:37:01 changed by rjollos

Note that I haven't yet addressed the issue in comment:7.

10/02/12 23:40:05 changed by rjollos

(In [12099]) Refs #840, #10263: FIX: the reply buttons were redirecting to the delete page as well.

10/18/12 22:11:58 changed by rjollos

  • keywords deleted.

Most likely, nothing will happen with this ticket unless someone provides a patch. Following #10263, we have a version of TicketDeletePlugin that utilizes ITemplateStreamFilter to modify the Genshi template. Since Trac 0.12 supports ticket deletion, I have little motivation to do additional work on this plugin. I'd welcome a patch though!

03/26/13 20:06:55 changed by rjollos

  • status changed from new to closed.
  • resolution set to wontfix.

Closing tickets for deprecated plugin.

Add/Change #840 (pressing delete link in ticket comments ticks the wrong comment)




Change Properties
Action