Fix: consistency and other minor things

deepwell/src/api.rs

@@ -254,7 +254,7 @@ async fn build_module(app_state: ServerState) -> anyhow::Result<RpcModule<Server
     register!("parent_remove", parent_remove);
     register!("parent_relationships_get", parent_relationships_get);
     register!("parent_get_all", parent_get_all);
-    register!("parent_modify", parent_modify);
+    register!("parent_update", parent_update);
 
     // Blob data
     register!("blob_get", blob_get);

deepwell/src/endpoints/page.rs

@@ -84,8 +84,8 @@ pub async fn page_get_score(
 
     info!("Getting score for page {reference:?} in site ID {site_id}");
     let page_id = PageService::get_id(ctx, site_id, reference).await?;
-    let rating = ScoreService::score(ctx, page_id).await?;
-    Ok(GetPageScoreOutput { page_id, rating })
+    let score = ScoreService::score(ctx, page_id).await?;
+    Ok(GetPageScoreOutput { page_id, score })
 }
 
 pub async fn page_edit(

deepwell/src/endpoints/parent.rs

@@ -22,8 +22,8 @@ use super::prelude::*;
 use crate::models::page_parent::Model as PageParentModel;
 use crate::services::page::GetPageReference;
 use crate::services::parent::{
-    GetParentRelationships, ModifyParentOutput, ParentDescription,
-    ParentModifyDescription, RemoveParentOutput,
+    GetParentRelationships, ParentDescription, RemoveParentOutput, UpdateParents,
+    UpdateParentsOutput,
 };
 use crate::web::Reference;
 
@@ -108,21 +108,21 @@ pub async fn parent_get_all(
 
     let pages: Vec<String> = PageService::get_pages(ctx, site_id, parents.as_slice())
         .await?
-        .iter()
-        .map(|p| p.slug.clone())
+        .into_iter()
+        .map(|p| p.slug)
         .collect();
 
     Ok(pages)
 }
 
-pub async fn parent_modify(
+pub async fn parent_update(
     ctx: &ServiceContext<'_>,
     params: Params<'static>,
-) -> Result<ModifyParentOutput> {
-    let input: ParentModifyDescription = params.parse()?;
+) -> Result<UpdateParentsOutput> {
+    let input: UpdateParents = params.parse()?;
 
     info!(
-        "Modifying multiple parental relationship for child {:?} in site ID {}",
+        "Updating multiple parental relationships for child {:?} in site ID {}",
         input.child, input.site_id,
     );
 
@@ -134,7 +134,7 @@ pub async fn parent_modify(
                     ctx,
                     ParentDescription {
                         site_id: input.site_id,
-                        parent: parent.clone(),
+                        parent: parent.to_owned(),
                         child: input.child.clone(),
                     },
                 )
@@ -170,7 +170,7 @@ pub async fn parent_modify(
         None => None,
     };
 
-    Ok(ModifyParentOutput {
+    Ok(UpdateParentsOutput {
         added: creation,
         removed: removal,
     })

deepwell/src/services/page/structs.rs

@@ -116,7 +116,7 @@ pub struct GetPageOutput {
 #[derive(Serialize, Debug, Clone)]
 pub struct GetPageScoreOutput {
     pub page_id: i64,
-    pub rating: ScoreValue,
+    pub score: ScoreValue,
 }
 
 #[derive(Deserialize, Debug, Clone)]

deepwell/src/services/parent/structs.rs

@@ -30,7 +30,7 @@ pub struct ParentDescription<'a> {
 }
 
 #[derive(Deserialize, Debug, Clone)]
-pub struct ParentModifyDescription<'a> {
+pub struct UpdateParents<'a> {
     pub site_id: i64,
     pub child: Reference<'a>,
     pub add: Option<Vec<Reference<'a>>>,
@@ -80,7 +80,7 @@ pub struct RemoveParentOutput {
 }
 
 #[derive(Serialize, Debug, Clone)]
-pub struct ModifyParentOutput {
+pub struct UpdateParentsOutput {
     pub added: Option<Vec<i64>>,
     pub removed: Option<Vec<bool>>,
 }

framerail/src/lib/server/deepwell/page.ts

@@ -166,14 +166,14 @@ export async function pageLayout(
   })
 }
 
-export async function pageParentModify(
+export async function pageParentUpdate(
   siteId: number,
   pageId: number,
   userId: number,
   add: string[],
   remove: string[]
 ): Promise<object> {
-  return client.request("parent_modify", {
+  return client.request("parent_update", {
     site_id: siteId,
     child: pageId,
     user_id: userId,

framerail/src/routes/[slug]/[...extra]/+server.ts

@@ -99,7 +99,7 @@ export async function POST(event) {
       let layout = data.get("layout")?.toString().trim() ?? null
 
       res = await page.pageLayout(siteId, pageId, session?.user_id, layout)
-    } else if (extra.includes("parentset")) {
+    } else if (extra.includes("parent-set")) {
       let oldParentStr = data.get("old-parents")?.toString().trim() ?? ""
       let oldParent = oldParentStr.split(" ")
       let newParentStr = data.get("new-parents")?.toString().trim() ?? ""
@@ -116,7 +116,7 @@ export async function POST(event) {
       }
 
       res = await page.pageParentModify(siteId, pageId, session?.user_id, added, removed)
-    } else if (extra.includes("parentget")) {
+    } else if (extra.includes("parent-get")) {
       res = await page.pageParentGet(siteId, pageId, slug)
     } else if (extra.includes("score")) {
       res = await page.pageScore(siteId, pageId, slug)

framerail/src/routes/[slug]/[...extra]/page.svelte

@@ -130,7 +130,7 @@
     let fdata = new FormData()
     fdata.set("site-id", $page.data.site.site_id)
     fdata.set("page-id", $page.data.page.page_id)
-    let res = await fetch(`/${$page.data.page.slug}/parentget`, {
+    let res = await fetch(`/${$page.data.page.slug}/parent-get`, {
       method: "POST",
       body: fdata
     }).then((res) => res.json())
@@ -151,7 +151,7 @@
     fdata.set("site-id", $page.data.site.site_id)
     fdata.set("page-id", $page.data.page.page_id)
     fdata.set("old-parents", parents)
-    let res = await fetch(`/${$page.data.page.slug}/parentset`, {
+    let res = await fetch(`/${$page.data.page.slug}/parent-set`, {
       method: "POST",
       body: fdata
     }).then((res) => res.json())
@@ -262,7 +262,7 @@
         message: res.message
       })
     } else {
-      voteRating = res.rating ?? 0
+      voteRating = res.score ?? 0
       showVote = true
     }
   }