bug(inventory): ui-selects are buggy

[jniles]

In the Inventory Group management page, the ui-selects are a bit buggy. Specifically:

1. They do not order the accounts correctly - they order by number instead of by the account tree.
2. After selecting an account, the number and label do not line up.

See the two problems below:

Fig 1: Inventory Account Ordering Bug - these should be grouped by Title Account

Fig 2: Inventory Account Rendering Bug - the labels and numbers should line up.

Fixes
Thankfully, these are easy to fix. Here is what needs to happen:

1. The AccountService gives us the option to use Accounts.order(accounts) on an array of accounts to put them in the correct order. The solution will look like this:

Accounts.read()
.then(function (accounts) {
 // before
 vm.accounts = accounts;

 // after
 vm.accounts = Accounts.order(accounts);
});

See the bhAccountSelect code for how it solves this issue.

2. The only problem with the rendering is that there the entire ui-select default choice should be wrapped in a <span> element. That should fix the view rendering.