From 89888f9f230c98bde926e849d5e42ed92da10c0d Mon Sep 17 00:00:00 2001 From: tkellner Date: Fri, 2 Sep 2011 16:01:17 +0000 Subject: Housekeeping * Generic types parameterized * SupressWarnings annotations added where parameterization impossible * Unused members commented out * SerialVersionUID added/removed where necessary * Superfluous imports removed * Other small changed to get rid of some compiler/validator warnings git-svn-id: https://joinup.ec.europa.eu/svn/mocca/trunk@976 8a26b1a7-26f0-462f-b9ef-d0e30c41f5a4 --- .../bku/accesscontroller/CommandParamChecker.java | 14 +- .../bku/accesscontroller/InfoboxParamChecker.java | 2 +- .../bku/binding/BindingProcessorManagerImpl.java | 4 +- .../main/java/at/gv/egiz/bku/binding/HttpUtil.java | 2 +- .../at/gv/egiz/bku/conf/IAIKLogAdapterFactory.java | 2 +- .../slcommands/impl/xsect/SignatureLocation.java | 3 +- .../gv/egiz/bku/spring/PKIProfileFactoryBean.java | 1 + bkucommon/src/test/java/moaspss/MOASPClient.java | 4 +- .../src/test/java/moaspss/generated/MOAFault.java | 5 + bkucommon/src/test/resources/MOA-SPSS-1.3.xsd | 938 ++++++++++----------- 10 files changed, 491 insertions(+), 484 deletions(-) (limited to 'bkucommon') diff --git a/bkucommon/src/main/java/at/gv/egiz/bku/accesscontroller/CommandParamChecker.java b/bkucommon/src/main/java/at/gv/egiz/bku/accesscontroller/CommandParamChecker.java index 427e9c37..e9bfdf0b 100644 --- a/bkucommon/src/main/java/at/gv/egiz/bku/accesscontroller/CommandParamChecker.java +++ b/bkucommon/src/main/java/at/gv/egiz/bku/accesscontroller/CommandParamChecker.java @@ -31,15 +31,15 @@ import at.gv.egiz.bku.slcommands.SLCommand; public abstract class CommandParamChecker { - protected List> paramList = new LinkedList>(); + protected List> paramList = new LinkedList>(); - public static class Tupel { + public static class Pair { private T key; private Q val; - public Tupel(T key, Q val) { + public Pair(T key, Q val) { if ((key == null) || (val == null)) { - throw new NullPointerException("Tupel key and value must not be null"); + throw new NullPointerException("Pair key and value must not be null"); } this.key = key; this.val = val; @@ -54,8 +54,8 @@ public abstract class CommandParamChecker { } public boolean equals(Object other) { - if (other instanceof Tupel) { - Tupel ot = (Tupel) other; + if (other instanceof Pair) { + Pair ot = (Pair) other; return (key.equals(ot.key) && val.equals(ot.val)); } return false; @@ -67,7 +67,7 @@ public abstract class CommandParamChecker { } public void addParameter(String key, String value) { - paramList.add(new Tupel(key, value)); + paramList.add(new Pair(key, value)); } public abstract boolean checkParameter(SLCommand cmd); diff --git a/bkucommon/src/main/java/at/gv/egiz/bku/accesscontroller/InfoboxParamChecker.java b/bkucommon/src/main/java/at/gv/egiz/bku/accesscontroller/InfoboxParamChecker.java index e6f03234..6a10a116 100644 --- a/bkucommon/src/main/java/at/gv/egiz/bku/accesscontroller/InfoboxParamChecker.java +++ b/bkucommon/src/main/java/at/gv/egiz/bku/accesscontroller/InfoboxParamChecker.java @@ -50,7 +50,7 @@ public class InfoboxParamChecker extends CommandParamChecker { if (cmd instanceof InfoboxReadCommand) { InfoboxReadCommand irc = (InfoboxReadCommand) cmd; - for (Tupel param : paramList) { + for (Pair param : paramList) { if (param.getKey().equals(INFOBOX_ID)) { if (!param.getVal().equals(irc.getInfoboxIdentifier())) { return false; diff --git a/bkucommon/src/main/java/at/gv/egiz/bku/binding/BindingProcessorManagerImpl.java b/bkucommon/src/main/java/at/gv/egiz/bku/binding/BindingProcessorManagerImpl.java index 590a937c..2278bfa9 100644 --- a/bkucommon/src/main/java/at/gv/egiz/bku/binding/BindingProcessorManagerImpl.java +++ b/bkucommon/src/main/java/at/gv/egiz/bku/binding/BindingProcessorManagerImpl.java @@ -286,11 +286,11 @@ public class BindingProcessorManagerImpl implements BindingProcessorManager, Com public void run() { Collection futures = submittedFutures.values(); List toBeRemoved = new ArrayList(); - int active = 0; +// int active = 0; for(BindingProcessorFuture future : futures) { BindingProcessor bindingProcessor = future.getBindingProcessor(); if (!future.isDone()) { - active++; +// active++; } if ((bindingProcessor.getLastAccessTime().getTime() - System .currentTimeMillis()) > maxAcceptedAge) { diff --git a/bkucommon/src/main/java/at/gv/egiz/bku/binding/HttpUtil.java b/bkucommon/src/main/java/at/gv/egiz/bku/binding/HttpUtil.java index 1678c5ae..152b007b 100644 --- a/bkucommon/src/main/java/at/gv/egiz/bku/binding/HttpUtil.java +++ b/bkucommon/src/main/java/at/gv/egiz/bku/binding/HttpUtil.java @@ -64,7 +64,7 @@ public class HttpUtil { boolean replaceNullWithDefault) { ParameterParser pf = new ParameterParser(); pf.setLowerCaseNames(true); - Map map = pf.parse(contentType, SEPERATOR); + Map map = pf.parse(contentType, SEPERATOR); String retVal = (String) map.get(CHAR_SET); if ((retVal == null) && (replaceNullWithDefault)) { if (map.containsKey(APPLICATION_URL_ENCODED)) { diff --git a/bkucommon/src/main/java/at/gv/egiz/bku/conf/IAIKLogAdapterFactory.java b/bkucommon/src/main/java/at/gv/egiz/bku/conf/IAIKLogAdapterFactory.java index 56719a77..0e9b9fe6 100644 --- a/bkucommon/src/main/java/at/gv/egiz/bku/conf/IAIKLogAdapterFactory.java +++ b/bkucommon/src/main/java/at/gv/egiz/bku/conf/IAIKLogAdapterFactory.java @@ -63,7 +63,7 @@ public class IAIKLogAdapterFactory extends LogFactory { * @see iaik.logging.LogFactory#getInstance(java.lang.Class) */ @Override - public synchronized Log getInstance(Class clazz) throws LogConfigurationException { + public synchronized Log getInstance(@SuppressWarnings("rawtypes") Class clazz) throws LogConfigurationException { Logger logger = LoggerFactory.getLogger(clazz); Log log = instances.get(logger); if (log == null) { diff --git a/bkucommon/src/main/java/at/gv/egiz/bku/slcommands/impl/xsect/SignatureLocation.java b/bkucommon/src/main/java/at/gv/egiz/bku/slcommands/impl/xsect/SignatureLocation.java index bf8b738a..d2702635 100644 --- a/bkucommon/src/main/java/at/gv/egiz/bku/slcommands/impl/xsect/SignatureLocation.java +++ b/bkucommon/src/main/java/at/gv/egiz/bku/slcommands/impl/xsect/SignatureLocation.java @@ -222,8 +222,9 @@ public class SignatureLocation { return namespaceContext.getPrefix(namespaceURI); } + @SuppressWarnings("unchecked") @Override - public Iterator getPrefixes(String namespaceURI) { + public Iterator getPrefixes(String namespaceURI) { return namespaceContext.getPrefixes(namespaceURI); } diff --git a/bkucommon/src/main/java/at/gv/egiz/bku/spring/PKIProfileFactoryBean.java b/bkucommon/src/main/java/at/gv/egiz/bku/spring/PKIProfileFactoryBean.java index 63b2ddeb..ee6583f7 100644 --- a/bkucommon/src/main/java/at/gv/egiz/bku/spring/PKIProfileFactoryBean.java +++ b/bkucommon/src/main/java/at/gv/egiz/bku/spring/PKIProfileFactoryBean.java @@ -92,6 +92,7 @@ public class PKIProfileFactoryBean implements FactoryBean, ResourceLoaderAware { return getURL(SSL_CA_DIRECTORY); } + @SuppressWarnings("unchecked") public List getRevocationServiceOrder() throws Exception { return configuration.getList(SSL_REVOCATION_SERVICE_ORDER); } diff --git a/bkucommon/src/test/java/moaspss/MOASPClient.java b/bkucommon/src/test/java/moaspss/MOASPClient.java index ff763c4a..4c5ec14c 100644 --- a/bkucommon/src/test/java/moaspss/MOASPClient.java +++ b/bkucommon/src/test/java/moaspss/MOASPClient.java @@ -91,7 +91,7 @@ public class MOASPClient { public static class ClientJAXBContextFactory implements JAXBContextFactory { public JAXBRIContext createJAXBContext(final SEIModel sei, - final List classesToBind, final List typeReferences) + @SuppressWarnings("rawtypes") final List classesToBind, final List typeReferences) throws JAXBException { System.out.println("Create Context"); @@ -168,7 +168,7 @@ public class MOASPClient { } @Override - public Bridge createBridge(TypeReference arg0) { + public Bridge createBridge(TypeReference arg0) { return context.createBridge(arg0); } }; diff --git a/bkucommon/src/test/java/moaspss/generated/MOAFault.java b/bkucommon/src/test/java/moaspss/generated/MOAFault.java index 2ee74919..60057183 100644 --- a/bkucommon/src/test/java/moaspss/generated/MOAFault.java +++ b/bkucommon/src/test/java/moaspss/generated/MOAFault.java @@ -39,6 +39,11 @@ public class MOAFault extends Exception { + /** + * + */ + private static final long serialVersionUID = 1L; + /** * Java type that goes as soapenv:Fault detail element. * diff --git a/bkucommon/src/test/resources/MOA-SPSS-1.3.xsd b/bkucommon/src/test/resources/MOA-SPSS-1.3.xsd index 756b5127..5a58e4f8 100644 --- a/bkucommon/src/test/resources/MOA-SPSS-1.3.xsd +++ b/bkucommon/src/test/resources/MOA-SPSS-1.3.xsd @@ -1,469 +1,469 @@ - - - - - - - - - - - - - - - - - - - - Ermöglichung der Stapelsignatur durch wiederholte Angabe dieses Elements - - - - - - - - - - - - - - - - - - - Auswahl: Entweder explizite Angabe des Signaturorts sowie ggf. sinnvoller Supplements im Zshg. mit der Signaturumgebung, oder Verweis auf ein benanntes Profil - - - - - - - - - - - - - - - - - - Kardinalität 1..oo erlaubt die Antwort auf eine Stapelsignatur-Anfrage - - - - Resultat, falls die Signaturerstellung erfolgreich war - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - mit diesem Profil wird eine Menge von vertrauenswürdigen Wurzelzertifikaten spezifiziert - - - - - - - - - - - only ds:X509Data and RetrievalMethod is supported; QualifiedCertificate is included as X509Data/any;publicAuthority is included as X509Data/any - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - Pro dsig:Reference-Element in der zu überprüfenden XML-Signatur muss hier ein ReferenceInfo-Element erscheinen. Die Reihenfolge der einzelnen ReferenceInfo Elemente entspricht jener der dsig:Reference Elemente in der XML-Signatur. - - - - - - - - - - mit diesem Profil wird eine Menge von vertrauenswürdigen Wurzelzertifikaten spezifiziert - - - - - - - - - - - only ds:X509Data and ds:RetrievalMethod is supported; QualifiedCertificate is included as X509Data/any; PublicAuthority is included as X509Data/any - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - Auswahl: Entweder explizite Angabe EINER Transformationskette inklusive ggf. sinnvoller Supplements oder Verweis auf ein benanntes Profil - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - Resultat, falls die Signaturerstellung gescheitert ist - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - Ein oder mehrere Transformationswege können von der Applikation an MOA mitgeteilt werden. Die zu prüfende Signatur hat zumindest einem dieser Transformationswege zu entsprechen. Die Angabe kann explizit oder als Profilbezeichner erfolgen. - - - - - Profilbezeichner für einen Transformationsweg - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - Die Angabe des Transformationsparameters (explizit oder als Hashwert) kann unterlassen werden, wenn die Applikation von der Unveränderlichkeit des Inhalts der in "Transformationsparamter", Attribut "URI" angegebenen URI ausgehen kann. - - - - Der Transformationsparameter explizit angegeben. - - - - - Der Hashwert des Transformationsparameters. - - - - - - - - - - - - - - - - - - - - - - Explizite Angabe des Transformationswegs - - - - - - - Alle impliziten Transformationsparameter, die zum Durchlaufen der oben angeführten Transformationskette bekannt sein müssen, müssen hier angeführt werden. Das Attribut "URI" bezeichnet den Transformationsparameter in exakt jener Weise, wie er in der zu überprüfenden Signatur gebraucht wird. - - - - - - - - - - - - - - - - + + + + + + + + + + + + + + + + + + + + Ermöglichung der Stapelsignatur durch wiederholte Angabe dieses Elements + + + + + + + + + + + + + + + + + + + Auswahl: Entweder explizite Angabe des Signaturorts sowie ggf. sinnvoller Supplements im Zshg. mit der Signaturumgebung, oder Verweis auf ein benanntes Profil + + + + + + + + + + + + + + + + + + Kardinalität 1..oo erlaubt die Antwort auf eine Stapelsignatur-Anfrage + + + + Resultat, falls die Signaturerstellung erfolgreich war + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + mit diesem Profil wird eine Menge von vertrauenswürdigen Wurzelzertifikaten spezifiziert + + + + + + + + + + + only ds:X509Data and RetrievalMethod is supported; QualifiedCertificate is included as X509Data/any;publicAuthority is included as X509Data/any + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + Pro dsig:Reference-Element in der zu überprüfenden XML-Signatur muss hier ein ReferenceInfo-Element erscheinen. Die Reihenfolge der einzelnen ReferenceInfo Elemente entspricht jener der dsig:Reference Elemente in der XML-Signatur. + + + + + + + + + + mit diesem Profil wird eine Menge von vertrauenswürdigen Wurzelzertifikaten spezifiziert + + + + + + + + + + + only ds:X509Data and ds:RetrievalMethod is supported; QualifiedCertificate is included as X509Data/any; PublicAuthority is included as X509Data/any + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + Auswahl: Entweder explizite Angabe EINER Transformationskette inklusive ggf. sinnvoller Supplements oder Verweis auf ein benanntes Profil + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + Resultat, falls die Signaturerstellung gescheitert ist + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + Ein oder mehrere Transformationswege können von der Applikation an MOA mitgeteilt werden. Die zu prüfende Signatur hat zumindest einem dieser Transformationswege zu entsprechen. Die Angabe kann explizit oder als Profilbezeichner erfolgen. + + + + + Profilbezeichner für einen Transformationsweg + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + Die Angabe des Transformationsparameters (explizit oder als Hashwert) kann unterlassen werden, wenn die Applikation von der Unveränderlichkeit des Inhalts der in "Transformationsparamter", Attribut "URI" angegebenen URI ausgehen kann. + + + + Der Transformationsparameter explizit angegeben. + + + + + Der Hashwert des Transformationsparameters. + + + + + + + + + + + + + + + + + + + + + + Explizite Angabe des Transformationswegs + + + + + + + Alle impliziten Transformationsparameter, die zum Durchlaufen der oben angeführten Transformationskette bekannt sein müssen, müssen hier angeführt werden. Das Attribut "URI" bezeichnet den Transformationsparameter in exakt jener Weise, wie er in der zu überprüfenden Signatur gebraucht wird. + + + + + + + + + + + + + + + + -- cgit v1.2.3