[Building Sakai] WHAT HAPPENED TO SERVER-CONFIGURATION-SERVICE ?!?

[Jim Eng]

It turns out that I was having problems due to stale artifacts in my local maven repo.  That was causing problems finding the inner interfaces. I would have had the same problem even if they were not inner classes.  That's fixed.

I use inner classes in some places.  But people had lots of problems with the inner Storage interfaces in Service classes when trying to create mocks in the early days of sakai2. Maybe I was wrong when I said you complained about that.  For a while there was general agreement not to use inner interfaces because of those problems.  Apparently those days are gone.

I am still asking that these two commits be backed out and reimplemented without destroying the history of the code.  

Please.

Jim 


On Sep 13, 2011, at 5:55 PM, Aaron Zeckoski wrote:

> I'm unaware of issues with inner class interfaces. Please explain.
> Thanks
> -AZ
> 
> 
> On Tue, Sep 13, 2011 at 5:53 PM, Jim Eng <jimeng at umich.edu> wrote:
>> Aaron, and all:
>> 
>> Yes, I am shouting!
>> 
>> It looks like two checkins were made to ServerConfigurationService recently, but I cannot tell what has changed because the entire contents of the interface was replaced though it looks like many of the methods did not change at all.  That means all history for that file is gone. Or maybe I should say useless.  We can no longer do a simple svn command to find out when a particular line of code changed and why. It looks like every single line of code except for the interface name changed in rev 98010.
>> 
>> http://source.sakaiproject.org/viewsvn/kernel/trunk/component-manager/src/main/java/org/sakaiproject/component/api/ServerConfigurationService.java?r1=66281&r2=98010&pathrev=98098
>> 
>> Then a few more changes were added in rev rev 98098.
>> 
>> http://source.sakaiproject.org/viewsvn/kernel/trunk/component-manager/src/main/java/org/sakaiproject/component/api/ServerConfigurationService.java?r1=98010&r2=98098&pathrev=98098
>> 
>> I looked this up because I am trying to update a mock in a contrib project, and someone has added inner interfaces to ServerConfigurationService, which I am having difficulty dealing with.  I thought we learned the problems with inner interfaces a long time ago, and I seem to remember Aaron Z complaining about the problems they created when implementing mocks.
>> 
>> I think revs 98010 and 98098 should be backed out until they are cleaned up.  The code should NOT be reformatted in this way that loses all of the history and masks the actual changes that are being made.  The inner interfaces should be moved out so they are separate interfaces. etc.
>> 
>> Please.
>> 
>> Jim
>> 
>> 
> 
> 
> 
> -- 
> Aaron Zeckoski - Software Architect - http://tinyurl.com/azprofile
> 
>