Merge pull request #467 from schrieveslaach/ignore-annotations

Ignore Annotations

src/ods.rs

@@ -616,6 +616,15 @@ fn get_datatype(
                 {
                     return Ok((Data::String(s), formula, true));
                 }
+                Ok(Event::Start(ref e)) if e.name() == QName(b"office:annotation") => loop {
+                    match reader.read_event_into(buf) {
+                        Ok(Event::End(ref e)) if e.name() == QName(b"office:annotation") => {
+                            break;
+                        }
+                        Err(e) => return Err(OdsError::Xml(e)),
+                        _ => (),
+                    }
+                },
                 Ok(Event::Start(ref e)) if e.name() == QName(b"text:p") => {
                     if first_paragraph {
                         first_paragraph = false;

tests/test.rs

@@ -1822,3 +1822,10 @@ fn issue_repeated_empty(#[case] fixture_path: &str) {
         ]
     );
 }
+
+#[test]
+fn ods_with_annotations() {
+    let mut ods: Ods<_> = wb("with-annotation.ods");
+    let range = ods.worksheet_range("table1").unwrap();
+    range_eq!(range, [[String("cell a.1".to_string())],]);
+}