[1138328] globally uncaught exception on storage metrics admin page
Although I was still unable to reproduce this bug I did find a couple of
places where we needed to shore things up, that perhaps contributed.
- fix getResourceIdFromStorageNode utility to throw the correct exceptions,
  this solves issues where caller catch definitions were not catching.
- when uninventorying a Storage Node Resource we did not pro-actively unlink
  it from the StorageNode entity.  We did the unlink in the async work but that
  that left the bad resource linked until async job ran or until
  it was re-linked to a newly imported SN resource.  Now unlinked up front
  (which is unfortunate as it adds another db query per batch).
Bug 1062789 - Set quoting to enabled for newly created script server resources

Changed "quotingEnabled" attribute default value
Made ScriptDiscoveryComponent implement ResourceUpgradeFacet
Tracking upgraded resources with an hidden plugin config property ("quotingEnabledNeedsUpgrade")
Logging message when a Script Server component is started and "quotingEnabled" property is missing
Updated the "getConfiguredEscapeCharacter" method of the Script Server component class