![]() |
|
|
|
|
Change By:
|
Christopher Zimmermann
(18/Dec/13 11:36 AM)
|
|
Description:
|
Many error messages are sent to all users via a call to AppInstanceController#broadcastMessage
This is not good because there is no reason for most users to see these messages and will be alarming and confusing for them, as well as cluttering their message inbox.
A short search reveals calls to broadcastMessage when actions fail in BaseDialogPresenter, DetailEditorPresenter, DetailPresenter. A complete search on entire codebase must still be performed.
A conservative solution which can be improved later is to implement a simple new message method on the AppInstanceController,
sendErrorMessage(), which will send the message to the user who triggered the error and the superuser user (not the group).
Eventually logic could be added to send the message appropriately in a MultiTenancy setup to the correct administrator of the user who triggered the error.
*
So
The current required
fix for this ticket is to
implement AppInstanceController#sendErrorMessage and to
find and replace usages of broadcastMessage with
sendErrorMessage
sendLocalMessage (which sends a message to the user who triggered the error)
.
|
|
|
|
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
|
----------------------------------------------------------------
For list details, see: http://www.magnolia-cms.com/community/mailing-lists.html
Alternatively, use our forums: http://forum.magnolia-cms.com/
To unsubscribe, E-mail to: <
[email protected]>
----------------------------------------------------------------
