Merge pull request 'Extract Post::ap_url()' (#918) from ap_url into main

Reviewed-on: #918

plume-models/src/posts.rs

@@ -73,12 +73,7 @@ impl Post {
             .execute(conn)?;
         let mut post = Self::last(conn)?;
         if post.ap_url.is_empty() {
-            post.ap_url = ap_url(&format!(
+            post.ap_url = Self::ap_url(post.get_blog(conn)?, &post.slug);
-                "{}/~/{}/{}/",
-                CONFIG.base_url,
-                post.get_blog(conn)?.fqn,
-                post.slug
-            ));
             let _: Post = post.save_changes(conn)?;
         }
 
@@ -254,6 +249,10 @@ impl Post {
             .map_err(Error::from)
     }
 
+    pub fn ap_url(blog: Blog, slug: &str) -> String {
+        ap_url(&format!("{}/~/{}/{}/", CONFIG.base_url, blog.fqn, slug))
+    }
+
     pub fn get_authors(&self, conn: &Connection) -> Result<Vec<User>> {
         use crate::schema::post_authors;
         use crate::schema::users;

src/routes/posts.rs

@@ -290,13 +290,7 @@ pub fn update(
             let newly_published = if !post.published && !form.draft {
                 post.published = true;
                 post.creation_date = Utc::now().naive_utc();
-                // TODO: share code with Post::insert()
+                post.ap_url = Post::ap_url(post.get_blog(&conn).unwrap(), &new_slug);
-                post.ap_url = plume_models::ap_url(&format!(
-                    "{}/~/{}/{}/",
-                    CONFIG.base_url,
-                    post.get_blog(&conn).unwrap().fqn,
-                    new_slug
-                ));
                 true
             } else {
                 false