Description

Presently, the RemoteEntityLookup class (implemented in repo/includes/RemoteEntity/RemoteEntityLookup.php) includes both a fetchEntity($conceptUri) method and a getEntity($conceptUri) method. fetchEntity fetches the value (first checking the local cache, then fetching from remote) without saving it to the local cache. getEntity fetches the value (first checking the local cache, then fetching from remote) and then saves it to the local cache.

Obviously, there is some overlapping code that could be replaced with a call to fetchEntity from within getEntity, but the question is do we need fetchEntity at all? Is RemoteEntityLookup following suit to some other lookup class that also implements both? Or can we (please) just do something like getEntity($conceptUri, bool $save = false)?

Acceptance Criteria

• Investigate and answer the above questions
• Write a follow-up implementation ticket