fix(pyo3-macros): allow pyclass named Probe

The `*` import was shadowing the name of the pyclass.

The PyMethods trait was only defined for the multiple-pymethods feature,
but it cannot be conditionally imported inside the macro.
I just defined the trait unconditionally.

Fixes #4792.

Author: maurosilber

pyo3-macros-backend/src/pyclass.rs

@@ -3010,13 +3010,14 @@ impl<'a> PyClassImplsBuilder<'a> {
                 type BaseNativeType = #base_nativetype;
 
                 fn items_iter() -> #pyo3_path::impl_::pyclass::PyClassItemsIter {
-                    use #pyo3_path::impl_::pyclass::*;
-                    let collector = PyClassImplCollector::<Self>::new();
-                    static INTRINSIC_ITEMS: PyClassItems = PyClassItems {
+                    use #pyo3_path::impl_::pyclass::PyMethods as _;
+                    use #pyo3_path::types::PyAnyMethods as _;
+                    let collector = #pyo3_path::impl_::pyclass::PyClassImplCollector::<Self>::new();
+                    static INTRINSIC_ITEMS: #pyo3_path::impl_::pyclass::PyClassItems = #pyo3_path::impl_::pyclass::PyClassItems {
                         methods: &[#(#default_method_defs),*],
                         slots: &[#(#default_slot_defs),* #(#freelist_slots),*],
                     };
-                    PyClassItemsIter::new(&INTRINSIC_ITEMS, #pymethods_items)
+                    #pyo3_path::impl_::pyclass::PyClassItemsIter::new(&INTRINSIC_ITEMS, #pymethods_items)
                 }
 
                 const RAW_DOC: &'static ::std::ffi::CStr = #doc;

src/impl_/pyclass.rs

@@ -1002,7 +1002,7 @@ pub trait PyClassInventory: inventory::Collect {
 }
 
 // Items from #[pymethods] if not using inventory.
-#[cfg(not(feature = "multiple-pymethods"))]
+// #[cfg(not(feature = "multiple-pymethods"))]
 pub trait PyMethods<T> {
     fn py_methods(self) -> &'static PyClassItems;
 }

tests/ui/pyclass_probe.rs

@@ -1,21 +1,36 @@
 #![deny(unused_imports)]
 use pyo3::prelude::*;
 
-#[pyclass]
-pub struct Probe {}
-
-#[pymethods]
-impl Probe {
-    #[new]
-    fn new() -> Self {
-        Self {}
+#[pymodule]
+mod probe_no_fields {
+    use pyo3::prelude::*;
+    #[pyclass]
+    pub struct Probe {}
+    
+    #[pymethods]
+    impl Probe {
+        #[new]
+        fn new() -> Self {
+            Self {}
+        }
     }
 }
 
 #[pymodule]
-fn probe(_py: Python<'_>, m: &Bound<'_, PyModule>) -> PyResult<()> {
-    m.add_class::<Probe>()?;
-    Ok(())
+mod probe_with_fields {
+    use pyo3::prelude::*;
+    #[pyclass(get_all)]
+    pub struct Probe {
+        field: u8,
+    }
+    
+    #[pymethods]
+    impl Probe {
+        #[new]
+        fn new() -> Self {
+            Self { field: 0 }
+        }
+    }
 }
 
 #[pyclass]