external-software/github_OpenIdentityPlatform_OpenDJ.git

Remove null checks on returned values of Entry.get*Attribute*() methods.

Jean-Noël Rouvignac

17.11.2015 14f94c13789b8ace4eae258b5f1d64494518f9c3

refs
author	Jean-Noël Rouvignac <jean-noel.rouvignac@forgerock.com>
	Thursday, December 17, 2015 11:11 +0100
committer	Jean-Noël Rouvignac <jean-noel.rouvignac@forgerock.com>
	Monday, December 21, 2015 15:04 +0100
commit	14f94c13789b8ace4eae258b5f1d64494518f9c3
tree	8ecf2a0f8c3e7b0ecfee35e427d3563d0f5995e0 tree \| zip \| gz
parent	23de2bc1524fe169fcca2c4a39201db78f6c6b9a view \| diff

Remove null checks on returned values of Entry.get*Attribute*() methods.

TODO CryptoManagerSync.modifyEntry()

Entry.java:
In get*Attribute*(), made sure the methods never return null.
Removed unused methods

*.java:
Removed now unnecessary null checks

87 files modified

	opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/task/ModifyEntryTask.java	9 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/guitools/controlpanel/ui/ViewEntryPanel.java	34 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/admin/AdministrationDataSync.java	10 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/admin/server/ServerManagementContext.java	17 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/api/AuthenticationPolicy.java	115 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/api/AuthenticationPolicyState.java	118 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/api/ClientConnection.java	62 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/AciHandler.java	34 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/authorization/dseecompat/AciList.java	16 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/backends/BackupBackend.java	41 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/backends/SchemaBackend.java	62 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/backends/TrustStoreBackend.java	4 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/AttributeIndex.java	14 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/backends/pluggable/VLVIndex.java	40 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/backends/task/RecurringTask.java	150 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/backends/task/Task.java	10 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/backends/task/TaskScheduler.java	2 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/config/ConfigEntry.java	9 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/core/PasswordPolicyState.java	255 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/core/SchemaConfigManager.java	7 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/core/SubentryPasswordPolicy.java	42 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/crypto/CryptoManagerSync.java	44 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/extensions/AttributeValuePasswordValidator.java	20 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/extensions/DynamicGroup.java	39 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/extensions/ExternalSASLMechanismHandler.java	24 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/extensions/LDAPPassThroughAuthenticationPolicyFactory.java	98 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/extensions/SMTPAccountStatusNotificationHandler.java	25 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/extensions/StaticGroup.java	32 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/extensions/UserAttributeNotificationMessageTemplateElement.java	31 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/extensions/VirtualStaticGroup.java	55 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/plugins/EntryUUIDPlugin.java	3 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/plugins/PasswordPolicyImportPlugin.java	18 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/plugins/ReferentialIntegrityPlugin.java	25 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/plugins/SevenBitCleanPlugin.java	16 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/plugins/UniqueAttributePlugin.java	118 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/replication/plugin/EntryHistorical.java	69 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/replication/plugin/FractionalLDIFImportPlugin.java	16 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/replication/plugin/LDAPReplicationDomain.java	10 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/replication/plugin/PersistentServerState.java	6 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/tasks/AddSchemaFileTask.java	13 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/tasks/BackupTask.java	44 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/tasks/DisconnectClientTask.java	67 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/tasks/SetGenerationIdTask.java	2 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/tasks/ShutdownTask.java	4 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/tasks/TaskUtils.java	25 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/tools/tasks/TaskEntry.java	11 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/types/Entry.java	178 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/types/SearchFilter.java	78 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/types/SubEntry.java	124 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/workflowelement/localbackend/LocalBackendAddOperation.java	2 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/workflowelement/localbackend/LocalBackendBindOperation.java	5 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/workflowelement/localbackend/LocalBackendCompareOperation.java	3 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/main/java/org/opends/server/workflowelement/localbackend/LocalBackendModifyOperation.java	7 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/TestCaseUtils.java	10 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/authorization/dseecompat/AciTestCase.java	5 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/backends/ChangelogBackendTestCase.java	2 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/backends/pluggable/TestDnKeyFormat.java	8 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/core/AddOperationTestCase.java	15 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/core/ModifyOperationTestCase.java	29 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/core/SearchOperationTestCase.java	12 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/core/SubentryManagerTestCase.java	5 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/core/TestModifyDNOperation.java	5 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/EntryDNVirtualAttributeProviderTestCase.java	11 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/EntryUUIDVirtualAttributeProviderTestCase.java	14 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/GoverningStructureRuleVirtualAttributeProviderTestCase.java	4 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/HasSubordinatesVirtualAttributeProviderTestCase.java	4 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/NumSubordinatesVirtualAttributeProviderTestCase.java	4 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/PasswordExpirationTimeVirtualAttributeProviderTestCase.java	1 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/StructuralObjectClassVirtualAttributeProviderTestCase.java	4 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/SubschemaSubentryVirtualAttributeProviderTestCase.java	4 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/extensions/UserDefinedVirtualAttributeProviderTestCase.java	26 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/plugins/EntryUUIDPluginTestCase.java	13 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/plugins/LDAPADListPluginTestCase.java	12 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/plugins/LastModPluginTestCase.java	20 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/plugins/SambaPasswordPluginTestCase.java	17 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/replication/UpdateOperationTest.java	2 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/replication/plugin/AssuredReplicationPluginTest.java	2 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/replication/plugin/AttrHistoricalMultipleTest.java	2 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/replication/plugin/AttrHistoricalSingleTest.java	2 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/replication/plugin/FractionalReplicationTest.java	32 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/replication/plugin/ModifyConflictTest.java	30 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/replication/plugin/NamingConflictTest.java	5 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/schema/GenericSchemaTestCase.java	27 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/tasks/DummyTask.java	12 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/tools/LDIFSearchTestCase.java	81 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/tools/makeldif/MakeLDIFTestCase.java	34 ●●●●● diff \| view \| raw \| blame \| history
	opendj-server-legacy/src/test/java/org/opends/server/types/TestEntry.java	478 ●●●●● diff \| view \| raw \| blame \| history