[jira] [Comment Edited] (OFBIZ-12405) Rename invoiceApplications menu-item label to Payments

2021-11-30 Thread Pierre Smits (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-12405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17451102#comment-17451102
 ] 

Pierre Smits edited comment on OFBIZ-12405 at 11/30/21, 1:30 PM:
-

Bon après-midi Jacques,
{noformat}
I finally think that Pierre's is right and we could simply use Payments there
rather than Applications and even "Payments applications" (could be 
debated...){noformat}
So, finaly, you concede that I was correct. You doubted me, despite the fact 
that I had told you way back when (even on OFBiz ML) that I did my studies in 
accounting, worked for an CPA (auditing accounting books, etc.) and had a 
career as a Financial Controller in sectors like retail, manufacturing and 
project organisations… So I know the accounting lingo, having worked not only 
in companies that used the Dutch variant, but also in international companies 
using English and/or German.

But still, despite the fact that you gave in that I was correct about *Invoice 
Payments* and thus my contribution was good ** , you still went ahead and
 * changed the label in the invoiceSubTabBar to AccountingPaymentApplications. 
 * changed the label definition from AccountingApplications to 
AccountingPaymentApplications, while at the same changing the translation for 
English from Applications (which is correct) to Payments (which is incorrect, 
applications != payments), and changed the French translation from 
'Enregristrements' to Enrigestrements des Paiements'', which both translate to 
'Registrations' and 'Registrations of Payments', not applications (in French: 
l'application)

In PaymentsScreens.xml the use of the Label AccountingApplications was correct, 
as in
{code:java}
{code}
With the change to 
{code:java}
{code}
You didn't improved it. You made it worse.

A Payment can be applicable to more than just an invoice. It can be applicable 
to a salary slip, a reimbursement of a voucher, etc. 

What a waste of precious time you cost us both. Rendering perfectly good 
commits and PR useless

And now with this [       4aa2d71     
|https://github.com/apache/ofbiz-framework/commit/4aa2d714328447787455abcd5887ca99bcbf1152]
 , while together with Michael complaining at my address ‘repeatedly and 
annoyingly’ in other tickets (and PRs), about mixing stuff (beyond a single 
ticket and/or PR) in commits, you do exactly that.


was (Author: pfm.smits):
Bon après-midi Jacques,
{noformat}
I finally think that Pierre's is right and we could simply use Payments there
rather than Applications and even "Payments applications" (could be 
debated...){noformat}
So, finaly, you concede that I was correct. You doubted me, despite the fact 
that I had told you way back when (even on OFBiz ML) that I did my studies in 
accounting, worked for an CPA (auditing accounting books, etc.) and had a 
career as a Financial Controller in sectors like retail, manufacturing and 
project organisations…

But still, despite the fact that you gave in that I was correct about *Invoice 
Payments* and thus my contribution was good ** , you still went ahead and
 * changed the label in the invoiceSubTabBar to AccountingPaymentApplications. 
 * changed the label definition from AccountingApplications to 
AccountingPaymentApplications, while at the same changing the translation for 
English from Applications (which is correct) to Payments (which is incorrect, 
applications != payments), and changed the French translation from 
'Enregristrements' to Enrigestrements des Paiements'', which both translate to 
'Registrations' and 'Registrations of Payments', not applications (in French: 
l'application)

In PaymentsScreens.xml the use of the Label AccountingApplications was correct, 
as in
{code:java}
{code}
With the change to 
{code:java}
{code}
You didn't improved it. You made it worse.

A Payment can be applicable to more than just an invoice. It can be applicable 
to a salary slip, a reimbursement of a voucher, etc. 

What a waste of precious time you cost us both. Rendering perfectly good 
commits and PR useless

And now with this [       4aa2d71     
|https://github.com/apache/ofbiz-framework/commit/4aa2d714328447787455abcd5887ca99bcbf1152]
 , while together with Michael complaining at my address ‘repeatedly and 
annoyingly’ in other tickets (and PRs), about mixing stuff (beyond a single 
ticket and/or PR) in commits, you do exactly that.

> Rename invoiceApplications menu-item label to Payments
> --
>
> Key: OFBIZ-12405
> URL: https://issues.apache.org/jira/browse/OFBIZ-12405
> Project: OFBiz
>  Issue Type: Improvement
>  Components: accounting
>Affects Versions: Trunk
>Reporter: Pierre Smits
>Assignee: Pierre Smits
>Priority: Major
>  Labels: invoice, usability
>
> Application(s) is generic 

[jira] [Comment Edited] (OFBIZ-12405) Rename invoiceApplications menu-item label to Payments

2021-11-30 Thread Pierre Smits (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-12405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17451108#comment-17451108
 ] 

Pierre Smits edited comment on OFBIZ-12405 at 11/30/21, 1:04 PM:
-

Please revert commit 
[4aa2d71|https://github.com/apache/ofbiz-framework/commit/4aa2d714328447787455abcd5887ca99bcbf1152]
 and merge the PR associated with this ticket.


was (Author: pfm.smits):
Please revert commit [       4aa2d71     
|https://github.com/apache/ofbiz-framework/commit/4aa2d714328447787455abcd5887ca99bcbf1152]

> Rename invoiceApplications menu-item label to Payments
> --
>
> Key: OFBIZ-12405
> URL: https://issues.apache.org/jira/browse/OFBIZ-12405
> Project: OFBiz
>  Issue Type: Improvement
>  Components: accounting
>Affects Versions: Trunk
>Reporter: Pierre Smits
>Assignee: Pierre Smits
>Priority: Major
>  Labels: invoice, usability
>
> Application(s) is generic (also used within contexts elsewhere) in OFBiz.
> Users have a better perception of context when looking at a menu-item 
> Payments in the screen of an invoice, than seeing Applications.
> The additional benefit for developers is that with the implemented change the 
> tech come closer to the language of the user



--
This message was sent by Atlassian Jira
(v8.20.1#820001)


[jira] [Comment Edited] (OFBIZ-12405) Rename invoiceApplications menu-item label to Payments

2021-11-26 Thread Jacques Le Roux (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-12405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17449567#comment-17449567
 ] 

Jacques Le Roux edited comment on OFBIZ-12405 at 11/26/21, 1:21 PM:


OK, it seems we have some time to spent on this.

A famous author asked this question: "What's in a name?". His answer was that 
it's irrelevant* and I more than agree with that**.

What I mean is end users will never see the name of the label 
(AccountingApplications). Only developpers will see it and that does not count 
much. What end users will see, and is important, is the translation in their 
languages. We should not confuse syntax (name) and semantic (content).

Now about the content, AccountingApplications is used in 3 places:
 # menu-item name="editInvoiceApplications"
 # Two containers in screen name="EditPayment" with respectively conditions: 
 vs 

First let's analyze #1. You say
{quote}These translations would be more appropriate for a trigger (button, 
menu-item in an menu for action to be performed) and thus for a specific label 
(if not already present, I did not check), not for a menu-item intended to show 
an overview of payments associated to an invoice.
{quote}
That's simply wrong. This menu-item does not show when an invoice has been paid 
(ie all payments are applied to the invoice). It shows when no payments are 
applied to the invoice. Depending on the status of the invoice (at least 
Approved or In-Process) it respectively leads to
 # a screen which title is: "Assign Payment to This Invoice". So "Payment 
application" is appropriate there.-
 # a screen which title is: "Possible payments to apply [payer] sent from 
[recipient]" (combination of AccountingListPaymentsNotYetApplied and 
AccountingPaymentSentForm labels) So "Payment application" is also appropriate 
there.-
In both cases the mention "No applications found.." is shown

I let you analyse #2 and possibly #1 more. It seems to me we are going a bit 
too deep for such a trivial issue, when there are much more important ones 
waiting...

For AccountingApplicationSuccess, we need 1st to agree (or definitely not) 
about what to do about AccountingApplications before speaking again about 
that...

Don't expect more comments on this from me...

\* "That which we call a rose by any other name would smell just as sweet."
\** For the fun: my name is "Le Roux", it means "The Redhead", but my hairs are 
black. Not only names (syntax) can be irrelevant but sometimes wrong. That's 
not a problem, what's count is their content (semantic), here me :)


was (Author: jacques.le.roux):
OK, it seems we have some time to spent on this.

A famous author asked this question: "What's in a name?". His answer was that 
it's irrelevant* and I more than agree with that**.

What I mean is end users will never see the name of the label 
(AccountingApplications). Only developpers will see it and that does not count 
much. What end users will see, and is important, is the translation in their 
languages. We should not confuse syntax (name) and semantic (content).

Now about the content, AccountingApplications is used in 3 places:
 # menu-item name="editInvoiceApplications"
 # Two containers in screen name="EditPayment" with respectively conditions: 
 vs 

First let's analyze #1. You say
{quote}These translations would be more appropriate for a trigger (button, 
menu-item in an menu for action to be performed) and thus for a specific label 
(if not already present, I did not check), not for a menu-item intended to show 
an overview of payments associated to an invoice.
{quote}
That's simply wrong. This menu-item does not show when an invoice has been paid 
(ie all payments are applied to the invoice). It shows when no payments are 
applied to the invoice. Depending on the status of the invoice (at least 
Approved or In-Process) it respectively leads to
 # a screen which title is: "Assign Payment to This Invoice". So "Payment 
application" is appropriate there.-
 # a screen which title is: "Possible payments to apply [payer] sent from 
[recipient]" (combination of AccountingListPaymentsNotYetApplied and 
AccountingPaymentSentForm labels) So "Payment application" is also appropriate 
there.-
In both cases the mention "No applications found.." is shown

I let you analyse #2 and possibly #1 more. It seems to me we are going a bit 
too deep for such a trivial issue, when there are much more important ones 
waiting...

For AccountingApplicationSuccess, we need 1st to agree (or definitely not) 
about what to do about AccountingApplications before speaking again about 
that...

Don't expect more comments on this from me...

* "That which we call a rose by any other name would smell just as sweet."
** For the fun: my name is "Le Roux", it means "The Redhead", but my hairs are 
black. Not only names (syntax) can be irrelevant but sometimes wrong. That's 
not a problem, 

[jira] [Comment Edited] (OFBIZ-12405) Rename invoiceApplications menu-item label to Payments

2021-11-26 Thread Jacques Le Roux (Jira)


[ 
https://issues.apache.org/jira/browse/OFBIZ-12405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17449475#comment-17449475
 ] 

Jacques Le Roux edited comment on OFBIZ-12405 at 11/26/21, 9:23 AM:


Hi Pierre,

I don't know much about the terms used in English accounting. If I refer to 
https://www.quora.com/What-is-the-difference-between-a-payment-application-invoice-and-Running-Accounts-Bill-in-construction-industry
 I'm not sure we should change that this way. Quoting it:
bq. Payment application is when a received payment is applied to an account 
balance.

For me, the payment alone is the action of a person sending some money to sold 
out an invoice. Which is different from applying a received payment to an 
account balance (aka an invoice for me).

So, since we are in nitpicking and I have all the time for that, I propose 
rather to change the label  to "Payment 
application" in English and (eg) "Enregistrer un paiement" in French.

You may also want to check AccountingApplicationSuccess. There I propose 
inFrench: "Enregistrer ${amountApplied?currency(${isoCode})} du paiement 
[${paymentId}] � ${toMessage}"

HTH


was (Author: jacques.le.roux):
Hi Pierre,

I don't know much about the terms used in English accounting. If I refer to 
https://www.quora.com/What-is-the-difference-between-a-payment-application-invoice-and-Running-Accounts-Bill-in-construction-industry
 I'm not sure we should change that this way. Quoting it:
bq. Payment application is when a received payment is applied to an account 
balance.

For mee, the payment alone is the action of a person sending some money to sold 
out an invoice. Which is different from applying a received payment to an 
account balance (aka an invoice for me).

So, since we are in nitpicking and I have all the time for that, I propose 
rather to change the label  to "Payment 
application" in English and (eg) "Enregistrer un paiement" in French.

You may also want to check AccountingApplicationSuccess. There I propose 
inFrench: "Enregistrer ${amountApplied?currency(${isoCode})} du paiement 
[${paymentId}] � ${toMessage}"

HTH

> Rename invoiceApplications menu-item label to Payments
> --
>
> Key: OFBIZ-12405
> URL: https://issues.apache.org/jira/browse/OFBIZ-12405
> Project: OFBiz
>  Issue Type: Improvement
>  Components: accounting
>Affects Versions: Trunk
>Reporter: Pierre Smits
>Assignee: Pierre Smits
>Priority: Major
>  Labels: invoice, usability
>
> Application(s) is generic (also used within contexts elsewhere) in OFBiz.
> Users have a better perception of context when looking at a menu-item 
> Payments in the screen of an invoice, than seeing Applications.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)