HI Gil,

Where do you see "uiLabelMap.MyComponent/CommonXXXXX"  in widgets and
templates? Nowhere, as far as I can tell. But in each (most?) component
you'll find
{code}<property-map resource="CommonUiLabels" map-name="uiLabelMap" global=
"true"/>
{code}

And for what it is worth: a product in manufacturing, accounting, party,
workeffort or any other component referencing a product from the product
component is a product. No difference in context.

Best regards,

Pierre Smits

ORRTIZ.COM <http://www.orrtiz.com>
OFBiz based solutions & services

OFBiz Extensions Marketplace
http://oem.ofbizci.net/oci-2/

On Wed, Sep 21, 2016 at 11:25 AM, gil portenseigne <
gil.portensei...@nereide.fr> wrote:

> Hi Pierre,
>
> I do not agree that FormFieldTitle_XXX is *only* for lazy programmer (and
> i remember a teacher that used to tell that to be good, you have to be sma
> rtly lazy :) ).
>
> The fact that this label contains default translation for XXX fields allow
> speed developpment and is far more readable than having
> title="uiLabelMap.MyComponent/CommonXXXXX" everywhere in your form
> file... And if you want to overload this label its very easy. I don't see
> the problem here.
>
> inline
> Le 21/09/2016 à 11:03, Pierre Smits a écrit :
>
> FormFieldTitle_XXX is for lazy programmers, only considering the OFBiz Demo
> implementation as the only adopter of the product. Who use
> {code}<auto-fields-entity entity-name="EntityName" default-field-type="XXXX"
> />{code}
> as their means to display entity elements (fields) in forms.
>
> Unfortunately that doesn't work for all.
>
> Furthermore, like I said 
> inhttps://issues.apache.org/jira/browse/OFBIZ-8121?focusedCommentId=15501193&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15501193
> A specific title applied processes faster than none applied.
>
> How much ? For display i guess it's not significant.
>
>
> Also, applying specific labels reduces the total number of labels, when one
> takes the context into consideration.
>
> Labels for products or others object could be different in each component,
> thus i don't like replacing all with CommonXXX
>
> Regards,
>
> Gil
>
>
> E.g.
>
> CommonProduct
>
> vs
>
> AccountingProduct
>
> ManufacturingProduct
>
> OrderProduct
>
> WorkEffortProduct
> PartyProduct
>
> FormFieldTitle_Product
>
> ScrumProduct
>
> EtcProduct
>
>
> Best regards,
>
> Pierre Smits
> ORRTIZ.COM <http://www.orrtiz.com> <http://www.orrtiz.com>
>
> OFBiz based solutions & services
>
> OFBiz Extensions Marketplacehttp://oem.ofbizci.net/oci-2/
>
> On Wed, Sep 21, 2016 at 9:47 AM, gil portenseigne 
> <gil.portensei...@nereide.fr> wrote:
>
>
> Hi Jacques,
> Like Nicolas said in previous Michael commit 
> answer:http://markmail.org/message/x4ulworuwgbotvrv?q=r1761332
>
> I do not understand these kind of improvements. Adding a title when and F
> ormFieldTitle_XXX properties exists is not good in my opinion (i did not
> check these ones).
>
> Moreover i liked Michael answer on this JIRA 
> :https://issues.apache.org/jira/browse/OFBIZ-8056?focusedComm
> entId=15501066&page=com.atlassian.jira.plugin.system.
> issuetabpanels:comment-tabpanel#comment-15501066
>
> Gil
>
> Le 21/09/2016 à 09:34, jler...@apache.org a écrit :
>
> Author: jleroux
> Date: Wed Sep 21 07:34:13 2016
> New Revision: 1761687
>
> URL: http://svn.apache.org/viewvc?rev=1761687&view=rev
> Log:
> Improves: Maximise the utilisation of common labels in various applications
> (OFBIZ-8110)
>
> There are many commonalities among entity field definitions. Often these field
> definitions have led to unique label definitions, where a shared (common) 
> label
> could have sufficed.
>
> As examples you can take:
> * the various Id fields (where for most label CommonId could be used)
> * the various Type fields (where for most label CommonType could be used)
>
> This is a placeholder ticket, intended to capture applicable issues as sub 
> tasks
>  to address the aspect of maximising the utilisation of labels in the
> CommonUiLabels.xml file and to track progress.
>
> Thanks: Pierre Smits
>
> Modified:
>     ofbiz/trunk/specialpurpose/myportal/widget/PortalAdmForms.xml
>
> Modified: ofbiz/trunk/specialpurpose/myportal/widget/PortalAdmForms.xml
> URL: 
> http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/myportal/widget/PortalAdmForms.xml?rev=1761687&r1=1761686&r2=1761687&view=diff
> ==============================================================================
> --- ofbiz/trunk/specialpurpose/myportal/widget/PortalAdmForms.xml (original)
> +++ ofbiz/trunk/specialpurpose/myportal/widget/PortalAdmForms.xml Wed Sep 21 
> 07:34:13 2016
> @@ -27,7 +27,7 @@ under the License.
>          <field name="originalPortalPageId" position="2"><text-find/></field>
>          <field name="portalPageName" 
> title="${uiLabelMap.CommonName}"><text-find/></field>
>          <field name="parentPortalPageId" position="2"><text-find/></field>
> -        <field name="description"><text-find/></field>
> +        <field name="description" 
> title="${uiLabelMap.CommonDescription}"><text-find/></field>
>          <field name="securityGroupId" position="2" 
> title="${uiLabelMap.CommonSecurityGroupId}"><text-find/></field>
>          <field name="searchButton" title="${uiLabelMap.CommonFind}" 
> widget-style="smallSubmit"><submit button-type="button"/></field>
>      </form>
> @@ -88,8 +88,8 @@ under the License.
>              <text/>
>          </field>
>          <field name="parentPortalPageId" position="2"><text/></field>
> -        <field name="portalPageName"><text/></field>
> -        <field name="description" position="2"><text size="60"/></field>
> +        <field name="portalPageName" 
> title="${uiLabelMap.CommonName}"><text/></field>
> +        <field name="description" title="${uiLabelMap.CommonDescription}" 
> position="2"><text size="60"/></field>
>          <field name="sequenceNum"><text/></field>
>          <field name="securityGroupId" >
>              <drop-down allow-empty="true">
>
>
>
>
>
>
>
>

Reply via email to