From 491cb3eb34d20140aed80dbb9edc39c4db5335d2 Mon Sep 17 00:00:00 2001
From: Owen Jacobson <owen@grimoire.ca>
Date: Sun, 15 Sep 2024 23:50:41 -0400
Subject: Consolidate most repository types into a repo module.

Having them contained in the individual endpoint groups conveyed an unintended sense that their intended scope was _only_ that endpoint group. It also made most repo-related import paths _quite_ long. This splits up the repos as follows:

* "General applicability" repos - those that are only loosely connected to a single task, and are likely to be shared between tasks - go in crate::repo.

* Specialized repos - those tightly connected to a specific task - go in the module for that task, under crate::PATH::repo.

In both cases, each repo goes in its own submodule, to make it easier to use the module name as a namespace.

Which category a repo goes in is a judgment call. `crate::channel::repo::broadcast` (formerly `channel::repo::messages`) is used outside of `crate::channel`, for example, but its main purpose is to support channel message broadcasts. It could arguably live under `crate::event::repo::channel`, but the resulting namespace is less legible to me.
---
 src/login/repo/logins.rs | 136 -----------------------------------------------
 1 file changed, 136 deletions(-)
 delete mode 100644 src/login/repo/logins.rs

(limited to 'src/login/repo/logins.rs')

diff --git a/src/login/repo/logins.rs b/src/login/repo/logins.rs
deleted file mode 100644
index 11ae50f..0000000
--- a/src/login/repo/logins.rs
+++ /dev/null
@@ -1,136 +0,0 @@
-use sqlx::{sqlite::Sqlite, SqliteConnection, Transaction};
-
-use crate::id::Id as BaseId;
-use crate::login::app::StoredHash;
-
-pub trait Provider {
-    fn logins(&mut self) -> Logins;
-}
-
-impl<'c> Provider for Transaction<'c, Sqlite> {
-    fn logins(&mut self) -> Logins {
-        Logins(self)
-    }
-}
-
-pub struct Logins<'t>(&'t mut SqliteConnection);
-
-// This also implements FromRequestParts (see `src/login/extract/login.rs`). As
-// a result, it can be used as an extractor.
-#[derive(Clone, Debug, serde::Serialize)]
-pub struct Login {
-    pub id: Id,
-    pub name: String,
-    // The omission of the hashed password is deliberate, to minimize the
-    // chance that it ends up tangled up in debug output or in some other chunk
-    // of logic elsewhere.
-}
-
-impl<'c> Logins<'c> {
-    pub async fn create(
-        &mut self,
-        name: &str,
-        password_hash: &StoredHash,
-    ) -> Result<Login, sqlx::Error> {
-        let id = Id::generate();
-
-        let login = sqlx::query_as!(
-            Login,
-            r#"
-                insert or fail
-                into login (id, name, password_hash)
-                values ($1, $2, $3)
-                returning
-                    id as "id: Id",
-                    name
-            "#,
-            id,
-            name,
-            password_hash,
-        )
-        .fetch_one(&mut *self.0)
-        .await?;
-
-        Ok(login)
-    }
-
-    pub async fn by_id(&mut self, id: &Id) -> Result<Login, sqlx::Error> {
-        let login = sqlx::query_as!(
-            Login,
-            r#"
-                select
-                    id as "id: Id",
-                    name
-                from login
-                where id = $1
-            "#,
-            id,
-        )
-        .fetch_one(&mut *self.0)
-        .await?;
-
-        Ok(login)
-    }
-
-    /// Retrieves a login by name, plus its stored password hash for
-    /// verification. If there's no login with the requested name, this will
-    /// return [None].
-    pub async fn for_login(
-        &mut self,
-        name: &str,
-    ) -> Result<Option<(Login, StoredHash)>, sqlx::Error> {
-        let found = sqlx::query!(
-            r#"
-                select
-                    id as "id: Id",
-                    name,
-                    password_hash as "password_hash: StoredHash"
-                from login
-                where name = $1
-            "#,
-            name,
-        )
-        .map(|rec| {
-            (
-                Login {
-                    id: rec.id,
-                    name: rec.name,
-                },
-                rec.password_hash,
-            )
-        })
-        .fetch_optional(&mut *self.0)
-        .await?;
-
-        Ok(found)
-    }
-}
-
-impl<'t> From<&'t mut SqliteConnection> for Logins<'t> {
-    fn from(tx: &'t mut SqliteConnection) -> Self {
-        Self(tx)
-    }
-}
-
-/// Stable identifier for a [Login]. Prefixed with `L`.
-#[derive(Clone, Debug, Eq, PartialEq, sqlx::Type, serde::Serialize)]
-#[sqlx(transparent)]
-pub struct Id(BaseId);
-
-impl From<BaseId> for Id {
-    fn from(id: BaseId) -> Self {
-        Self(id)
-    }
-}
-
-impl Id {
-    pub fn generate() -> Self {
-        BaseId::generate("L")
-    }
-}
-
-impl std::fmt::Display for Id {
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        self.0.fmt(f)
-    }
-}
-- 
cgit v1.2.3