From e1c4bb8fc724065a52c8432f6a9d0d6201b72161 Mon Sep 17 00:00:00 2001 From: Rohan Sircar Date: Wed, 21 Apr 2021 13:32:03 +0530 Subject: [PATCH] Rename UserDTO to UserDto --- src/actions/users.rs | 16 ++++++++-------- src/models/users.rs | 2 +- src/services/user_service.rs | 16 ++++++++-------- 3 files changed, 17 insertions(+), 17 deletions(-) diff --git a/src/actions/users.rs b/src/actions/users.rs index 744f1fa..835567b 100644 --- a/src/actions/users.rs +++ b/src/actions/users.rs @@ -7,13 +7,13 @@ use bcrypt::{hash, verify, DEFAULT_COST}; pub fn find_user_by_uid( uid: i32, conn: &SqliteConnection, -) -> Result, errors::DomainError> { +) -> Result, errors::DomainError> { use crate::schema::users::dsl::*; let maybe_user = users .select((name, created_at)) .find(uid) - .first::(conn) + .first::(conn) .optional(); Ok(maybe_user?) @@ -22,9 +22,9 @@ pub fn find_user_by_uid( pub fn _find_user_by_name( user_name: String, conn: &SqliteConnection, -) -> Result, errors::DomainError> { +) -> Result, errors::DomainError> { let maybe_user = query::_get_user_by_name(&user_name) - .first::(conn) + .first::(conn) .optional(); Ok(maybe_user?) @@ -32,11 +32,11 @@ pub fn _find_user_by_name( pub fn get_all( conn: &SqliteConnection, -) -> Result>, errors::DomainError> { +) -> Result>, errors::DomainError> { use crate::schema::users::dsl::*; Ok(users .select((name, created_at)) - .load::(conn) + .load::(conn) .optional()?) } @@ -44,7 +44,7 @@ pub fn get_all( pub fn insert_new_user( nu: models::NewUser, conn: &SqliteConnection, -) -> Result { +) -> Result { // It is common when using Diesel with Actix web to import schema-related // modules inside a function's scope (rather than the normal module's scope) // to prevent import collisions and namespace pollution. @@ -57,7 +57,7 @@ pub fn insert_new_user( diesel::insert_into(users).values(&nu).execute(conn)?; let user = - query::_get_user_by_name(&nu.name).first::(conn)?; + query::_get_user_by_name(&nu.name).first::(conn)?; Ok(user) } diff --git a/src/models/users.rs b/src/models/users.rs index 6a5991e..485e4d5 100644 --- a/src/models/users.rs +++ b/src/models/users.rs @@ -21,7 +21,7 @@ pub struct NewUser { } #[derive(Debug, Clone, Serialize, Deserialize, Queryable)] -pub struct UserDTO { +pub struct UserDto { pub name: String, pub registration_date: chrono::NaiveDateTime, } diff --git a/src/services/user_service.rs b/src/services/user_service.rs index add8b10..09083f3 100644 --- a/src/services/user_service.rs +++ b/src/services/user_service.rs @@ -4,20 +4,20 @@ pub trait UserService { fn find_user_by_uid( &self, uid: i32, - ) -> Result, errors::DomainError>; + ) -> Result, errors::DomainError>; fn _find_user_by_name( &self, user_name: String, - ) -> Result, errors::DomainError>; + ) -> Result, errors::DomainError>; fn get_all( &self, - ) -> Result>, errors::DomainError>; + ) -> Result>, errors::DomainError>; fn insert_new_user( &self, nu: models::NewUser, - ) -> Result; + ) -> Result; // fn woot(&self) -> i32; @@ -37,7 +37,7 @@ impl UserService for UserServiceImpl { fn find_user_by_uid( &self, uid: i32, - ) -> Result, errors::DomainError> { + ) -> Result, errors::DomainError> { let conn = self.pool.get()?; actions::find_user_by_uid(uid, &conn) } @@ -45,14 +45,14 @@ impl UserService for UserServiceImpl { fn _find_user_by_name( &self, user_name: String, - ) -> Result, errors::DomainError> { + ) -> Result, errors::DomainError> { let conn = self.pool.get()?; actions::_find_user_by_name(user_name, &conn) } fn get_all( &self, - ) -> Result>, errors::DomainError> { + ) -> Result>, errors::DomainError> { let conn = self.pool.get()?; actions::get_all(&conn) } @@ -60,7 +60,7 @@ impl UserService for UserServiceImpl { fn insert_new_user( &self, nu: models::NewUser, - ) -> Result { + ) -> Result { let conn = self.pool.get()?; actions::insert_new_user(nu, &conn) }