Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035576Openbravo ERP09. Financial managementpublic2017-03-17 14:012017-03-30 11:37
Reportermaite 
Assigned Tomarkmm82 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR16Q4.4Fixed in Version3.0PR16Q4.4 
Merge Request Status
Review Assigned Toaferraz
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2015-04-24
Regression introduced in release3.0PR15Q3
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/1881cfa4bb9c [^]
Triggers an Emergency PackNo
Summary0035576: "Adjustment Period" not used when posting a GL Journal
DescriptionIn case Adjustment Period is defined for your calendar and user tries to post gl journal document in that period it is not properly done
Steps To Reproduce1. Access Fiscal Calendar "US Calendar", go to Year 2017 and create new Period named "adjustment period", starting date=ending date=31-12-2017 and type= adjustment perio"
2. Access Open/Close period control window filter by "US Calendar" and "2017" year and run "Open period" process in adjustment period
3. Register GL Journal for "F&B US East Coast" organization, Date=31-12-2017, period= adjustment period and currency=USD
4. Add header setting "F&B International Group US/A/US Dollar" general ledger and verify that adjustment period is set
5. Add line for any account and any debit amount
6. Add another line for different account and same credit amount
7. Process header and post it
8. Access "Accounting transaction details" window and verify that period set is "Dec-17" instead of "adjustment period"

Moreover, you can verify that if you unpost journal, close Dec-17 period and try to post it again you will receive "Period closed" error. So period set in GL Journal document is not being considered
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0035560 closed markmm82 "Adjustment Period" not used when posting a GL Journal 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-03-20 15:00markmm82Typedefect => backport
2017-03-20 15:00markmm82Target Version => 3.0PR16Q4.4
2017-03-20 18:56hgbotCheckin
2017-03-20 18:56hgbotNote Added: 0095443
2017-03-20 18:56hgbotStatusscheduled => resolved
2017-03-20 18:56hgbotResolutionopen => fixed
2017-03-20 18:56hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR16Q4.4/rev/2e50c32a4286f0ec4b9d868ccfa3cf1cb95597eb [^]
2017-03-20 19:06markmm82Assigned Toluisrs8608 => markmm82
2017-03-20 19:09markmm82Review Assigned To => markmm82
2017-03-20 19:09markmm82Statusresolved => closed
2017-03-20 19:09markmm82Fixed in Version => 3.0PR16Q4.4
2017-03-30 09:23aferrazReview Assigned Tomarkmm82 => aferraz
2017-03-30 11:36hgbotCheckin
2017-03-30 11:36hgbotNote Added: 0095700
2017-03-30 11:37aferrazNote Added: 0095702

Notes
(0095443)
hgbot   
2017-03-20 18:56   
Repository: erp/backports/3.0PR16Q4.4
Changeset: 2e50c32a4286f0ec4b9d868ccfa3cf1cb95597eb
Author: Mark <markmm82 <at> gmail.com>
Date: Mon Mar 20 13:14:16 2017 -0400
URL: http://code.openbravo.com/erp/backports/3.0PR16Q4.4/rev/2e50c32a4286f0ec4b9d868ccfa3cf1cb95597eb [^]

Fixes issue 35576: "Adjustment Period" not used when posting a GL Journal

In case Adjustment Period was defined for a calendar and user tries to post gl journal document in that period it was not properly done.

It was fixed re-structuring the affected queries using the C_PERIOD_ID column of GL_JOURNAL and C_PERIOD tables, always retrieving this value as C_PERIOD_ID instead of "period" alias

---
M src/org/openbravo/erpCommon/ad_forms/DocGLJournal_data.xsql
---
(0095700)
hgbot   
2017-03-30 11:36   
Repository: erp/backports/3.0PR16Q4.4
Changeset: 8260e60d91d1d6260253eb697ea72898e256f837
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Thu Mar 30 10:25:21 2017 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR16Q4.4/rev/8260e60d91d1d6260253eb697ea72898e256f837 [^]

Related to issue 35576: Code review improvements

Backout fixes of issues 35576 & 28973 and fix issue 28973 properly:
DocGLJournal.isperiodOpen method was not correctly checking if DocGLJournalData.periodOpen retrieves no period.

---
M src/org/openbravo/erpCommon/ad_forms/DocGLJournal.java
M src/org/openbravo/erpCommon/ad_forms/DocGLJournal_data.xsql
---
(0095702)
aferraz   
2017-03-30 11:37   
Code review + Testing OK