Fixed errors newly exposed by pylint 2.4.0
    
    Newest Pylint introduced additional checks [1]:
    
    - import-outside-toplevel [2]
    
    > This check warns when modules are imported from places other
    than a module toplevel, e.g. inside a function or a class.
    
    - no-else-continue [3]
    
    > These checks highlight unnecessary else and elif blocks after
    break and continue statements.
    
    - unnecessary-comprehension [4]
    
    > This check is emitted when pylint finds list-, set- or
    dict-comprehensions, that are unnecessary and can be rewritten
    with the list-, set- or dict-constructors.
    
    [1] https://github.com/PyCQA/pylint/blob/pylint-2.4.0/doc/whatsnew/2.4.rst
    [2] https://github.com/PyCQA/pylint/issues/3067
    [3] https://github.com/PyCQA/pylint/issues/2327
    [4] https://github.com/PyCQA/pylint/issues/2905
    
    Fixes: https://pagure.io/freeipa/issue/8077
    Signed-off-by: Stanislav Levin <slev@altlinux.org>
    Reviewed-By: Christian Heimes <cheimes@redhat.com>