Minor Correction

Author: DESKTOP-U434MT0\hiro

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "umya-spreadsheet"
-version = "0.6.0"
+version = "0.7.0"
 authors = ["MathNya <umya.net.info@gmail.com>"]
 repository = "https://github.com/MathNya/umya-spreadsheet"
 keywords = ["excel", "spreadsheet", "xlsx", "reader", "writer"]

src/structs/cell.rs

@@ -391,7 +391,11 @@ impl Cell {
         shared_string_table: Arc<RwLock<SharedStringTable>>,
         stylesheet: &mut Stylesheet,
     ) {
-        let empty_flag = self.cell_value.is_empty();
+        let empty_flag_value = self.cell_value.is_empty();
+        let empty_flag_style = self.style.is_empty();
+        if empty_flag_value && empty_flag_style {
+            return;
+        }
 
         // c
         let mut attributes: Vec<(&str, &str)> = Vec::new();
@@ -407,8 +411,8 @@ impl Cell {
             attributes.push(("s", &xf_index_str));
         }
 
-        if !empty_flag {
-            write_start_tag(writer, "c", attributes, empty_flag);
+        if !empty_flag_value {
+            write_start_tag(writer, "c", attributes, false);
             // f
             match &self.cell_value.formula {
                 Some(v) => {
@@ -439,6 +443,8 @@ impl Cell {
             write_end_tag(writer, "v");
 
             write_end_tag(writer, "c");
+        } else {
+            write_start_tag(writer, "c", attributes, true);
         }
     }
 }

src/structs/style.rs

@@ -194,6 +194,25 @@ impl Style {
         self
     }
 
+    pub(crate) fn is_empty(&self) -> bool {
+        if self.font.is_some() {
+            return false;
+        }
+        if self.fill.is_some() {
+            return false;
+        }
+        if self.borders.is_some() {
+            return false;
+        }
+        if self.alignment.is_some() {
+            return false;
+        }
+        if self.numbering_format.is_some() {
+            return false;
+        }
+        true
+    }
+
     pub(crate) fn get_defalut_value() -> Self {
         let mut def = Self::default();
         def.set_font(Font::get_defalut_value());

src/structs/worksheet.rs

@@ -430,6 +430,11 @@ impl Worksheet {
         self.cell_collection.get_mut(col, row).get_style_mut()
     }
 
+    pub fn set_style(&mut self, coordinate: &str, style: Style) -> &mut Self {
+        let (col, row) = index_from_coordinate_simple(coordinate);
+        self.set_style_by_column_and_row(&col, &row, style)
+    }
+
     /// Set the style by specifying the column number and row number.
     /// # Arguments
     /// * `col` - Specify the column number. (first column number is 1)

tests/integration_test.rs

@@ -750,6 +750,28 @@ fn new_file_and_edit() {
         .get_color_mut()
         .set_argb("00FF0000");
 
+    // change background color.
+    book.get_sheet_by_name_mut("Sheet2")
+        .unwrap()
+        .get_style_mut("A1")
+        .get_fill_mut()
+        .get_pattern_fill_mut()
+        .get_foreground_color_mut()
+        .set_argb(umya_spreadsheet::Color::COLOR_BLUE);
+
+    // change background color part2.
+    let mut color = umya_spreadsheet::Color::default();
+    color.set_argb(umya_spreadsheet::Color::COLOR_BLUE);
+    let mut pattern_fill = umya_spreadsheet::PatternFill::default();
+    pattern_fill.set_foreground_color(color);
+    let mut fill = umya_spreadsheet::Fill::default();
+    fill.set_pattern_fill(pattern_fill);
+    let mut style = umya_spreadsheet::Style::default();
+    style.set_fill(fill);
+    book.get_sheet_by_name_mut("Sheet2")
+        .unwrap()
+        .set_style("A2", style);
+    
     let worksheet = book.get_sheet_by_name_mut("Sheet3").unwrap();
     worksheet.get_column_dimension_mut("A").set_auto_width(true);