1309: A little more hacking to get the Login page to render more then once

Author: bdemers

extensions/servlet/src/main/java/com/stormpath/sdk/servlet/mvc/LoginController.java

@@ -174,11 +174,11 @@ public boolean isNotAllowedIfAuthenticated() {
     protected void appendModel(HttpServletRequest request, HttpServletResponse response, Form form, List<ErrorModel> errors,
                                Map<String, Object> model) {
 
-        final List<AccountStoreModel> accountStores = accountStoreModelFactory.getAccountStores(request);
+//        final List<AccountStoreModel> accountStores = accountStoreModelFactory.getAccountStores(request);
 
         // 748: If stormpath.web.idSite.enabled is false and stormpath.web.callback.enabled is false AND
         // there are SAML directories mapped to the application, that is a configuration error.
-        if (!idSiteEnabled && !callbackEnabled && containsSaml(accountStores)) {
+        if (!idSiteEnabled && !callbackEnabled ) { // && containsSaml(accountStores)) {
             String errorMsg = "ID Site is disabled and callbacks are disabled, yet this application has SAML directories. Please enable callbacks or remove SAML directories.";
             log.warn(errorMsg);
             if (errors == null) {
@@ -190,7 +190,7 @@ protected void appendModel(HttpServletRequest request, HttpServletResponse respo
             }
         }
 
-        model.put("accountStores", accountStores);
+//        model.put("accountStores", accountStores);
 
         if (isHtmlPreferred(request, response)) {
             model.put("forgotPasswordEnabled", forgotPasswordEnabled);

extensions/servlet/src/test/groovy/com/stormpath/sdk/servlet/mvc/LoginControllerTest.groovy

@@ -195,7 +195,7 @@ class LoginControllerTest {
 
         List<AccountStoreModel> accountStores = new ArrayList<AccountStoreModel>()
         accountStores.add(new DefaultAccountStoreModel(null, new DefaultProviderModel('foo', 'saml'), null))
-        expect(accountStoreModelFactory.getAccountStores(request)).andReturn(accountStores)
+//        expect(accountStoreModelFactory.getAccountStores(request)).andReturn(accountStores)
         expect(request.getAttribute(UserAgents.USER_AGENT_REQUEST_ATTRIBUTE_NAME)).andReturn new DefaultUserAgent(request)
         expect(request.getParameter("status")).andReturn null
         expect(request.getHeader("Accept")).andReturn "text/html"