diff options
Diffstat (limited to 'src/net/java/sip/communicator/impl/protocol/gibberish')
-rw-r--r-- | src/net/java/sip/communicator/impl/protocol/gibberish/ProtocolProviderFactoryGibberishImpl.java | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/net/java/sip/communicator/impl/protocol/gibberish/ProtocolProviderFactoryGibberishImpl.java b/src/net/java/sip/communicator/impl/protocol/gibberish/ProtocolProviderFactoryGibberishImpl.java index 0d9e68d..d6c7806 100644 --- a/src/net/java/sip/communicator/impl/protocol/gibberish/ProtocolProviderFactoryGibberishImpl.java +++ b/src/net/java/sip/communicator/impl/protocol/gibberish/ProtocolProviderFactoryGibberishImpl.java @@ -144,6 +144,14 @@ public class ProtocolProviderFactoryGibberishImpl public void modifyAccount( ProtocolProviderService protocolProvider, Map accountProperties) { + // Make sure the specified arguments are valid. + if (protocolProvider == null) + throw new NullPointerException( + "The specified Protocol Provider was null"); + if (accountProperties == null) + throw new NullPointerException( + "The specified property map was null"); + BundleContext context = GibberishActivator.getBundleContext(); @@ -151,10 +159,6 @@ public class ProtocolProviderFactoryGibberishImpl throw new NullPointerException( "The specified BundleContext was null"); - if (protocolProvider == null) - throw new NullPointerException( - "The specified Protocol Provider was null"); - GibberishAccountID accountID = (GibberishAccountID) protocolProvider.getAccountID(); @@ -172,10 +176,6 @@ public class ProtocolProviderFactoryGibberishImpl accountProperties.put(USER_ID, accountID.getUserID()); - if (accountProperties == null) - throw new NullPointerException( - "The specified property map was null"); - if (!accountProperties.containsKey(PROTOCOL)) accountProperties.put(PROTOCOL, ProtocolNames.GIBBERISH); |