grishka/lemmy
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Removing some TODOS.

Browse source
This commit is contained in:
Dessalines 2021-03-15 18:18:50 -04:00
parent b9f483bc27
commit 270ce539bf
2 changed files with 0 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
crates/api/src/lib.rs
View file

@ -72,15 +72,6 @@ pub(crate) async fn is_mod_or_admin(
Ok(()) Ok(())
} }
// TODO this probably isn't necessary anymore
// pub async fn is_admin(pool: &DbPool, person_id: i32) -> Result<(), LemmyError> {
// let user = blocking(pool, move |conn| LocalUser::read(conn, person_id)).await??;
// if !user.admin {
// return Err(ApiError::err("not_an_admin").into());
// }
// Ok(())
// }
pub fn is_admin(local_user_view: &LocalUserView) -> Result<(), LemmyError> { pub fn is_admin(local_user_view: &LocalUserView) -> Result<(), LemmyError> {
if !local_user_view.local_user.admin { if !local_user_view.local_user.admin {
return Err(ApiError::err("not_an_admin").into()); return Err(ApiError::err("not_an_admin").into());

2
crates/api_structs/src/lib.rs
View file

@ -68,7 +68,6 @@ pub async fn send_local_notifs(
Ok(ids) Ok(ids)
} }
// TODO should this really use person_ids as recipient ids? or local_user_ids ?
fn do_send_local_notifs( fn do_send_local_notifs(
conn: &PgConnection, conn: &PgConnection,
mentions: &[MentionData], mentions: &[MentionData],
@ -85,7 +84,6 @@ fn do_send_local_notifs(
.filter(|m| m.is_local() && m.name.ne(&person.name)) .filter(|m| m.is_local() && m.name.ne(&person.name))
.collect::<Vec<&MentionData>>() .collect::<Vec<&MentionData>>()
{ {
// TODO do a local user fetch
if let Ok(mention_user_view) = LocalUserView::read_from_name(&conn, &mention.name) { if let Ok(mention_user_view) = LocalUserView::read_from_name(&conn, &mention.name) {
// TODO // TODO
// At some point, make it so you can't tag the parent creator either // At some point, make it so you can't tag the parent creator either