digitalSTROM Server

when a raise-eventsubscription has a _override parameter, it should be transfered to the event, regardless what is set before ... also _default parameter should be provided to the raised event

If you look at that code-snippet als the second line before end ( newEvent->setProperties(_event.getProperties());)
, you will clearly see, that all _default parameters and _override parameters are completly wasted. please 1. move up the line one time and 2. make a "adding" instead of "replacing" call

void EventInterpreterPluginRaiseEvent::handleEvent(Event& _event, const EventSubscription& _subscription) {
    boost::shared_ptr<Event> newEvent(new Event(_subscription.getOptions().getParameter("event_name")));
    if(_subscription.getOptions().hasParameter("time")) {
      std::string timeParam = _subscription.getOptions().getParameter("time");
      if(!timeParam.empty()) {
        Logger::getInstance()->log("RaiseEvent: Event has time");
        newEvent->setTime(timeParam);
      }
    }
    applyOptionsWithSuffix(_subscription.getOptions(), "_default", newEvent);
    if(_subscription.getOptions().hasParameter(EventPropertyLocation)) {
      std::string location = _subscription.getOptions().getParameter(EventPropertyLocation);
      if(!location.empty()) {
        Logger::getInstance()->log("RaiseEvent: Event has location");
        newEvent->setLocation(location);
      }
    }
    applyOptionsWithSuffix(_subscription.getOptions(), "_override", newEvent);
    newEvent->setProperties(_event.getProperties());
    getEventInterpreter().getQueue().pushEvent(newEvent);
  } // handleEvent