rh-che #1488: Adding NPE check in the 'FactoryUrlSetterInterceptor' t…

…o avoid failures of workspaces started from devfile

Signed-off-by: Ilya Buziuk <ibuziuk@redhat.com>

plugins/che-plugin-analytics/che-plugin-analytics-wsmaster/src/main/java/com/redhat/che/plugin/analytics/wsmaster/FactoryUrlSetterInterceptor.java

@@ -16,6 +16,7 @@
 import org.aopalliance.intercept.MethodInterceptor;
 import org.aopalliance.intercept.MethodInvocation;
 import org.eclipse.che.api.factory.shared.dto.FactoryDto;
+import org.eclipse.che.api.workspace.shared.dto.WorkspaceConfigDto;
 
 /**
  * This interceptor must be bound for the method {@link
@@ -32,11 +33,14 @@ public Object invoke(MethodInvocation invocation) throws Throwable {
     @SuppressWarnings("unchecked")
     final Map<String, String> parameters = (Map<String, String>) args[0];
     FactoryDto factory = (FactoryDto) invocation.proceed();
-    Map<String, String> attributes = factory.getWorkspace().getAttributes();
-    parameters.forEach(
-        (k, v) -> {
-          attributes.put("factory.parameter." + k, v);
-        });
+    WorkspaceConfigDto workspace = factory.getWorkspace();
+    if (workspace != null) {
+      Map<String, String> attributes = workspace.getAttributes();
+      parameters.forEach(
+          (k, v) -> {
+            attributes.put("factory.parameter." + k, v);
+          });
+    }
     return factory;
   }
 }