[4204] Prevent the changeDescriptionSink from crashing because of the…

… related representation refresh Bug: #4204 Signed-off-by: Denis Nikiforov <[email protected]>
eclipse-sirius · Jan 7, 2025 · 9aaf607 · 9aaf607
1 parent 3f2ad65
commit 9aaf607
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/CHANGELOG.adoc b/CHANGELOG.adoc
@@ -30,6 +30,7 @@ This may have some consequences for downstream applications which are embedding
 === Bug fixes
 
 - https://github.com/eclipse-sirius/sirius-web/issues/4282[#4282] [diagram] Fixed a warning on the frontend about non-unique GraphQL fragment name
+- https://github.com/eclipse-sirius/sirius-web/issues/4204[#4204] [core] Prevent the changeDescriptionSink from crashing because of the related representation refresh
 
 === New Features
 

diff --git a/.../eclipse/sirius/components/collaborative/editingcontext/EditingContextEventProcessor.java b/.../eclipse/sirius/components/collaborative/editingcontext/EditingContextEventProcessor.java
@@ -157,18 +157,18 @@ private Disposable setupChangeDescriptionSinkConsumer() {
 
             var refreshRepresentationSample = Timer.start(this.meterRegistry);
 
-            RepresentationEventProcessorEntry representationEventProcessorEntry = this.representationEventProcessors.get(changeDescription.getSourceId());
-            if (representationEventProcessorEntry != null) {
-                try {
+            try {
+                RepresentationEventProcessorEntry representationEventProcessorEntry = this.representationEventProcessors.get(changeDescription.getSourceId());
+                if (representationEventProcessorEntry != null) {
                     IRepresentationEventProcessor representationEventProcessor = representationEventProcessorEntry.getRepresentationEventProcessor();
                     representationEventProcessor.refresh(changeDescription);
                     IRepresentation representation = representationEventProcessor.getRepresentation();
                     this.applicationEventPublisher.publishEvent(new RepresentationRefreshedEvent(this.editingContext.getId(), representation));
-                } catch (Exception exception) {
-                    this.logger.warn(exception.getMessage(), exception);
                 }
+                this.refreshOtherRepresentations(changeDescription);
+            } catch (Exception exception) {
+                this.logger.warn(exception.getMessage(), exception);
             }
-            this.refreshOtherRepresentations(changeDescription);
 
             var timer = this.meterRegistry.timer(Monitoring.TIMER_REFRESH_REPRESENTATION, "changeDescription", changeDescription.getSourceId());
             refreshRepresentationSample.stop(timer);

diff --git a/...omponents-interpreter/src/main/java/org/eclipse/sirius/components/interpreter/Result.java b/...omponents-interpreter/src/main/java/org/eclipse/sirius/components/interpreter/Result.java
@@ -147,6 +147,6 @@ public Optional<Boolean> asBoolean() {
 
     @Override
     public String toString() {
-        return MessageFormat.format("Result { status: {0}, rawValue: {1}}", this.status, this.rawValue);
+        return MessageFormat.format("Result '{' status: {0}, rawValue: {1} '}'", this.status, this.rawValue);
     }
 }