From 6812cad59660d20a73a986577782c152a48432dd Mon Sep 17 00:00:00 2001 From: CounterPillow Date: Fri, 9 May 2014 16:58:47 +0200 Subject: [PATCH] Fix validator for empty lists (the default value) Whoops. --- overviewer_core/settingsValidators.py | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/overviewer_core/settingsValidators.py b/overviewer_core/settingsValidators.py index fd9f757..29553a7 100644 --- a/overviewer_core/settingsValidators.py +++ b/overviewer_core/settingsValidators.py @@ -167,17 +167,18 @@ def validateOptImg(optimizers): raise ValidationException("What you passed to optimizeimg is not a list. "\ "Make sure you specify them like [foo()], with square brackets.") - for opt, next_opt in zip(optimizers, optimizers[1:]) + [(optimizers[-1], None)]: - if not isinstance(opt, Optimizer): - raise ValidationException("Invalid Optimizer!") + if optimizers: + for opt, next_opt in zip(optimizers, optimizers[1:]) + [(optimizers[-1], None)]: + if not isinstance(opt, Optimizer): + raise ValidationException("Invalid Optimizer!") - opt.check_availability() + opt.check_availability() - # Check whether the chaining is somewhat sane - if next_opt: - if opt.is_crusher() and not next_opt.is_crusher(): - logging.warning("You're feeding a crushed output into an optimizer that does not crush. "\ - "This is most likely pointless, and wastes time.") + # Check whether the chaining is somewhat sane + if next_opt: + if opt.is_crusher() and not next_opt.is_crusher(): + logging.warning("You're feeding a crushed output into an optimizer that does not crush. "\ + "This is most likely pointless, and wastes time.") return optimizers