🎨 make legacy property name consistent with the new style

Author: pmonks

README.md

@@ -40,7 +40,7 @@ There are a couple of static classes that help common usage scenarios:
 `Spdx-Java-Library` can be configured using either Java system properties or a Java properties file located in the runtime CLASSPATH at `/resources/spdx-java-library.properties`.
 
 The library has these configuration options:
-1. `SPDXParser.OnlyUseLocalLicenses` - a boolean that controls whether the (potentially out of date) listed license information bundled inside the JAR is used (true), vs the library downloading the latest files from the SPDX website (false). Default is false (always download the latest files from the SPDX website).
+1. `org.spdx.useJARLicenseInfoOnly` - a boolean that controls whether the (potentially out of date) listed license information bundled inside the JAR is used (true), vs the library downloading the latest files from the SPDX website (false). Default is false (always download the latest files from the SPDX website).
 2. `org.spdx.downloadCacheEnabled` - a boolean that enables or disables the download cache. Defaults to `false` (the cache is disabled). The cache location is determined as per the [XDG Base Directory Specification](https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html) (i.e. `${XDG_CACHE_HOME}/Spdx-Java-Library` or `${HOME}/.cache/Spdx-Java-Library`).
 3. `org.spdx.downloadCacheCheckIntervalSecs` - a long that controls how often each cache entry is rechecked for staleness, in units of seconds. Defaults to 86,400 seconds (24 hours). Set to 0 (zero) to have each cache entry checked every time (note: this will result in a lot more network I/O and negatively impact performance, albeit there is still a substantial performance saving vs not using the cache at all).
 

resources/spdx-java-library.properties.sample

@@ -1,6 +1,6 @@
-# if true, only use license information from the set of licenses stored within the JAR file at release time, rather than
-# downloading the latest versions from the SPDX website
-SPDXParser.OnlyUseLocalLicenses=false
+# if true, only use license information stored within the JAR file at release time, rather than downloading the latest
+# versions from the SPDX website
+org.spdx.useJARLicenseInfoOnly=false
 
 # if true, enable the download cache
 org.spdx.downloadCacheEnabled=false

src/main/java/org/spdx/library/model/license/ListedLicenses.java

@@ -59,11 +59,11 @@ public class ListedLicenses {
 	 * This constructor should only be called by the getListedLicenses method
 	 */
 	private ListedLicenses() {
-		// Note: this code confusingly uses different property values depending on the source (Java system property vs properties file),
-		// so we have to check both names in order to not break downstream consumers' legacy configurations.  This is _NOT_ recommended for
-		// any new code that leverages the Configuration class.
-		onlyUseLocalLicenses = Boolean.parseBoolean(Configuration.getInstance().getProperty("SPDXParser.OnlyUseLocalLicenses",
-				                                    Configuration.getInstance().getProperty("OnlyUseLocalLicenses", "false")));
+		// Note: this code is confusing as this property changed names several times over time, and we want to preserve
+		// backwards compatibility for those downstream library users who are using the old/deprecated names
+		onlyUseLocalLicenses = Boolean.parseBoolean(Configuration.getInstance().getProperty("org.spdx.useJARLicenseInfoOnly",
+				                                      Configuration.getInstance().getProperty("SPDXParser.OnlyUseLocalLicenses",
+				                                        Configuration.getInstance().getProperty("OnlyUseLocalLicenses", "false"))));
 		initializeLicenseModelStore();
 	}