SummaryPage-Module: Bugfix, move countrylist.parse() back in try statement since we need to make sure it is parseable due to automatic year change feature
This commit is contained in:
@@ -102,7 +102,6 @@ class SummaryPageEntry():
|
|||||||
# Check if parsing country list is needed
|
# Check if parsing country list is needed
|
||||||
if( self.countrylist.is_parsing_needed( self.countrylist_revid )):
|
if( self.countrylist.is_parsing_needed( self.countrylist_revid )):
|
||||||
|
|
||||||
self.countrylist.parse()
|
|
||||||
self.correct_chartein()
|
self.correct_chartein()
|
||||||
|
|
||||||
self.update_params()
|
self.update_params()
|
||||||
@@ -135,12 +134,20 @@ class SummaryPageEntry():
|
|||||||
try:
|
try:
|
||||||
self.countrylist = CountryList( self.countrylist_wikilink )
|
self.countrylist = CountryList( self.countrylist_wikilink )
|
||||||
|
|
||||||
|
if( self.countrylist and \
|
||||||
|
self.countrylist.is_parsing_needed( self.countrylist_revid )):
|
||||||
|
self.countrylist.parse()
|
||||||
|
|
||||||
# Maybe fallback to last years list
|
# Maybe fallback to last years list
|
||||||
except CountryListError:
|
except CountryListError:
|
||||||
|
|
||||||
self.countrylist_wikilink.title = link_title
|
self.countrylist_wikilink.title = link_title
|
||||||
self.countrylist = CountryList( self.countrylist_wikilink )
|
self.countrylist = CountryList( self.countrylist_wikilink )
|
||||||
|
|
||||||
|
if( self.countrylist and \
|
||||||
|
self.countrylist.is_parsing_needed( self.countrylist_revid )):
|
||||||
|
self.countrylist.parse()
|
||||||
|
|
||||||
if not self.countrylist:
|
if not self.countrylist:
|
||||||
raise SummaryPageEntryError( "CountryList does not exists!" )
|
raise SummaryPageEntryError( "CountryList does not exists!" )
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user