No Code Attached Yet
avatar chmst
chmst
4 Apr 2020

Steps to reproduce the issue

Make a tree of categories. Move and copy these categories. It is possible to make wrong moves, then a warning appears like this one.

screen shot 2020-04-04 at 21 07 11

Expected result

The warning should be informative for the user.

Actual result

For non-developers, the warning looks harsh.

System information (as much as possible)

Additional comments

avatar chmst chmst - open - 4 Apr 2020
avatar joomla-cms-bot joomla-cms-bot - labeled - 4 Apr 2020
avatar Quy
Quy - comment - 4 Apr 2020

Please test PR #20943 for J3

avatar jwaisner jwaisner - change - 4 Apr 2020
Status New Confirmed
avatar richard67
richard67 - comment - 4 Apr 2020

@Quy Not related, or only indirectly. Here it is just the text of the warning which is an issue.

avatar richard67
richard67 - comment - 4 Apr 2020

Maybe I'm too much a developer, because to me the message is very clear.

avatar Quy
Quy - comment - 4 Apr 2020

It might be related in that it may fix the issue to not get the error message.

avatar richard67
richard67 - comment - 4 Apr 2020

Well, it might be that I got this issue wrong.

avatar chmst
chmst - comment - 5 Apr 2020

I think the error message itself is necessary, but the message could be less "development". Something like "Moving the category to this place is not possble".

avatar mbabker
mbabker - comment - 5 Apr 2020

This is why it's a bad idea for errors to go uncaught throughout the system and them eventually get shown to all end users. This is the type of error message that is appropriate for debugging at a development level, but this is also the same type of error that should be getting caught in the component MVC and being transformed into something more appropriate for end user display. The right fix is not to "dumb down" Exception messages because they might get shown to an end user who isn't the intended audience.

avatar richard67
richard67 - comment - 5 Apr 2020

Agree.

avatar chmst chmst - change - 2 Nov 2022
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2022-11-02 15:18:32
Closed_By chmst
Labels Added: No Code Attached Yet
Removed: ?
avatar chmst chmst - close - 2 Nov 2022
avatar chmst
chmst - comment - 2 Nov 2022

In J4 no longer valid.

Add a Comment

Login with GitHub to post a comment