✨ Add better file feature handling on file retrieval

backend/src/app/rpc/commands/files.clj

@@ -189,6 +189,8 @@
 ;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 
 (defn check-features-compatibility!
+  "Function responsible to check if provided features are supported by
+  the current backend"
   [features]
   (let [not-supported (set/difference features supported-features)]
     (when (seq not-supported)
@@ -248,28 +250,10 @@
        (into #{} (comp (filter pmap/pointer-map?)
                        (map pmap/get-id)))))
 
-;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
-;; QUERY COMMANDS
-;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
-
-(defn handle-file-features!
-  [conn {:keys [id features data] :as file} client-features]
-
-  (when (and (contains? features "components/v2")
-             (not (contains? client-features "components/v2")))
-    (ex/raise :type :restriction
-              :code :feature-mismatch
-              :feature "components/v2"
-              :hint "file has 'components/v2' feature enabled but frontend didn't specifies it"))
-
-  ;; NOTE: this operation is needed because the components migration
-  ;; generates a new page with random id which is returned to the
-  ;; client; without persisting the migration this can cause that two
-  ;; simultaneous clients can have a different view of the file data
-  ;; and end persisting two pages with main components and breaking
-  ;; the whole file
-  (let [file (if (and (contains? client-features "components/v2")
-                      (not (contains? features "components/v2")))
+;; FIXME: file locking
+(defn- process-components-v2-feature
+  "A special case handling of the components/v2 feature."
+  [conn {:keys [id features data] :as file}]
   (binding [pmap/*tracked* (atom {})]
     (let [data        (ctf/migrate-to-components-v2 data)
           features    (conj features "components/v2")
@@ -281,13 +265,43 @@
       (persist-pointers! conn id)
       (-> file
           (assoc :features features)
-                       (assoc :data data))))
-               file)]
+          (assoc :data data)))))
+
+(defn handle-file-features!
+  [conn {:keys [features] :as file} client-features]
+
+  ;; Check features compatibility between the currently supported features on
+  ;; the current backend instance and the file retrieved from the database
+  (check-features-compatibility! features)
 
   (cond-> file
+    (and (contains? features "components/v2")
+         (not (contains? client-features "components/v2")))
+    (as-> file (ex/raise :type :restriction
+                         :code :feature-mismatch
+                         :feature "components/v2"
+                         :hint "file has 'components/v2' feature enabled but frontend didn't specifies it"
+                         :file-id (:id file)))
+
+    ;; This operation is needed because the components migration generates a new
+    ;; page with random id which is returned to the client; without persisting
+    ;; the migration this can cause that two simultaneous clients can have a
+    ;; different view of the file data and end persisting two pages with main
+    ;; components and breaking the whole file."
+    (and (contains? client-features "components/v2")
+         (not (contains? features "components/v2")))
+    (as-> file (process-components-v2-feature conn file))
+
+    ;; This operation is needed for backward comapatibility with frontends that
+    ;; does not support pointer-map resolution mechanism; this just resolves the
+    ;; pointers on backend and return a complete file.
     (and (contains? features "storage/pointer-map")
          (not (contains? client-features "storage/pointer-map")))
-      (process-pointers deref))))
+    (process-pointers deref)))
+
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
+;; QUERY COMMANDS
+;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;
 
 ;; --- COMMAND QUERY: get-file (by id)
 

common/src/app/common/fressian.clj

@@ -323,3 +323,13 @@
   :rfn (fn [^Reader rdr]
          (let [^List x (read-object! rdr)]
            (Matrix. (.get x 0) (.get x 1) (.get x 2) (.get x 3) (.get x 4) (.get x 5))))})
+
+
+;; Backward compatibility for 1.19 with v1.20;
+
+(add-handlers!
+ {:name "penpot/geom/rect"
+  :rfn read-map-like}
+ {:name "penpot/shape"
+  :rfn read-map-like})
+

common/src/app/common/types/container.cljc

@@ -100,6 +100,9 @@
      (nil? shape)
      nil
 
+     (= uuid/zero (:id shape))
+     nil
+
      (and (not (ctk/in-component-copy? shape)) (not allow-main?))
      nil