Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0007111Openbravo ERP05. Production managementpublic2009-01-23 15:242012-03-20 10:58
plujan 
dmiguelez 
urgentminorhave not tried
closedout of date 
30
2.50alpha-r6 
 
Core
No
0007111: On Work Incidence, Starting Time field has no Time format
I can complete Starting Time of Work incidence and no time format is required. See attach.
Since is an starting time, it should have a Time format. If no time format is required, the label should be changed.
OB3-Reviewed, QA_review_pre_beta, VMA-Reviewed
related to defect 0009757 closed harikrishnan API change:api check: build 62 fails 
png BUG_250-a-r6-WrongHourFormat.PNG (45,073) 2009-01-23 15:24
https://issues.openbravo.com/file_download.php?file_id=716&type=bug
png
Issue History
2009-01-23 15:24plujanNew Issue
2009-01-23 15:24plujanAssigned To => rafaroda
2009-01-23 15:24plujansf_bug_id0 => 2531166
2009-01-23 15:24plujanFile Added: BUG_250-a-r6-WrongHourFormat.PNG
2009-01-23 15:24plujanRegression testing => No
2009-01-23 15:25plujanTag Attached: QA_review_pre_beta
2009-01-30 15:01rafarodaAssigned Torafaroda => AinhoaPagola
2009-01-30 15:01rafarodaPrioritynormal => low
2009-01-30 15:01rafarodaStatusnew => scheduled
2009-03-09 18:05AinhoaPagolaNote Added: 0014532
2009-06-10 11:57rafarodaAssigned ToAinhoaPagola => sathiyan
2009-06-26 12:13sathiyanAssigned Tosathiyan => harikrishnan
2009-06-30 13:00hgbotCheckin
2009-06-30 13:00hgbotNote Added: 0017670
2009-06-30 13:00hgbotStatusscheduled => resolved
2009-06-30 13:00hgbotResolutionopen => fixed
2009-06-30 13:00hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/1d333b902d5803c291688fe6a7def920a6c97b43 [^]
2009-07-03 11:15shuehnerRelationship addedrelated to 0009757
2009-07-03 19:22pjuvaraStatusresolved => new
2009-07-03 19:22pjuvaraResolutionfixed => open
2009-07-03 19:22pjuvaraNote Added: 0017902
2009-07-03 19:22pjuvaraNote Edited: 0017902
2009-07-03 19:31pjuvaraNote Added: 0017905
2009-07-07 12:53hgbotCheckin
2009-07-07 12:53hgbotNote Added: 0017964
2009-07-07 12:53hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/1d333b902d5803c291688fe6a7def920a6c97b43 [^] => http://code.openbravo.com/erp/devel/pi/rev/e2aa45438b62f2f32aca9eac409e6f8a7b2700bb [^]
2009-07-10 16:18rafarodaStatusnew => scheduled
2009-07-10 16:18rafarodaFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/e2aa45438b62f2f32aca9eac409e6f8a7b2700bb [^] =>
2011-10-27 19:16psarobeTag Attached: OB3-Reviewed
2011-10-28 18:32psarobeAssigned Toharikrishnan => jonalegriaesarte
2011-10-28 18:32psarobePrioritylow => urgent
2011-10-28 18:32psarobeFixed in Version => 3.0MP6
2011-10-31 11:15jonalegriaesarteFixed in Version3.0MP6 =>
2011-10-31 11:15jonalegriaesarteTarget Version => 3.0MP6
2011-11-25 17:24jonalegriaesarteNote Added: 0043354
2011-11-25 17:24jonalegriaesarteTarget Version3.0MP6 => 3.0MP7
2011-12-30 10:23jonalegriaesarteTarget Version3.0MP7 => 3.0MP8
2012-01-25 09:48jonalegriaesarteTarget Version3.0MP8 =>
2012-02-15 19:03iciordiaAssigned Tojonalegriaesarte => vmromanos
2012-02-17 08:54vmromanosTag Attached: VMA-Reviewed
2012-03-20 09:24dmiguelezAssigned Tovmromanos => dmiguelez
2012-03-20 10:56dmiguelezNote Added: 0046531
2012-03-20 10:56dmiguelezStatusscheduled => closed
2012-03-20 10:56dmiguelezResolutionopen => out of date
2012-03-20 10:58dmiguelezNote Added: 0046533

Notes
(0014532)
AinhoaPagola   
2009-03-09 18:05   
Starting time label is wrong as this field should contain standard time.
It should be a number and actually is a String.
(0017670)
hgbot   
2009-06-30 13:00   
Repository: erp/devel/pi
Changeset: 1d333b902d5803c291688fe6a7def920a6c97b43
Author: harikrishnan
Date: Tue Jun 30 16:23:03 2009 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/1d333b902d5803c291688fe6a7def920a6c97b43 [^]

Fixes Issue 7111: On Work Incidence, Starting Time field has no Time format

---
M src-db/database/model/tables/MA_INCIDENCE.xml
M src-db/database/sourcedata/AD_COLUMN.xml
---
(0017902)
pjuvara   
2009-07-03 19:22   
The fix that has been implemented is incorrect as it breaks a public API.

It needs to be backed out as soon as possible.

(0017905)
pjuvara   
2009-07-03 19:31   
Comment added to 9757:

After the comment by shuehner, I now understand that the fix was technically correct.
Yet, I believe it is not appropriate to break a public API in order to fix a minor issue that was not reported by a customer (7111 was reported internally by QA).

So I confirm my decision to reject this exception and I request the fix to be backed out.

Harikrishnan should investigate an alternative fix for 7111 that does not involve breaking the API. If that is not possible, we should postpone the fix for 7111 to after we have created the maintenance repository for 2.50 so that we can fix the issue for the next release without breaking the existing API.
(0017964)
hgbot   
2009-07-07 12:53   
Repository: erp/devel/pi
Changeset: e2aa45438b62f2f32aca9eac409e6f8a7b2700bb
Author: Harikrishnan Raja <harikrishnan.raja <at> openbravo.com>
Date: Tue Jul 07 16:16:16 2009 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/e2aa45438b62f2f32aca9eac409e6f8a7b2700bb [^]

Its a backed out commit for the issue 7111

---
M src-db/database/model/tables/MA_INCIDENCE.xml
M src-db/database/sourcedata/AD_COLUMN.xml
---
(0043354)
jonalegriaesarte   
2011-11-25 17:24   
Targer version changed to MP7 due to other priorities
(0046531)
dmiguelez   
2012-03-20 10:56   
The label is already renamed to "Work Incidence Downtime"
In the User Documentation is explained that the information is not taken into account for any cost calculation and is up to the user to decide the unit of mesaure of the field.
(0046533)
dmiguelez   
2012-03-20 10:58   
Test Plan:

Login as Openbravo/openbravo

Go to the Work Incidence Window and create a new Work Incidence.
Notice that there is a field name "Work Incidence Downtime"