Uploaded image for project: 'OpenMRS Core'
  1. OpenMRS Core
  2. TRUNK-5134

Replace path constructions using separator char

    XMLWordPrintable

Details

    • Task
    • Status: Code Review (Initial)
    • Should
    • Resolution: Unresolved
    • None
    • None
    • None

    Description

      Find ALL occurrences where we construct a file path using

      • File.separatorChar
      • System.getProperty("file.separator")

      and string concatenation like for example when creating a FileInputStream

      FileInputStream fis = new FileInputStream(appDataDir + System.getProperty("file.separator")
      + DatabaseUtil.ORDER_ENTRY_UPGRADE_SETTINGS_FILENAME);
      

      and replace the construction of the file path with proper use of Java's Path constructor which can easily do this for us, like so

      FileInputStream fis = new FileInputStream(Paths.get("a", "b", "c").toFile());
      

      To clarify:
      The task is to replace the string concatenation with proper use of Java's Paths class which can construct paths for us. The above code sample is just an example from the code base, this task does not mean you need to replace anything with FileInputStream.

      Gliffy Diagrams

        Attachments

          Activity

            People

              wikumc Wikum Weerakutti
              teleivo Ivo Ulrich
              Daniel Kayiwa Daniel Kayiwa
              Votes:
              2 Vote for this issue
              Watchers:
              18 Start watching this issue

              Dates

                Created:
                Updated:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Time Spent - 1 day, 2 hours, 30 minutes Remaining Estimate - 30 minutes
                  30m
                  Logged:
                  Time Spent - 1 day, 2 hours, 30 minutes Remaining Estimate - 30 minutes
                  1d 2h 30m