You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by ad...@apache.org on 2016/01/13 11:12:22 UTC

svn commit: r1724388 - /james/project/trunk/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/JmapResponseWriterImpl.java

Author: aduprat
Date: Wed Jan 13 10:12:22 2016
New Revision: 1724388

URL: http://svn.apache.org/viewvc?rev=1724388&view=rev
Log:
JAMES-1648 Refactor toFieldNames lambda. Contributed by Baechler <ma...@gmail.com>

Modified:
    james/project/trunk/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/JmapResponseWriterImpl.java

Modified: james/project/trunk/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/JmapResponseWriterImpl.java
URL: http://svn.apache.org/viewvc/james/project/trunk/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/JmapResponseWriterImpl.java?rev=1724388&r1=1724387&r2=1724388&view=diff
==============================================================================
--- james/project/trunk/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/JmapResponseWriterImpl.java (original)
+++ james/project/trunk/server/protocols/jmap/src/main/java/org/apache/james/jmap/methods/JmapResponseWriterImpl.java Wed Jan 13 10:12:22 2016
@@ -27,6 +27,7 @@ import javax.inject.Inject;
 
 import org.apache.james.jmap.model.Property;
 import org.apache.james.jmap.model.ProtocolResponse;
+import org.apache.james.util.streams.Collectors;
 
 import com.fasterxml.jackson.databind.Module;
 import com.fasterxml.jackson.databind.ObjectMapper;
@@ -57,16 +58,6 @@ public class JmapResponseWriterImpl impl
         });
     }
     
-    private FilterProvider buildPropertiesFilter(Optional<? extends Set<? extends Property>> properties) {
-        PropertyFilter filter = properties
-                .map(x -> x.stream()
-                        .map(Property::asFieldName)
-                        .collect(java.util.stream.Collectors.toSet()))
-                .map(SimpleBeanPropertyFilter::filterOutAllExcept)
-                .orElse(SimpleBeanPropertyFilter.serializeAll());
-        return new SimpleFilterProvider().addFilter("propertiesFilter", filter);
-    }
-    
     private ObjectMapper newConfiguredObjectMapper(JmapResponse jmapResponse) {
         ObjectMapper objectMapper = new ObjectMapper().registerModules(jacksonModules)
                 .configure(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS, false);
@@ -75,4 +66,18 @@ public class JmapResponseWriterImpl impl
 
         return objectMapper;
     }
+
+    private FilterProvider buildPropertiesFilter(Optional<? extends Set<? extends Property>> properties) {
+        PropertyFilter filter = properties
+                .map(this::toFieldNames)
+                .map(SimpleBeanPropertyFilter::filterOutAllExcept)
+                .orElse(SimpleBeanPropertyFilter.serializeAll());
+        return new SimpleFilterProvider().addFilter("propertiesFilter", filter);
+    }
+    
+    private Set<String> toFieldNames(Set<? extends Property> properties) {
+        return properties.stream()
+            .map(Property::asFieldName)
+            .collect(Collectors.toImmutableSet());
+    }
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org