mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

Nicolas Capponi
02.33.2014 e114a899bbc0df59f5135f195d24797bf6c1b778
refs
author Nicolas Capponi <nicolas.capponi@forgerock.com>
Thursday, January 2, 2014 10:33 +0100
committer Nicolas Capponi <nicolas.capponi@forgerock.com>
Thursday, January 2, 2014 10:33 +0100
commite114a899bbc0df59f5135f195d24797bf6c1b778
tree a5192981bc3f788e2a3ebfe437c654a7dd4e9518 tree | zip | gz
parent 348d8980a6317cc0c4192c8b8d00bc796bbd76b2 view | diff
Checkpoint commit for OPENDJ-1235 : Migrate configuration framework

Refactor ClassPropertyDefinition and AttributeTypePropertyDefinition classes to
replace static setup methods by dynamic setup through new argument in methods

* Introduce new class PropertyDefinitionsOptions to hold options
** Provide an immutable version with no validation as constant of the class as it is
needed in tests and some classes.

* Pass PropertyDefinitionsOptions as argument to PropertyDefinition#decodeValue
and PropertyDefinition#validateValue methods

* Update all classes using PropertyDefinition#decodeValue and
PropertyDefinition#validateValue methods
** For server classes, inject PropertyDefinitionsOptions to ServerManagementContext
and propagate to classes needing options
** For client classes, inject PropertyDefinitionsOptions to LDAPManagementContext
and LDAPDriver classes and propagate to classes needing options
** Exception for three classes - use an immutable PropertyDefinitionsOptions object
with no validation, as primary purpose is not validation of value :
*** Class DefaultManagedObject : call decodeValue to copy value
*** Class Reference : call decodeValue to normalize value
*** Class ContainsCondition : call decode value to compare value

* Update all test classes to use dynamic setup through PropertyDefinitionsOptions
when needed instead of static call in setup method
54 files modified
1 files added
1405 ■■■■ changed files
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/ACIPropertyDefinition.java 4 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/AggregationPropertyDefinition.java 6 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/AttributeTypePropertyDefinition.java 37 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/BooleanPropertyDefinition.java 4 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/ClassPropertyDefinition.java 61 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/DNPropertyDefinition.java 7 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/DefaultManagedObject.java 3 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/DurationPropertyDefinition.java 6 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/EnumPropertyDefinition.java 4 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/IPAddressMaskPropertyDefinition.java 4 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/IPAddressPropertyDefinition.java 4 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/IntegerPropertyDefinition.java 6 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/PropertyDefinition.java 12 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/PropertyDefinitionsOptions.java 137 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/Reference.java 3 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/SizePropertyDefinition.java 6 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/StringPropertyDefinition.java 6 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/client/IllegalManagedObjectNameException.java 16 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/client/ldap/LDAPDriver.java 39 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/client/ldap/LDAPManagedObject.java 3 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/client/ldap/LDAPManagementContext.java 42 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/client/spi/AbstractManagedObject.java 19 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/client/spi/Driver.java 16 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/client/spi/PropertySet.java 8 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/condition/ContainsCondition.java 3 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/doc/ConfigGuideGeneration.java 4 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/main/java/org/opends/server/admin/server/ServerManagementContext.java 40 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/forgerock/opendj/config/ConfigTestCase.java 10 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/forgerock/opendj/config/ConfigurationMock.java 13 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/forgerock/opendj/config/ConfigurationMockTest.java 14 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/AbstractManagedObjectDefinitionTest.java 1 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/AdminTestCase.java 4 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/AggregationPropertyDefinitionTest.java 1 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/AttributeTypePropertyDefinitionTest.java 25 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/BooleanPropertyDefinitionTest.java 9 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/ClassPropertyDefinitionTest.java 6 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/DNPropertyDefinitionTest.java 14 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/DurationPropertyDefinitionTest.java 21 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/EnumPropertyDefinitionTest.java 7 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/IntegerPropertyDefinitionTest.java 16 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/LDAPProfileTest.java 6 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/ManagedObjectDefinitionI18NResourceTest.java 1 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/ManagedObjectPathTest.java 6 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/RelativeInheritedDefaultBehaviorProviderTest.java 1 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/SizePropertyDefinitionTest.java 16 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/StringPropertyDefinitionTest.java 17 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/TestTopCfgDefnTest.java 6 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/ValidateConfigDefinitionsTest.java 1 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/client/ldap/AggregationClientTest.java 21 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/client/ldap/LDAPClientTest.java 242 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/server/AggregationServerTest.java 143 ●●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/server/ConstraintTest.java 95 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/server/DNBuilderTest.java 1 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/server/DefaultBehaviorTest.java 113 ●●●● diff | view | raw | blame | history
opendj-sdk/opendj-admin/src/test/java/org/opends/server/admin/server/ListenerTest.java 95 ●●●● diff | view | raw | blame | history