Improved reactive fassade

frontend/app/components/work-packages/work-package-cache.service.ts

@@ -58,17 +58,20 @@ export class WorkPackageCacheService {
 
   updateWorkPackageList(list: WorkPackageResource[]) {
     for (const wp of list) {
-      // var cached = this.workPackageCache[wp.id];
 
-      // TODO RR Question: This check doesn't seem to be right!?
+      // var cached = this.workPackageCache[wp.id];
       // if (cached && cached.dirty) {
       //   this.workPackageCache[wp.id] = cached;
       // } else {
       //   this.workPackageCache[wp.id] = wp;
       // }
 
+      // TODO Roman: clarify with Jens/Oliver. Is this check still ok?
+      if (!wp.dirty) {
         states.workPackages.put(wp.id.toString(), wp);
       }
+
+    }
     // this.workPackagesSubject.onNext(this.workPackageCache);
   }