Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-12301

View flag for disabling growls should disable client side growls as well

    Details

    • Type: Task
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 2.6
    • Security Level: Public (Public: Anyone can view)
    • Labels:
    • Rice Team:
      Framework
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Include in Release Notes?:
      Yes

      Description

      The view component has property growlMessagingEnabled. This was originally added when adding the Growl type to MessageMap. The intention was if growls were added to the mesage map, to allow them to be displayed as info messages instead.

      However, growls are also shown client side in places (like the validation framework), and there is no way to prevent them without overriding code. It makes sense for this view property to disable all growls for a view regardless of where they come from. Therefore we need to modify the client side code to check this view setting.

      Note the related Jira on validation handling. It might be decided that growls should not be used at all for validations, therefore this would become irrelevant.

      Finally, the javadocs isGrowlMessagingEnabled and setGrowlMessagingEnabled have some content and formatting issues that need cleaned up.

        Attachments

          Issue Links

            Activity

            Hide
            jkneal Jerry Neal (Inactive) added a comment -

            Another possibility. We now only use growls to display JS error notifications. However, we are considering how those notifications are given.

            Show
            jkneal Jerry Neal (Inactive) added a comment - Another possibility. We now only use growls to display JS error notifications. However, we are considering how those notifications are given.

              People

              • Assignee:
                Unassigned
                Reporter:
                jkneal Jerry Neal (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:

                  Time Tracking

                  Estimated:
                  Original Estimate - 1 day
                  1d
                  Remaining:
                  Remaining Estimate - 1 day
                  1d
                  Logged:
                  Time Spent - Not Specified
                  Not Specified