patchwork_csv_utils 0.1.13-arm64-darwin → 0.1.15-arm64-darwin

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 54fc4d5750dd83ab95eebe09a6270e8b38e409cc0cf9ed2beb5fb0eeddd096ec
-  data.tar.gz: 7dc07de1618c260cb80c2feee25ace49fe9fba817c63b18f3dd2aa0fd0a4d2a2
+  metadata.gz: 14f80b793a82a72259a5109f10fe4819209d5678b0dfb9e9625f1dcbe369032c
+  data.tar.gz: 88eb4c82a01f9aefed47a4004b0ee3aab5c06c797f13037782379d2652aabb97
 SHA512:
-  metadata.gz: 28050396ee306f0512d7f5c689b0a098f2df0fcacc0d6eda085eb070a6cfe2f8097cadb4508b2bc36a1a601f3f73755c6f528bee9b424f89c59af709b497b785
-  data.tar.gz: 7907c50d50888bcac36868296616ae741e9a5d8855ec9007733b45be2e227f1f96a4248df710e52897f97d197bfda0001cc5eb44e3e7954d5d31ae3e270c8dfd
+  metadata.gz: ab3d39f423189c11331d585572f6a574d2e77a6b66f67172c4ab82b1c0185b4a251a1cee58e4cbbee361d5a8651a9bf08b7a7d9867cc1e45f47d2e06cd1a8a3c
+  data.tar.gz: b20cfb0ac285189a58f879b4ded074bac5886c780afb7bef983809e8689e9ee24d13f923ec633094e150be528d84702ba3585e28f8951974b4495727c6efb4d7

data/Gemfile.lock

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

data/README.md

@@ -14,8 +14,8 @@ gem install patchwork_csv_utils
 ```irb
 require 'csv_utils'
 CsvUtils.dedup('file1.csv', 'file2.csv', 'output.csv', ['mandatory_headers'])
-CsvUtils.to_csv('file1.xls', 'output_file1.csv', ['request_ids_to_skip'], ['mandatory_headers'])
-CsvUtils.transform_csv('file1.xls', 'output_file1.csv', ['request_ids_to_skip'], ['mandatory_headers'])
+CsvUtils.to_csv('file1.xls', 'output_file1.csv', ['request_ids_to_skip'], ['mandatory_headers'], ['status_exclusions'])
+CsvUtils.transform_csv('file1.xls', 'output_file1.csv', ['request_ids_to_skip'], ['mandatory_headers'], ['status_exclusions'])
 ```
 
 ## Release

data/ext/csv_utils/src/lib.rs

@@ -9,7 +9,7 @@ pub mod utils;
 fn init() -> Result<(), magnus::Error> {
     let module = define_module("CsvUtils")?;
     module.define_singleton_method("dedup", function!(dedup, 4))?;
-    module.define_singleton_method("to_csv", function!(to_csv, 4))?;
-    module.define_singleton_method("transform_csv", function!(transform_csv, 4))?;
+    module.define_singleton_method("to_csv", function!(to_csv, 5))?;
+    module.define_singleton_method("transform_csv", function!(transform_csv, 5))?;
     Ok(())
 }

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

@@ -1,6 +1,6 @@
 use std::collections::HashMap;
 use std::fs::File;
-
+use calamine::Data;
 use chrono::{NaiveDate, NaiveDateTime, NaiveTime, Utc};
 use csv::{Reader, StringRecord, Writer};
 use magnus::{Error, RArray, Ruby};
@@ -9,12 +9,14 @@ use crate::utils::{FileExtension, magnus_err, missing_header, to_datetime_error,
 
 pub fn transform_csv(ruby: &Ruby, csv_path: String,
                      target_path: String, exclusions: RArray,
-                     mandatory_headers: RArray, ) -> magnus::error::Result<()> {
+                     mandatory_headers: RArray,
+                     status_exclusions: RArray) -> magnus::error::Result<()> {
     if !csv_path.has_extension(&["csv"]) {
         return Err(Error::new(ruby.exception_standard_error(), "csv_path must be a csv file".to_string()));
     }
 
     let exclusions = RArray::to_vec(exclusions)?;
+    let status_exclusions = RArray::to_vec(status_exclusions)?;
     let mandatory_headers: Vec<String> = RArray::to_vec(mandatory_headers)?;
 
     let csv_file = File::open(csv_path).map_err(|e| magnus_err(ruby, e, "csv_path"))?;
@@ -38,12 +40,14 @@ pub fn transform_csv(ruby: &Ruby, csv_path: String,
     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 status = header_map.get("Status");
 
     let mandatory_records = get_mandatory_records(&ruby, &mut csv, &headers_list, &mandatory_headers)?;
 
     for (ri, record) in mandatory_records.iter().enumerate() {
 
         if skip_excluded_rows(request_id, &record, &exclusions) { continue; }
+        if skip_excluded_status_rows(&status, &record, &status_exclusions) { continue; }
         if has_empty_row_skip(&record) { continue; }
         if has_empty_first_col_skip_row(&record) { continue; }
 
@@ -124,6 +128,12 @@ fn skip_excluded_rows(request_id: &usize, r: &StringRecord, exclusions: &Vec<Str
     exclusions.contains(&value.to_string())
 }
 
+fn skip_excluded_status_rows(status: &Option<&usize>, r: &StringRecord, exclusions: &Vec<String>) -> bool {
+    status
+        .map(|index| exclusions.contains(&r[*index].to_string()))
+        .unwrap_or(false)
+}
+
 fn string_to_datetime(s: &str) -> Option<NaiveDateTime> {
     let maybe_correct = correct_datetime(s);
     if maybe_correct.is_some() { return maybe_correct; }

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

@@ -11,7 +11,8 @@ use crate::utils::{FileExtension, magnus_err, missing_header, to_datetime_error,
 pub fn to_csv(ruby: &Ruby, xls_path: String,
               target_path: String,
               exclusions: RArray,
-              mandatory_headers: RArray
+              mandatory_headers: RArray,
+              status_exclusions: RArray,
 ) -> 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()));
@@ -19,6 +20,7 @@ pub fn to_csv(ruby: &Ruby, xls_path: String,
 
     let exclusions = RArray::to_vec(exclusions)?;
     let mandatory_headers: Vec<String> = RArray::to_vec(mandatory_headers)?;
+    let status_exclusions = RArray::to_vec(status_exclusions)?;
 
     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_at(0)
@@ -26,7 +28,7 @@ pub fn to_csv(ruby: &Ruby, xls_path: 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 headers_list : Vec<String> = headers.iter().map(|h| h.to_string()).collect();
+    let headers_list: Vec<String> = headers.iter().map(|h| h.to_string()).collect();
 
     if let Some(value) =
         check_mandatory_headers(ruby, &headers_list, &mandatory_headers, "csv") { return value; }
@@ -35,13 +37,14 @@ pub fn to_csv(ruby: &Ruby, xls_path: String,
     let csv_out_file = File::create(target_path.clone()).map_err(|e| magnus_err(ruby, e, format!("could not create csv file: {}", target_path).as_str()))?;
     let mut dest = BufWriter::new(csv_out_file);
 
-    write_csv(ruby, &mut dest, &range, header_map, exclusions, mandatory_headers, headers_list)
+    write_csv(ruby, &mut dest, &range, header_map, exclusions, mandatory_headers, headers_list, status_exclusions)
 }
 
 fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>,
                        header_map: HashMap<String, usize>, exclusions: Vec<String>,
                        mandatory_headers: Vec<String>,
-                       headers_list: Vec<String>) -> magnus::error::Result<()> {
+                       headers_list: Vec<String>,
+                       status_exclusions: Vec<String>) -> magnus::error::Result<()> {
     let n = mandatory_headers.len() - 1;
     let request_id = header_map.get("Request Id").ok_or(missing_header(ruby, "Request Id"))?;
     let date = header_map.get("Date").ok_or(missing_header(ruby, "Date"))?;
@@ -49,6 +52,7 @@ fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>,
     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 status = header_map.get("Status");
 
     let mandatory_rows = get_mandatory_records(ruby, range, &headers_list, &mandatory_headers)?;
 
@@ -56,6 +60,7 @@ fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>,
         let mut date_value = Utc::now().naive_utc();
 
         if skip_excluded_rows(&request_id, &r, &exclusions) { continue; }
+        if skip_excluded_status_rows(&status, &r, &status_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) {
@@ -63,7 +68,6 @@ fn write_csv<W: Write>(ruby: &Ruby, dest: &mut W, range: &Range<Data>,
         }
 
         for (i, c) in mandatory_headers.iter().enumerate() {
-
             let column_index = header_map.get(c).ok_or(missing_header(ruby, c))?;
             let c = r.get(*column_index).ok_or(missing_value(ruby, c))?;
 
@@ -125,6 +129,12 @@ fn skip_excluded_rows(request_id: &usize, r: &Vec<&Data>, exclusions: &Vec<Strin
     exclusions.contains(&value.to_string())
 }
 
+fn skip_excluded_status_rows(status: &Option<&usize>, r: &Vec<&Data>, exclusions: &Vec<String>) -> bool {
+    status
+        .map(|index| exclusions.contains(&r[*index].to_string()))
+        .unwrap_or(false)
+}
+
 fn skip_empty_rows(r: &Vec<&Data>) -> bool {
     r.into_iter().all(|c| c == &&Data::Empty)
 }
@@ -149,7 +159,6 @@ fn clean_strings(s: &str) -> String {
     s.replace("\n", " ")
         .replace("\r", " ")
         .replace("\"", "")
-        .replace("'", "")
 }
 
 struct XlsMandatoryColumn<'a> {

data/lib/csv_utils/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: patchwork_csv_utils
 version: !ruby/object:Gem::Version
-  version: 0.1.13
+  version: 0.1.15
 platform: arm64-darwin
 authors:
 - kingsley.hendrickse
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-09-04 00:00:00.000000000 Z
+date: 2024-10-31 00:00:00.000000000 Z
 dependencies: []
 description: Deduplication of CSV files and XLS to CSV conversion.
 email: