Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0044933 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
backport | [Openbravo ERP] A. Platform | major | have not tried | 2020-08-17 13:50 | 2020-08-31 11:50 | |||
Reporter | cberner | View Status | public | |||||
Assigned To | cberner | |||||||
Priority | immediate | Resolution | fixed | Fixed in Version | PR20Q2.2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | |||||
Projection | none | ETA | none | Target Version | PR20Q2.2 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | approved | |||||||
Review Assigned To | ||||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | Production - Confirmed Stable | |||||||
Regression date | 2019-10-15 | |||||||
Regression introduced in release | 3.0PR20Q1 | |||||||
Regression introduced by commit | https://gitlab.com/openbravo/product/openbravo/-/commit/dded7dec5f17b8b1fc8f6c118af985c5274670a5 [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0044933: Scheduling a process without start date or start time fails | |||||||
Description | When trying to schedule or reschedule a process request without start date or start time it fails with a parse date exception. In logs: Couldn't create quartz trigger for process Acct Server Process java.text.ParseException: Could not parse date at org.openbravo.scheduling.SchedulerTimeUtils.parse(SchedulerTimeUtils.java:81) ~[classes/:?] | |||||||
Steps To Reproduce | - Open Process Request - Create new Process (e.g. Log Clean Up) - Set to scheduled and empty Start Date or Start Time fields - Save Process Request and try to schedule it Check logs -> There's should be a parse date error. | |||||||
Proposed Solution | If date/time is missing, use default = now in SchedulerTimeUtils parse function. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
||||||||
|
![]() |
|
(0122469) hgbot (developer) 2020-08-31 10:06 |
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/130 [^] |
(0122476) hgbot (developer) 2020-08-31 11:50 |
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/130 [^] |
(0122477) hgbot (developer) 2020-08-31 11:50 |
Directly closing issue as related merge request is already approved. Repository: https://gitlab.com/openbravo/product/openbravo [^] Changeset: a5968298257987264d8e61e056f812bedfda647c Author: Cristian Berner <cristian.berner@openbravo.com> Date: 2020-08-31T10:05:26+02:00 URL: https://gitlab.com/openbravo/product/openbravo/-/commit/a5968298257987264d8e61e056f812bedfda647c [^] Fixes ISSUE-44933: Allow scheduling a process without start date or time Previously it was possible to schedule a process without start date or time, assuming current date and current time. This was lost in a refactor. A new method getCurrentDateTime has been added to SchedulerTimeUtils to get the missing part of date/time provided assuming "now" when missing. It will return the date + time in the expected format for timestamp and parse. Fixes regression introduced in commit dded7dec5f17b8b1fc8f6c118af985c5274670a5 --- M src/org/openbravo/scheduling/SchedulerTimeUtils.java M src/org/openbravo/scheduling/trigger/LaterTriggerGenerator.java M src/org/openbravo/scheduling/trigger/ScheduledTriggerGenerator.java --- |
![]() |
|||
Date Modified | Username | Field | Change |
2020-08-31 10:02 | cberner | Type | defect => backport |
2020-08-31 10:02 | cberner | Target Version | => PR20Q2.2 |
2020-08-31 10:06 | hgbot | Note Added: 0122469 | |
2020-08-31 11:50 | hgbot | Note Added: 0122476 | |
2020-08-31 11:50 | hgbot | Resolution | open => fixed |
2020-08-31 11:50 | hgbot | Status | scheduled => closed |
2020-08-31 11:50 | hgbot | Fixed in Version | => PR20Q2.2 |
2020-08-31 11:50 | hgbot | Note Added: 0122477 |
Copyright © 2000 - 2009 MantisBT Group |