Project

General

Profile

Actions

Feature #1725

closed

Delete button on comments

Added by Derek Montgomery over 15 years ago. Updated about 8 years ago.

Status:
Closed
Priority:
Normal
Category:
Issues
Target version:
Start date:
2008-07-31
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

Description

I know people with rights can just remove all content of a comment to get rid of it but it is not as "evident" as a delete icon, especially when quote and edit icons are present.


Files

add_a_delete_link_to_journal_entries.patch (968 Bytes) add_a_delete_link_to_journal_entries.patch Adds a delete button to issue journal entries Felix Schäfer, 2010-07-07 20:00
delete-note.png (47.3 KB) delete-note.png screenshot Go MAEDA, 2016-01-17 16:24
add_a_delete_link_to_journal_entries_v2.patch (1.67 KB) add_a_delete_link_to_journal_entries_v2.patch patch against 3.2.0.devel.15067 Go MAEDA, 2016-01-17 16:25

Related issues

Related to Redmine - Feature #1665: Ability to delete tracker commentsClosed2008-07-20

Actions
Related to Redmine - Patch #7610: Patch providing issue journal rollbackNew2011-02-11

Actions
Has duplicate Redmine - Feature #11255: Add delete button for commentClosed

Actions
Actions #1

Updated by Felix Schäfer over 13 years ago

  • Assignee set to Felix Schäfer

I've been wondering about this too, will have a look at it and see how quickly feasible it is.

Actions #2

Updated by Felix Schäfer over 13 years ago

The supplied patch adds a delete button to an issue's journal entries, only for users with the right to edit the journal entry.

Actions #3

Updated by Eric Davis over 13 years ago

  • Status changed from Resolved to 7
Actions #4

Updated by Eric Davis over 13 years ago

  • Assignee changed from Felix Schäfer to Eric Davis
  • Target version deleted (1.0.1)

There is a problem with this patch. Journals are only deleted if:

  1. The notes are empty and
  2. The details are empty

All this patch does is to clear the notes. If a Journal had details, it wouldn't be deleted. There are two options we can do here:

  1. Add a new action to really delete a Journal and provide a link to it, or
  2. Change this icon so it's clear that it only clears the note.

I've added Holger to this issue, I know he has an opinion about deleting things like Journals and I would like to hear what he thinks.

Actions #5

Updated by Felix Schäfer over 13 years ago

The scope of the patch wasn't to delete journals but the notes attached to it, which is currently possible to if you edit the notes to an empty string. The current functions quote and edit only work on the notes too, so I don't think that would be that misleading.

Actions #6

Updated by Eric Davis over 13 years ago

Maybe the title attribute should say "Delete notes" then. What do you think?

Actions #7

Updated by Felix Schäfer over 13 years ago

  • Assignee changed from Eric Davis to Felix Schäfer

Can do that, I'll change the other titles (edit and quote) too to reflect that if it's ok with you?

Actions #8

Updated by Felix Schäfer over 13 years ago

  • Assignee changed from Felix Schäfer to Eric Davis

Eric, I slightly reworked the patch to have the title say "Delete notes", the confirmation pop-up now also states that the changes to the issue will not be reverted if the journal details are not empty.

See http://github.com/thegcat/redmine/commit/5be3c6e9b405bf03ad69226e8235e3d6618adbf5 for the change.

Actions #9

Updated by Eric Davis over 13 years ago

  • Assignee deleted (Eric Davis)
Actions #10

Updated by Brian Lindahl about 13 years ago

created a related patch for rolling back journal entries #7610

This rolls back the changes made to the issue as well as removing the journal entry. Note that this capability is only enabled for the last journal entry (for obvious reasons).

Actions #11

Updated by rm user over 11 years ago

another 4 years issue.. :(

Actions #12

Updated by Jean-Philippe Lang about 11 years ago

  • Status changed from 7 to New

Assigned issue with no assignee back to New status.

Actions #14

Updated by Terry Cullen almost 10 years ago

+1

Actions #15

Updated by rm user almost 10 years ago

what's up with this? (5 years old and counting...)

i managed to delete WHOLE issue instead of the single comment.. :(

Actions #16

Updated by Go MAEDA about 8 years ago

Updated patch for 3.2.0.devel.15067.

The procedure to delete note is a little bit difficult to understand. This change improve usability. In addition, we can avoid accidental deletion of a issue by clicking delete button under a latest note.

Please include this feature in Redmine 3.3.0.

screenshot

Actions #17

Updated by Jean-Philippe Lang about 8 years ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed
Feature added in r15072.
I made slight changes to the patch:
  • removed image_tag regarding #21256
  • use of existing confirmation message
  • AJAX call (similar to how the journal edit works)

Thanks.

Actions #18

Updated by rm user about 8 years ago

wow! 7years old issue is finally solved!

big up Jean-Philippe and Go MAEDA!

what about permissions btw? is there a permission where user can delete only his own comments on issues?

Actions #19

Updated by Jean-Philippe Lang about 8 years ago

rm user wrote:

what about permissions btw? is there a permission where user can delete only his own comments on issues?

As noted above, users with "edit notes" or "edit own notes" were already able to delete notes by making them empty. The delete button is just some kind of shortcut for doing this. The permissions are the same, so the answer is yes, you can allow users to edit/delete only their own notes.

Actions

Also available in: Atom PDF