Clean up title setting logic.
[pelican-mode.git] / pelican-mode.el
index ddc21ea..15db6cf 100644 (file)
@@ -1,7 +1,24 @@
-;;; pelican-mode.el --- Minor mode for editing pages and posts in Pelican sites
+;;; pelican-mode.el --- Minor mode for editing Pelican sites -*- lexical-binding: t -*-
 ;;
-;; Author: Joe Wreschnig
-;; This code is released into the public domain.
+;; Copyright 2013-2017 Joe Wreschnig
+;;
+;; Author: Joe Wreschnig <joe.wreschnig@gmail.com>
+;; Package-Version: 20170618
+;; Package-Requires: ((emacs "25"))
+;; Keywords: convenience, editing
+;;
+;; This program is free software; you can redistribute it and/or modify
+;; it under the terms of the GNU General Public License as published by
+;; the Free Software Foundation, either version 3 of the License, or
+;; (at your option) any later version.
+;;
+;; This program is distributed in the hope that it will be useful,
+;; but WITHOUT ANY WARRANTY; without even the implied warranty of
+;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;; GNU General Public License for more details.
+;;
+;; You should have received a copy of the GNU General Public License
+;; along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 ;;; Commentary:
 ;;
 ;; tested it on networked drives and the lookup for pelicanconf.py
 ;; might slow it down considerably.
 
+
 ;;; Code:
 
-(defun pelican-timestamp-now ()
-  "Generate a Pelican-compatible timestamp."
-  (format-time-string "%Y-%m-%d %H:%M"))
+(require 'subr-x)
+
+(defun pelican-timestamp (&optional time)
+  "Generate a Pelican-compatible timestamp for TIME."
+  (format-time-string "%Y-%m-%d %H:%M" time))
 
 (defun pelican-is-markdown ()
   "Check if the buffer is likely using Markdown."
-  (eq major-mode 'markdown-mode))
+  (derived-mode-p 'markdown-mode))
 
 (defun pelican-field (name value)
-  "Helper to format a field NAME and VALUE."
-  (if value (format "%s: %s\n" name value) ""))
-
-(defun pelican-markdown-header (title date status category tags slug)
-  "Generate a Pelican Markdown header.
-
-All parameters but TITLE may be nil to omit them. DATE may be a
-string or 't to use the current date and time."
-  (let ((title (format "Title: %s\n" title))
-        (status (pelican-field "Status" status))
-        (category (pelican-field "Category" category))
-        (tags (pelican-field "Tags" tags))
-        (slug (pelican-field "Slug" slug))
-        (date (if date (format "Date: %s\n"
-                               (if (stringp date) date
-                                 (pelican-timestamp-now)))
-                "")))
-    (concat title date status tags category slug "\n")))
-
-(defun pelican-rst-header (title date status category tags slug)
-  "Generate a Pelican reStructuredText header.
-
-All parameters but TITLE may be nil to omit them. DATE may be a
-string or 't to use the current date and time."
-  (let ((title (format "%s\n%s\n\n" title
-                       (make-string (string-width title) ?#)))
-        (status (pelican-field ":status" status))
-        (category (pelican-field ":category" category))
-        (tags (pelican-field ":tags" tags))
-        (slug (pelican-field ":slug" slug))
-        (date (if date (format ":date: %s\n"
-                               (if (stringp date) date
-                                 (pelican-timestamp-now)))
-                "")))
-    (concat title date status tags category slug "\n")))
+  "Format a line for a field NAME with a VALUE."
+  (if value
+      (cond ((derived-mode-p 'markdown-mode)
+             (format "%s: %s\n" (capitalize name) value))
+            ((derived-mode-p 'rst-mode)
+             (format ":%s: %s\n" (downcase name) value))
+            (t (error "Unsupported major mode %S" major-mode)))
+    ""))
+
+(defun pelican-rst-title (title)
+  "Create a ReSt version of TITLE."
+  (concat title "\n" (make-string (string-width title) ?#) "\n\n"))
+
+(defun pelican-title (title)
+  "Format a TITLE for the current document, according to major mode."
+  (cond ((derived-mode-p 'markdown-mode)
+         (pelican-field "title" title))
+        ((derived-mode-p 'rst-mode)
+         (pelican-rst-title title))
+        (t (error "Unsupported major mode %S" major-mode))))
+
+(defun pelican-header (title date status category tags slug)
+  "Create a Pelican header."
+  ;; TODO: Use a property list (-> alist via seq-partition) instead.
+  (when (eq date t)
+    (setq date (pelican-timestamp)))
+  
+  (concat (pelican-title title)
+          (pelican-field "date" date)
+          (pelican-field "status" status)
+          (pelican-field "tags" tags)
+          (pelican-field "category" category)
+          (pelican-field "slug" slug)
+          "\n"))
 
 (defun pelican-insert-draft-post-header (title tags)
   "Insert a Pelican header for a draft post."
   (interactive "sPost title: \nsTags: ")
-  (let ((slug (pelican-default-slug))
-        (header (if (pelican-is-markdown)
-                    'pelican-markdown-header 'pelican-rst-header)))
+  (let ((slug (pelican-default-slug)))
     (save-excursion
       (goto-char 0)
-      (insert (funcall header title 't "draft" nil tags slug)))))
+      (insert (pelican-header title 't "draft" nil tags slug)))))
 
 (defun pelican-insert-page-header (title hidden)
   "Insert a Pelican header for a page."
@@ -72,12 +89,10 @@ string or 't to use the current date and time."
    (list (read-string "Page title: ")
          (y-or-n-p "Hidden? ")))
   (let ((slug (pelican-default-slug))
-        (hidden (if hidden "hidden" nil))
-        (header (if (pelican-is-markdown)
-                    'pelican-markdown-header 'pelican-rst-header)))
+        (hidden (if hidden "hidden" nil)))
     (save-excursion
       (goto-char 0)
-      (insert (funcall header title nil hidden nil nil slug)))))
+      (insert (pelican-header title nil hidden nil nil slug)))))
 
 (defun pelican-insert-header ()
   "Insert a Pelican header for a page or post."
@@ -86,54 +101,58 @@ string or 't to use the current date and time."
                           'pelican-insert-page-header
                         'pelican-insert-draft-post-header)))
 
+(defun pelican-set-field (field value)
+  "Set FIELD to VALUE."
+  (save-excursion
+    (goto-char 0)
+    (if (re-search-forward (concat "^" (pelican-field field ".+*")) nil t)
+        (replace-match (pelican-field field value))
+      (re-search-forward "^$")
+      (replace-match (pelican-field field value)))))
+
+(defun pelican-set-title (title)
+  "Set the title to TITLE."
+  (if (pelican-is-markdown)
+      (pelican-set-field "title" title)
+    (save-excursion
+      (goto-char 0)
+      (let ((header (pelican-rst-title title)))
+        (if (looking-at ".*\n#+\n+")
+            (replace-match header)
+          (insert header))))))
+
 (defun pelican-update-date ()
   "Update a Pelican date header."
   (interactive)
-  (save-excursion
-    (goto-char 0)
-    (let* ((field (if (pelican-is-markdown) "Date" ":date"))
-           (re (format "^%s: [-0-9 :]+\n" field))
-           (date (pelican-timestamp-now)))
-      (if (re-search-forward re nil t)
-          (replace-match (format "%s: %s\n" field date))
-        (message "This doesn't look like a Pelican page.")))))
+  (pelican-set-field "date" (pelican-timestamp)))
 
 (defun pelican-publish-draft ()
   "Remove draft status from a Pelican post."
   (interactive)
-  (save-excursion
-    (goto-char 0)
-    (let* ((field (if (pelican-is-markdown) "Status" ":status"))
-           (re (format "^%s: draft\n" field)))
-      (if (re-search-forward re nil t)
-          (progn
-            (replace-match (format ""))
-            (pelican-update-date))
-        (message "This doesn't look like a Pelican draft.")))))
+  (pelican-set-field "status" nil)
+  (pelican-update-date))
 
 (defun pelican-is-page ()
   "Guess the current buffer is a Pelican page (vs. a post or neither)."
-  (let ((pelican-base (pelican-find-root)))
-    (if pelican-base
-        (let* ((relative (file-relative-name buffer-file-name pelican-base))
-               (components (split-string relative "/")))
-          (string= "pages" (car (cdr components)))))))
+  (when-let (pelican-base (pelican-find-root))
+    (let* ((relative (file-relative-name buffer-file-name pelican-base))
+           (components (split-string relative "/")))
+      (equal "pages" (cadr components)))))
 
 (defun pelican-default-slug ()
   "Generate a Pelican post/page slug for the current buffer."
-  (let ((pelican-base (pelican-find-root))
-        (file-name (file-name-sans-extension buffer-file-name)))
-    (if pelican-base
-        (let* ((relative (file-relative-name file-name pelican-base))
-               (components (cdr (split-string relative "/")))
-               (components (if (string= "pages" (car components))
-                               (cdr components) components)))
-          (mapconcat 'identity components "/"))
-      (format "%s/%s"
-              (file-name-nondirectory
-               (directory-file-name
-                (file-name-directory file-name)))
-              (file-name-base file-name)))))
+  (if-let ((pelican-base (pelican-find-root))
+           (file-name (file-name-sans-extension buffer-file-name)))
+      (let* ((relative (file-relative-name file-name pelican-base))
+             (components (cdr (split-string relative "/")))
+             (components (if (string= "pages" (car components))
+                             (cdr components) components)))
+        (mapconcat 'identity components "/"))
+    (format "%s/%s"
+            (file-name-nondirectory
+             (directory-file-name
+              (file-name-directory file-name)))
+            (file-name-base file-name))))
 
 (defun pelican-find-in-parents (file-name)
   "Find FILE-NAME in the default directory or one of its parents, or nil."
@@ -146,29 +165,28 @@ string or 't to use the current date and time."
 
 (defun pelican-find-root ()
   "Return the root of the buffer's Pelican site, or nil."
-  (let ((conf (pelican-find-in-parents "pelicanconf.py")))
-    (if conf (file-name-directory conf))))
+  (when-let (conf (pelican-find-in-parents "pelicanconf.py"))
+    (file-name-directory conf)))
 
 (defun pelican-is-in-site ()
   "Check if this buffer is under a Pelican site."
-  (not (not (pelican-find-root))))
+  (not (null (pelican-find-root))))
 
 (defun pelican-enable-if-site ()
   "Enable `pelican-mode' if this buffer is under a Pelican site."
-  (if (pelican-is-in-site)
-      (pelican-mode 1)))
+  (when (pelican-is-in-site)
+    (pelican-mode 1)))
 
 (defun pelican-make (target)
   "Execute TARGET in a Makefile at the root of the site."
   (interactive "sMake Pelican target: ")
-  (let ((default-directory (pelican-find-root)))
-    (if default-directory
-        (let ((output (get-buffer-create "*Pelican Output*")))
-          (display-buffer output)
-          (pop-to-buffer output)
-          (compilation-mode)
-          (start-process "Pelican Makefile" output "make" target))
-      (message "This doesn't look like a Pelican site."))))
+  (if-let ((default-directory (pelican-find-root)))
+      (let ((output (get-buffer-create "*Pelican Output*")))
+        (display-buffer output)
+        (pop-to-buffer output)
+        (compilation-mode)
+        (start-process "Pelican Makefile" output "make" target))
+    (message "This doesn't look like a Pelican site.")))
 
 (defun pelican-make-html ()
   "Generate HTML via a Makefile at the root of the site."
@@ -182,32 +200,34 @@ string or 't to use the current date and time."
 
 (defconst pelican-keymap (make-sparse-keymap)
   "The default keymap used in Pelican mode.")
-(define-key pelican-keymap [?\C-x ?p ?n]
+(define-key pelican-keymap (kbd "C-c P n")
   'pelican-insert-header)
-(define-key pelican-keymap [?\C-x ?p ?p]
+(define-key pelican-keymap (kbd "C-c P p")
   'pelican-publish-draft)
-(define-key pelican-keymap [?\C-x ?p ?t]
+(define-key pelican-keymap (kbd "C-c P t")
   'pelican-update-date)
-(define-key pelican-keymap [?\C-x ?p ?h]
+(define-key pelican-keymap (kbd "C-c P h")
   'pelican-make-html)
-(define-key pelican-keymap [?\C-x ?p ?u]
+(define-key pelican-keymap (kbd "C-c P u")
   'pelican-make-rsync-upload)
 
+
+;;;###autoload
 (define-minor-mode pelican-mode
   "Toggle Pelican mode.
 
 Interactively with no argument, this command toggles the mode.
-to show buffer size and position in mode-line.
-"
+to show buffer size and position in mode-line."
   :init-value nil
   :lighter " Pelican"
   :keymap pelican-keymap
-  :group 'pelican
-  )
+  :group 'pelican)
 
+;;;###autoload
 (add-hook 'markdown-mode-hook 'pelican-enable-if-site)
+
+;;;###autoload
 (add-hook 'rst-mode-hook 'pelican-enable-if-site)
 
 (provide 'pelican-mode)
-
 ;;; pelican-mode.el ends here