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 bt...@apache.org on 2018/05/28 01:06:22 UTC

[12/13] james-project git commit: JAMES-2403 Correct method visibility in ScanningQuotaSearcher

JAMES-2403 Correct method visibility in ScanningQuotaSearcher


Project: http://git-wip-us.apache.org/repos/asf/james-project/repo
Commit: http://git-wip-us.apache.org/repos/asf/james-project/commit/95c44232
Tree: http://git-wip-us.apache.org/repos/asf/james-project/tree/95c44232
Diff: http://git-wip-us.apache.org/repos/asf/james-project/diff/95c44232

Branch: refs/heads/master
Commit: 95c44232b33c14c8de2481ad6087a6e1ba7d3ad3
Parents: 35b7ec7
Author: benwa <bt...@linagora.com>
Authored: Thu May 24 11:57:03 2018 +0700
Committer: benwa <bt...@linagora.com>
Committed: Mon May 28 08:05:35 2018 +0700

----------------------------------------------------------------------
 .../james/quota/search/scanning/ClauseConverter.java    | 12 ++++++------
 .../quota/search/scanning/ScanningQuotaSearcher.java    |  4 ++--
 2 files changed, 8 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/james-project/blob/95c44232/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ClauseConverter.java
----------------------------------------------------------------------
diff --git a/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ClauseConverter.java b/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ClauseConverter.java
index 43e158c..f1d9035 100644
--- a/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ClauseConverter.java
+++ b/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ClauseConverter.java
@@ -62,34 +62,34 @@ public class ClauseConverter {
             .orElse(user -> true);
     }
 
-    public Predicate<User> toPredicate(QuotaClause clause) {
+    private Predicate<User> toPredicate(QuotaClause clause) {
         return toPredicates.get(clause.getClass())
             .apply(clause);
     }
 
-    public Predicate<User> moreThanToPredicate(QuotaClause clause) {
+    private Predicate<User> moreThanToPredicate(QuotaClause clause) {
         QuotaClause.MoreThan moreThan = (QuotaClause.MoreThan) clause;
         return user -> retrieveUserRatio(user) >= moreThan.getQuotaBoundary().getRatio();
     }
 
-    public Predicate<User> lessThanToPredicate(QuotaClause clause) {
+    private Predicate<User> lessThanToPredicate(QuotaClause clause) {
         QuotaClause.LessThan lessThan = (QuotaClause.LessThan) clause;
         return user -> retrieveUserRatio(user) <= lessThan.getQuotaBoundary().getRatio();
     }
 
-    public Predicate<User> hasDomainToPredicate(QuotaClause clause) {
+    private Predicate<User> hasDomainToPredicate(QuotaClause clause) {
         QuotaClause.HasDomain hasDomain = (QuotaClause.HasDomain) clause;
         return user -> user.getDomainPart()
             .map(hasDomain.getDomain()::equals)
             .orElse(false);
     }
 
-    public Predicate<User> andToPredicate(QuotaClause clause) {
+    private Predicate<User> andToPredicate(QuotaClause clause) {
         QuotaClause.And and = (QuotaClause.And) clause;
         return andToPredicate(and);
     }
 
-    public double retrieveUserRatio(User user) {
+    private double retrieveUserRatio(User user) {
         try {
             QuotaRoot quotaRoot = quotaRootResolver.forUser(user);
             Quota<QuotaSize> storageQuota = quotaManager.getStorageQuota(quotaRoot);

http://git-wip-us.apache.org/repos/asf/james-project/blob/95c44232/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ScanningQuotaSearcher.java
----------------------------------------------------------------------
diff --git a/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ScanningQuotaSearcher.java b/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ScanningQuotaSearcher.java
index 8d388d0..9aaf2c2 100644
--- a/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ScanningQuotaSearcher.java
+++ b/mailbox/plugin/quota-search-scanning/src/main/java/org/apache/james/quota/search/scanning/ScanningQuotaSearcher.java
@@ -58,13 +58,13 @@ public class ScanningQuotaSearcher implements QuotaSearcher {
             .collect(Guavate.toImmutableList());
     }
 
-    public Stream<User> limit(Stream<User> results, Limit limit) {
+    private Stream<User> limit(Stream<User> results, Limit limit) {
         return limit.getValue()
             .map(results::limit)
             .orElse(results);
     }
 
-    public Iterator<String> listUsers() {
+    private Iterator<String> listUsers() {
         try {
             return usersRepository.list();
         } catch (UsersRepositoryException e) {


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