Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0023947
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementminoralways2013-05-30 17:052013-06-20 02:07
ReportermiruritaView Statuspublic 
Assigned Topramakrishnan 
PriorityhighResolutionfixedFixed in Version
StatusclosedFix in branchpiFixed in SCM revision3d39d5fbf846
ProjectionnoneETAnoneTarget Version
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_18
OS VersionCommunity ApplianceDatabase version8.3.9Ant version1.7.1
Product VersionpiSCM revision 
Review Assigned ToSandrahuguet
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0023947: Recalculate amortization process: when searching for existing amortization lines starting date should be ignored

DescriptionRecalculate amortization process: when searching for existing amortization lines starting date should be ignored

The amortization period is taking based only in the ending date.
Do the same for amortization line search.
Steps To Reproduce1) Create a new asset
Monthly - 5 months - Every Month is 30 days
Start date: 01-10-2013
Amount: 100

2) Calculate amortization
At this point the amortization of December 2013 contains one line.

3) Create a new asset
Percentage - 50%
Start date: 01-01-2013
Amount: 500

4) Calculate amortization

5) Remove the amortization line of 2014 year

6) Click on recalculate amortization button
Realize that the amortization line for 2013 year is created again.

When checking if the amortization line of 2013 exists it does not find because it is using the starting date as filtering criteria.
The start date for this period (it is yearly) is 01-01-2013 but the already existing amortization period (created in step 2) the starting date is 01-12-2013.
Proposed SolutionOnly take in account the ending date when searching an existing amortization line.
(see attached diff)
TagsNo tags attached.
Attached Filesdiff file icon issue23947_proposedsolution.diff [^] (1,762 bytes) 2013-05-30 17:08 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]

-  Notes
(0059187)
pramakrishnan (developer)
2013-06-05 15:16

Test Plan:
1) Create a new asset
Percentage - 50%
Start date: 01-01-2013
Amount: 500
2) Calculate amortization
3) Remove the amortization line of 2014 year
4) Click on recalculate amortization button
Realize that the amortization line for 2013 year is not created again
(0059257)
hgbot (developer)
2013-06-07 08:50

Repository: erp/devel/pi
Changeset: 3d39d5fbf84618f038deb2f81aa6092700e631be
Author: Pandeeswari Ramakrishnan <pandeeswari.ramakrishnan <at> openbravo.com>
Date: Mon Jun 03 11:01:15 2013 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/3d39d5fbf84618f038deb2f81aa6092700e631be [^]

Fixes Issue 23947: Recalculate amortization process
When searching for existing amortization lines starting date should be ignored

---
M src/org/openbravo/erpCommon/ad_process/assets/AssetLinearDepreciationMethodProcess.java
---
(0059262)
Sandrahuguet (developer)
2013-06-07 08:55

Code review + testing OK
(0059552)
hudsonbot (developer)
2013-06-20 02:07

A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/5caf871044ad [^]

Maturity status: Test

- Issue History
Date Modified Username Field Change
2013-05-30 17:05 mirurita New Issue
2013-05-30 17:05 mirurita Assigned To => dmiguelez
2013-05-30 17:05 mirurita Modules => Core
2013-05-30 17:05 mirurita Triggers an Emergency Pack => No
2013-05-30 17:08 mirurita File Added: issue23947_proposedsolution.diff
2013-05-30 17:08 mirurita Proposed Solution updated
2013-06-03 07:30 pramakrishnan Assigned To dmiguelez => pramakrishnan
2013-06-03 07:34 pramakrishnan Status new => scheduled
2013-06-03 07:34 pramakrishnan fix_in_branch => pi
2013-06-05 15:16 pramakrishnan Note Added: 0059187
2013-06-07 08:50 hgbot Checkin
2013-06-07 08:50 hgbot Note Added: 0059257
2013-06-07 08:50 hgbot Status scheduled => resolved
2013-06-07 08:50 hgbot Resolution open => fixed
2013-06-07 08:50 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/3d39d5fbf84618f038deb2f81aa6092700e631be [^]
2013-06-07 08:55 Sandrahuguet Review Assigned To => Sandrahuguet
2013-06-07 08:55 Sandrahuguet Note Added: 0059262
2013-06-07 08:55 Sandrahuguet Status resolved => closed
2013-06-20 02:07 hudsonbot Checkin
2013-06-20 02:07 hudsonbot Note Added: 0059552


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker