Restore original behavior of MasterScriptProvider::hasByName

Commit f3ce30ec changed this to return
after first non-throwing XNameContainer::hasByName, regardless if it
returned true or false.

This changes it back, to iterate over all the providers again.

Change-Id: I74fa4d4aa8760cad509442226601ab4842312b80
Reviewed-on: https://gerrit.libreoffice.org/65471
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
(cherry picked from commit 09f80ade)
Reviewed-on: https://gerrit.libreoffice.org/65473Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

scripting/source/provider/MasterScriptProvider.cxx

@@ -466,7 +466,8 @@ template <typename Proc> bool FindProviderAndApply(ProviderCache& rCache, Proc p
             try
             {
                 bResult = p(xCont);
-                break;
+                if (bResult)
+                    break;
             }
             catch (Exception& e)
             {