ICEfaces
  1. ICEfaces
  2. ICE-2095

selectInputDate: popupDateFormat not working as expected

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6.1
    • Fix Version/s: 1.6.2, 1.7DR#1, 1.7
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      any

      Description

      guess there's a page containing a simple selectInputDate component like:

      <ice:selectInputDate renderAsPopup="true"
      value="#{TestBean.date}"
      id="dateField"
      popupDateFormat="EEE, d MMM yyyy">
      </ice:selectInputDate>

      when the page is entered initially and there is a pre-existing value for the date then
      the format is not applied to this component. Instead the date will be shown with the
      component's default format settings. The correct format will be applied when
      the JSF lifecycle is initiated i.e. by a partialSubmit of another component. the problem
      then is that a conversion error occurs, because the default format doesn't match the
      value of the popupDateFormat attribute.

        Activity

        Hide
        Mark Collette added a comment -

        The actual DateFormat object is now properly derived from the String popupDateFormat property which is its pattern.

        Subversion 14821
        icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDate.java
        icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDateRenderer.java

        Show
        Mark Collette added a comment - The actual DateFormat object is now properly derived from the String popupDateFormat property which is its pattern. Subversion 14821 icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDate.java icefaces\component\src\com\icesoft\faces\component\selectinputdate\SelectInputDateRenderer.java
        Hide
        Ken Fyten added a comment -

        Needs to be backported to the 1.6 branch.

        Show
        Ken Fyten added a comment - Needs to be backported to the 1.6 branch.
        Hide
        Mark Collette added a comment -

        Backported to ICEfaces 1.6.2, along with ICE-2150.

        Show
        Mark Collette added a comment - Backported to ICEfaces 1.6.2, along with ICE-2150 .
        Hide
        Marcin Dabrowski added a comment -

        When I include backport sources from ICE-2150 to my 1.6.1 version, it seems that during validation for selectInputDate it not check is it required.
        Can you test it with yours 1.6.2 development branch?

        Show
        Marcin Dabrowski added a comment - When I include backport sources from ICE-2150 to my 1.6.1 version, it seems that during validation for selectInputDate it not check is it required. Can you test it with yours 1.6.2 development branch?
        Hide
        Sebastien Pirlot added a comment -

        I'm having the exact opposing issue using 1.7DR1!

        when the page is entered initially and there is a pre-existing value for the date then the format is applied, but as soon as a partialsubmit occurs, the format will revert to the component's default format settings (DD/mm/yyyy).

        Show
        Sebastien Pirlot added a comment - I'm having the exact opposing issue using 1.7DR1! when the page is entered initially and there is a pre-existing value for the date then the format is applied, but as soon as a partialsubmit occurs, the format will revert to the component's default format settings (DD/mm/yyyy).

          People

          • Assignee:
            Unassigned
            Reporter:
            Michael Thiem
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: