Different behavior of `new_locked_contract()` than documented

According to [new_locked_contract()](https://github.com/casper-network/casper-node/blob/e688fbc75206652dce440c3ddb9570c965382d37/smart_contracts/contract/src/contract_api/storage.rs#L114) documentation:

> if `hash_name` is provided, puts contract hash in current context's named keys under `hash_name`

That is not true, because **ContractPackage** is stored at given key name. I would rather expect to see there `HashAddr` of contract itself (not package).

However, if you store _contract hash_ as `Key::Hash(HashAddr)`, then querying it will not give you the address:


I was able to store it manually via URef, but this seems to be quite counter intuitive.

_**Note:** Documentation/behavior of `new_contract()` is incorrect as well, but I consider upgradable contracts as anti-feature._