480473 - Deny editing page and include notification message on empty string

Status: VERIFIED FIXED

(support.mozilla.org :: General, defect)

Description

[krupa raj[:krupa]]

Attachment: inaccurate_user_messg

Steps to reproduce:

1.Login to support.mozilla.com
2.Click on "How to Contribute"
3.Click on "Writing new Knowledge Base articles" link under "Improve the
Knowledge Base"
4.Click "edit" without entering the article name
5.Notice the user message

expected result
User message requests the user to specify the name of the article which they need to edit.

actual result:
User message- "Error.Permission denied you cannot edit this page" is displayed.

Also,If this user message is retained-its missing a semi-colon/full stop.It should be "Permission denied.You cannot edit this page."

Comment 1

[Paul Craciunoiu [:paulc]]

As a contributor, I can edit the page when the title of the page is blank, which is bad. As a user, I can reproduce the reported error.
In any case, for both, there should be a message like:
"You cannot create a page with an empty title."

Suggestions on the user message?

Comment 2

[krupa raj[:krupa]]

How about "Empty article names are not allowed. Please specify a valid article name" as the user message?

Comment 3

[Chris Ilias [:cilias]]

I think it emphasize what to do about the problem.
"Please us the Back button in your browser and make sure you have entered an article title."

Comment 4

[Paul Craciunoiu [:paulc]]

r22958 / r22959
Done, added Chris' message. Let me know if that works.

Comment 5

[krupa raj[:krupa]]

Well,the behavior seems to be fixed but why should the user message say "Please use the Back button in your browser..." when there is "Go back" link right below the user message?

So keeping the bug open.

Comment 6

[Chris Ilias [:cilias]]

"Empty article names are not allowed. Please click on "Go back" and make sure you have entered an article title."

You like?

Comment 7

[krupa raj[:krupa]]

much better.me likey :)

Comment 8

[Paul Craciunoiu [:paulc]]

r22979 / r22980

Comment 9

[krupa raj[:krupa]]

Verified fixed.