Allow borrowed dictionary in replace_text

src/document/body.rs

@@ -1,5 +1,6 @@
 use derive_more::From;
 use hard_xml::{XmlRead, XmlWrite};
+use std::borrow::Borrow;
 
 use crate::__xml_test_suites;
 use crate::document::{Paragraph, Run, Table, TableCell};
@@ -45,15 +46,14 @@
     where
         S: AsRef<str>,
     {
-        let dic = (old, new);
-        let dic = vec![dic];
-        let _d = self.replace_text(&dic);
+        let _d = self.replace_text(&[(old, new)]);
     }
 
-    pub fn replace_text<'b, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
+    pub fn replace_text<'b, I, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
     where
         S: AsRef<str> + 'b,
-        T: IntoIterator<Item = &'b (S, S)> + std::marker::Copy,
+        T: IntoIterator<Item = I> + Copy,
+        I: Borrow<(S, S)>,
     {
         for content in self.content.iter_mut() {
             match content {
@@ -94,7 +94,7 @@
 /// A set of elements that can be contained in the body
 #[derive(Debug, From, XmlRead, XmlWrite, Clone)]
 #[cfg_attr(test, derive(PartialEq))]
 pub enum BodyContent<'a> {
     #[xml(tag = "w:p")]
     Paragraph(Paragraph<'a>),
     #[xml(tag = "w:tbl")]

src/document/header.rs

@@ -4,6 +4,7 @@
 //!
 
 use hard_xml::{XmlRead, XmlResult, XmlWrite, XmlWriter};
+use std::borrow::Borrow;
 use std::io::Write;
 
 use crate::__xml_test_suites;
@@ -29,15 +30,14 @@ impl<'a> Header<'a> {
     where
         S: AsRef<str>,
     {
-        let dic = (old, new);
-        let dic = vec![dic];
-        let _d = self.replace_text(&dic);
+        let _d = self.replace_text(&[(old, new)]);
     }
 
-    pub fn replace_text<'b, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
+    pub fn replace_text<'b, I, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
     where
         S: AsRef<str> + 'b,
-        T: IntoIterator<Item = &'b (S, S)> + std::marker::Copy,
+        T: IntoIterator<Item = I> + Copy,
+        I: Borrow<(S, S)>,
     {
         for content in self.content.iter_mut() {
             match content {

src/document/paragraph.rs

@@ -1,7 +1,7 @@
 #![allow(unused_must_use)]
 use derive_more::From;
 use hard_xml::{XmlRead, XmlWrite};
-use std::borrow::Cow;
+use std::borrow::{Borrow, Cow};
 
 use crate::{
     __setter, __xml_test_suites,
@@ -110,13 +110,14 @@
             .flatten()
     }
 
-    pub fn replace_text<'b, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
+    pub fn replace_text<'b, I, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
     where
         S: AsRef<str> + 'b,
-        T: IntoIterator<Item = &'b (S, S)> + std::marker::Copy,
+        T: IntoIterator<Item = I> + Copy,
+        I: Borrow<(S, S)>,
     {
         for content in self.content.iter_mut() {
             match content {
                 ParagraphContent::Run(r) => {
                     r.replace_text(dic)?;
                 }

src/document/run.rs

@@ -1,7 +1,7 @@
 #![allow(unused_must_use)]
 use derive_more::From;
 use hard_xml::{XmlRead, XmlWrite};
-use std::borrow::Cow;
+use std::borrow::{Borrow, Cow};
 
 use crate::{
     __setter, __xml_test_suites,
@@ -151,22 +151,21 @@
     where
         S: AsRef<str>,
     {
-        let dic = (old, new);
-        let dic = vec![dic];
-        self.replace_text(&dic);
+        self.replace_text(&[(old, new)]);
     }
 
-    pub fn replace_text<'b, T, S>(&mut self, dic: T) -> DocxResult<()>
+    pub fn replace_text<'b, I, T, S>(&mut self, dic: T) -> DocxResult<()>
     where
         S: AsRef<str> + 'b,
-        T: IntoIterator<Item = &'b (S, S)> + std::marker::Copy,
+        T: IntoIterator<Item = I> + Copy,
+        I: Borrow<(S, S)>,
     {
         for c in self.content.iter_mut() {
             match c {
                 RunContent::Text(t) => {
                     let mut tc = t.text.to_string();
                     for p in dic {
-                        tc = tc.replace(p.0.as_ref(), p.1.as_ref());
+                        tc = tc.replace(p.borrow().0.as_ref(), p.borrow().1.as_ref());
                     }
                     t.text = tc.into();
                 }
@@ -181,7 +180,7 @@
 /// A set of elements that can be contained as the content of a run.
 #[derive(Debug, From, XmlRead, XmlWrite, Clone)]
 #[cfg_attr(test, derive(PartialEq))]
 pub enum RunContent<'a> {
     #[xml(tag = "w:br")]
     Break(Break),
     #[xml(tag = "w:t")]

src/document/table.rs

@@ -1,6 +1,6 @@
 #![allow(unused_must_use)]
 use hard_xml::{XmlRead, XmlWrite};
-use std::borrow::Cow;
+use std::borrow::{Borrow, Cow};
 
 use crate::{
     __setter, __xml_test_suites,
@@ -48,10 +48,11 @@ impl<'a> Table<'a> {
             .flat_map(|content| content.iter_text_mut())
     }
 
-    pub fn replace_text<'b, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
+    pub fn replace_text<'b, I, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
     where
         S: AsRef<str> + 'b,
-        T: IntoIterator<Item = &'b (S, S)> + std::marker::Copy,
+        T: IntoIterator<Item = I> + Copy,
+        I: Borrow<(S, S)>,
     {
         for row in self.rows.iter_mut() {
             row.replace_text(dic)?;

src/document/table_cell.rs

@@ -1,5 +1,5 @@
 #![allow(unused_must_use)]
-use std::borrow::Cow;
+use std::borrow::{Borrow, Cow};
 
 use derive_more::From;
 use hard_xml::{XmlRead, XmlWrite};
@@ -38,7 +38,7 @@
     }
 
     pub fn iter_text(&self) -> impl Iterator<Item = &Cow<'a, str>> {
         self.content
             .iter()
             .filter_map(|content| match content {
                 TableCellContent::Paragraph(p) => Some(p.iter_text()),
@@ -47,7 +47,7 @@
     }
 
     pub fn iter_text_mut(&mut self) -> impl Iterator<Item = &mut Cow<'a, str>> {
         self.content
             .iter_mut()
             .filter_map(|content| match content {
                 TableCellContent::Paragraph(p) => Some(p.iter_text_mut()),
@@ -55,10 +55,11 @@
             .flatten()
     }
 
-    pub fn replace_text<'b, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
+    pub fn replace_text<'b, I, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
     where
         S: AsRef<str> + 'b,
-        T: IntoIterator<Item = &'b (S, S)> + std::marker::Copy,
+        T: IntoIterator<Item = I> + Copy,
+        I: Borrow<(S, S)>,
     {
         for content in self.content.iter_mut() {
             match content {

src/document/table_row.rs

@@ -1,5 +1,5 @@
 #![allow(unused_must_use)]
-use std::borrow::Cow;
+use std::borrow::{Borrow, Cow};
 
 use super::SDT;
 
@@ -83,10 +83,11 @@ impl<'a> TableRow<'a> {
             .flatten()
     }
 
-    pub fn replace_text<'b, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
+    pub fn replace_text<'b, I, T, S>(&mut self, dic: T) -> crate::DocxResult<()>
     where
         S: AsRef<str> + 'b,
-        T: IntoIterator<Item = &'b (S, S)> + std::marker::Copy,
+        T: IntoIterator<Item = I> + Copy,
+        I: Borrow<(S, S)>,
     {
         for cell in self.cells.iter_mut() {
             if let TableRowContent::TableCell(c) = cell {

tests/integration_test.rs

@@ -1,11 +1,12 @@
 extern crate docx_rust;
-use std::fs::read_dir;
 
 use docx_rust::{
     document::{BodyContent, ParagraphContent, RunContent},
     rels::TargetMode,
     DocxFile,
 };
+use std::collections::HashMap;
+use std::fs::read_dir;
 
 #[test]
 fn read_and_replace() {
@@ -19,6 +20,32 @@ fn read_and_replace() {
     let _d = docx.write_file(path2).unwrap();
 }
 
+#[test]
+fn replace_text_multiple() {
+    // reader
+    let path = std::path::Path::new("./tests/aaa/aa.docx");
+    let book = DocxFile::from_file(path).unwrap();
+    let mut docx = book.parse().unwrap();
+
+    let map = HashMap::from([("好日子", "好天气")]);
+    docx.document.body.replace_text(&map).unwrap();
+
+    let map = HashMap::from([("好日子".to_string(), "好天气".to_string())]);
+    docx.document.body.replace_text(&map).unwrap();
+
+    let slice = [("好日子", "好天气")];
+    docx.document.body.replace_text(&slice).unwrap();
+
+    let slice = [("好日子".to_string(), "好天气".to_string())];
+    docx.document.body.replace_text(&slice).unwrap();
+
+    let vec = vec![("好日子", "好天气")];
+    docx.document.body.replace_text(&vec).unwrap();
+
+    let vec = vec![("好日子".to_string(), "好天气".to_string())];
+    docx.document.body.replace_text(&vec).unwrap();
+}
+
 #[test]
 fn read_list() {
     let path = std::path::Path::new("./tests/aaa/aa_list.docx");