ICEmobile
  1. ICEmobile
  2. MOBI-176

Mobi dateSpinner can not handle mm/dd/yyyy pattern

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0 Williams
    • Fix Version/s: 1.0 Williams
    • Component/s: None
    • Labels:
      None
    • Environment:
      any

      Description

      The date spinner has an issue when given date patterns that contain slashes. The current JavaScript does not check for this type of pattern. The other issue with dateSpinner is that if users specifidies MM/dd/yyyy format the popup dialog will not changes its display, always showing yyyy MM dd as the default input order.

      One more issue that can occur is in how the pattern attribute is used on the <mobi:dateSpinner/> in combination with a dateTimeConverter. Currently the pattern attributes wins over the converter which will results in conversion error the end user can't fix unless they manually correct then date in the input fields. The pattern attribute if different from the converter should lose out of the converter.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #27961 Tue Feb 21 14:54:21 MST 2012 patrick.corless MOBI-176 updated the date spinner to accept patterns with / delimiters, input buttons will order to pattern ordering, updated pattern/converter logic to be consistent for encode and decode.
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/components/component/src/org/icefaces/mobi/component/datespinner/DateSpinnerRenderer.java
        Commit graph MODIFY /icemobile/trunk/icemobile/components/component/src/org/icefaces/mobi/component/datespinner/DateSpinnerMeta.java
        Commit graph MODIFY /icemobile/trunk/icemobile/components/component/src/org/icefaces/mobi/component/datespinner/DateSpinner.java
        Patrick Corless created issue -
        Hide
        Patrick Corless added a comment -

        I've update the dateSpinner on a few fronts. First I sorted out the pattern logic so that if for some reason the pattern or dateTimeConverter differ, then the converter will win. I also updated the component so that if only a dateTimeConverter is used it's pattern will take effect for display and conversion.

        The renderer has been updated so that the popup panel input buttons will align with the pattern value. So if MM/dd/yyy is specified as the pattern then the input will be in the order MM DD YYYY.

        And finally the datespinner.js has been updated to handle patterns with the '/' as well as the original '-' delimiter.

        Show
        Patrick Corless added a comment - I've update the dateSpinner on a few fronts. First I sorted out the pattern logic so that if for some reason the pattern or dateTimeConverter differ, then the converter will win. I also updated the component so that if only a dateTimeConverter is used it's pattern will take effect for display and conversion. The renderer has been updated so that the popup panel input buttons will align with the pattern value. So if MM/dd/yyy is specified as the pattern then the input will be in the order MM DD YYYY. And finally the datespinner.js has been updated to handle patterns with the '/' as well as the original '-' delimiter.
        Patrick Corless made changes -
        Field Original Value New Value
        Assignee Steve Maryka [ steve.maryka ] Patrick Corless [ patrick.corless ]
        Hide
        Patrick Corless added a comment -

        Marking issue as resolved.

        Show
        Patrick Corless added a comment - Marking issue as resolved.
        Patrick Corless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #27969 Wed Feb 22 07:40:59 MST 2012 patrick.corless MOBI-176 updated the date spinner to accept patterns with / delimiters, input buttons will order to pattern ordering, updated pattern/converter logic to be consistent for encode and decode.
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/components/component/resources/org.icefaces.component.datespinner/datespinner.js
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #28060 Tue Feb 28 09:12:16 MST 2012 patrick.corless MOBI-176 updated the date spinner to accept patterns with / delimiters, input buttons will order to pattern ordering, updated pattern/converter logic to be consistent for encode and decode.
        Files Changed
        Commit graph MODIFY /icemobile/branches/icemobile-1.0.0.A/icemobile/components/component/src/org/icefaces/mobi/component/datespinner/DateSpinner.java
        Commit graph MODIFY /icemobile/branches/icemobile-1.0.0.A/icemobile/components/component/src/org/icefaces/mobi/component/datespinner/DateSpinnerRenderer.java
        Commit graph MODIFY /icemobile/branches/icemobile-1.0.0.A
        Commit graph MODIFY /icemobile/branches/icemobile-1.0.0.A/icemobile/components/component/src/org/icefaces/mobi/component/datespinner/DateSpinnerMeta.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #28062 Tue Feb 28 09:13:48 MST 2012 patrick.corless MOBI-176 updated the date spinner to accept patterns with / delimiters, input buttons will order to pattern ordering, updated pattern/converter logic to be consistent for encode and decode.
        Files Changed
        Commit graph MODIFY /icemobile/branches/icemobile-1.0.0.A
        Commit graph MODIFY /icemobile/branches/icemobile-1.0.0.A/icemobile/components/component/resources/org.icefaces.component.datespinner/datespinner.js
        Migration made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Patrick Corless
            Reporter:
            Patrick Corless
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: