Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0020368
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 07. Sales managementminoralways2012-04-26 18:542012-05-25 12:11
ReporterrgorisView Statuspublic 
Assigned ToSandrahuguet 
PriorityhighResolutionfixedFixed in Version
StatusclosedFix in branchpiFixed in SCM revisiondc8359e908b0
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0020368: Label "Payment In/Out" is incorrect in Sales Invoice and Purchase Invoice window

DescriptionThe 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 ReproduceA completed sales invoice or purchase invoice shows the button "Add Payment In/Out"
Proposed SolutionCreate 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
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0020405 closednaiaramartinez Change all "Payment In Plan" and "Payment Out Plan" labels to "Payment Plan" 

-  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
Powered by Mantis Bugtracker