diff --git a/kmymoney/plugins/views/reports/core/querytable.h b/kmymoney/plugins/views/reports/core/querytable.h
--- a/kmymoney/plugins/views/reports/core/querytable.h
+++ b/kmymoney/plugins/views/reports/core/querytable.h
@@ -68,7 +68,7 @@
   void sumInvestmentValues(const ReportAccount &account, QList<CashFlowList> &cfList, QList<MyMoneyMoney> &shList) const;
   void constructPerformanceRow(const ReportAccount& account, TableRow& result, CashFlowList &all) const;
   void constructCapitalGainRow(const ReportAccount& account, TableRow& result) const;
-  MyMoneyMoney helperROI(const MyMoneyMoney& buys, const MyMoneyMoney& sells, const MyMoneyMoney& startingBal, const MyMoneyMoney& endingBal, const MyMoneyMoney& cashIncome) const;
+  QString helperROI(const MyMoneyMoney& buys, const MyMoneyMoney& sells, const MyMoneyMoney& startingBal, const MyMoneyMoney& endingBal, const MyMoneyMoney& cashIncome) const;
   QString helperIRR(const CashFlowList& all) const;
   void constructSplitsTable();
   bool linkEntries() const final override { return true; }
diff --git a/kmymoney/plugins/views/reports/core/querytable.cpp b/kmymoney/plugins/views/reports/core/querytable.cpp
--- a/kmymoney/plugins/views/reports/core/querytable.cpp
+++ b/kmymoney/plugins/views/reports/core/querytable.cpp
@@ -357,7 +357,7 @@
               if (subtotal == ctReturnInvestment)
                 totalsRow[subtotal] = helperROI((*currencyGrp).at(i + 1).value(ctBuys) - (*currencyGrp).at(i + 1).value(ctReinvestIncome), (*currencyGrp).at(i + 1).value(ctSells),
                                                 (*currencyGrp).at(i + 1).value(ctStartingBalance), (*currencyGrp).at(i + 1).value(ctEndingBalance) + (*currencyGrp).at(i + 1).value(ctMarketValue),
-                                                (*currencyGrp).at(i + 1).value(ctCashIncome)).toString();
+                                                (*currencyGrp).at(i + 1).value(ctCashIncome));
               else if (subtotal == ctPercentageGain)
                 totalsRow[subtotal] = (((*currencyGrp).at(i + 1).value(ctBuys) + (*currencyGrp).at(i + 1).value(ctMarketValue)) / (*currencyGrp).at(i + 1).value(ctBuys).abs()).toString();
               else if (subtotal == ctPrice)
@@ -417,7 +417,7 @@
           if (subtotal == ctReturnInvestment)
             totalsRow[subtotal] = helperROI((*currencyGrp).at(0).value(ctBuys) - (*currencyGrp).at(0).value(ctReinvestIncome), (*currencyGrp).at(0).value(ctSells),
                                             (*currencyGrp).at(0).value(ctStartingBalance), (*currencyGrp).at(0).value(ctEndingBalance) + (*currencyGrp).at(0).value(ctMarketValue),
-                                            (*currencyGrp).at(0).value(ctCashIncome)).toString();
+                                            (*currencyGrp).at(0).value(ctCashIncome));
           else if (subtotal == ctPercentageGain)
             totalsRow[subtotal] = (((*currencyGrp).at(0).value(ctBuys) + (*currencyGrp).at(0).value(ctMarketValue)) / (*currencyGrp).at(0).value(ctBuys).abs()).toString();
           else if (subtotal == ctPrice)
@@ -1054,15 +1054,14 @@
   }
 }
 
-MyMoneyMoney QueryTable::helperROI(const MyMoneyMoney &buys, const MyMoneyMoney &sells, const MyMoneyMoney &startingBal, const MyMoneyMoney &endingBal, const MyMoneyMoney &cashIncome) const
+QString QueryTable::helperROI(const MyMoneyMoney &buys, const MyMoneyMoney &sells, const MyMoneyMoney &startingBal, const MyMoneyMoney &endingBal, const MyMoneyMoney &cashIncome) const
 {
   MyMoneyMoney returnInvestment;
   if (!buys.isZero() || !startingBal.isZero()) {
     returnInvestment = (sells + buys + cashIncome + endingBal - startingBal) / (startingBal - buys);
-    returnInvestment = returnInvestment.convert(10000);
+    return returnInvestment.convert(10000).toString();
   } else
-    returnInvestment = MyMoneyMoney(); // if no investment then no return on investment
-  return returnInvestment;
+    return QString();
 }
 
 QString QueryTable::helperIRR(const CashFlowList &all) const
@@ -1404,11 +1403,9 @@
     break;
   }
 
-  MyMoneyMoney returnInvestment = helperROI(buysTotal - reinvestIncomeTotal, sellsTotal, startingBal, endingBal, cashIncomeTotal);
-
   result[ctBuys] = buysTotal.toString();
   result[ctReturn] = helperIRR(all);
-  result[ctReturnInvestment] = returnInvestment.toString();
+  result[ctReturnInvestment] = helperROI(buysTotal - reinvestIncomeTotal, sellsTotal, startingBal, endingBal, cashIncomeTotal);
   result[ctEquityType] = MyMoneySecurity::securityTypeToString(file->security(account.currencyId()).securityType());
 }