fixed notional calculation for commodityforwards

Author: rkapl123

OREData/ored/portfolio/commodityforward.cpp

@@ -146,8 +146,7 @@ void CommodityForward::build(const boost::shared_ptr<EngineFactory>& engineFacto
     instrument_ = boost::make_shared<VanillaInstrument>(commodityForward);
     npvCurrency_ = fixingDate_==Date() ? currency_ : payCcy_;
 
-    // notional_ = strike_ * quantity_;
-    notional_ = Null<Real>(); // is handled by override of notional()
+    notional_ = strike_ * quantity_;
     notionalCurrency_ = currency_;
 
     additionalData_["quantity"] = quantity_;
@@ -167,18 +166,6 @@ void CommodityForward::build(const boost::shared_ptr<EngineFactory>& engineFacto
     additionalData_["isdaTransaction"] = string("");  
 }
 
-Real CommodityForward::notional() const {
-    // try to get the notional from the additional results of the instrument
-    try {
-        return instrument_->qlInstrument(true)->result<Real>("currentNotional");
-    } catch (const std::exception& e) {
-        if (strcmp(e.what(), "currentNotional not provided"))
-            ALOG("error when retrieving notional: " << e.what());
-    }
-    // if not provided, return null
-    return Null<Real>();
-}
-
 std::map<AssetClass, std::set<std::string>>
 CommodityForward::underlyingIndices(const boost::shared_ptr<ReferenceDataManager>& referenceDataManager) const {
     return {{AssetClass::COM, std::set<std::string>({commodityName_})}};