...
- event pack should not have to declare a dependency on conditions-impl. This is in there now in order to avoid a
ClassdefNotFound
error when we ask theNotificationService
to reconstitute aResourceReleaseRule
from the database. It should be able to get aClass
object from the SakaiComponentManager
instead. Look inBaseNotificationService
for where it does this:Code Block m_action = (NotificationAction) Class.forName(className).newInstance();
GradebookExternalAssessmentServiceImpl
should not use a static cover of theEventTrackingService
. This is a case of doing the quick thing rather than the right thing. There's a Spring bean configuration file in there. We just need to modify it to add an injectedEventTrackingService
.- The behavior of the content tool UI should not depend on the index number of items in the
<select>
tags. There are hard-coded references to magic indexes inResourcesAction
,ListItem
, andsakai_properties_scripts.vm
. It's ugly.