Make WordPress Core

Opened 13 months ago

Closed 9 months ago

#58168 closed defect (bug) (wontfix)

Need to focus on comment validation in backend

Reported by: ravipatel's profile ravipatel Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Comments Keywords: needs-patch close
Focuses: accessibility, administration Cc:

Description

1) Edit comment
2) Remove all content and save

i think there have need to add some validation please add content.
Right now form submitted without any data.

Also need to validate email and weburl when use Quick edit.

Attachments (3)

Edit-comment-1.png (77.4 KB) - added by ravipatel 13 months ago.
Edit comment form
ajax-edit-comment.png (36.3 KB) - added by ravipatel 13 months ago.
Ajax edit comment validation required
58168.patch (670 bytes) - added by ravipatel 13 months ago.
Edit comment form validation

Download all attachments as: .zip

Change History (9)

@ravipatel
13 months ago

Edit comment form

@ravipatel
13 months ago

Ajax edit comment validation required

@ravipatel
13 months ago

Edit comment form validation

#1 @sabernhardt
13 months ago

  • Component changed from General to Comments
  • Focuses accessibility added
  • Version trunk deleted

This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.


13 months ago

This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.


10 months ago

#4 @joedolson
10 months ago

  • Keywords close added

Thanks for opening this ticket, @ravipatel!

There are legitimate cases for an admin to want to remove comment text without removing the comment itself. For example, if you want to be tracking the participation of an abusive visitor but removing what they are saying, you might delete their comment text but retain the record of their participation.

An admin could also replace the text with boilerplate (e.g., this comment has been removed), but if I was doing that, I'd actually set it up using a filter that showed the boilerplate for any empty comment text field, as that would be more consistent and less work.

This should be taken into consideration for whether this is actually something we should do. Recommending for closure.

This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.


9 months ago

#6 @joedolson
9 months ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.