Fix: 'OrderedDict' object has no attribute 'startswith'

[remichristiaan]

Excluding an id when another state has a requisite depending on e.g.
a file that would be created by that excluded id/state. An exception is raised.

What does this PR do?

When excluding an id when another state has a requisite depending on e.g. a file that would be created by that excluded id/state. An exception is raised which does not give any hints what's wrong.

This fix will allow the (high)state run to finish and show the failed id(s) in the result.

What issues does this PR fix or reference?

Fixes: #57999

Previous Behavior

AttributeError: 'OrderedDict' object has no attribute 'startswith'

New Behavior

<snip>
----------
          ID: docker-restart
    Function: cmd.run
        Name: /bin/systemctl daemon-reload && /bin/systemctl restart docker.service
      Result: False
     Comment: The following requisites were not found:
                                 onchanges:
                                     file: /etc/systemd/system/docker.service.d/block-external-access.conf
     Started: 11:21:56.707274
    Duration: 0.015 ms
     Changes:

<snip>
Summary for myhost
-------------
Succeeded: 66 (unchanged=1)
Failed:     1
-------------
Total states run:     67
Total run time:   28.343 s

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

• Docs
• Changelog - https://docs.saltstack.com/en/master/topics/development/changelog.html
• Tests written/updated

Commits signed with GPG?

Yes/No

Please review Salt's Contributing Guide for best practices.

See GitHub's page on GPG signing for more information about signing commits with GPG.

[remichristiaan]

Unfortunately I do not have the time to setup my system for the pre-commit and lint stuff. Anyone that does, be my guest.

[dwoz]

Closing this due to inactivity. Anyone should feel free to re-open it if they want to see it through to the end in one release cycle.