Make sure only flush db if there are redfams
To prevent from doing unnecessary stuff and trying to use not existing db connection Related Task: [https://fs.golderweb.de/index.php?do=details&task_id=75 FS#75] Related Task: [https://fs.golderweb.de/index.php?do=details&task_id=75 FS#75]
This commit is contained in:
@@ -102,18 +102,31 @@ class DiscussionParserBot(
|
||||
# Check whether parsing is needed
|
||||
if red_page.is_parsing_needed():
|
||||
|
||||
# Count families for failure analysis
|
||||
fam_counter = 0
|
||||
|
||||
# Iterate over returned generator with redfam sections
|
||||
for fam in red_page.parse():
|
||||
|
||||
# Run RedFamParser on section text
|
||||
redfam.RedFamParser.parser( fam, red_page.page._pageid,
|
||||
red_page.is_archive() )
|
||||
|
||||
fam_counter += 1
|
||||
|
||||
else:
|
||||
# If successfully parsed whole page, flush
|
||||
# db write cache
|
||||
if( fam_counter ):
|
||||
redfam.RedFamParser.flush_db_cache()
|
||||
jogobot.output( "Page [[{reddisc}]] parsed".format(
|
||||
reddisc=red_page.page.title() ) )
|
||||
else:
|
||||
jogobot.output(
|
||||
"\03{red} Page [[{reddisc}]], ".format(
|
||||
reddisc=red_page.page.title() ) +
|
||||
"containing no redfam, parsed!",
|
||||
"WARNING" )
|
||||
|
||||
|
||||
def apply_conf_cat_generators( genFactory ):
|
||||
|
||||
Reference in New Issue
Block a user