Move functions to query/modify db to connection object.

Makes tha API a bit nicer by asking the database rather than
passing the database on to each model. Reserve models for
method/functions that work on the model structs themselves.

1 files changed, 1 insertions, 17 deletions

diff --git a/src/models/film.rs b/src/models/film.rs
index 8207e5d..8f17d42 100644
--- a/src/models/film.rs
+++ b/src/models/film.rs
@@ -16,11 +16,7 @@
     along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 
-use crate::{
-    db::Connection,
-    schema::films,
-};
-use diesel::prelude::*;
+use crate::schema::films;
 use serde_derive::{Deserialize, Serialize};
 
 #[derive(Deserialize, Identifiable, PartialEq, Serialize, Queryable)]
@@ -36,15 +32,3 @@ pub struct NewFilm<'a> {
     pub title: &'a str,
     pub url: Option<&'a str>,
 }
-
-pub fn get_all(db: &Connection) -> QueryResult<Vec<Film>> {
-    use crate::schema::films::dsl::*;
-    films.load::<Film>(&**db)
-}
-
-pub fn by_title(ftitle: &str, db: &Connection) -> QueryResult<Film> {
-    use crate::schema::films::dsl::*;
-    films
-        .filter(title.eq(&ftitle))
-        .get_result::<Film>(&**db)
-}