Browse Source

Merge pull request #1811 from return42/fix-paper.html

[fix] doc of the paper.html template (isbn, issn)
Alexandre Flament 2 years ago
parent
commit
7097508a67
2 changed files with 5 additions and 4 deletions
  1. 2 2
      docs/dev/engine_overview.rst
  2. 3 2
      searx/engines/core.py

+ 2 - 2
docs/dev/engine_overview.rst

@@ -386,11 +386,11 @@ the parameter ``template`` must be set to the desired type.
      - DOI number (like ``10.1038/d41586-018-07848-2``)
 
    * - issn
-     - :py:class:`str`
+     - :py:class:`List <list>`\ [\ :py:class:`str`\ ]
      - ISSN number like ``1476-4687``
 
    * - isbn
-     - :py:class:`str`
+     - :py:class:`List <list>`\ [\ :py:class:`str`\ ]
      - ISBN number like ``9780201896831``
 
    * - pdf_url

+ 3 - 2
searx/engines/core.py

@@ -77,6 +77,7 @@ def response(resp):
         if url is None:
             continue
 
+        publishedDate = None
         time = source['publishedDate'] or source['depositedDate']
         if time:
             publishedDate = datetime.fromtimestamp(time / 1000)
@@ -106,8 +107,8 @@ def response(resp):
                 # 'pages' : '',
                 # 'number': '',
                 'doi': source['doi'],
-                'issn': source['issn'],
-                'isbn': source.get('isbn'),  # exists in the rawRecordXml
+                'issn': [x for x in [source.get('issn')] if x],
+                'isbn': [x for x in [source.get('isbn')] if x],  # exists in the rawRecordXml
                 'pdf_url': source.get('repositoryDocument', {}).get('pdfOrigin'),
             }
         )