patchwork_csv_utils 0.1.3-arm64-darwin → 0.1.5-arm64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cc5b0905c8210cfdad65db324e93a40623ad1aaefe2a2d05165b8df796272186
-  data.tar.gz: b1e4a1262e1e49ed970482433ac193610ec105d1bf024d123551063607bdbd40
+  metadata.gz: ccdea0ba6abfc936797f007183000983de9b15b9c195dfcdcfcd6ed3e63b5119
+  data.tar.gz: af20ff750dbbf82f86a3aaad20242770e1327db98218dd5e93381f3627f744bb
 SHA512:
-  metadata.gz: c7a88eed7af4705c554dfb41b1159b2eeba01657daf572fb2d188fd86d051977e5e6c401ae43a5b5401756a2bef8a56cbe28eb93dca389760ac0004946746478
-  data.tar.gz: b98944299828a21085c531f9d76602a3dc1dfb4ef9db03cf93a2d44fd1d9c55481617c16a5fe65cbb8b9509794430916252abfbfa98df18b6cce7d1aa2e2a0b8
+  metadata.gz: d134cfdfafab51d74fef17480c0aad78c1033d0fc3ae443cd57ad5ce838a76fdf44a05970c295177eb6b9b26f556c2d37a8a2b04f05d67fec194f22991da8628
+  data.tar.gz: dc188247d6c270c0640f3f3b88173b4e3ff97d23c4b077749219fcf33c0394616b1b95b3d22e9fd5126fb2741f72bea316c1d0f09a7798484e775663661c0daa

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    patchwork_csv_utils (0.1.3)
+    patchwork_csv_utils (0.1.5)
 
 GEM
   remote: https://rubygems.org/

data/ext/csv_utils/src/lib.rs

@@ -8,6 +8,6 @@ pub mod utils;
 fn init() -> Result<(), magnus::Error> {
     let module = define_module("CsvUtils")?;
     module.define_singleton_method("dedup", function!(dedup, 3))?;
-    module.define_singleton_method("to_csv", function!(to_csv, 3))?;
+    module.define_singleton_method("to_csv", function!(to_csv, 2))?;
     Ok(())
 }

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

@@ -33,12 +33,20 @@ pub fn dedup(ruby: &Ruby, previous_csv_path: String, new_csv_path: String, targe
     let mut previous_records = vec![];
     for previous_record in previous_csv.records() {
         let previous_record = previous_record.map_err(|e| magnus_err(ruby, e, "previous_record"))?;
+
+        if has_empty_row_skip(&previous_record) { continue; }
+        if has_empty_first_col_skip_row(&previous_record) { continue; }
+
         let previous_record = previous_record.into_iter().map(|r| r.trim_end()).collect::<StringRecord>();
         previous_records.push(previous_record)
     }
 
     for new_record in new_csv.records() {
         let new_record = new_record.map_err(|e| magnus_err(ruby, e, "new_record"))?;
+
+        if has_empty_row_skip(&new_record) { continue; }
+        if has_empty_first_col_skip_row(&new_record) { continue; }
+
         let new_record = new_record.into_iter().map(|r| r.trim_end()).collect::<StringRecord>();
         if !previous_records.contains(&new_record) {
             wtr.write_byte_record(new_record.as_byte_record()).unwrap();
@@ -50,4 +58,18 @@ pub fn dedup(ruby: &Ruby, previous_csv_path: String, new_csv_path: String, targe
     Ok(())
 }
 
+fn has_empty_first_col_skip_row(previous_record: &StringRecord) -> bool {
+    if previous_record[0].is_empty() {
+        return true;
+    }
+    false
+}
+
+fn has_empty_row_skip(record: &StringRecord) -> bool {
+    if record.iter().all(|r| r.is_empty()) {
+        return true;
+    }
+    false
+}
+
 

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

@@ -8,13 +8,15 @@ use magnus::Ruby;
 
 use crate::utils::{FileExtension, magnus_err};
 
-pub fn to_csv(ruby: &Ruby, xls_path: String, target_path: String, sheet_name: String) -> magnus::error::Result<()> {
+pub fn to_csv(ruby: &Ruby, xls_path: String, target_path: String) -> magnus::error::Result<()> {
     if !xls_path.has_extension(&["xls"]) {
         return Err(magnus::Error::new(ruby.exception_standard_error(), "xls_path must be an xls file".to_string()));
     }
 
     let mut workbook: Xls<_> = open_workbook(xls_path.clone()).map_err(|e| magnus_err(ruby, e, format!("could not open xls: {}", xls_path).as_str()))?;
-    let range = workbook.worksheet_range(sheet_name.as_str()).map_err(|e| magnus_err(ruby, e, "Error"))?;
+    let range = workbook.worksheet_range_at(0)
+        .ok_or(magnus::Error::new(ruby.exception_standard_error(), "no worksheet found in xls".to_string()))
+        .and_then(|r| r.map_err(|e| magnus_err(ruby, e, "could not read worksheet range")))?;
 
     let headers = range.headers().ok_or(magnus::Error::new(ruby.exception_standard_error(), "no headers found in xls".to_string()))?;
     let header_map: HashMap<String, usize> = headers.iter().enumerate().map(|(i, h)| (h.to_string(), i)).collect();
@@ -29,17 +31,16 @@ fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>, header_ma
     for (ri, r) in range.rows().enumerate() {
         let mut date_value = Utc::now().naive_utc();
 
-        if r.iter().all(|c| c == &Data::Empty) {
-            continue;
-        }
+        if skip_empty_rows(r) { continue; }
+        if skip_rows_with_no_request_id(&header_map, r) { continue; }
 
-        for (i, c) in r.iter().enumerate() {
-            let date = header_map.get("Date").ok_or(missing_header(ruby, "Date"))?;
-            let start = header_map.get("Start").ok_or(missing_header(ruby, "Start"))?;
-            let end = header_map.get("End").ok_or(missing_header(ruby, "End"))?;
-            let actual_start = header_map.get("Actual Start").ok_or(missing_header(ruby, "Actual Start"))?;
-            let actual_end = header_map.get("Actual End").ok_or(missing_header(ruby, "Actual End"))?;
+        let date = header_map.get("Date").ok_or(missing_header(ruby, "Date"))?;
+        let start = header_map.get("Start").ok_or(missing_header(ruby, "Start"))?;
+        let end = header_map.get("End").ok_or(missing_header(ruby, "End"))?;
+        let actual_start = header_map.get("Actual Start").ok_or(missing_header(ruby, "Actual Start"))?;
+        let actual_end = header_map.get("Actual End").ok_or(missing_header(ruby, "Actual End"))?;
 
+        for (i, c) in r.iter().enumerate() {
             match *c {
                 Data::Empty => Ok(()),
                 Data::String(ref s) | Data::DateTimeIso(ref s) | Data::DurationIso(ref s) => {
@@ -68,6 +69,22 @@ fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>, header_ma
     Ok(())
 }
 
+fn skip_empty_rows(r: &[Data]) -> bool {
+    if r.iter().all(|c| c == &Data::Empty) {
+        return true;
+    }
+    false
+}
+
+fn skip_rows_with_no_request_id(header_map: &HashMap<String, usize>, r: &[Data]) -> bool {
+    if let Some(request_id) = header_map.get("Request Id") {
+        if r[*request_id] == Data::Empty {
+            return true;
+        }
+    }
+    false
+}
+
 fn missing_header(ruby: &Ruby, header: &str) -> magnus::Error {
     magnus::Error::new(ruby.exception_standard_error(), format!("Missing '{}' header in xls", header))
 }

data/lib/csv_utils/version.rb

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

metadata

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