Merge pull request #203 from Netflix/feature/OPLAliasInputForMacro

Fix issue with using cpl virtual track alias
Netflix · Aug 21, 2017 · fab4bbb · fab4bbb
2 parents a0fdd39 + 7e7b4d6
commit fab4bbb
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/main/java/com/netflix/imflibrary/st2067_100/OutputProfileList.java b/src/main/java/com/netflix/imflibrary/st2067_100/OutputProfileList.java
@@ -316,8 +316,7 @@ private static Map<String, Handle> populateCPLVirtualTrackHandles(ApplicationCom
                     StringBuilder handleBuilder = new StringBuilder();
                     handleBuilder.append("cpl/virtual-tracks/" + virtualTrack.getTrackID());
                     Handle handleType = new VirtualTrackHandle(handleBuilder.toString(), virtualTrack);
-                    handleMap.put(handleBuilder.toString(), handleType);
-                }
+                    handleMap.put(handleBuilder.toString(), handleType);                }
                 break;
 
                 case MainAudioSequence: {
@@ -410,10 +409,13 @@ private void populateMacroHandles(Map<String, Handle> handleMap) {
          */
         for(String handle: this.aliasMap.values()) {
             Handle handleType = handleMap.get(handle);
-            if (handleType == null && !handle.contains("/inputs/")) {
+            // Ignore input aliases as they are not needed for dependency resolution
+            // Ignore cpl/virtual track aliases too. All track IDs are not available for OPL and hence cannot validate.
+            if (handleType == null && !handle.contains("/inputs/") && !handle.startsWith("cpl/virtual-tracks/")) {
                 imfErrorLogger.addError(IMFErrorLogger.IMFErrors.ErrorCodes.IMF_OPL_ERROR, IMFErrorLogger.IMFErrors.ErrorLevels.NON_FATAL,
                         String.format("Invalid handle %s in alias", handle));
             }
+
         }
     }