Update everywhere that accesses an arg on policy to use {} as default arg for safety, since we can't use optional chaining

…money-request-updates

…saction violations. Fix and make that more clear with argument names

…money-request-updates

…money-request-updates

…money-request-updates

…money-request-updates

…money-request-updates

…money-request-updates

This component was typed before the actions file, and so it's not
inferring the correct type for the policy arg. Should be fixed when
IOU.js is typed.

…money-request-updates

Problems remaining:
1. Inconsistent types
2. Mismatch between Onyx types and what's actually sent, and my code
   based on what's actually sent doesn't account for bad types:

```
> new.expensify@1.4.36-5 typecheck
> tsc

src/components/ReportActionItem/MoneyRequestView.tsx:159:109 - error TS2345: Argument of type 'OnyxEntry<Policy>' is not assignable to parameter of type 'Policy'.
  Type 'null' is not assignable to type 'Policy'.

159             IOU.updateMoneyRequestBillable(transaction?.transactionID ?? '', report?.reportID, newBillable, policy, policyTags, policyCategories);
                                                                                                                ~~~~~~

Found 1 error in src/components/ReportActionItem/MoneyRequestView.tsx:159
```

It's never actually null, it's empty object

3. Need to double-check that everything is set by default when it needs
   to be

Partway through updating code, realizing that `updateMoneyRequest` is
now a completely dead code branch

…money-request-updates

This reverts commit 3493040.

…money-request-updates

…money-request-updates

Co-authored-by: Carlos Alvarez <cead22@gmail.com>