core: Extend list method to handle all types

Author: konac-hamza

src/api/v3/user/types.rs

@@ -353,8 +353,7 @@ impl From<UserListParameters> for identity_types::UserListParameters {
             domain_id: value.domain_id,
             name: value.name,
             unique_id: value.unique_id,
-            user_type: None,
-            //    limit: value.limit,
+            ..Default::default() //    limit: value.limit,
         }
     }
 }

src/identity/backends/sql.rs

@@ -64,22 +64,13 @@ impl IdentityBackend for SqlBackend {
         state: &ServiceState,
         params: &UserListParameters,
     ) -> Result<Vec<UserResponse>, IdentityProviderError> {
-        // Determine user type to call appropriate listing function
-        if let Some(val) = &params.user_type {
-            match val {
-                UserType::Local => {
-                    return Ok(local_user::list(&self.config, &state.db, params).await?);
-                }
-                UserType::NonLocal => {
-                    // return Ok(nonlocal_user::list(&self.config, &state.db, params).await?);
-                }
-                UserType::Federated => {
-                    // return Ok(federated_user::list(&self.config, &state.db, params).await?);
-                }
-                UserType::All => { /* continue to general listing */ }
-            }
+        // Create a modified copy if user_type is None
+        let mut modified_params = params.clone();
+        if modified_params.user_type.is_none() {
+            modified_params.user_type = Some(UserType::All);
         }
-        Ok(user::list(&self.config, &state.db, params).await?)
+
+        Ok(user::list(&self.config, &state.db, &modified_params).await?)
     }
 
     /// Get single user by ID

src/identity/backends/sql/local_user.rs

@@ -17,11 +17,9 @@ use crate::identity::types::*;
 
 mod create;
 mod get;
-mod list;
 mod load;
 mod set;
 
-pub use list::list;
 pub use load::load_local_user_with_passwords;
 pub use load::load_local_users_passwords;
 pub use set::reset_failed_auth;