Matthew Jones

Yeah, thanks. I think I'd wanted the default to be true, not sure why I'd left it false. Just wanted it to be able to be disabled in-case of performance or other concerns.

Yeah, thanks. I think I'd wanted the default to be true, not sure why I'd left it false. Just wanted it to be able to be disabled in-case of performance or other concerns.

SAK-25556 - Annnouncements: Assignment Open Date announcement in wrong order in Home > Recent Announcements
SAK-25556 - Annnouncements: Assignment Open Date announcement in wrong order in Home > Recent Announcements
SAK-22384 - Contributed patch from UWO - Support MathJax rendering in Sakai
SAK-22384 - Contributed patch from UWO - Support MathJax rendering in Sakai
SAK-23143 - Clean up new properties on pages and tools
SAK-23143 - Clean up new properties on pages and tools
No I'm not even sure how I'd test .anon or .auth? This original patch (long ago) set up a security advisor in the session that gives access to the special private melete docs folder. The bug here w...

No I'm not even sure how I'd test .anon or .auth? This original patch (long ago) set up a security advisor in the session that gives access to the special private melete docs folder. The bug here was that because the advisor was pushed at the beginning of the method (if it existed) that the user didn't have access to any other content. So it has to only push it when trying ot get to the special private content.

If you have a test case let me know.

Yea that's a good comment, I can try to pop that specific one, though it shouldn't be an issue

Yea that's a good comment, I can try to pop that specific one, though it shouldn't be an issue

SAK-24209 - Private collections (melete docs) not supported by iSyllabus)
SAK-24209 - Private collections (melete docs) not supported by iSyllabus)
I'd rename and prefix this with chat, something like chat.delivery.ttl=60. Eventually we'll want to clean up all of these properties.

I'd rename and prefix this with chat, something like chat.delivery.ttl=60. Eventually we'll want to clean up all of these properties.

Okay, I updated the source in rev 129945 and this looks like it reflects that. Thanks for the feedback. Also caught a hanging i.

Okay, I updated the source in rev 129945 and this looks like it reflects that. Thanks for the feedback. Also caught a hanging i.

Yea, I can never seem to get complete consistency with tabs/spaces. I generally use eclipse to edit and haven't changed the defaults but sometimes the file has tabs other times it has spaces and it...

Yea, I can never seem to get complete consistency with tabs/spaces. I generally use eclipse to edit and haven't changed the defaults but sometimes the file has tabs other times it has spaces and it always looks perfect but never looks perfect in the diff. It looks like Sakai project is doing 8 spaces to a tab and in a few places it was expanding the tabs to 4 spaces.

I can fix all of these but not 100% sure how it happens.

Not really a bug, just surprised that SCS doesn't have a getLong method which makes this conversion pattern required.

Not really a bug, just surprised that SCS doesn't have a getLong method which makes this conversion pattern required.

Review SAK-23897 Site Info -> Import from site needs to run in a seperate thread and email user when finished
Review SAK-23897 Site Info -> Import from site needs to run in a seperate thread and email user when finished
I reverted CategoryDefintion change because it was unneeded, the category information was pulled from Assignment. Some refactoring might be useful to pull more information from CategoryDefinition t...

I reverted CategoryDefintion change because it was unneeded, the category information was pulled from Assignment. Some refactoring might be useful to pull more information from CategoryDefinition though.

SAK-23896 - Import from site with EC category results in NPE
SAK-23896 - Import from site with EC category results in NPE
  • More
  • CR-4
  • Abandoned
  • More
  • CR-2
  • Abandoned