Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-9571

[DEPRECATION] Replace BigDecimal.ROUND_* by RoundingMode.*

    Details

    • Type: Improvement
    • Status: In Progress
    • Priority: Trivial
    • Resolution: Unresolved
    • Affects Version/s: Trunk
    • Fix Version/s: None
    • Component/s: ALL COMPONENTS
    • Labels:
      None

      Description

      3 years ago I wrote about that in dev ML: https://s.apache.org/EVxA

      I just read at http://docs.oracle.com/javase/7/docs/api/java/math/BigDecimal.html : <<Using the integer fields in this class (such as ROUND_HALF_UP) to represent rounding mode is largely obsolete; the enumeration values of the RoundingMode enum, (such as RoundingMode.HALF_UP) should be used instead.

      It's time for action...

        Issue Links

          Activity

          Hide
          jacques.le.roux Jacques Le Roux added a comment - - edited

          In a 1st step at revision: 1804953, I did an easy global replacement. But it also needs to import the RoundingMode class. It was 26 files so I decided to rather clean all the imports in all files with the help of Eclipse.

          In a 2nd step, I will need to closely look at 2 special classes: UtilNumber and Calculate.

          Show
          jacques.le.roux Jacques Le Roux added a comment - - edited In a 1st step at revision: 1804953, I did an easy global replacement. But it also needs to import the RoundingMode class. It was 26 files so I decided to rather clean all the imports in all files with the help of Eclipse. In a 2nd step, I will need to closely look at 2 special classes: UtilNumber and Calculate.

            People

            • Assignee:
              jacques.le.roux Jacques Le Roux
              Reporter:
              jacques.le.roux Jacques Le Roux
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:

                Development