svn commit: r1131429 - /mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java

[srowen]

Author: srowen
Date: Sat Jun  4 16:31:50 2011
New Revision: 1131429

URL: http://svn.apache.org/viewvc?rev=1131429&view=rev
Log:
Follow up on MAHOUT-705, additional check for empty args

Modified:
    
mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java

Modified: 
mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java
URL: 
http://svn.apache.org/viewvc/mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java?rev=1131429&r1=1131428&r2=1131429&view=diff
==============================================================================
--- 
mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java
 (original)
+++ 
mahout/trunk/integration/src/main/java/org/apache/mahout/cf/taste/impl/model/mongodb/MongoDBDataModel.java
 Sat Jun  4 16:31:50 2011
@@ -748,8 +748,10 @@ public final class MongoDBDataModel impl
                          boolean add) throws NoSuchUserException, 
NoSuchItemException {
     Preconditions.checkNotNull(userID);
     Preconditions.checkNotNull(items);
+    Preconditions.checkArgument(userID.length() > 0);
     for (List<String> item : items) {
       Preconditions.checkNotNull(item.get(0));
+      Preconditions.checkArgument(item.get(0).length() > 0);
     }
     if (userIsObject && !ID_PATTERN.matcher(userID).matches()) {
       throw new IllegalArgumentException();