You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@chemistry.apache.org by fm...@apache.org on 2011/11/07 20:32:58 UTC
svn commit: r1198887 -
/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/cookies/CmisHttpCookie.java
Author: fmui
Date: Mon Nov 7 19:32:57 2011
New Revision: 1198887
URL: http://svn.apache.org/viewvc?rev=1198887&view=rev
Log:
More robust cookie handling
Modified:
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/cookies/CmisHttpCookie.java
Modified: chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/cookies/CmisHttpCookie.java
URL: http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/cookies/CmisHttpCookie.java?rev=1198887&r1=1198886&r2=1198887&view=diff
==============================================================================
--- chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/cookies/CmisHttpCookie.java (original)
+++ chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/cookies/CmisHttpCookie.java Mon Nov 7 19:32:57 2011
@@ -236,10 +236,7 @@ public final class CmisHttpCookie implem
}
Setter setter = cookie.attributeSet.get(attrName.toLowerCase());
- if (null == setter) {
- throw new IllegalArgumentException();
- }
- if (!setter.isSet()) {
+ if (setter != null && !setter.isSet()) {
String attrValue = matcher.group(2);
setter.validate(attrValue, cookie);
setter.setValue(matcher.group(2), cookie);