SakaiContrib

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Done

Done

Done.

Done.

Done

Done

Done

Done

Done

Done

fix formatting

fix formatting

Always use StringBuffer for concatenation - a lot less expensive: directURL = new StringBuffer( ServerConfigurationService.getPortalUrl() ); directURL.append( serverUrl); etc return directURL.toStr...

Always use StringBuffer for concatenation - a lot less expensive:
directURL = new StringBuffer( ServerConfigurationService.getPortalUrl() );
directURL.append( serverUrl);
etc
return directURL.toString()

Use constant (see above)

Use constant (see above)

I see you're just repeating a pattern all over the code of hard-coding this constant again and again, but it's still bad practice https://crucible.sakaiproject.org/static/nsjxqv/2static/images/wiki...

I see you're just repeating a pattern all over the code of hard-coding this constant again and again, but it's still bad practice
Please add this as a constant to MnemeService, and be sure to differentiate it from the APPLICATION_ID – I'd call it "SAKAI_MNEME_TOOL_ID.

Date should be 2015

Date should be 2015

Agreed

Agreed

Should the function name be: getMyAssessmentsForAllSites()?

Should the function name be: getMyAssessmentsForAllSites()?

UMICH-1182 extending the entity feed to have dates/ direct url
UMICH-1182 extending the entity feed to have dates/ direct url
OK

OK

The thing is the mneme feed should not crash if the GB tool is associated or not associated with it. So i put better comment next time

The thing is the mneme feed should not crash if the GB tool is associated or not associated with it. So i put better comment next time

These statements say the exact opposite of each other: "Mneme RESTful feed crashes without gradebook tool" and "Mneme api feed won't crash if gradebook tool is not associated with it"

These statements say the exact opposite of each other: "Mneme RESTful feed crashes without gradebook tool" and "Mneme api feed won't crash if gradebook tool is not associated with it"

This comment is probably not necessary. The symbols are well-named, so it's easy to understand what's happening there.

This comment is probably not necessary. The symbols are well-named, so it's easy to understand what's happening there.

UMICH-1171 Mneme api feed won't crash if gradebook tool is not associated with it
UMICH-1171 Mneme api feed won't crash if gradebook tool is not associated with it
No comments – looks good!

No comments – looks good!

UMICH-1152 Update Inactivate Tool to work with deprecated tools
UMICH-1152 Update Inactivate Tool to work with deprecated tools
Also include a version of the URLs that would work for the command line server version. I suspect that people will try that first.

Also include a version of the URLs that would work for the command line server version. I suspect that people will try that first.

Some place needs to explain what the purpose / contents of the test files.

Some place needs to explain what the purpose / contents of the test files.

An quick overview section on how data flows through the stub server would be helpful. E.g. list what query types it responds to, what data matters, how it looks for the data to return, what needs t...

An quick overview section on how data flows through the stub server would be helpful. E.g. list what query types it responds to, what data matters, how it looks for the data to return, what needs to be in the data files.

Add "FROM THE COMMAND LINE" to the title

Add "FROM THE COMMAND LINE" to the title