Openbravo Issue Tracking System - Retail Modules | |||||||||||||||||||||||||||||||||||||||||||||
View Issue Details | |||||||||||||||||||||||||||||||||||||||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||||||||||||||||||||||||||||||||||
0033671 | Retail Modules | Web POS | public | 2016-08-10 10:39 | 2016-08-26 16:55 | ||||||||||||||||||||||||||||||||||||||||
Reporter | vmromanos | ||||||||||||||||||||||||||||||||||||||||||||
Assigned To | mario_castello | ||||||||||||||||||||||||||||||||||||||||||||
Priority | immediate | Severity | major | Reproducibility | always | ||||||||||||||||||||||||||||||||||||||||
Status | closed | Resolution | fixed | ||||||||||||||||||||||||||||||||||||||||||
Platform | OS | 5 | OS Version | ||||||||||||||||||||||||||||||||||||||||||
Product Version | |||||||||||||||||||||||||||||||||||||||||||||
Target Version | Fixed in Version | RR16Q4 | |||||||||||||||||||||||||||||||||||||||||||
Merge Request Status | |||||||||||||||||||||||||||||||||||||||||||||
Review Assigned To | guilleaer | ||||||||||||||||||||||||||||||||||||||||||||
OBNetwork customer | No | ||||||||||||||||||||||||||||||||||||||||||||
Support ticket | |||||||||||||||||||||||||||||||||||||||||||||
Regression level | |||||||||||||||||||||||||||||||||||||||||||||
Regression date | |||||||||||||||||||||||||||||||||||||||||||||
Regression introduced in release | |||||||||||||||||||||||||||||||||||||||||||||
Regression introduced by commit | |||||||||||||||||||||||||||||||||||||||||||||
Triggers an Emergency Pack | No | ||||||||||||||||||||||||||||||||||||||||||||
Summary | 0033671: FIN_PaymentScheduleDetail created from POS doesn't include the Business Partner | ||||||||||||||||||||||||||||||||||||||||||||
Description | The FIN_PaymentScheduleDetail created from POS doesn't include the Business Partner. The ERP always sets the business partner information when creating the PSD from an invoice or an order (through extension points on C_Invoice_Post and C_Order_Post). However the POS doesn't fill it. When the order is directly paid from the POS there is no problem. However, credit orders generated from POS couldn't be paid in the ERP after fixing 0033629, because that fix requires to have the Business Partner information. | ||||||||||||||||||||||||||||||||||||||||||||
Steps To Reproduce | 1. Create and pay and order in POS 2. Go to the ERP and search for the associated payment (Sales Order | Payment Plan | Payment Details, and browse to the Payment In window) 3. Inside the Payment In window, go to Lines tab and verify Business Partner field is empty. It should have the order's business partner | ||||||||||||||||||||||||||||||||||||||||||||
Proposed Solution | Modify OrderLoader Ensure created PSD has set the business partner: paymentScheduleDetail.setBusinessPartner(order.getBusinessPartner()); Wrong data PSD should be updated. You can reuse module script UpdateBpPaymentLine. It's recommended to modify the update queries to filter by isinvoicepaid = 'N', so it performs faster: update fin_payment_scheduledetail psd set c_bpartner_id = (select c_bpartner_id from c_invoice i, fin_payment_schedule ps where ps.fin_payment_schedule_id = fin_payment_schedule_invoice and ps.c_invoice_id = i.c_invoice_id) where c_bpartner_id is null and fin_payment_schedule_invoice is not null and isinvoicepaid = 'N'; update fin_payment_scheduledetail psd set c_bpartner_id = (select c_bpartner_id from c_order o, fin_payment_schedule ps where ps.fin_payment_schedule_id = fin_payment_schedule_order and ps.c_order_id = o.c_order_id) where c_bpartner_id is null and fin_payment_schedule_order is not null and fin_payment_schedule_invoice is null and isinvoicepaid = 'N'; | ||||||||||||||||||||||||||||||||||||||||||||
Additional Information | |||||||||||||||||||||||||||||||||||||||||||||
Tags | No tags attached. | ||||||||||||||||||||||||||||||||||||||||||||
Relationships |
| ||||||||||||||||||||||||||||||||||||||||||||
Attached Files | ![]() https://issues.openbravo.com/file_download.php?file_id=9706&type=bug | ||||||||||||||||||||||||||||||||||||||||||||
Issue History | |||||||||||||||||||||||||||||||||||||||||||||
Date Modified | Username | Field | Change | ||||||||||||||||||||||||||||||||||||||||||
2016-08-10 10:39 | vmromanos | New Issue | |||||||||||||||||||||||||||||||||||||||||||
2016-08-10 10:39 | vmromanos | Assigned To | => Retail | ||||||||||||||||||||||||||||||||||||||||||
2016-08-10 10:39 | vmromanos | OBNetwork customer | => No | ||||||||||||||||||||||||||||||||||||||||||
2016-08-10 10:39 | vmromanos | Triggers an Emergency Pack | => No | ||||||||||||||||||||||||||||||||||||||||||
2016-08-10 10:39 | vmromanos | Relationship added | blocks 0033629 | ||||||||||||||||||||||||||||||||||||||||||
2016-08-10 10:43 | vmromanos | Steps to Reproduce Updated | bug_revision_view_page.php?rev_id=12816#r12816 | ||||||||||||||||||||||||||||||||||||||||||
2016-08-10 10:43 | vmromanos | Proposed Solution updated | |||||||||||||||||||||||||||||||||||||||||||
2016-08-10 11:09 | Orekaria | Status | new => scheduled | ||||||||||||||||||||||||||||||||||||||||||
2016-08-10 11:46 | simbu94 | File Added: Issue_33671.patch | |||||||||||||||||||||||||||||||||||||||||||
2016-08-11 11:47 | Orekaria | Status | scheduled => resolved | ||||||||||||||||||||||||||||||||||||||||||
2016-08-11 11:47 | Orekaria | Resolution | open => fixed | ||||||||||||||||||||||||||||||||||||||||||
2016-08-11 11:47 | Orekaria | Status | resolved => new | ||||||||||||||||||||||||||||||||||||||||||
2016-08-11 11:47 | Orekaria | Resolution | fixed => open | ||||||||||||||||||||||||||||||||||||||||||
2016-08-12 12:44 | migueldejuana | Assigned To | Retail => Orekaria | ||||||||||||||||||||||||||||||||||||||||||
2016-08-17 08:24 | guilleaer | Assigned To | Orekaria => mario_castello | ||||||||||||||||||||||||||||||||||||||||||
2016-08-17 14:34 | mario_castello | Status | new => acknowledged | ||||||||||||||||||||||||||||||||||||||||||
2016-08-17 17:20 | mario_castello | Status | acknowledged => scheduled | ||||||||||||||||||||||||||||||||||||||||||
2016-08-18 16:29 | hgbot | Checkin | |||||||||||||||||||||||||||||||||||||||||||
2016-08-18 16:29 | hgbot | Note Added: 0089217 | |||||||||||||||||||||||||||||||||||||||||||
2016-08-18 16:29 | hgbot | Status | scheduled => resolved | ||||||||||||||||||||||||||||||||||||||||||
2016-08-18 16:29 | hgbot | Resolution | open => fixed | ||||||||||||||||||||||||||||||||||||||||||
2016-08-18 16:29 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/5f2056b6faf440501a4ae97a68e7b1268ccdb006 [^] | ||||||||||||||||||||||||||||||||||||||||||
2016-08-18 17:30 | guilleaer | Relationship added | related to 0033744 | ||||||||||||||||||||||||||||||||||||||||||
2016-08-23 15:15 | guilleaer | Note Added: 0089306 | |||||||||||||||||||||||||||||||||||||||||||
2016-08-23 15:15 | guilleaer | Status | resolved => new | ||||||||||||||||||||||||||||||||||||||||||
2016-08-23 15:15 | guilleaer | Resolution | fixed => open | ||||||||||||||||||||||||||||||||||||||||||
2016-08-23 15:15 | guilleaer | Status | new => scheduled | ||||||||||||||||||||||||||||||||||||||||||
2016-08-24 16:15 | hgbot | Checkin | |||||||||||||||||||||||||||||||||||||||||||
2016-08-24 16:15 | hgbot | Note Added: 0089360 | |||||||||||||||||||||||||||||||||||||||||||
2016-08-24 16:15 | hgbot | Status | scheduled => resolved | ||||||||||||||||||||||||||||||||||||||||||
2016-08-24 16:15 | hgbot | Resolution | open => fixed | ||||||||||||||||||||||||||||||||||||||||||
2016-08-24 16:15 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/5f2056b6faf440501a4ae97a68e7b1268ccdb006 [^] => http://code.openbravo.com/tools/automation/pi-mobile/rev/0b8451075de8c93507136a9b03668864db980536 [^] | ||||||||||||||||||||||||||||||||||||||||||
2016-08-24 16:16 | hgbot | Checkin | |||||||||||||||||||||||||||||||||||||||||||
2016-08-24 16:16 | hgbot | Note Added: 0089361 | |||||||||||||||||||||||||||||||||||||||||||
2016-08-25 08:44 | guilleaer | Review Assigned To | => guilleaer | ||||||||||||||||||||||||||||||||||||||||||
2016-08-25 08:44 | guilleaer | Status | resolved => closed | ||||||||||||||||||||||||||||||||||||||||||
2016-08-25 08:44 | guilleaer | Fixed in Version | => RR16Q4 | ||||||||||||||||||||||||||||||||||||||||||
2016-08-26 16:55 | hgbot | Checkin | |||||||||||||||||||||||||||||||||||||||||||
2016-08-26 16:55 | hgbot | Note Added: 0089470 | |||||||||||||||||||||||||||||||||||||||||||
2017-07-13 11:12 | ngarcia | Relationship added | related to 0033776 |
Notes | |||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|
||||
|
|||||
|
|