[Building Sakai] 2.6.x getCurrentAcademicSessions problem

Zach A. Thomas zach at aeroplanesoftware.com
Wed Apr 22 13:22:34 PDT 2009


On Apr 22, 2009, at 2:41 PM, Ray Davis wrote:

> Wow, if Zach thinks he's missing something obvious and Jon thinks he's
> having a stroke, things are pretty bad!  :)
>
> * Since this was improved functionality that combined a DB change  
> and an
> API change, I didn't request that it go into 2.5.x. Others can make  
> that
> argument if they want.
>
> * There's no "isCurrent()" or "setCurrent(boolean)" on the
> AcademicSession, true. The original read-only CourseManagementService
> API supplied:
>
>   List<AcademicSession> getCurrentAcademicSessions()
>
> And so I just added the equivalent method to the write-only
> CourseManagementAdministration API:
>
>   setCurrentAcademicSessions(List<String> academicSessionEids)
>
> This minimized changes to the existing API, which seemed important  
> to me
> given that the interface may have been locally re-implemented by
> multiple institutions.
>
> Best,
> Ray

I just added a few lines of code to ensure I call  
setCurrentAcademicSessions where we provision our AcademicSessions,  
and I'm all better now.

Jon, here's to your health.

Zach


More information about the sakai-dev mailing list