0032711: SL_Depreciate callout is not working having AssetGroup with no default values

Description

SL_Depreciate callout is not working when field Depreciate field is not checked in Asset Group and no values are set in not mandatory columns, for example "Depreciation Type" and "Calculate Type"

Steps To Reproduce

1) Set columns "Depreciation Type" and "Calculate Type" as NOT MANDATORY in "FinancialMgmtAssetGroup" table
2) Create new "Asset Group" having Depreciate field unchecked
3) Create new "Asset" with this new "Asset Category"
4) Check Depreciate checkbox, not able to display the list until go to another tab and come back.

Proposed Solution

Create condition in "SL_Depreciate" callout to check the values only if the field isDepreciate() is set:

if (assetGroup.isDepreciate()) {
          info.addResult("inpisdepreciated",assetGroup.isDepreciate());
          info.addResult("inpamortizationtype",assetGroup.getDepreciationType());
    ...
}

Tags

No tags attached.

Attached Files

Relationships [ Relation Graph ] [ Dependency Graph ]

Notes
(0096086) CSousa (developer) 2017-04-20 13:02	1) Set columns "Depreciation Type" and "Calculate Type" as NOT MANDATORY in "FinancialMgmtAssetGroup" table 2) Create new "Asset Group" having Depreciate field unchecked 3) Create new "Asset" with this new "Asset Category" 4) Check Depreciate checkbox, able to display the list.

(0099129) aferraz (manager) 2017-09-18 17:13	Callout works properly if we don't change the definition of FinancialMgmtAssetGroup table.

Issue History
Date Modified	Username	Field	Change
2016-04-19 12:53	JONHM	New Issue
2016-04-19 12:53	JONHM	Assigned To	=> Triage Finance
2016-04-19 12:53	JONHM	Modules	=> Core
2016-04-19 12:53	JONHM	Triggers an Emergency Pack	=> No
2017-04-20 13:02	CSousa	Note Added: 0096086
2017-05-08 13:19	CSousa	Status	new => scheduled
2017-05-08 13:22	CSousa	Assigned To	Triage Finance => CSousa
2017-09-18 17:13	aferraz	Review Assigned To	=> aferraz
2017-09-18 17:13	aferraz	Note Added: 0099129
2017-09-18 17:13	aferraz	Status	scheduled => closed
2017-09-18 17:13	aferraz	Resolution	open => no change required