Complete migration to V2 of TrigComposite Aux

!20600 (merged) missed the one-letter change to put this V2 EDM change into action.

This MR remedies this.

TrigCostMTSvc 0 0 WARNING Failed to append one or more details to trigger cost TC
ERROR xAOD::ByteStreamAuxContainer_v1::getData Unknown variable type (NSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEE) requested
xAOD::TrigComposite_v1::setDetail ERROR Internal logic error found: [SG::ExcBadAuxVar: Attempt to retrieve nonexistent aux data item `::store' (390).]
ERROR xAOD::ByteStreamAuxContainer_v1::getData Unknown variable type (s) requested
xAOD::TrigComposite_v1::setDetail ERROR Internal logic error found: [SG::ExcBadAuxVar: Attempt to retrieve nonexistent aux data item `::view' (391).]
...

http://atlas-computing.web.cern.ch/atlas-computing/links/distDirectory/gitwww/GITWebArea/nightlies/master/2019-02-06T2349/Athena/x86_64-slc6-gcc8-dbg/Athena/UT/TrigUpgradeTest.log.html

Caused by a move to a fully template-ed interface and removal of type massaging.

I touch a file in TrigUpgradeTest this time to trigger the unit tests. This wasn't done in !20600 (merged) which is why it didn't get caught.

Marking urgent as causes an error in CI

Trigger/TrigMonitoring/TrigCostMonitorMT/src/TrigCostMTSvc.cxx

@@ -131,7 +131,7 @@ StatusCode TrigCostMTSvc::endEvent(const EventContext& context, SG::WriteHandle<
     for ( const xAOD::TrigComposite* tc : *outputHandle ) {
       ATH_MSG_INFO("Algorithm:'" << tc->name() << "'");
       ATH_MSG_INFO("  Store:'" << tc->getDetail<std::string>("store") << "'");
-      ATH_MSG_INFO("  View ID:" << tc->getDetail<int>("view"));
+      ATH_MSG_INFO("  View ID:" << tc->getDetail<int16_t>("view"));
       ATH_MSG_INFO("  Thread ID Hash:" << tc->getDetail<uint32_t>("threadID") );
       ATH_MSG_INFO("  Start Time Seconds:" << tc->getDetail<uint32_t>("startTimeSeconds") << " s");
       ATH_MSG_INFO("  Start Time Microseconds:" << tc->getDetail<uint32_t>("startTimeMicroseconds") << " mu s");