Finance | Project Operations, Human Resources, ...

Answered

Error Chain of command method must contain one next call.

(3) Share

Report

Posted on by Shabir Ahmad

3,110

Hi experts,

I want to add 4 new methods to Standard class "CurrencyExchangeHelper" . I already tried by getting error "Error Chain of command method must contain one next call." I have allready checked some posted questions but didn't get it could you please tell me how to do it below are 4 methods

public static Amount roundWithRuleTypeCurrencyTable(Amount _amount, Currency _currency, CurrencyRoundingRuleType _currencyRoundingRuleType)
{
Amount result;

if (_amount == 0)
{
return 0;
}

switch (_currencyRoundingRuleType)
{
case CurrencyRoundingRuleType::Amount:

result = CurrencyExchangeHelper::roundWithRule(
_amount,
_currency.RoundingPrecision,
RoundOffType::Ordinary);

break;

case CurrencyRoundingRuleType::Price:

result = CurrencyExchangeHelper::roundWithRule(
_amount,
_currency.RoundOffPrice,
_currency.RoundOffTypePrice);

break;

case CurrencyRoundingRuleType::PurchaseOrder:

result = CurrencyExchangeHelper::roundWithRule(
_amount,
_currency.RoundOffPurch,
_currency.RoundOffTypePurch);

break;

case CurrencyRoundingRuleType::SalesOrder:

result = CurrencyExchangeHelper::roundWithRule(
_amount,
_currency.RoundOffSales,
_currency.RoundOffTypeSales);

break;

// <GJP>
case CurrencyRoundingRuleType::AssetDep_JP:

result = CurrencyExchangeHelper::roundWithRule(
_amount,
_currency.RoundOffAssetDep_JP,
_currency.RoundOffTypeAssetDep_JP);

break;
// </GJP>

default:

throw error(Error::wrongUseOfFunction(funcname()));
}

return result;
}

public static Amount roundWithRuleType(Amount _amount, CurrencyCode _currency, CurrencyRoundingRuleType _currencyRoundingRuleType)
{
Currency currency;

if (_amount == 0)
{
return 0;
}

if (_currency)
{
currency = Currency::find(_currency);
}
else
{
currency = Currency::find(Ledger::accountingCurrency());
}

return CurrencyExchangeHelper::roundWithRuleTypeCurrencyTable(_amount, currency, _currencyRoundingRuleType);
}

public static Amount roundWithRule(Amount _amount, RoundOff _roundOffAmount, RoundOffType _roundOffType)
{
RoundOff roundOffAmountLoc = _roundOffAmount;
Amount result;

if (_amount == 0)
{
return 0;
}

switch (_roundOffType)
{
case RoundOffType::Ordinary:

if (!roundOffAmountLoc)
{
roundOffAmountLoc = 0.01;
}

result = round(_amount, roundOffAmountLoc);

break;

case RoundOffType::RoundDown:

if (!roundOffAmountLoc)
{
roundOffAmountLoc = 1.0;
}

// round to zero since from a financial standpoint that's what
// we mean by rounding down
result = roundZero(_amount, roundOffAmountLoc);

break;

case RoundOffType::RoundUp:

if (!roundOffAmountLoc)
{
roundOffAmountLoc = 1.0;
}

if (_amount >= 0)
{
result = roundUp(_amount, roundOffAmountLoc);
}
else
{
result = roundDown(_amount, roundOffAmountLoc);
}

break;

default:

throw error(Error::wrongUseOfFunction(funcname()));
}

return result;
}

public static Amount round(Amount _amount, CurrencyCode _currency)
{
return CurrencyExchangeHelper::roundWithRuleType(
_amount,
_currency,
CurrencyRoundingRuleType::Amount);
}

Thanks and Regards,

Shabir Ahmad

Categories:

Dynamics 365 Finance

I have the same question (0)

All responses (13)

Answers (2)

Suggested answer

Waed Ayyad 9,039 Super User 2025 Season 2 on at

Like (0)

Report
Hi,

In Chain of Command, if you make an extension of standard method, you should use next() keyword before or after your logic, so in your code can you tell me if any of the four methods that you added exist in the standard method?

For Example, if amount method is a standard method, then you should write the following code:
public static Amount round(Amount _amount, CurrencyCode _currency) { Amount roundedAmount = next round(_amount,_currency); roundedAmount = CurrencyExchangeHelper::roundWithRuleType( _amount, _currency, CurrencyRoundingRuleType::Amount); return roundedAmount; }

To read more about COC, Check the following link:
https://dynamics365musings.com/chain-of-command-class-methods/

Thanks,

Waed Ayyad

If this helped, please mark it as "Verified" for others facing the same issue

Was this reply helpful? Yes No
Martin Dráb 237,965 Most Valuable Professional on at

Like (0)

Report

What's the whole point of your customization? It looks like you merely copied methods from CurrencyExchange, which gives you no benefit. You can simply use the standard methods instead of your own.

If you think you need an extension, please show us the header of your class. It sounds to me like if you;re extending CurrencyExchange, not CurrencyExchangeHelper.

Moved from Integration, Dataverse, and general topics forum.

Was this reply helpful? Yes No
Shabir Ahmad 3,110 on at

Like (1)

Report

Thanks to both of you ,

Actually, the methods which i have added here is exist in dynamics ax 2012. But in D365 FO "CurrencyExchangeHelper" Class these methods are not here i just tried to bring from ax 2012 to d365 FO. These methods are not existing methods. it is actually new in d365 FO.

Thanks and Regards,

Shabir Ahmad

Was this reply helpful? Yes No
Waed Ayyad 9,039 Super User 2025 Season 2 on at

Like (0)

Report

Hi,

Does CurrencyExchangeHelper class extend another class? Can you show us the error message that you received and your code from the beginning?

Thanks,

Waed Ayyad

Was this reply helpful? Yes No
Verified answer

Martin Dráb 237,965 Most Valuable Professional on at

Like (0)

Report

The methods already exist, you're just using a wrong class. You'll find them in CurrencyExchange class, not CurrencyExchangeHelper class.

Was this reply helpful? Yes No
Shabir Ahmad 3,110 on at

Like (0)

Report

Hi ,

Was this reply helpful? Yes No
Shabir Ahmad 3,110 on at

Like (0)

Report

Hi,

these screen shots are from D365 FO

Was this reply helpful? Yes No
Shabir Ahmad 3,110 on at

Like (0)

Report

Was this reply helpful? Yes No
Verified answer

Layan Jwei 8,118 Super User 2025 Season 2 on at

Like (0)

Report
Hi Shabir,

"CurrencyExchangeHelper" class extends "CurrencyExchange" class, which means it inherits all of it's methods

if you go inside the class "CurrencyExchange" then you will see that those methods are already there, there is no need to add them

so all you need to do is to call one of the methods like this:
CurrencyExchange::round(123, "GBP");

Thanks,
Layan Jweihan
Please mark this answer as "Verified" if it solved your issue. In order to help others who will face a similar issue in the future

Was this reply helpful? Yes No
Shabir Ahmad 3,110 on at

Like (1)

Report

Thanks to both of you

Was this reply helpful? Yes No