ICEpdf
  1. ICEpdf
  2. PDF-1233

Incorrect usage of new preferences api in example code.

          Details

          • Type: Bug Bug
          • Status: Resolved
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 6.3
          • Fix Version/s: 6.3.1
          • Component/s: Examples
          • Labels:
            None
          • Environment:
            any

            Description

            While working the forums with a community member an issue was raised with usability of the new Preferences api that is now part of the PropertiesManager.
            The examples are using a call to:

                            properties.checkAndStoreFloatProperty(PropertiesManager.PROPERTY_DEFAULT_ZOOM_LEVEL, 1.25f);

            instead of the correct call of:

            properties.getPreferences().putFloat(PropertiesManager.PROPERTY_DEFAULT_ZOOM_LEVEL, 1.25f);

            Under the new api its important to call PropertiesManager.getInstance().getPreference() to directly access the preferences api.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Fix Version/s 6.3.1 [ 13113 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #52310 Thu Feb 15 07:30:13 MST 2018 patrick.corless PDF-1233 correct usage of PropertiesManager.getInstance().getPreferences()
              for direct manipulation of properties.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/examples/component/src/main/java/org/icepdf/os/examples/component/ViewerComponentExample.java
              Commit graph MODIFY /icepdf/trunk/icepdf/examples/javafx/src/main/java/org/icepdf/os/examples/javafx/PdfFXViewer.java
              Hide
              Permalink
              Patrick Corless added a comment -

              Marking as fixed.

              Show
              Patrick Corless added a comment - Marking as fixed.
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: