Item constructor cleanup

This is purely a code cleanup, nothing functionality-wise is changed.
The main change by LOC is invoking the item constructor like:
```item newitem("item_id");```
instead of:
```item newitem(itypes["item_id"]);```
The rest are minor issues I noticed while I was doing that.