Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0023947 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 09. Financial management | minor | always | 2013-05-30 17:05 | 2013-06-20 02:07 | |||
Reporter | mirurita | View Status | public | |||||
Assigned To | pramakrishnan | |||||||
Priority | high | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | pi | Fixed in SCM revision | 3d39d5fbf846 | |||
Projection | none | ETA | none | Target Version | ||||
OS | Linux 32 bit | Database | PostgreSQL | Java version | 1.6.0_18 | |||
OS Version | Community Appliance | Database version | 8.3.9 | Ant version | 1.7.1 | |||
Product Version | pi | SCM revision | ||||||
Merge Request Status | ||||||||
Review Assigned To | Sandrahuguet | |||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0023947: Recalculate amortization process: when searching for existing amortization lines starting date should be ignored | |||||||
Description | Recalculate 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 Reproduce | 1) 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 Solution | Only take in account the ending date when searching an existing amortization line. (see attached diff) | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
|
![]() |
|
(0059187) pramakrishnan (viewer) 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 (viewer) 2013-06-07 08:55 |
Code review + testing OK |
(0059552) hudsonbot (viewer) 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 |
![]() |
|||
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 | OBNetwork customer | => No |
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 |