ICEfaces
  1. ICEfaces
  2. ICE-10744

showcase - demos that use ace:messages not clearing errors

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 4.0
          • Fix Version/s: 4.1
          • Component/s: Sample Apps
          • Labels:
            None
          • Environment:
            ICEfaces4 trunk revision# 45077
            Server: tomcat7
            Browser: IE7, FF3.6/34 & Chrome43
            JSF: Mojarra 2.2.10
          • Assignee Priority:
            P2
          • Affects:
            Sample App./Tutorial

            Description

            The showcase demos that use a messages component to show errors (ace:messages) are not clearing these errors from the messages when the user navigates to another demo in the showcase. This is confusing for the user as the new demo renders with the old error messages from the previous demo.

            For example:

            showcase > FileEntry > Callback:
            When navigating to the Callback demo all messages generated on previous fileEntry demos are rendered inside a single message panel, like in attached screen shot.
            1. callback.PNG
              32 kB
            2. callback1.PNG
              19 kB

              Issue Links

              depends on

              Improvement - An improvement or enhancement to an existing feature or task. ICE-10877 Add ability for ace:messages & ace:growlMessages components to only render messages for components in the current view

              • Major - Major loss of function.
              • Closed
              is duplicated by

              Bug - A problem which impairs or prevents the functions of the product. ICE-10534 showcase - message type ACE components - messages are not cleared from messages queue

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Judy Guglielmin added a comment -

                rev 46259
                note that the fileEntry examples all had the same id and so used <ace:message for="...." with a unique id for each example to ensure that the actual message (since the fileEntry is the only component on the form don't need ace:messages), is tied to a unique component id.
                For the growlMessages and ace:messages example use the following
                <icecore:config messagePersistence="false"/>
                to clear the messages between requests.

                Show
                Judy Guglielmin added a comment - rev 46259 note that the fileEntry examples all had the same id and so used <ace:message for="...." with a unique id for each example to ensure that the actual message (since the fileEntry is the only component on the form don't need ace:messages), is tied to a unique component id. For the growlMessages and ace:messages example use the following <icecore:config messagePersistence="false"/> to clear the messages between requests.
                Hide
                Permalink
                Carmen Cristurean added a comment - - edited

                Re-opening because of regressions found on ICEfaces4 trunk r46259:
                1) showcase > fileEntry - Callback:
                The error message when trying to upload a .pdf file fails to explain what is wrong with the upload; it only prints the name of the file (see callback1.PNG attached).

                2) messages > Overview:
                When tabbing through the entries, only the last message is rendered (all browsers).

                Show
                Carmen Cristurean added a comment - - edited Re-opening because of regressions found on ICEfaces4 trunk r46259: 1) showcase > fileEntry - Callback: The error message when trying to upload a .pdf file fails to explain what is wrong with the upload; it only prints the name of the file (see callback1.PNG attached). 2) messages > Overview: When tabbing through the entries, only the last message is rendered (all browsers).
                Hide
                Permalink
                Judy Guglielmin added a comment -

                rev 46306

                Show
                Judy Guglielmin added a comment - rev 46306
                Hide
                Permalink
                Carmen Cristurean added a comment -

                ICEfaces4 trunk r46307: issue 2) above can still be reproduced.

                Show
                Carmen Cristurean added a comment - ICEfaces4 trunk r46307: issue 2) above can still be reproduced.
                Hide
                Permalink
                Judy Guglielmin added a comment - - edited

                sorry. I had forgot to modify the showcase examples, so checked it in under ICE-10877 rev 46308, please update your showcase to that revision. (you may have tested the showcase without using the new value on the "for" attribute).

                Show
                Judy Guglielmin added a comment - - edited sorry. I had forgot to modify the showcase examples, so checked it in under ICE-10877 rev 46308, please update your showcase to that revision. (you may have tested the showcase without using the new value on the "for" attribute).
                Hide
                Permalink
                Carmen Cristurean added a comment -

                Verified on r46309 showcase > growlMessages, messages, fileEntry, no issues found (FF41, IE11, Chrome46).

                Show
                Carmen Cristurean added a comment - Verified on r46309 showcase > growlMessages, messages, fileEntry, no issues found (FF41, IE11, Chrome46).

                  People

                  • Assignee:
                    Judy Guglielmin
                    Reporter:
                    Carmen Cristurean
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: