Sling
  1. Sling
  2. SLING-2446

Add support for BigDecimal JCR values

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: JCR Resource 2.1.0
    • Component/s: JCR
    • Labels:

      Description

      JCR supports the typehint

      {Decimal}

      for java.math.BigDecimal, but JcrPropertyMap doesn't know about it so doing a map.get(propName, BigDecimal.class) returns null.

      Likewise, you should be able to adapt a Property Resource into a BigDecimal.

        Activity

        Jeff Young created issue -
        Justin Edelson made changes -
        Field Original Value New Value
        Assignee Justin Edelson [ justinedelson ]
        Justin Edelson made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s JCR Resource 2.1.0 [ 12316202 ]
        Resolution Fixed [ 1 ]
        Justin Edelson made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Justin Edelson made changes -
        Summary JCR-1609 added DecimalType, but it didn't get in to Sling's JcrPropertyMap Add support for BigDecimal JCR values
        Description JCR supports the typehint {Decimal} for java.math.BigDecimal, but JcrPropertyMap doesn't know about it so doing a map.get(propName, BigDecimal.class) returns null. JCR supports the typehint {Decimal} for java.math.BigDecimal, but JcrPropertyMap doesn't know about it so doing a map.get(propName, BigDecimal.class) returns null.

        Likewise, you should be able to adapt a Property Resource into a BigDecimal.
        Justin Edelson made changes -
        Issue Type Bug [ 1 ] New Feature [ 2 ]
        Justin Edelson made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Jeff Young made changes -
        Attachment BigDecimal_tests_.patch [ 12519637 ]
        Carsten Ziegeler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Gavin made changes -
        Workflow no-reopen-closed,doc-test-required [ 12659304 ] Copy of no-reopen-closed,doc-test-required [ 12762552 ]
        Gavin made changes -
        Workflow Copy of no-reopen-closed,doc-test-required [ 12762552 ] no-reopen-closed,doc-test-required [ 12765564 ]
        Gavin made changes -
        Workflow no-reopen-closed,doc-test-required [ 12765564 ] re-open possible,doc-test-required [ 12787462 ]
        Gavin made changes -
        Workflow re-open possible,doc-test-required [ 12787462 ] no-reopen-closed,doc-test-required [ 12790361 ]

          People

          • Assignee:
            Justin Edelson
            Reporter:
            Jeff Young
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development