[Building Sakai] Listing Course Sites in reverse chronological order

Matthew Jones matthew at longsight.com
Mon Dec 9 13:58:12 PST 2013


Yea, that looks like it would work too, I guess we just figured to do it in
the service instead as who wouldn't want that list descending in all cases?
It was annoying getting it fixed because the hibernate HQL was returning
one way but then it was resorting again the other way later.

In any case the most important thing is to make sure it's cached in the
service.


On Mon, Dec 9, 2013 at 4:37 PM, Kirschner, Beth <bkirschn at umich.edu> wrote:

> Hi Matt,
>
> Great - We don't have SAK-23544 in our CTools bulld (I see now you
> referenced it from SAK-19464). You can see my suggested patch with
> SAK-25448, but I like just having the reverse order as the default. I'll
> close my JIRA as "won't fix" or "duplicate".
>
> - Beth
>
> On Dec 9, 2013, at 3:58 PM, Matthew Jones <matthew at longsight.com> wrote:
>
> > It should have the most recent first, at least it does using the OOTB
> course management service. What did you think needed to be changed?
> >
> > https://jira.sakaiproject.org/browse/SAK-23544. The call to
> getCurrentAcademicSessions/getAcademicSessions was changed to return the
> order descending based on start date.Though this is only fixed in 2.9.x. If
> you have your own provider which implements this, don't have the start
> dates in correct order, or aren't all the way up-to-date with edu-services
> you might have different results?
> >
> > There is also a short term cache placed on this (
> https://jira.sakaiproject.org/browse/SAK-23570) so if you change the
> startDates live, you have to wait or reset you cache to see results. This
> was because it was hitting this a-lot and slowing down the portal.
> >
> > Also I *think* when you create a site it creates two properties Term and
> Term Eid as site properties based on Title and EID in
> CM_ACADEMIC_SESSION_T. The Term has to exactly (case sensitive) match the
> Title in that getAcademicSessions returns, which was confusing. This is the
> same as the value that was previously set in the manual property for
> (portal.term.order). I think it would have been *nice* if it could have
> matched either but in our case the TITLE==ENTERPRISE_ID in every case in
> CM_ACADEMIC_SESSION_T which makes this much easier.
> >
> > Shot of a client using this patch.
> >
> > <Ordered terms.PNG>
> >
> >
> >
> > On Mon, Dec 9, 2013 at 3:36 PM, Kirschner, Beth <bkirschn at umich.edu>
> wrote:
> > It looks like SAK-19464 implemented a default sort order for sites in
> the "More Sites" tab of the portal, which is great! But it also looks like
> the default sort order is not what was originally requested (and not what
> we'd like to see either). What was requested was _reverse_ chronological
> order (with most recent terms first), but what what implemented was
> chronological order (most recent terms last).
> >
> > I have a simple change proposed to fix this in SAK-25448, but wanted to
> first check what others think. I'll check the change into trunk by the end
> of the week unless I hear objections.
> >
> > Thanks!
> > - Beth
> >
> > https://jira.sakaiproject.org/browse/SAK-19464
> > https://jira.sakaiproject.org/browse/SAK-25448
> >
> > _______________________________________________
> > sakai-dev mailing list
> > sakai-dev at collab.sakaiproject.org
> > http://collab.sakaiproject.org/mailman/listinfo/sakai-dev
> >
> > TO UNSUBSCRIBE: send email to
> sakai-dev-unsubscribe at collab.sakaiproject.org with a subject of
> "unsubscribe"
> >
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://collab.sakaiproject.org/pipermail/sakai-dev/attachments/20131209/646a69ea/attachment.html 


More information about the sakai-dev mailing list