Cacheable annotation no longer allows groovy null check

[cdausmus]

Found when testing stats for institution admin, null pointer exceptions thrown when null value for institution was used. The Groovy null check operator ? no longer works in the Cacheable annotation:

@Cacheable(value = 'StatsTranscriptionTime', key = { "${institution?.id?.toString() ?: '-1'}-${startDate?.getTime() ?: '-1'}-${endDate?.getTime() ?: '-1'}" }) def getTranscriptionTimeByProjectType(Date startDate, Date endDate, Institution institution) {}

Needs to be updated to check for nulls the long way: "${(institution ? institution.id.toString() : '-1')}"

Testing check:

• Stats; Hourly Contribution
• Stats; Transcription Time
• Stats; Main Volunteer Stats
• Stats; Main Volunteer Contribution
• Get Audio Meta Data
• Get Image Meta Data
• Get Image Meta Data from File
• Find Max Sequence No
• User Has Validator
• User Has CAS Role