Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0020368 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 07. Sales management | minor | always | 2012-04-26 18:54 | 2012-05-25 12:11 | |||
Reporter | rgoris | View Status | public | |||||
Assigned To | Sandrahuguet | |||||||
Priority | high | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | pi | Fixed in SCM revision | dc8359e908b0 | |||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | ||||||||
Web browser | ||||||||
Modules | Core | |||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0020368: Label "Payment In/Out" is incorrect in Sales Invoice and Purchase Invoice window | |||||||
Description | The label "Payment In/Out" is incorrect and confusing. Is it a payment that goes both ways (in and out)? Then, after having pressed the button, you get a dialogue that shows only Payment In for a sales invoice. There is no payment out here. The title of the dialogue also says "Payment In". The button has a label that communicates two opposite actions. It is like having a shower with one button that says "hot" and "cold". The reason is that the same button is used for both windows and that translation is easier into Spanish ("AƱadir cobro/pago") but that should not be a reason to use an incorrect label in English. | |||||||
Steps To Reproduce | A completed sales invoice or purchase invoice shows the button "Add Payment In/Out" | |||||||
Proposed Solution | Create separate buttons for Sales Invoices and Purchase Invoices so we can properly label the buttons as follows: - "Add Payment In" for Sales Invoices - "Add Payment Out" for Purchase Invoices | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||
|
Notes | |
(0048232) eduardo_Argal (developer) 2012-05-02 14:11 |
To avoid creating two different processes and then create another column with a display logic, we have decided to change the label to "Add Payment". This will make translation more difficult as translator will need to have a look at the context to do a proper translation. I guess this is the standard procedure for translators, so it should not include too much overhead. |
(0048235) rgoris (developer) 2012-05-02 16:10 edited on: 2012-05-02 16:10 |
Can you also change the "Modify Payment In Plan" to "Modify Payment Plan" and "Modify Payment Out Plan" to "Modify Payment Plan"? |
(0048416) Sandrahuguet (developer) 2012-05-07 17:27 |
Test Plan 1: -Go to sales invoice window -Select a line (complete=yes and payment complete=no) -You can see the button Add Payment Test Plan 2: -Go to purchase invoice window -Select a line (complete=yes and payment complete=no) -You can see the button Add Payment |
(0048550) hgbot (developer) 2012-05-10 11:56 |
Repository: erp/devel/pi Changeset: dc8359e908b0782b9c69e144c18bd2657d8bc9aa Author: Sandra Huguet <sandra.huguet <at> openbravo.com> Date: Mon May 07 17:43:03 2012 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/dc8359e908b0782b9c69e144c18bd2657d8bc9aa [^] Fixed bug 20368 Change label "Payment In/Out" Change label "Payment In/Out" in Sales Invoice and Purchase Invoice window --- M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_COLUMN.xml M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_ELEMENT.xml M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_FIELD.xml M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_PROCESS.xml --- |
(0048551) mirurita (developer) 2012-05-10 11:57 |
code review + testing OK The button label in Sales/Purchase Invoice is: "Add Payment" |
(0049089) hudsonbot (developer) 2012-05-25 12:11 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/5401e185a8b0 [^] Maturity status: Test |
Issue History | |||
Date Modified | Username | Field | Change |
2012-04-26 18:54 | rgoris | New Issue | |
2012-04-26 18:54 | rgoris | Assigned To | => eduardo_Argal |
2012-04-26 18:54 | rgoris | Modules | => Core |
2012-05-02 14:11 | eduardo_Argal | Note Added: 0048232 | |
2012-05-02 16:10 | rgoris | Note Added: 0048235 | |
2012-05-02 16:10 | rgoris | Note Edited: 0048235 | View Revisions |
2012-05-07 15:57 | Sandrahuguet | Assigned To | eduardo_Argal => Sandrahuguet |
2012-05-07 15:57 | Sandrahuguet | Status | new => scheduled |
2012-05-07 15:57 | Sandrahuguet | fix_in_branch | => pi |
2012-05-07 16:53 | Sandrahuguet | Relationship added | related to 0020405 |
2012-05-07 17:27 | Sandrahuguet | Note Added: 0048416 | |
2012-05-10 11:56 | hgbot | Checkin | |
2012-05-10 11:56 | hgbot | Note Added: 0048550 | |
2012-05-10 11:56 | hgbot | Status | scheduled => resolved |
2012-05-10 11:56 | hgbot | Resolution | open => fixed |
2012-05-10 11:56 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/dc8359e908b0782b9c69e144c18bd2657d8bc9aa [^] |
2012-05-10 11:57 | mirurita | Note Added: 0048551 | |
2012-05-10 11:57 | mirurita | Status | resolved => closed |
2012-05-25 12:11 | hudsonbot | Checkin | |
2012-05-25 12:11 | hudsonbot | Note Added: 0049089 |
Copyright © 2000 - 2009 MantisBT Group |