ofbiz-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Pierre Smits <pierre.sm...@gmail.com>
Subject Re: svn commit: r1761687 - /ofbiz/trunk/specialpurpose/myportal/widget/PortalAdmForms.xml
Date Wed, 21 Sep 2016 09:03:33 GMT
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 in
https://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.

Also, applying specific labels reduces the total number of labels, when one
takes the context into consideration.
E.g.

CommonProduct

vs

AccountingProduct

ManufacturingProduct

OrderProduct

WorkEffortProduct
PartyProduct

FormFieldTitle_Product

ScrumProduct

EtcProduct


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 9:47 AM, gil portenseigne <
gil.portenseigne@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, jleroux@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">
>
>
>
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message