[nongnu] elpa/rubocop f5fd18a 64/64: Add the ability to format on save

[ELPA Syncer]

branch: elpa/rubocop
commit f5fd18aa810c3d3269188cbbd731ddc09006f8f5
Author: Bozhidar Batsov <bozhidar@batsov.com>
Commit: Bozhidar Batsov <bozhidar@batsov.com>

    Add the ability to format on save
    
    Basically, that's a subset of the autocorrect on save functionality.
---
 README.md  |  9 +++++++++
 rubocop.el | 23 ++++++++++++++++++++---
 2 files changed, 29 insertions(+), 3 deletions(-)

diff --git a/README.md b/README.md
index 1268dfe..78a7970 100644
--- a/README.md
+++ b/README.md
@@ -71,6 +71,15 @@ The variable `rubocop-autocorrect-on-save` controls whether 
to auto-correct auto
 (setq rubocop-autocorrect-on-save t)
 ```
 
+Alternatively you can enable only automatic code formatting on save 
(effectively that's a subset of
+the full auto-correct):
+
+``` emacs-lisp
+(setq rubocop-format-on-save t)
+```
+
+**Note:** Generally you shouldn't enable `rubocop-format-on-save` if 
`rubocop-autocorrect-on-save` is enabled.
+
 You can change the shell command used by `rubocop-check-*` commands via 
`rubocop-check-command`:
 
 ``` emacs-lisp
diff --git a/rubocop.el b/rubocop.el
index c7065a0..6353abd 100644
--- a/rubocop.el
+++ b/rubocop.el
@@ -82,6 +82,12 @@ It's basically auto-correction limited to layout cops."
   :type 'boolean
   :package-version '(rubocop . "0.6.0"))
 
+(defcustom rubocop-format-on-save nil
+  "Runs `rubocop-format-current-file' automatically on save."
+  :group 'rubocop
+  :type 'boolean
+  :package-version '(rubocop . "0.7.0"))
+
 (defcustom rubocop-prefer-system-executable nil
   "Runs rubocop with the system executable even if inside a bundled project."
   :group 'rubocop
@@ -232,6 +238,12 @@ See also `rubocop-autocorrect-on-save'."
   (interactive)
   (rubocop--file-command rubocop-format-command))
 
+(defun rubocop-format-current-file-silent ()
+  "This command is used by the minor mode to format on save.
+See also `rubocop-format-on-save' and `rubocop-autocorrect-on-save'."
+  (when (and rubocop-format-on-save (not rubocop-autocorrect-on-save))
+    (save-window-excursion (rubocop-format-current-file))))
+
 (defun rubocop-bundled-p ()
   "Check if RuboCop has been bundled."
   (let ((gemfile-lock (expand-file-name "Gemfile.lock" 
(rubocop-project-root))))
@@ -269,9 +281,14 @@ See also `rubocop-autocorrect-on-save'."
   :lighter " RuboCop"
   :keymap rubocop-mode-map
   :group 'rubocop
-  (cond
-   (rubocop-mode (add-hook 'before-save-hook 
'rubocop-autocorrect-current-file-silent nil t))
-   (t (remove-hook 'before-save-hook 'rubocop-autocorrect-current-file-silent 
t))))
+  (if rubocop-mode
+      ;; on mode enable
+      (progn
+        (add-hook 'before-save-hook 'rubocop-autocorrect-current-file-silent 
nil t)
+        (add-hook 'before-save-hook 'rubocop-format-current-file-silent nil t))
+    ;; on mode disable
+    (remove-hook 'before-save-hook 'rubocop-autocorrect-current-file-silent t)
+    (remove-hook 'before-save-hook 'rubocop-format-current-file-silent t)))
 
 (provide 'rubocop)