Merge pull request #125 from spdx/issue124

Add warning if a LicenseRef is not present

Author: goneall

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

@@ -134,10 +134,13 @@ protected List<String> _verify(List<String> verifiedIds, String specVersion) {
 			}
 		}
 		try {
-		String licenseText = this.getExtractedText();
+			String licenseText = this.getExtractedText();
 			if (licenseText == null || licenseText.isEmpty()) {
 				retval.add("Missing required license text for " + id);
 			}
+			if (LicenseExpressionParser.UNINITIALIZED_LICENSE_TEXT.equals(licenseText)) {
+				retval.add("License not found for " + id);
+			}
 		} catch (InvalidSPDXAnalysisException ex) {
 			retval.add("Unable to fetch license text: "+ex.getMessage());
 		}

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

@@ -52,6 +52,7 @@ enum Operator {
 	static final String LEFT_PAREN = "(";
 	static final String RIGHT_PAREN = ")";
 	static final Map<String, Operator> OPERATOR_MAP = new HashMap<>();
+	public static final String UNINITIALIZED_LICENSE_TEXT = "[Initialized with license Parser.  The actual license text is not available]";
 
 	static {
 		OPERATOR_MAP.put("+", Operator.OR_LATER);
@@ -293,7 +294,7 @@ private static AnyLicenseInfo parseSimpleLicenseToken(String token, IModelStore
 			} else {
 				localLicense = (ExtractedLicenseInfo) SpdxModelFactory.createModelObject(
 						store, documentUri, token, SpdxConstants.CLASS_SPDX_EXTRACTED_LICENSING_INFO, copyManager);
-				localLicense.setExtractedText("[Initialized with license Parser.  The actual license text is not available]");
+				localLicense.setExtractedText(UNINITIALIZED_LICENSE_TEXT);
 			}
 			return localLicense;
 		}