foobara-local-files-crud-driver 0.0.6 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a99fcad243c2ddf376c26cc20ca3189edc8f52e07b8398f0b0f46d8abcc6c356
-  data.tar.gz: e77ed2685961edd375e6ebf19a84e76e648d8f3480d7d0b87b8376f994e79dc9
+  metadata.gz: ac3f878951193768f4270053f52ef083fdded58e48365dfe02884495a365c9e2
+  data.tar.gz: a0bcaa7c8d7565842d64c5d0efa2e7cea005c168b6e2d71bd192a5b09dc67b00
 SHA512:
-  metadata.gz: 1a910cafedacbdd319c1e44c2162cbca5f4e34dbd91a1119bbef4dffff14dfe97b63118f49158c10d64c126d581419ee304522e9329afdeac6aa1dd47a7a5747
-  data.tar.gz: 51ef2628e11aa128131a42e0fb906aa40ff92308b0bf23c16ce966bfdf5af5bcfce099336e61cdd64f2b7754ec12514411d0ea01fd7378dcb50859187c7346f9
+  metadata.gz: 1f69ebb37262f861fa0f8881dedc0f8d663ab2c8c74ba51ae3afe37c0e53fe684dfd44a74c0c7c49431ff5b0b44c244d304c91d5bf0da00828fcb9dd47eeaab9
+  data.tar.gz: 06da6c67ba0d4891841757ea60b8326fa344749a5a12b3a62250e28554f39a40b333cb9bb202bf70176df273b2aac8421e63d43634ec262055c44a11c25508cc

data/CHANGELOG.md

@@ -1,3 +1,11 @@
+## [0.0.8] - 2025-03-29
+
+- Fix bug where writing out one table erases another
+
+## [0.0.7] - 2025-03-15
+
+- Make sure instances of Time and Date are written out in YAML-loadable ways
+
 ## [0.0.6] - 2025-03-15
 
 - Safety improvement: do not truncate file if we fail to generate record yaml

data/src/foobara/local_files_crud_driver.rb

@@ -4,7 +4,7 @@ require "yaml"
 module Foobara
   # TODO: lots of duplication in this file. Need to DRY this up a bit.
   class LocalFilesCrudDriver < Persistence::EntityAttributesCrudDriver
-    attr_accessor :data_path, :format, :fsync, :multi_process
+    attr_accessor :data_path, :format, :fsync, :multi_process, :raw_data
 
     def initialize(data_path: "#{Dir.pwd}/local_data/records.yml", format: :yaml, fsync: false, multi_process: false)
       self.data_path = data_path
@@ -64,7 +64,8 @@ module Foobara
       end
 
       def insert(attributes)
-        attributes = Util.deep_dup(attributes)
+        attributes = prepare_attributes_for_write(attributes)
+
         record_id = record_id_for(attributes)
 
         unless record_id
@@ -91,6 +92,8 @@ module Foobara
       end
 
       def update(attributes)
+        attributes = prepare_attributes_for_write(attributes)
+
         with_writeable_raw_data do |raw_data|
           attributes = Util.deep_dup(attributes)
 
@@ -135,6 +138,27 @@ module Foobara
 
       private
 
+      def raw_data
+        crud_driver.raw_data
+      end
+
+      def prepare_attributes_for_write(value)
+        case value
+        when ::Time
+          value.to_f
+        when ::Date
+          value.to_s
+        when ::Array
+          value.map { |element| prepare_attributes_for_write(element) }
+        when ::Hash
+          value.to_h do |key, inner_value|
+            [prepare_attributes_for_write(key), prepare_attributes_for_write(inner_value)]
+          end
+        else
+          value
+        end
+      end
+
       def data_path
         crud_driver.data_path
       end
@@ -157,11 +181,11 @@ module Foobara
             f.flock(lock)
             yaml = f.read
             raw_data = yaml.empty? ? {} : YAML.load(yaml)
-            @raw_data = raw_data unless multi_process
+            crud_driver.raw_data = raw_data unless multi_process
             yield raw_data, f
           end
-        elsif defined?(@raw_data)
-          yield @raw_data
+        elsif self.raw_data
+          yield self.raw_data
         else
           with_readable_raw_data(load: true, lock:, mode:, &)
         end
@@ -180,19 +204,19 @@ module Foobara
           else
             FileUtils.mkdir_p File.dirname(data_path)
 
-            File.open(data_path, "a+") do |f|
-              f.flock(File::LOCK_EX)
-              f.rewind
-              retval = yield raw_data, f
+            File.open(data_path, "a+") do |file|
+              file.flock(File::LOCK_EX)
+              file.rewind
+              retval = yield raw_data, file
               file_contents = YAML.dump(raw_data)
-              f.truncate(0)
-              f.rewind
-              f.write(file_contents)
+              file.truncate(0)
+              file.rewind
+              file.write(file_contents)
             end
           end
 
           unless multi_process
-            @raw_data = raw_data
+            crud_driver.raw_data = raw_data
           end
         end
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: foobara-local-files-crud-driver
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.8
 platform: ruby
 authors:
 - Miles Georgi
 bindir: bin
 cert_chain: []
-date: 2025-03-16 00:00:00.000000000 Z
+date: 2025-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: foobara
@@ -57,7 +57,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.5
+rubygems_version: 3.6.6
 specification_version: 4
 summary: Stores all record data in a yaml file in a local directory
 test_files: []