server: Simplify from_path methods

Author: A6GibKm

server/src/collection.rs

@@ -253,13 +253,7 @@ impl Collection {
     pub async fn item_from_path(&self, path: &ObjectPath<'_>) -> Option<item::Item> {
         let items = self.items.lock().await;
 
-        for item in items.iter() {
-            if item.path() == path {
-                return Some(item.clone());
-            }
-        }
-
-        None
+        items.iter().find(|i| i.path() == path).cloned()
     }
 
     pub async fn set_locked(&self, locked: bool) -> Result<(), ServiceError> {

server/src/service.rs

@@ -457,13 +457,7 @@ impl Service {
     pub async fn collection_from_path(&self, path: &ObjectPath<'_>) -> Option<Collection> {
         let collections = self.collections.lock().await;
 
-        for collection in collections.iter() {
-            if *collection.path() == **path {
-                return Some(collection.clone());
-            }
-        }
-
-        None
+        collections.iter().find(|c| c.path() == path).cloned()
     }
 
     pub async fn session_index(&self) -> u32 {