libre.sh/keycloak-scim
4
0
Fork 0
Code Issues 15 Pull requests Releases 1 Activity Actions

KEYCLOAK-2253 - Use string representation for boolean properties in GenericUserFederationCtrl.

Browse source 
Adapted due to code review.
This commit is contained in:
Thomas Darimont 2016-01-04 23:07:08 +01:00
parent 3ec516d14c
commit edcc39d906
2 changed files with 2 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
forms/common-themes/src/main/resources/theme/base/admin/resources/js/controllers/users.js
View file

@ -649,8 +649,7 @@ module.controller('GenericUserFederationCtrl', function($scope, $location, Notif
for (var i = 0; i < providerFactory.properties.length; i++) { for (var i = 0; i < providerFactory.properties.length; i++) {
var configProperty = providerFactory.properties[i]; var configProperty = providerFactory.properties[i];
var configValue = configProperty.type == "boolean" ? (configProperty.defaultValue === true ? 'true' : 'false') : configProperty.defaultValue; instance.config[configProperty.name] = configProperty.defaultValue;
instance.config[configProperty.name] = configValue;
} }
} }

2
testsuite/integration/src/test/java/org/keycloak/testsuite/DummyUserFederationProviderFactory.java
View file

@ -103,7 +103,7 @@ public class DummyUserFederationProviderFactory implements UserFederationProvide
ProviderConfigProperty prop2 = new ProviderConfigProperty(); ProviderConfigProperty prop2 = new ProviderConfigProperty();
prop2.setName("prop2"); prop2.setName("prop2");
prop2.setLabel("Prop2"); prop2.setLabel("Prop2");
prop2.setDefaultValue(true); prop2.setDefaultValue("true");
prop2.setHelpText("Prop2 HelpText"); prop2.setHelpText("Prop2 HelpText");
prop2.setType(ProviderConfigProperty.BOOLEAN_TYPE); prop2.setType(ProviderConfigProperty.BOOLEAN_TYPE);