Project

General

Profile

Actions
Copy link

Defect #32664

open

Adding an empty reply to a forum message does not show any error

Added by Go MAEDA almost 5 years ago. Updated almost 5 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Forums
Target version:
Candidate for next major release
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Affected version:

Description

Steps to reproduce:

1. Open any forum topic and click "Reply"
2. Attach any file and click "Submit"
3. You will see "Successful update" message but the reply is not added. The attachment is silently discarded

Expected behavior:

"Content cannot be blank" error should be shown.

Files

Download all files
show_any_error_when_reply.png (210 KB) show_any_error_when_reply.png Yuichi HARADA, 2019-12-25 02:30
32664_show_errormsg_submitting_reply.patch (3.92 KB) 32664_show_errormsg_submitting_reply.patch Yuichi HARADA, 2019-12-25 02:32
32664_show_errormsg_submitting_reply-v2.patch (5.89 KB) 32664_show_errormsg_submitting_reply-v2.patch Yuichi HARADA, 2020-01-16 06:04
Actions
Copy link
 #1

Updated by Yuichi HARADA almost 5 years ago

Show any error message when submitting a reply. I attached a patch.
However, I'm worried that if the number of replies increases, the error message, and the reply input fields will be separated.

Actions
Copy link
 #2

Updated by Go MAEDA almost 5 years ago

Yuichi HARADA wrote:

However, I'm worried that if the number of replies increases, the error message, and the reply input fields will be separated.

You are right. Thank you for pointing it out.

And I found that the behavior that showing an error is not consistent with issues. When updating issues, no error is shown even if note is empty. Its behavior is similar to forums but the only difference is the behavior when adding attachments with an empty note. In the case, attachments are added to the issue.

I think the following behavior is better for forums. It is consistent with the behavior of issues.

  • Empty message without attachments: Almost the same behavior with the current behavior but "Successful update" message should not be shown
  • empty message with attachments: Attach the files instead of discarding them
Actions
Copy link
 #3

Updated by Yuichi HARADA almost 5 years ago

Go MAEDA wrote:

I think the following behavior is better for forums. It is consistent with the behavior of issues.

  • Empty message without attachments: Almost the same behavior with the current behavior but "Successful update" message should not be shown
  • empty message with attachments: Attach the files instead of discarding them

Thank you for pointing it out. I attached a patch that incorporates the above behaviors.

Actions
Copy link
 #4

Updated by Go MAEDA almost 5 years ago

  • Target version set to Candidate for next major release
Actions
Copy link

Also available in: Atom PDF