svn commit: r958689 - /mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java

[srowen]

Author: srowen
Date: Mon Jun 28 19:37:20 2010
New Revision: 958689

URL: http://svn.apache.org/viewvc?rev=958689&view=rev
Log:
Missed actually using new PreferenceArray in one possible branch of update

Modified:
    
mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java

Modified: 
mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java
URL: 
http://svn.apache.org/viewvc/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java?rev=958689&r1=958688&r2=958689&view=diff
==============================================================================
--- 
mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java
 (original)
+++ 
mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/file/FileDataModel.java
 Mon Jun 28 19:37:20 2010
@@ -438,16 +438,17 @@ public class FileDataModel extends Abstr
         if (!exists) {
           if (prefs == null) {
             prefs = new GenericUserPreferenceArray(1);
-            ((FastByIDMap<PreferenceArray>) data).put(userID, prefs);
           } else {
             PreferenceArray newPrefs = new 
GenericUserPreferenceArray(prefs.length() + 1);
             for (int i = 0, j = 1; i < prefs.length(); i++, j++) {
               newPrefs.set(j, prefs.get(i));
             }
+            prefs = newPrefs;
           }
           prefs.setUserID(0, userID);
           prefs.setItemID(0, itemID);
           prefs.setValue(0, preferenceValue);
+          ((FastByIDMap<PreferenceArray>) data).put(userID, prefs);          
         }
       }