SXS-37: Replace net.sf.cglib with org.springframework.cglib

[wikumChamith]

Ticket: https://openmrs.atlassian.net/browse/SXS-37

[dkayiwa]

Shouldn't this pull request also contain the fix for? https://pastebin.com/ky1BTRGQ

[wikumChamith]

Shouldn't this pull request also contain the fix for? https://pastebin.com/ky1BTRGQ

I don't think it is related to this. Is it ok if we fix it in a separate PR?

[ibacher]

Shouldn't this import be ordered with the other org imports?

[wikumChamith]

@ibacher, I updated the PR according to this. However, few places in our code base do not follow this:

openmrs-module-serialization.xstream/api/src/main/java/org/openmrs/module/serialization/xstream/XStreamSerializer.java

Lines 63 to 64 in 87f07a9

	import org.springframework.beans.factory.annotation.Autowired;
	import org.springframework.stereotype.Component;

openmrs-module-serialization.xstream/api/src/main/java/org/openmrs/module/serialization/xstream/converter/HibernateCollectionConverter.java

Lines 3 to 13 in 87f07a9

	import org.apache.commons.logging.Log;
	import org.apache.commons.logging.LogFactory;
	import org.springframework.beans.factory.annotation.Autowired;
	import com.thoughtworks.xstream.converters.Converter;
	import com.thoughtworks.xstream.converters.ConverterLookup;
	import com.thoughtworks.xstream.converters.MarshallingContext;
	import com.thoughtworks.xstream.converters.UnmarshallingContext;
	import com.thoughtworks.xstream.io.HierarchicalStreamReader;
	import com.thoughtworks.xstream.io.HierarchicalStreamWriter;
	import org.springframework.stereotype.Component;

[ibacher]

We should eventually fix those. Technically the import order is part of our coding standards, but it's not well-enforced across projects.

[wikumChamith]

Shouldn't this pull request also contain the fix for? https://pastebin.com/ky1BTRGQ

This PR fixes that issue: openmrs/openmrs-module-referencemetadata#70