I made some changes based on what your feedback.

Thanks,
Michael 
On Feb 7, 2014, at 5:31 PM, Mihai Codoban <notifications@github.com> wrote:

> In edu.oregonstate.cope.eclipse.operations/src/edu/illinois/codingtracker/operations/OperationDeserializer.java:
> 
> > +			JSONObject jsonObj = jsonObjs.get(j);
> > +			//CHECK IF CLASSPATH IS BEFORE PROJECT
> > +			if(jsonObj.get("eventType").toString().equals("resourceAdded")){
> > +				if(jsonObj.get("entityAddress").toString().contains(".classpath")){
> > +					//check if next entry is .project
> > +					int currLoc = jsonObjs.indexOf(jsonObj);
> > +					JSONObject nextObj = jsonObjs.get(currLoc+1);
> > +					if(nextObj.get("eventType").toString().equals("resourceAdded") && 
> > +							nextObj.get("entityAddress").toString().contains(".project")){
> > +						jsonObjs.set(currLoc, nextObj);
> > + 						jsonObjs.set(currLoc+1, jsonObj);
> > +					}
> > +				}
> > +			}
> > +			
> > +			//CHECK FOR FILE EVENTS BEFORE RESOURCECREATED
> For readability, you could extract these two concerns to 2 methods that take as input the array and J. They would spit out the new J after they've made the modification to the array.
> 
> —
> Reply to this email directly or view it on GitHub.
>