If we go through the call stack, we can see that the cache is actually being updated each time. However, some linters will alert the user if a return statement is also being used to assign. So this is something that I may look to rectify. Thanks :)