Fix generation of trigger configurations

There is a consistency check in PropertyConfig::initProperties, which attempts to verify that toString / fromString are each others inverse. With Gaudi v27r1 (2016-patches), the check does not seem to work (fromString seems to have no effect on the property). (edit: the check does seem to be working, but the update handler does not seem to be passed to the clone ) . With Gaudi v28r0, this check is actually working and fromString triggers a call to the property's update handler.

The latter is a problem for LoKi filters, because it triggers the functor decoding, which can't succeed at this stage of execution (e.g. see this nighly):

HltGenConfig                              ERROR failure in update handler of 'Preambulo': HltPV3D:: Unable to decode the functor!

My solution is to remove the update handler from the cloned property before doing the check. This will potentially miss bad behaviour in the update handler, like modifying the property value.

Kernel/HltInterfaces/src/PropertyConfig.cpp

@@ -13,7 +13,10 @@ void PropertyConfig::initProperties(const IProperty& obj) {
         m_properties.emplace_back(i->name(),i->toString());
         // verify that toString / fromString are each others inverse...
         // WARNING: this does not guarantee that we don't loose information -- toString may be lossy!!!!
+        // WARNING: we remove the update handler (if any), which means that weird behaviour there (like
+        //          changing the property that was just updated) will be missed.
         std::unique_ptr<Property> clone( i->clone() );
+        clone->declareUpdateHandler(nullptr);
         if ( clone->fromString(m_properties.back().second).isFailure() ) {
             std::cerr << "Property::fromString fails to parse Property::toString" << std::endl;
             std::cerr << "component: " << kind() << " / " << name() << ", property type: " << i->type()<< " : " << i->name() << " -> " << i->toString() << std::endl;