Don't call ItemRepository::close() twice in a row

ItemRepositoryRegistry::unRegisterRepository() is called only from
within ~ItemRepository(). ~ItemRepository() unconditionally calls
close() right after unRegisterRepository(). There is no need to lock the
repository's mutex in its destructor. So removing the duplicate close()
call along with the repository's mutex lock from unRegisterRepository()
must be safe.