From 027bf8114a23f0a9552edbee0bcbc2fb4e71630c Mon Sep 17 00:00:00 2001 From: Surya Gaddipati Date: Mon, 16 May 2016 13:25:53 -0500 Subject: [PATCH] formatting and variable name changes from CR --- .../jenkins/mongo/CauseActionConverter.java | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) diff --git a/src/main/java/com/groupon/jenkins/mongo/CauseActionConverter.java b/src/main/java/com/groupon/jenkins/mongo/CauseActionConverter.java index a48f0249..0e0b0ced 100644 --- a/src/main/java/com/groupon/jenkins/mongo/CauseActionConverter.java +++ b/src/main/java/com/groupon/jenkins/mongo/CauseActionConverter.java @@ -36,32 +36,33 @@ of this software and associated documentation files (the "Software"), to deal public class CauseActionConverter extends TypeConverter implements SimpleValueConverter { - public CauseActionConverter(){ + public CauseActionConverter() { super(CauseAction.class); } @Override public CauseAction decode(Class targetClass, Object fromDBObject, MappedField optionalExtraInfo) { - if(fromDBObject == null) return null; + if (fromDBObject == null) return null; List causes = new ArrayList(); List rawList = (List) ((DBObject) fromDBObject).get("causes"); - for(Object obj : rawList) { + for (Object obj : rawList) { DBObject dbObj = (DBObject) obj; - causes.add(getMapper().fromDBObject(optionalExtraInfo.getSubClass(), dbObj, getMapper().createEntityCache())); + Object cause = getMapper().fromDBObject(optionalExtraInfo.getSubClass(), dbObj, getMapper().createEntityCache()); + causes.add(cause); } return new CauseAction(causes); } @Override public Object encode(Object value, MappedField optionalExtraInfo) { - if(value == null) return null; + if (value == null) return null; CauseAction action = (CauseAction) value; - List core = new BasicDBList(); + List causes = new BasicDBList(); - for(Object obj : action.getCauses()) { - core.add(getMapper().toDBObject(obj)); + for (Object obj : action.getCauses()) { + causes.add(getMapper().toDBObject(obj)); } - return BasicDBObjectBuilder.start("causes",core).add("className",CauseAction.class.getName()).get(); + return BasicDBObjectBuilder.start("causes", causes).add("className", CauseAction.class.getName()).get(); } }