Do not update content when stale.

5 years ago · cac9d9df26
2 changed files with 11 additions and 2 deletions
--- a/backend/src/Squidex.Infrastructure/Commands/DomainObject.cs
+++ b/backend/src/Squidex.Infrastructure/Commands/DomainObject.cs
@ -315,7 +315,16 @@ namespace Squidex.Infrastructure.Commands

                if (persistence.IsSnapshotStale)
                {
-                    await persistence.WriteSnapshotAsync(Snapshot);
+                    try
+                    {
+                        await persistence.WriteSnapshotAsync(Snapshot);
+                    }
+                    catch (Exception ex)
+                    {
+                        log.LogError(ex, w => w
+                            .WriteProperty("action", "RepairSnapshot")
+                            .WriteProperty("status", "Failed"));
+                    }
                }
            }
        }
--- a/backend/src/Squidex.Infrastructure/States/Persistence.cs
+++ b/backend/src/Squidex.Infrastructure/States/Persistence.cs
@ -46,7 +46,7 @@ namespace Squidex.Infrastructure.States

        public bool IsSnapshotStale
        {
-            get => persistenceMode == PersistenceMode.SnapshotsAndEventSourcing && versionSnapshot < versionEvents;
+            get => persistenceMode == PersistenceMode.SnapshotsAndEventSourcing && versionSnapshot < versionEvents && versionSnapshot > EtagVersion.Empty;
        }

        public Persistence(DomainId ownerKey, Type ownerType,