# -*- coding: utf-8 -*-
"""
The matcher subsystem needs a function called 'confirm_top', which
takes the data passed to a top file environment and determines if that
data matches this minion.
"""
from __future__ import absolute_import

import logging

import salt.loader

log = logging.getLogger(__file__)


def confirm_top(match, data, nodegroups=None):
    """
    Takes the data passed to a top file environment and determines if the
    data matches this minion
    """
    matcher = 'compound'
    if not isinstance(data, list):
        log.error('Received bad data when setting the match from the top '
                  'file')
        return False
    for item in data:
        if isinstance(item, dict):
            if "match" in item:
                matcher = item["match"]

    matchers = salt.loader.matchers(__opts__)
    funcname = matcher + "_match.match"
    if matcher == "nodegroup":
        return matchers[funcname](match, nodegroups)
    else:
        m = matchers[funcname]
        return m(match)
    # except TypeError, KeyError:
    #     log.error('Attempting to match with unknown matcher: %s', matcher)

@@ -507,30 +507,12 @@
 not 
-isinstance(data, list)
+data
 :%0A  

Revert "Check the type of the data passed to salt.matcher.confirm_top is a list instead of assuming an implicit false is a "bad data" error."