Fix for empty html-description-class

reuteras · Aug 13, 2023 · c2443c4 · c2443c4
1 parent afc01f8
commit c2443c4
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/README.md b/README.md
@@ -219,7 +219,7 @@ rssfixer --title "Nucleus CISA KEV" --output nucleus.xml  --html --filter-type d
 
 # NCSC-SE
 # https://www.ncsc.se/publikationer/
-rssfixer --html --filter-type div --filter-name 'page-container' --html-entries div --html-entries-class "news-text" --html-title h2 --html-title-class "" --html-description p --html-url a --base-url https://www.ncsc.se --stdout  --atom --title "Feed for NCSC-SE" https://www.ncsc.se/publikationer/
+rssfixer --html --filter-type div --filter-name 'page-container' --html-entries div --html-entries-class "news-text" --html-title h2 --html-title-class "" --html-description p --html-description-class "" --html-url a --base-url https://www.ncsc.se --stdout  --atom --title "Feed for NCSC-SE" https://www.ncsc.se/publikationer/
 ```
 
 If you have other example use case please add them in [show usage examples][sue] in discussions.

diff --git a/src/rssfixer/rss.py b/src/rssfixer/rss.py
@@ -151,10 +151,13 @@ def extract_links_html(soup, arguments):
             # Continue if URL or title is not found to find other entries
             continue
         try:
-            description = entry.find(
-                arguments.html_description,
-                re.compile(arguments.html_description_class),
-            ).text.strip()
+            if not arguments.html_description_class:
+                description = entry.find(arguments.html_description).text.strip()
+            else:
+                description = entry.find(
+                    arguments.html_description,
+                    re.compile(arguments.html_description_class),
+                ).text.strip()
         except (KeyError, AttributeError):
             # Ignore description if it's not found
             description = ""