Definition: fix loadToCache to return `id`

Fix the bug introduced by 47837ff where `Definition.loadToCache`
no longer returned the id of the created instance. That caused
`Facet.loadIntoCache` to throw because facetId was undefined.