Should you save the Session.get 'currentGroupId' into a variable first so that you don't have to make a call to Session.get again in this line? I'm more asking for my own code to know what is the preferred way - two calls and less lines of code or the extra line of code to save the call to a variable and reusing it below? I've done the second approach, you do the first, so we should probably consolidate to one.