Check for documents in all main commands

papis/commands/addto.py

@@ -28,6 +28,7 @@
 import logging
 import papis.cli
 import click
+import papis.strings
 
 
 def run(document, filepaths):
@@ -100,6 +101,10 @@ def run(document, filepaths):
 def cli(query, files, file_name):
     """Add files to an existing document"""
     documents = papis.database.get().query(query)
+    logger = logging.getLogger('cli:addto')
+    if not documents:
+        logger.warning(papis.strings.no_documents_retrieved_message)
+        return
     document = papis.api.pick_doc(documents)
     if not document:
         return status.file_not_found

papis/commands/browse.py

@@ -22,6 +22,8 @@
 import papis.api
 import click
 import papis.database
+import papis.strings
+import logging
 
 
 def run(document):
@@ -41,6 +43,12 @@ def run(document):
 def cli(query, key):
     """Open document's url in a browser"""
     documents = papis.database.get().query(query)
+    logger = logging.getLogger('cli:browse')
+
+    if len(documents) == 0:
+        logger.warning(papis.strings.no_documents_retrieved_message)
+        return 0
+
     document = papis.api.pick_doc(documents)
     if len(key):
         papis.config.set('browse-key', key)

papis/commands/edit.py

@@ -16,6 +16,7 @@
 import papis.cli
 import click
 import logging
+import papis.strings
 
 
 def run(document, wait=True):
@@ -66,6 +67,7 @@ def cli(
         documents = [document] if document else []
 
     if len(documents) == 0:
+        logger.warning(papis.strings.no_documents_retrieved_message)
         return 0
 
     for document in documents:

papis/commands/explore.py

@@ -92,6 +92,7 @@
 from papis.document import from_folder
 import papis.config
 import papis.bibtex
+import papis.strings
 import papis.cli
 import click
 import logging
@@ -457,6 +458,10 @@ def citations(ctx, query, doc_folder, max_citations, save, rmfile):
             search=query
         )
 
+    if not documents:
+        logger.warning(papis.strings.no_documents_retrieved_message)
+        return
+
     doc = papis.api.pick_doc(documents)
     db = papis.database.get()
     citations_file = os.path.join(doc.get_main_folder(), 'citations.yaml')

papis/commands/export.py

@@ -54,6 +54,7 @@
 import papis.cli
 import papis.api
 import papis.database
+import papis.strings
 import logging
 
 
@@ -181,6 +182,10 @@ def cli(
     if json and folder or yaml and folder:
         logger.warning("Only --folder flag will be considered")
 
+    if not documents:
+        logger.warning(papis.strings.no_documents_retrieved_message)
+        return
+
     if not all:
         document = papis.api.pick_doc(documents)
         if not document:

papis/commands/list.py

@@ -71,6 +71,7 @@
 from papis.api import status
 import os
 import papis.utils
+import papis.strings
 import papis.config
 import papis.database
 import papis.downloaders.utils
@@ -100,6 +101,7 @@ def run(
     if library is None:
         library = papis.config.get_lib()
     config = papis.config.get_configuration()
+    logger = logging.getLogger('cli:list')
     db = papis.database.get(library)
     if template is not None:
         if not os.path.exists(template):
@@ -122,6 +124,8 @@ def run(
         ]
 
     documents = db.query(query)
+    if not documents:
+        logger.warning(papis.strings.no_documents_retrieved_message)
 
     if pick:
         documents = filter(lambda x: x, [papis.api.pick_doc(documents)])

papis/commands/mv.py

@@ -15,6 +15,7 @@
 import logging
 import papis.cli
 import papis.api
+import papis.strings
 import click
 
 
@@ -61,6 +62,9 @@ def cli(query, git):
     logger = logging.getLogger('cli:mv')
 
     documents = papis.database.get().query(query)
+    if not documents:
+        logger.warning(papis.strings.no_documents_retrieved_message)
+        return
 
     document = papis.api.pick_doc(documents)
     if not document:

papis/commands/open.py

@@ -80,6 +80,7 @@
 import click
 import logging
 from papis.document import from_folder
+import papis.strings
 
 
 def run(document, opener=None, folder=False, mark=False):
@@ -170,8 +171,8 @@ def cli(query, doc_folder, tool, folder, all, mark):
         documents = [from_folder(doc_folder)]
 
     if not documents:
-        click.echo("No documents found with that name.")
-        return 1
+        logger.warning(papis.strings.no_documents_retrieved_message)
+        return 0
 
     if not all:
         documents = [papis.api.pick_doc(documents)]

papis/commands/rename.py

@@ -14,6 +14,7 @@
 import click
 import papis.cli
 import papis.database
+import papis.strings
 
 
 def run(document, new_name, git=False):
@@ -52,6 +53,10 @@ def cli(query, git):
     """Rename entry"""
 
     documents = papis.database.get().query(query)
+    logger = logging.getLogger('cli:rename')
+
+    if not documents:
+        logger.warning(papis.strings.no_documents_retrieved_message)
     document = papis.api.pick_doc(documents)
     if not document:
         return 0

papis/commands/rm.py

@@ -12,8 +12,10 @@
 import papis.config
 import papis.document
 import papis.cli
+import papis.strings
 import papis.database
 import click
+import logging
 
 
 def run(
@@ -57,6 +59,12 @@ def cli(
         ):
     """Delete command for several objects"""
     documents = papis.database.get().query(query)
+    logger = logging.getLogger('cli:rm')
+
+    if not documents:
+        logger.warning(papis.strings.no_documents_retrieved_message)
+        return 0
+
     document = papis.api.pick_doc(documents)
     if not document:
         return status.file_not_found
@@ -80,6 +88,7 @@ def cli(
             toolbar = 'The folder {0} would be removed'.format(
                 document.get_main_folder()
             )
+            logger.warning("This document will be removed, check it")
             papis.utils.text_area(
                 title=toolbar, text=document.dump(), lexer_name='yaml'
             )

papis/commands/update.py

@@ -43,6 +43,7 @@
 import urllib.error
 import logging
 import papis.utils
+import papis.strings
 import papis.bibtex
 import papis.downloaders.utils
 import papis.document
@@ -166,12 +167,17 @@ def cli(
     documents = papis.database.get().query(query)
     data = dict()
     logger = logging.getLogger('cli:update')
+    if not documents:
+        logger.warning(papis.strings.no_documents_retrieved_message)
 
     if doc_folder:
         documents = [from_folder(doc_folder)]
 
     if not all:
-        documents = [papis.api.pick_doc(documents)]
+        documents = filter(
+            lambda d: d,
+            [papis.api.pick_doc(documents)]
+        )
 
     for document in documents:
         if all: