Logging

code/moya-web/src/main/java/fi/codecrew/moya/web/cdiview/reader/ReaderNameContainer.java

@@ -25,6 +25,9 @@ import java.util.List;
 import javax.enterprise.context.SessionScoped;
 import javax.enterprise.context.SessionScoped;
 import javax.inject.Named;
 import javax.inject.Named;
 
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 @Named
 @Named
 @SessionScoped
 @SessionScoped
 public class ReaderNameContainer implements Serializable {
 public class ReaderNameContainer implements Serializable {
@@ -32,48 +35,53 @@ public class ReaderNameContainer implements Serializable {
 	private static final long serialVersionUID = 571747919767505523L;
 	private static final long serialVersionUID = 571747919767505523L;
 
 
 	//private Integer readerId;
 	//private Integer readerId;
-	
+
 	// handling first user when using via getReaderId and setReaderId;
 	// handling first user when using via getReaderId and setReaderId;
 	private final List<Integer> readers = new ArrayList<Integer>();;
 	private final List<Integer> readers = new ArrayList<Integer>();;
-	
+
 	private boolean autopoll = false;
 	private boolean autopoll = false;
 
 
+	private static final Logger logger = LoggerFactory.getLogger(ReaderNameContainer.class);
+
 	/**
 	/**
-	 * 	use this function when using readers in "only one reader" -mode
+	 * use this function when using readers in "only one reader" -mode
 	 * 
 	 * 
 	 * @return first selected reader, or null if no reader selected
 	 * @return first selected reader, or null if no reader selected
 	 */
 	 */
 	public Integer getReaderId() {
 	public Integer getReaderId() {
-		if(getReaders().size() == 0)
+		if (getReaders().size() == 0)
 			return null;
 			return null;
-		
+
 		return readers.get(0);
 		return readers.get(0);
 	}
 	}
-	
 
 
 	/**
 	/**
 	 * Use this function when using readers in "only one reader" -mode
 	 * Use this function when using readers in "only one reader" -mode
-	 * @param readerId to set, or null to clear all readers 
+	 * 
+	 * @param readerId
+	 *            to set, or null to clear all readers
 	 */
 	 */
 	public void setReaderId(Integer readerId) {
 	public void setReaderId(Integer readerId) {
-		if(readerId == null)
+		if (readerId == null)
 			clearReaders();
 			clearReaders();
-		
-		if(getReaders().size() == 0) {
+
+		if (getReaders().isEmpty()) {
 			getReaders().add(readerId);
 			getReaders().add(readerId);
 		} else {
 		} else {
-			getReaders().set(0, readerId);
+			Integer replacedItem = getReaders().set(0, readerId);
+			logger.warn("Replaced item {} wit {}", readerId, replacedItem);
+
 		}
 		}
 	}
 	}
-	
+
 	public void addReader(Integer readerId) {
 	public void addReader(Integer readerId) {
-		if(!getReaders().contains(readerId)) {
+		if (!getReaders().contains(readerId)) {
 			getReaders().add(readerId);
 			getReaders().add(readerId);
 		}
 		}
 	}
 	}
-	
+
 	public List<Integer> getReaders() {
 	public List<Integer> getReaders() {
-		
+
 		return readers;
 		return readers;
 	}
 	}
 
 
@@ -84,7 +92,7 @@ public class ReaderNameContainer implements Serializable {
 	public void setAutopoll(boolean autopoll) {
 	public void setAutopoll(boolean autopoll) {
 		this.autopoll = autopoll;
 		this.autopoll = autopoll;
 	}
 	}
-	
+
 	public void clearReaders() {
 	public void clearReaders() {
 		readers.clear();
 		readers.clear();
 	}
 	}