From bd10288adfaabdeb763fecb2f66d7dc4d35b37ee Mon Sep 17 00:00:00 2001
From: HampusM <hampus@hampusmat.com>
Date: Thu, 19 Jan 2023 20:21:17 +0100
Subject: refactor!: make binding builder & configurator methods take self
 ownership

BREAKING CHANGE: The methods of BindingBuilder, AsyncBuilder, BindingScopeConfigurator, AsyncBindingScopeConfigurator, BindingWhenConfigurator, AsyncBindingWhenConfigurator now take ownership of self
---
 .../blocking/binding/scope_configurator.rs         | 29 +++++++++++++---------
 1 file changed, 17 insertions(+), 12 deletions(-)

(limited to 'src/di_container/blocking/binding/scope_configurator.rs')

diff --git a/src/di_container/blocking/binding/scope_configurator.rs b/src/di_container/blocking/binding/scope_configurator.rs
index 6c6c32d..ede0f6b 100644
--- a/src/di_container/blocking/binding/scope_configurator.rs
+++ b/src/di_container/blocking/binding/scope_configurator.rs
@@ -64,19 +64,12 @@ where
     /// This is the default.
     #[allow(clippy::must_use_candidate)]
     pub fn in_transient_scope(
-        &self,
+        self,
     ) -> BindingWhenConfigurator<Interface, DIContainerType, DependencyHistoryType>
     {
-        self.di_container.set_binding::<Interface>(
-            None,
-            Box::new(TransientTypeProvider::<
-                Implementation,
-                DIContainerType,
-                DependencyHistoryType,
-            >::new()),
-        );
+        self.set_in_transient_scope();
 
-        BindingWhenConfigurator::new(self.di_container.clone())
+        BindingWhenConfigurator::new(self.di_container)
     }
 
     /// Configures the binding to be in a singleton scope.
@@ -84,7 +77,7 @@ where
     /// # Errors
     /// Will return Err if resolving the implementation fails.
     pub fn in_singleton_scope(
-        &self,
+        self,
     ) -> Result<
         BindingWhenConfigurator<Interface, DIContainerType, DependencyHistoryType>,
         BindingScopeConfiguratorError,
@@ -101,7 +94,19 @@ where
         self.di_container
             .set_binding::<Interface>(None, Box::new(SingletonProvider::new(singleton)));
 
-        Ok(BindingWhenConfigurator::new(self.di_container.clone()))
+        Ok(BindingWhenConfigurator::new(self.di_container))
+    }
+
+    pub(crate) fn set_in_transient_scope(&self)
+    {
+        self.di_container.set_binding::<Interface>(
+            None,
+            Box::new(TransientTypeProvider::<
+                Implementation,
+                DIContainerType,
+                DependencyHistoryType,
+            >::new()),
+        );
     }
 }
 
-- 
cgit v1.2.3-18-g5258