Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0030740 | Openbravo ERP | 05. Production management | public | 2015-09-02 12:56 | 2016-06-28 16:06 |
|
| Reporter | shuehner | |
| Assigned To | Sanjota | |
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | 3.0PR16Q3 | |
| Merge Request Status | |
| Review Assigned To | aferraz |
| 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 | 0030740: pl-function ma_copy_version does not work with non (only-numeric' document numbers) |
| Description | That function has a datatype bug.
It contains the following variable declaration:
v_DocumentNo NUMERIC;
Assigned like this:
SELECT * INTO v_DocumentNo FROM Ad_Sequence_Doc('DocumentNo_MA_ProcessPlan_Version', v_Client_ID, 'Y') ;
used for insert into MA_Processplan_Version.documentno
Both teh ad_sequence_doc + the documentno field are varchar's are document number definition can and do contain prefixes & suffixes.
So that variable declaration in the pl function is looks just wrong.
Found using plpgsql_check tool |
| Steps To Reproduce | Read sourcecode of that pl function or run plpgsql_check tool |
| Proposed Solution | |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files | |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2015-09-02 12:56 | shuehner | New Issue | |
| 2015-09-02 12:56 | shuehner | Assigned To | => Triage Finance |
| 2015-09-02 12:56 | shuehner | OBNetwork customer | => No |
| 2015-09-02 12:56 | shuehner | Modules | => Core |
| 2015-09-02 12:56 | shuehner | Triggers an Emergency Pack | => No |
| 2016-06-06 18:36 | aferraz | Assigned To | Triage Finance => Sanjota |
| 2016-06-10 14:11 | Sanjota | Note Added: 0087154 | |
| 2016-06-27 12:38 | hgbot | Checkin | |
| 2016-06-27 12:38 | hgbot | Note Added: 0087976 | |
| 2016-06-27 12:38 | hgbot | Status | new => resolved |
| 2016-06-27 12:38 | hgbot | Resolution | open => fixed |
| 2016-06-27 12:38 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/f8552f6b006ede6e412370d4a5e268f938b60724 [^] |
| 2016-06-27 12:38 | hgbot | Checkin | |
| 2016-06-27 12:38 | hgbot | Note Added: 0087977 | |
| 2016-06-27 12:44 | aferraz | Review Assigned To | => aferraz |
| 2016-06-27 12:44 | aferraz | Note Added: 0087980 | |
| 2016-06-27 12:44 | aferraz | Status | resolved => closed |
| 2016-06-27 12:44 | aferraz | Fixed in Version | => 3.0PR16Q3 |
| 2016-06-28 16:06 | hudsonbot | Checkin | |
| 2016-06-28 16:06 | hudsonbot | Note Added: 0088055 | |
| 2016-06-28 16:06 | hudsonbot | Checkin | |
| 2016-06-28 16:06 | hudsonbot | Note Added: 0088056 | |
|
Notes |
|
|
|
Test Plan:
*Changed the datatype of v_documentNo from Numeric to var char(30).
* So pl-Function ma_copy_version() will accept NoN also. |
|
|
|
(0087976)
|
|
hgbot
|
|
2016-06-27 12:38
|
|
Repository: erp/devel/pi
Changeset: f8552f6b006ede6e412370d4a5e268f938b60724
Author: Sanjota <sanjota.nelagi <at> promantia.com>
Date: Fri Jun 10 17:30:18 2016 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/f8552f6b006ede6e412370d4a5e268f938b60724 [^]
Fixes issue 30740: pl-function ma_copy_version does not work with non
Changed the datatype of v_documentNo from Numeric to var char(30).
Since the function fails incase of insert MA_Processplan_Version statemment
---
M src-db/database/model/functions/MA_COPY_VERSION.xml
---
|
|
|
|
(0087977)
|
|
hgbot
|
|
2016-06-27 12:38
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|