patchwork_csv_utils 0.1.7-arm64-darwin → 0.1.8-arm64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f0a84b0c5c4d9eee5d49fdb63c5f1381967eda72a68a16b35c602fa0143cd0e2
-  data.tar.gz: d1e0f1077c2abbda112ddcee7dae55e198666566097e95bd96aa6665c363eec7
+  metadata.gz: cee57c41f3e282435e1d7de609e9674dfd8f12b3b58a800ec5d5208c4563956f
+  data.tar.gz: 1c14864e23f63a738cb88f077d272a15d6efdba60ee0ed086cd7f3e99cd6ccc1
 SHA512:
-  metadata.gz: 011f0adde455db75eeb27492bbbb8617f2a406ef56278c54024fb3c874a7038935e11cd70176e2706c1e4700f9a3d6dc1cfde1abc076bbe2561c3c886f1140ec
-  data.tar.gz: a5d81faa6efd2b56941003638bd2c5c0829670179ee8a76a475724b6db688c6430bea189397ebd7c53a7b790f8f8ae5a38b4049c2aef7bb7a4c62dcd152021c3
+  metadata.gz: d54494b5517a1624c2904ca9ff74a1d66ae64cefb3cd3eabd2452c0daa034e69af6292b307629ad7075974e5cb59aff44a2843d5d7f2c18e0c08e604481d1d32
+  data.tar.gz: 25837083730e79c5f161ef4a5cef157659662f639d5c3c6cd0601cbd53fdabf8a8c303458927071163a2ff7e8b15ecdf07d04d01aefebf797f83e96d43a0d2b3

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    patchwork_csv_utils (0.1.7)
+    patchwork_csv_utils (0.1.8)
 
 GEM
   remote: https://rubygems.org/
@@ -67,4 +67,4 @@ DEPENDENCIES
   rubocop (~> 1.21)
 
 BUNDLED WITH
-   2.4.4
+   2.4.10

data/ext/csv_utils/src/utils/csv.rs

@@ -19,6 +19,7 @@ pub fn transform_csv(ruby: &Ruby, csv_path: String, target_path: String, exclusi
     let mut wtr = Writer::from_path(target_path).map_err(|e| magnus_err(ruby, e, "target_path"))?;
     let headers = csv.headers().map_err(|e| magnus_err(ruby, e, "csv_path headers"))?;
     let header_map: HashMap<String, usize> = headers.iter().enumerate().map(|(i, h)| (h.to_string(), i)).collect();
+    let inverse_header_map: HashMap<usize, String> = headers.iter().enumerate().map(|(i, h)| (i, h.to_string())).collect();
 
     wtr.write_byte_record(headers.as_byte_record()).map_err(|e| magnus_err(ruby, e, "write_byte_record"))?;
 
@@ -41,11 +42,14 @@ pub fn transform_csv(ruby: &Ruby, csv_path: String, target_path: String, exclusi
         let record = record.iter().enumerate().map(|(i, c)| {
             let c = c.trim_end();
             if i == *date {
-                let current = string_to_datetime(c).ok_or(to_datetime_error(ruby, c, ri, i))?;
+                let current = string_to_datetime(c).ok_or(to_datetime_error(ruby, c, ri, "Date"))?;
                 date_value = current;
                 Ok(current.to_string())
             } else if i == *start || i == *end || i == *actual_start || i == *actual_end {
-                let current_time = string_to_time(c).ok_or(to_datetime_error(ruby, c, ri, i))?;
+                if c.is_empty() { return Ok(c.to_string()); }
+                let unknown = "Unknown".to_string();
+                let column_name = inverse_header_map.get(&i).unwrap_or(&unknown);
+                let current_time = string_to_time(c).ok_or(to_datetime_error(ruby, c, ri, column_name))?;
                 let datetime = transform_time_to_datetime(date_value, current_time);
                 Ok(datetime.to_string())
             } else {
@@ -79,7 +83,7 @@ fn transform_time_to_datetime(t1: NaiveDateTime, t2: NaiveTime) -> NaiveDateTime
     NaiveDateTime::new(t1.date(), t2)
 }
 
-fn to_datetime_error(ruby: &Ruby, value: &str, row: usize, col: usize) -> magnus::Error {
+fn to_datetime_error(ruby: &Ruby, value: &str, row: usize, col: &str) -> magnus::Error {
     magnus::Error::new(ruby.exception_standard_error(), format!("Could not parse datetime '{}', row: {}, col: {}", value, row, col))
 }
 

data/ext/csv_utils/src/utils/xls.rs

@@ -44,6 +44,9 @@ fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>, header_ma
         if skip_excluded_rows(&request_id, r, &exclusions) { continue; }
         if skip_empty_rows(r) { continue; }
         if skip_rows_with_no_request_id(&request_id, r) { continue; }
+        if date_value_is_not_present(&date, r) {
+            return Err(magnus::Error::new(ruby.exception_standard_error(), format!("Date value is not present in row: {}", ri)));
+        }
 
         for (i, c) in r.iter().enumerate() {
             match *c {
@@ -53,7 +56,7 @@ fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>, header_ma
                 }
                 Data::Float(ref f) => write!(dest, "{}", f),
                 Data::DateTime(ref d) => {
-                    let mut current = d.as_datetime().unwrap_or_default();
+                    let mut current = d.as_datetime().ok_or(to_datetime_error(ruby, &d.to_string(), ri, "Date"))?;
                     if i == *date {
                         date_value = current;
                     } else if i == *start || i == *end || i == *actual_start || i == *actual_end {
@@ -74,6 +77,10 @@ fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>, header_ma
     Ok(())
 }
 
+fn date_value_is_not_present(date: &usize, r: &[Data]) -> bool {
+    r[*date] == Data::Empty
+}
+
 fn skip_excluded_rows(request_id: &usize, r: &[Data], exclusions: &Vec<String>) -> bool {
     let value = r[*request_id].to_string();
     exclusions.contains(&value.to_string())
@@ -91,6 +98,10 @@ fn transform_time_to_datetime(t1: NaiveDateTime, t2: NaiveDateTime) -> NaiveDate
     NaiveDateTime::new(t1.date(), t2.time())
 }
 
+fn to_datetime_error(ruby: &Ruby, value: &str, row: usize, col: &str) -> magnus::Error {
+    magnus::Error::new(ruby.exception_standard_error(), format!("Could not parse datetime '{}', row: {}, col: {}", value, row, col))
+}
+
 fn handle_commas<W: Write>(dest: &mut W, s: &str) -> std::io::Result<()> {
     if s.contains(",") {
         write!(dest, "{:?}", clean_strings(s).trim_end())

data/lib/csv_utils/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module CsvUtils
-  VERSION = '0.1.7'
+  VERSION = '0.1.8'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: patchwork_csv_utils
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
 platform: arm64-darwin
 authors:
 - kingsley.hendrickse