Change Attributes::has_nil to accept NamespaceResolver instead of Reader<R>

Author: Mingun

Changelog.md

@@ -23,6 +23,7 @@
 - [#908]: Increase minimal supported `serde` version from 1.0.139 to 1.0.180.
 - [#913]: Deprecate `.prefixes()`, `.resolve()`, `.resolve_attribute()`, and `.resolve_element()`
   of `NsReader`. Use `.resolver().bindings()` and `.resolver().resolve()` methods instead.
+- [#913]: `Attributes::has_nil` now accepts `NamespaceResolver` instead of `Reader<R>`.
 
 [#908]: https://github.com/tafia/quick-xml/pull/908
 [#913]: https://github.com/tafia/quick-xml/pull/913

src/de/mod.rs

@@ -3469,7 +3469,7 @@ impl<'i, R: BufRead> XmlRead<'i> for IoReader<R> {
     }
 
     fn has_nil_attr(&self, start: &BytesStart) -> bool {
-        start.attributes().has_nil(&self.reader)
+        start.attributes().has_nil(self.reader.resolver())
     }
 }
 
@@ -3537,7 +3537,7 @@ impl<'de> XmlRead<'de> for SliceReader<'de> {
     }
 
     fn has_nil_attr(&self, start: &BytesStart) -> bool {
-        start.attributes().has_nil(&self.reader)
+        start.attributes().has_nil(self.reader.resolver())
     }
 }
 

src/events/attributes.rs

@@ -5,8 +5,7 @@
 use crate::encoding::Decoder;
 use crate::errors::Result as XmlResult;
 use crate::escape::{escape, resolve_predefined_entity, unescape_with};
-use crate::name::{LocalName, Namespace, QName};
-use crate::reader::NsReader;
+use crate::name::{LocalName, Namespace, NamespaceResolver, QName};
 use crate::utils::{is_whitespace, Bytes};
 
 use std::fmt::{self, Debug, Display, Formatter};
@@ -367,7 +366,7 @@ impl<'a> Attributes<'a> {
     ///             e => panic!("Unexpected event {:?}", e),
     ///         };
     ///         assert_eq!(
-    ///             (event.name(), event.attributes().has_nil(&$reader)),
+    ///             (event.name(), event.attributes().has_nil($reader.resolver())),
     ///             (QName($name.as_bytes()), $value),
     ///         );
     ///     };
@@ -377,7 +376,7 @@ impl<'a> Attributes<'a> {
     ///     Event::Start(e) => e,
     ///     e => panic!("Unexpected event {:?}", e),
     /// };
-    /// assert_eq!(root.attributes().has_nil(&reader), false);
+    /// assert_eq!(root.attributes().has_nil(reader.resolver()), false);
     ///
     /// // definitely true
     /// check!(reader, "true",          true);
@@ -394,12 +393,12 @@ impl<'a> Attributes<'a> {
     /// ```
     ///
     /// [`xsi:nil`]: https://www.w3.org/TR/xmlschema-1/#xsi_nil
-    pub fn has_nil<R>(&mut self, reader: &NsReader<R>) -> bool {
+    pub fn has_nil(&mut self, resolver: &NamespaceResolver) -> bool {
         use crate::name::ResolveResult::*;
 
         self.any(|attr| {
             if let Ok(attr) = attr {
-                match reader.resolver().resolve_attribute(attr.key) {
+                match resolver.resolve_attribute(attr.key) {
                     (
                         Bound(Namespace(b"http://www.w3.org/2001/XMLSchema-instance")),
                         LocalName(b"nil"),