style and changelog for #525

beets/ui/commands.py

@@ -1157,14 +1157,14 @@ def modify_items(lib, mods, query, write, move, album, confirm):
 
     # Still something to do?
     if not changed:
-        print_('No modification to perform.')
+        print_('No changes to make.')
         return
-    else:
-        # Confirm.
-        if confirm:
-            extra = ' and write tags' if write else ''
-            if not ui.input_yn('Really modify%s (Y/n)?' % extra):
-                return
+
+    # Confirm action.
+    if confirm:
+        extra = ' and write tags' if write else ''
+        if not ui.input_yn('Really modify%s (Y/n)?' % extra):
+            return
 
     # Apply changes to database.
     with lib.transaction():

docs/changelog.rst

@@ -65,6 +65,8 @@ Other little fixes:
 * :doc:`/plugins/convert`: Transcoding should now work on Windows.
 * :doc:`/plugins/duplicates`: The ``move`` and ``copy`` destination arguments
   are now treated as directories. Thanks to Pedro Silva.
+* The :ref:`modify-cmd` command now skips confirmation and prints a message if
+  no changes are necessary. Thanks to brilnius.
 
 
 1.3.2 (December 22, 2013)