Hi,
While looking at SLING-4207, I have noticed that the current implementation
of tenant creation does not match with the API description in case the
tenant already exists.
Indeed, the API stipulates
* @throws IllegalArgumentException if a tenant with the same {@code
tentantId} already exists.
org.apache.sling.tenant.TenantManager#create
where in fact the implementation does return null in this case.
I have opened SLING-4234 to track matching the implementation to the API.
The question is, should the Javadoc change to match what the implementation
does (arguably, lots of code has already be written to support the null
reference) or align the implementation on the API (potentially breaking
existing code).
Regards,
Timothee
