Hello Brad, In your patch, wouldn't it be easier to catch the return value from: Provider provider = providers [ProviderName]; And if null, report that the provider was not found, instead of adding an extra loop?