[jira] [Commented] (OFBIZ-10772) OFBiz On a Diet

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-10772) OFBiz On a Diet

Nicolas Malin (Jira)

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

Michael Brohl commented on OFBIZ-10772:
---------------------------------------

While I appreciate this effort, I'd like to say that we should be really careful here.

"amount" for example translates to 10+ different words in German, this is highly dependent on the context. We should also have in mind that even if we can reduce the labels in the standard, users might have the need to adapt the labels to their own terminology. The diverse use of labels is a great benefit for customizing and we should be careful to not over-optimize to the disadvantage of this flexibility.

I'd suggest to make some examples as patches before any mass update activity to be sure that the effort is reasonable.

> OFBiz On a Diet
> ---------------
>
>                 Key: OFBIZ-10772
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10772
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: ALL COMPONENTS
>    Affects Versions: Trunk
>            Reporter: Pierre Smits
>            Priority: Major
>
> OFBiz is laden with superfluous redundancies and approaches for 'novice' developers, when it comes to handling labels (titles) for screens and fields in forms in both Freemarker templates and widgets.
> In screens we have defined elements such as: 
> {code:java}
> <set field="titleProperty" value="ManufacturingCreateProductionRun"/>
> {code}
> and
>  
> {code:java}
> <screenlet title="${uiLabelMap.PageTitleAddFeatureInterAction"}">
> {code}
> Equally so we have defined elements in widget forms and templates, such as:
> {code:java}
> ${uiLabelMap.FormFieldTitle_<WhatEverTheFieldNameIs>}
> {code}
> or
> {code:java}
> ${uiLabelMap.<ComponentName><VeryLenghtyExplanationAsLabelForAField>}
> {code}
> This all has led to a situation where we have a product that, label-wise, overflown with exceptions on exceptions, and unnecessary 'explanations' - such as the 'componentName', or 'PageTitle' or 'FormFieldTitle_', in any of the files.
> Equally so, having to have the name of the component in the identifier of a label is unnecessary and reduces reuse. An 'Amount' label has the same meaning everywhere (given its context), so there is no need to have definitions like:
>  * AccountingAmount
>  * PaymentAmount
>  * ProductAmount
>  * OrderAmount
>  * TermAmount
>  * FormFieldTitle_amount
>  * etc.
> Having done a quick assessment I get to over 90.000 unnecessary identifiers in our files that  hampers us in reusing labels as much as possible.
> We can save 100.000s of bytes in our product by removing these unnecessary 'novice' developer requirements, resulting in cost- and time-saving benefits for our adopters, their developers and our contributors.
> This is the umbrella ticket to capture related sub-tasks (tickets/sub-tasks).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)