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

          Activity

          Hide
          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
          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
          Judy Guglielmin added a comment -

          rev 46306

          Show
          Judy Guglielmin added a comment - rev 46306
          Hide
          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
          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
          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: