[Portfolio] OSP SiteEventListener.java

Bryan Holladay holladay at longsight.com
Thu Sep 20 12:19:58 PDT 2012


I removed the listened locally and created a new site without any
tools.  Then added all OSP tools, checked permissions and they were
all there.  Then removed and re-added the tools again and still there.
 This makes sense since OSP used to have a different permission
handling system and now that its on Sakai's it should behave the same.
 I can commit this to trunk if everyone else agrees (let me know).
Also, I think this should go into 2.9.x since it's a huge slowdown for
siteService.save(site).

-Bryan

On Wed, Sep 19, 2012 at 10:48 PM, Maurer, Christopher Wayne
<chmaurer at iupui.edu> wrote:
> I believe that's the listener that allowed osp tools to get default
> permissions after adding a tool to the site.  I know that it was used back
> in the day before we integrated with the normal sakai template permission
> stuff, so it's possible that it isn't necessary anymore.  Might try not
> registering it and see if tool permissions still look right when
> adding/removing tools.
>
> Chris
>
> On 9/19/12 4:15 PM, "Bryan Holladay" <holladay at longsight.com> wrote:
>
>>What does this class do?
>>https://source.sakaiproject.org/svn/osp/trunk/common/api-impl/src/java/org
>>/theospi/portfolio/worksite/mgt/impl/SiteEventListener.java
>>
>>I am running Yourkit and this stood out as 79% of the load when
>>SiteService.save(Site) is called.  I've attached a screen shot.  Is it
>>ok to turn off or disable w/sakai.property?
>>
>>Thanks,
>>Bryan
>


More information about the portfolio mailing list