beetbox#1676 Add a more descriptive error message

A useful error message if we get an OperationalError from SQLite that indicates a permissions problem on write.

Author: joelstanner

beets/dbcore/db.py

@@ -668,8 +668,17 @@ def mutate(self, statement, subvals=()):
         """Execute an SQL statement with substitution values and return
         the row ID of the last affected row.
         """
-        cursor = self.db._connection().execute(statement, subvals)
-        return cursor.lastrowid
+        try:
+            cursor = self.db._connection().execute(statement, subvals)
+            return cursor.lastrowid
+        except sqlite3.OperationalError as exe:
+            if exe.message == "unable to open database file":
+                print("ERROR: ", exe)
+                print(
+                    "Sorry, but there is a problem opening your database file."
+                    " Make sure the directory containing your database file "
+                    "has write permissions.")
+            raise
 
     def script(self, statements):
         """Execute a string containing multiple SQL statements."""

docs/changelog.rst

@@ -1,6 +1,9 @@
 Changelog
 =========
 
+* Give some helpful extra info in the event of a
+  "sqlite3.OperationalError: unable to open database file". :bug:`1676`
+
 1.3.17 (in development)
 -----------------------