.

roktas · Sep 14, 2024 · c6f2633 · c6f2633
1 parent 4ef276f
commit c6f2633
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 9 deletions.
diff --git a/graphics/lib/sevgi/graphics/attribute.rb b/graphics/lib/sevgi/graphics/attribute.rb
@@ -4,21 +4,21 @@
 
 module Sevgi
   module Graphics
+    ATTRIBUTE_INTERNAL_PREFIX = "-"
+    ATTRIBUTE_UPDATE_SUFFIX   = "+"
+
     module Attribute
       module Ident
-        INTERNAL_PREFIX = "_"
-        UPDATE_SUFFIX   = "+"
-
         def internal?(given)
-          (@internal ||= {})[given] ||= given.start_with?(INTERNAL_PREFIX)
+          (@internal ||= {})[given] ||= given.start_with?(ATTRIBUTE_INTERNAL_PREFIX)
         end
 
         def id(given)
-          (@id ||= {})[given] ||= (updateable?(given) ? given.to_s.delete_suffix(UPDATE_SUFFIX) : given).to_sym
+          (@id ||= {})[given] ||= (updateable?(given) ? given.to_s.delete_suffix(ATTRIBUTE_UPDATE_SUFFIX) : given).to_sym
         end
 
         def updateable?(given)
-          (@updateable ||= {})[given] ||= given.end_with?(UPDATE_SUFFIX)
+          (@updateable ||= {})[given] ||= given.end_with?(ATTRIBUTE_UPDATE_SUFFIX)
         end
       end
 

diff --git a/graphics/lib/sevgi/graphics/mixtures/duplicate.rb b/graphics/lib/sevgi/graphics/mixtures/duplicate.rb
@@ -10,7 +10,7 @@ def Duplicate(dx: nil, dy: nil, parent: nil, &block)
           duplicated.Traverse do |element|
             element.children = element.children.map(&:dup)
             id = (element.attributes = element.attributes.dup).delete(:id)
-            element[:_id] = id if id
+            element[:"#{ATTRIBUTE_INTERNAL_PREFIX}id"] = id if id
             block&.call(element)
           end
 

diff --git a/graphics/lib/sevgi/graphics/mixtures/save.rb b/graphics/lib/sevgi/graphics/mixtures/save.rb
@@ -10,7 +10,7 @@ def Out(*, **, &filter)
           F.out(self.(**), *, &filter)
         end
 
-        def Save(path = nil, default: nil, &filter)
+        def Save(path = nil, default: nil, backup_suffix: nil, &filter)
           default ||= F.subext(EXT, caller_locations(1..1).first.path)
 
           if path
@@ -19,6 +19,8 @@ def Save(path = nil, default: nil, &filter)
             default
           end => path
 
+          ::FileUtils.cp(path, "#{path}#{backup_suffix}") if backup_suffix && !backup_suffix.empty? && ::File.exist?(path)
+
           Out(F.touch(path), &filter)
         end
       end

diff --git a/graphics/test/graphics/mixtures/duplicable_test.rb b/graphics/test/graphics/mixtures/duplicable_test.rb
@@ -30,7 +30,7 @@ def test_duplicate_deletes_id_attribute_by_default
         def test_duplicate_can_produce_new_id_attribute
           doc = SVG do
             line(id: "original", "data-var": "main var").Duplicate do |element|
-              element[:id] = "#{element[:_id]}-copy" if element[:_id]
+              element[:id] = "#{element[:"#{ATTRIBUTE_INTERNAL_PREFIX}id"]}-copy" if element[:"#{ATTRIBUTE_INTERNAL_PREFIX}id"]
             end
           end