Re: svn commit: r1184785 - in /ofbiz/trunk/applications/product: config/ProductEntityLabels.xml webapp/catalog/WEB-INF/controller.xml widget/catalog/FeatureForms.xml widget/catalog/FeatureScreens.xml

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

Re: svn commit: r1184785 - in /ofbiz/trunk/applications/product: config/ProductEntityLabels.xml webapp/catalog/WEB-INF/controller.xml widget/catalog/FeatureForms.xml widget/catalog/FeatureScreens.xml

Jacques Le Roux
Administrator
Why not keeping the translations? It would be only overriden when reloading demo data. What do I miss, how/why does it prevent to
change values?
And then why not remove them completly?

Thanks

Jacques

From: <[hidden email]>

> Author: hansbak
> Date: Sun Oct 16 09:00:07 2011
> New Revision: 1184785
>
> URL: http://svn.apache.org/viewvc?rev=1184785&view=rev
> Log:
> enable the update of the feature category description and remove translation because it is demo data and would not allow update
>
> Modified:
>    ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
>    ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
>    ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
>    ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
>
> Modified: ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/config/ProductEntityLabels.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/product/config/ProductEntityLabels.xml (original)
> +++ ofbiz/trunk/applications/product/config/ProductEntityLabels.xml Sun Oct 16 09:00:07 2011
> @@ -1519,7 +1519,8 @@
>         <value xml:lang="zh">æ ‡å‡†</value>
>         <value xml:lang="zh_TW">標准</value>
>     </property>
> -    <property key="ProductFeatureCategory.description.1000">
> +    <!-- this is not seed databut demo and should not be translated, otherwise it cannot be updated in the screens -->
> +    <!-- property key="ProductFeatureCategory.description.1000">
>         <value xml:lang="en">Hazmat Features</value>
>         <value xml:lang="es">Características de Peligroso</value>
>         <value xml:lang="fr">Substances dangeureuse</value>
> @@ -1575,7 +1576,7 @@
>         <value xml:lang="it">Caratteristiche widget</value>
>         <value xml:lang="zh">预留名称特征</value>
>         <value xml:lang="zh_TW">é ç•™åç¨±ç‰¹å¾</value>
> -    </property>
> +    </property-->
>     <property key="ProductFeatureIactnType.description.FEATURE_IACTN_DEPEND">
>         <value xml:lang="de">Abhängiges Merkmal</value>
>         <value xml:lang="en">Dependent Feature</value>
>
> Modified: ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml (original)
> +++ ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml Sun Oct 16 09:00:07 2011
> @@ -1104,8 +1104,7 @@ under the License.
>     <request-map uri="UpdateFeatureCategory">
>         <security https="true" auth="true"/>
>         <event type="service" path="" invoke="updateProductFeatureCategory"/>
> -        <response name="success" type="view" value="EditFeatureCategories"/>
> -        <response name="error" type="view" value="EditFeatureCategories"/>
> +        <response name="success" type="request-redirect-noparam" value="EditFeatureCategories"/>
>     </request-map>
>
>     <request-map uri="EditFeatureCategoryFeatures">
>
> Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml (original)
> +++ ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml Sun Oct 16 09:00:07 2011
> @@ -292,12 +292,12 @@
>
>     <form name="FindProductFeature" type="single" target="EditFeatureCategories"
>         header-row-style="header-row" default-table-style="basic-table">
> -        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="25"></text-find></field>
> +        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="20"
> maxlength="255"></text-find></field>
>         <field name="description" title="${uiLabelMap.CommonDescription}"><text-find size="25"></text-find></field>
>         <field name="searchButton" title="${uiLabelMap.CommonFind}" widget-style="smallSubmit find"><submit/></field>
>     </form>
>
> -    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
> +    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
> target="UpdateFeatureCategory"
>         odd-row-style="alternate-row" default-table-style="basic-table hover-bar" separate-columns="true">
>         <actions>
>             <set field="parameters.noConditionFind" value="Y"/>
> @@ -314,8 +314,8 @@
>                 <parameter param-name="productFeatureCategoryId"/>
>             </hyperlink>
>         </field>
> -        <field name="description" title="${uiLabelMap.CommonDescription}"><display/></field>
> -
> +        <field name="description" title="${uiLabelMap.CommonDescription}"><text/></field>
> + <field name="update" title="${uiLabelMap.CommonUpdate}"><submit/></field>
>     </form>
>
>     <form name="CreateProductFeature" type="single" target="CreateFeatureCategory">
>
> Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
> URL:
> http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml (original)
> +++ ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml Sun Oct 16 09:00:07 2011
> @@ -215,8 +215,8 @@
>                 <set field="titleProperty" value="PageTitleEditProductFeatureCategories"/>
>                 <set field="headerItem" value="featurecats"/>
>                 <set field="tabButtonItem" value="FeatureCategory"/>
> -                <set field="viewIndex" from-field="parameters.VIEW_INDEX" type="Integer" default-value="0"/>
> -                <set field="viewSize" from-field="parameters.VIEW_SIZE" type="Integer" default-value="10"/>
> +                <set field="viewIndex" from-field="parameters.VIEW_INDEX_1" type="Integer" default-value="0"/>
> +                <set field="viewSize" from-field="parameters.VIEW_SIZE_1" type="Integer" default-value="10"/>
>
>                 <!--script location="component://product/webapp/catalog/WEB-INF/actions/feature/EditFeatureCategories.groovy"/-->
>             </actions>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1184785 - in /ofbiz/trunk/applications/product: config/ProductEntityLabels.xml webapp/catalog/WEB-INF/controller.xml widget/catalog/FeatureForms.xml widget/catalog/FeatureScreens.xml

hans_bakker
Jacques,

feature categories can be added, but not modified which is an
inconsistency i corrected.

However, when there are translations for records these cannot be
modified because the description is taken from the uilabel file, not
from the database which very confusing....

I order for people to comment i left the descriptions in the uilablel
file. Sure we can remove the descriptions.

Regards,
Hans

On Sun, 2011-10-16 at 23:34 +0200, Jacques Le Roux wrote:

> Why not keeping the translations? It would be only overriden when reloading demo data. What do I miss, how/why does it prevent to
> change values?
> And then why not remove them completly?
>
> Thanks
>
> Jacques
>
> From: <[hidden email]>
> > Author: hansbak
> > Date: Sun Oct 16 09:00:07 2011
> > New Revision: 1184785
> >
> > URL: http://svn.apache.org/viewvc?rev=1184785&view=rev
> > Log:
> > enable the update of the feature category description and remove translation because it is demo data and would not allow update
> >
> > Modified:
> >    ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
> >    ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
> >    ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
> >    ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
> >
> > Modified: ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
> > URL:
> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/config/ProductEntityLabels.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> > ==============================================================================
> > --- ofbiz/trunk/applications/product/config/ProductEntityLabels.xml (original)
> > +++ ofbiz/trunk/applications/product/config/ProductEntityLabels.xml Sun Oct 16 09:00:07 2011
> > @@ -1519,7 +1519,8 @@
> >         <value xml:lang="zh">æ ‡å‡†</value>
> >         <value xml:lang="zh_TW">標准</value>
> >     </property>
> > -    <property key="ProductFeatureCategory.description.1000">
> > +    <!-- this is not seed databut demo and should not be translated, otherwise it cannot be updated in the screens -->
> > +    <!-- property key="ProductFeatureCategory.description.1000">
> >         <value xml:lang="en">Hazmat Features</value>
> >         <value xml:lang="es">Características de Peligroso</value>
> >         <value xml:lang="fr">Substances dangeureuse</value>
> > @@ -1575,7 +1576,7 @@
> >         <value xml:lang="it">Caratteristiche widget</value>
> >         <value xml:lang="zh">预留名称特征</value>
> >         <value xml:lang="zh_TW">é ç•™åç¨±ç‰¹å¾</value>
> > -    </property>
> > +    </property-->
> >     <property key="ProductFeatureIactnType.description.FEATURE_IACTN_DEPEND">
> >         <value xml:lang="de">Abhängiges Merkmal</value>
> >         <value xml:lang="en">Dependent Feature</value>
> >
> > Modified: ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
> > URL:
> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> > ==============================================================================
> > --- ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml (original)
> > +++ ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml Sun Oct 16 09:00:07 2011
> > @@ -1104,8 +1104,7 @@ under the License.
> >     <request-map uri="UpdateFeatureCategory">
> >         <security https="true" auth="true"/>
> >         <event type="service" path="" invoke="updateProductFeatureCategory"/>
> > -        <response name="success" type="view" value="EditFeatureCategories"/>
> > -        <response name="error" type="view" value="EditFeatureCategories"/>
> > +        <response name="success" type="request-redirect-noparam" value="EditFeatureCategories"/>
> >     </request-map>
> >
> >     <request-map uri="EditFeatureCategoryFeatures">
> >
> > Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
> > URL:
> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> > ==============================================================================
> > --- ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml (original)
> > +++ ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml Sun Oct 16 09:00:07 2011
> > @@ -292,12 +292,12 @@
> >
> >     <form name="FindProductFeature" type="single" target="EditFeatureCategories"
> >         header-row-style="header-row" default-table-style="basic-table">
> > -        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="25"></text-find></field>
> > +        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="20"
> > maxlength="255"></text-find></field>
> >         <field name="description" title="${uiLabelMap.CommonDescription}"><text-find size="25"></text-find></field>
> >         <field name="searchButton" title="${uiLabelMap.CommonFind}" widget-style="smallSubmit find"><submit/></field>
> >     </form>
> >
> > -    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
> > +    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
> > target="UpdateFeatureCategory"
> >         odd-row-style="alternate-row" default-table-style="basic-table hover-bar" separate-columns="true">
> >         <actions>
> >             <set field="parameters.noConditionFind" value="Y"/>
> > @@ -314,8 +314,8 @@
> >                 <parameter param-name="productFeatureCategoryId"/>
> >             </hyperlink>
> >         </field>
> > -        <field name="description" title="${uiLabelMap.CommonDescription}"><display/></field>
> > -
> > +        <field name="description" title="${uiLabelMap.CommonDescription}"><text/></field>
> > + <field name="update" title="${uiLabelMap.CommonUpdate}"><submit/></field>
> >     </form>
> >
> >     <form name="CreateProductFeature" type="single" target="CreateFeatureCategory">
> >
> > Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
> > URL:
> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> > ==============================================================================
> > --- ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml (original)
> > +++ ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml Sun Oct 16 09:00:07 2011
> > @@ -215,8 +215,8 @@
> >                 <set field="titleProperty" value="PageTitleEditProductFeatureCategories"/>
> >                 <set field="headerItem" value="featurecats"/>
> >                 <set field="tabButtonItem" value="FeatureCategory"/>
> > -                <set field="viewIndex" from-field="parameters.VIEW_INDEX" type="Integer" default-value="0"/>
> > -                <set field="viewSize" from-field="parameters.VIEW_SIZE" type="Integer" default-value="10"/>
> > +                <set field="viewIndex" from-field="parameters.VIEW_INDEX_1" type="Integer" default-value="0"/>
> > +                <set field="viewSize" from-field="parameters.VIEW_SIZE_1" type="Integer" default-value="10"/>
> >
> >                 <!--script location="component://product/webapp/catalog/WEB-INF/actions/feature/EditFeatureCategories.groovy"/-->
> >             </actions>
> >
> >

--
Ofbiz on twitter: http://twitter.com/apache_ofbiz
Alternative ofbiz website: http://www.ofbiz.info
http://www.antwebsystems.com : Quality services for competitive rates.

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1184785 - in /ofbiz/trunk/applications/product: config/ProductEntityLabels.xml webapp/catalog/WEB-INF/controller.xml widget/catalog/FeatureForms.xml widget/catalog/FeatureScreens.xml

Jacques Le Roux
Administrator
Thanks Hans,

After a test to be sure, I know understand the problem and removed the whole block of labels at r1184991

Could you please have a look at the 2 other demands I posted on dev ML?
* Hardcoded sentences in <default-message> tags
* r1130415

Jacques

From: "Hans Bakker" <[hidden email]>

> Jacques,
>
> feature categories can be added, but not modified which is an
> inconsistency i corrected.
>
> However, when there are translations for records these cannot be
> modified because the description is taken from the uilabel file, not
> from the database which very confusing....
>
> I order for people to comment i left the descriptions in the uilablel
> file. Sure we can remove the descriptions.
>
> Regards,
> Hans
>
> On Sun, 2011-10-16 at 23:34 +0200, Jacques Le Roux wrote:
>> Why not keeping the translations? It would be only overriden when reloading demo data. What do I miss, how/why does it prevent to
>> change values?
>> And then why not remove them completly?
>>
>> Thanks
>>
>> Jacques
>>
>> From: <[hidden email]>
>> > Author: hansbak
>> > Date: Sun Oct 16 09:00:07 2011
>> > New Revision: 1184785
>> >
>> > URL: http://svn.apache.org/viewvc?rev=1184785&view=rev
>> > Log:
>> > enable the update of the feature category description and remove translation because it is demo data and would not allow update
>> >
>> > Modified:
>> >    ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
>> >    ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
>> >    ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
>> >    ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
>> >
>> > Modified: ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
>> > URL:
>> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/config/ProductEntityLabels.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
>> > ==============================================================================
>> > --- ofbiz/trunk/applications/product/config/ProductEntityLabels.xml (original)
>> > +++ ofbiz/trunk/applications/product/config/ProductEntityLabels.xml Sun Oct 16 09:00:07 2011
>> > @@ -1519,7 +1519,8 @@
>> >         <value xml:lang="zh">æ ‡å‡†</value>
>> >         <value xml:lang="zh_TW">標准</value>
>> >     </property>
>> > -    <property key="ProductFeatureCategory.description.1000">
>> > +    <!-- this is not seed databut demo and should not be translated, otherwise it cannot be updated in the screens -->
>> > +    <!-- property key="ProductFeatureCategory.description.1000">
>> >         <value xml:lang="en">Hazmat Features</value>
>> >         <value xml:lang="es">Características de Peligroso</value>
>> >         <value xml:lang="fr">Substances dangeureuse</value>
>> > @@ -1575,7 +1576,7 @@
>> >         <value xml:lang="it">Caratteristiche widget</value>
>> >         <value xml:lang="zh">预留名称特征</value>
>> >         <value xml:lang="zh_TW">é ç•™åç¨±ç‰¹å¾</value>
>> > -    </property>
>> > +    </property-->
>> >     <property key="ProductFeatureIactnType.description.FEATURE_IACTN_DEPEND">
>> >         <value xml:lang="de">Abhängiges Merkmal</value>
>> >         <value xml:lang="en">Dependent Feature</value>
>> >
>> > Modified: ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
>> > URL:
>> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
>> > ==============================================================================
>> > --- ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml (original)
>> > +++ ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml Sun Oct 16 09:00:07 2011
>> > @@ -1104,8 +1104,7 @@ under the License.
>> >     <request-map uri="UpdateFeatureCategory">
>> >         <security https="true" auth="true"/>
>> >         <event type="service" path="" invoke="updateProductFeatureCategory"/>
>> > -        <response name="success" type="view" value="EditFeatureCategories"/>
>> > -        <response name="error" type="view" value="EditFeatureCategories"/>
>> > +        <response name="success" type="request-redirect-noparam" value="EditFeatureCategories"/>
>> >     </request-map>
>> >
>> >     <request-map uri="EditFeatureCategoryFeatures">
>> >
>> > Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
>> > URL:
>> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
>> > ==============================================================================
>> > --- ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml (original)
>> > +++ ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml Sun Oct 16 09:00:07 2011
>> > @@ -292,12 +292,12 @@
>> >
>> >     <form name="FindProductFeature" type="single" target="EditFeatureCategories"
>> >         header-row-style="header-row" default-table-style="basic-table">
>> > -        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="25"></text-find></field>
>> > +        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="20"
>> > maxlength="255"></text-find></field>
>> >         <field name="description" title="${uiLabelMap.CommonDescription}"><text-find size="25"></text-find></field>
>> >         <field name="searchButton" title="${uiLabelMap.CommonFind}" widget-style="smallSubmit find"><submit/></field>
>> >     </form>
>> >
>> > -    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
>> > +    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
>> > target="UpdateFeatureCategory"
>> >         odd-row-style="alternate-row" default-table-style="basic-table hover-bar" separate-columns="true">
>> >         <actions>
>> >             <set field="parameters.noConditionFind" value="Y"/>
>> > @@ -314,8 +314,8 @@
>> >                 <parameter param-name="productFeatureCategoryId"/>
>> >             </hyperlink>
>> >         </field>
>> > -        <field name="description" title="${uiLabelMap.CommonDescription}"><display/></field>
>> > -
>> > +        <field name="description" title="${uiLabelMap.CommonDescription}"><text/></field>
>> > + <field name="update" title="${uiLabelMap.CommonUpdate}"><submit/></field>
>> >     </form>
>> >
>> >     <form name="CreateProductFeature" type="single" target="CreateFeatureCategory">
>> >
>> > Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
>> > URL:
>> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
>> > ==============================================================================
>> > --- ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml (original)
>> > +++ ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml Sun Oct 16 09:00:07 2011
>> > @@ -215,8 +215,8 @@
>> >                 <set field="titleProperty" value="PageTitleEditProductFeatureCategories"/>
>> >                 <set field="headerItem" value="featurecats"/>
>> >                 <set field="tabButtonItem" value="FeatureCategory"/>
>> > -                <set field="viewIndex" from-field="parameters.VIEW_INDEX" type="Integer" default-value="0"/>
>> > -                <set field="viewSize" from-field="parameters.VIEW_SIZE" type="Integer" default-value="10"/>
>> > +                <set field="viewIndex" from-field="parameters.VIEW_INDEX_1" type="Integer" default-value="0"/>
>> > +                <set field="viewSize" from-field="parameters.VIEW_SIZE_1" type="Integer" default-value="10"/>
>> >
>> >                 <!--script
>> > location="component://product/webapp/catalog/WEB-INF/actions/feature/EditFeatureCategories.groovy"/-->
>> >             </actions>
>> >
>> >
>
> --
> Ofbiz on twitter: http://twitter.com/apache_ofbiz
> Alternative ofbiz website: http://www.ofbiz.info
> http://www.antwebsystems.com : Quality services for competitive rates.
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1184785 - in /ofbiz/trunk/applications/product: config/ProductEntityLabels.xml webapp/catalog/WEB-INF/controller.xml widget/catalog/FeatureForms.xml widget/catalog/FeatureScreens.xml

hans_bakker
Ok Jacques,

The other comments you had, sure we will try to use the translation
feature....about the jquery, not sure if we are better in this. The guy
who did the tree has left us so if you can correct it, sure that would
be nice.

Regards,
Hans

On Mon, 2011-10-17 at 06:21 +0200, Jacques Le Roux wrote:

> Thanks Hans,
>
> After a test to be sure, I know understand the problem and removed the whole block of labels at r1184991
>
> Could you please have a look at the 2 other demands I posted on dev ML?
> * Hardcoded sentences in <default-message> tags
> * r1130415
>
> Jacques
>
> From: "Hans Bakker" <[hidden email]>
> > Jacques,
> >
> > feature categories can be added, but not modified which is an
> > inconsistency i corrected.
> >
> > However, when there are translations for records these cannot be
> > modified because the description is taken from the uilabel file, not
> > from the database which very confusing....
> >
> > I order for people to comment i left the descriptions in the uilablel
> > file. Sure we can remove the descriptions.
> >
> > Regards,
> > Hans
> >
> > On Sun, 2011-10-16 at 23:34 +0200, Jacques Le Roux wrote:
> >> Why not keeping the translations? It would be only overriden when reloading demo data. What do I miss, how/why does it prevent to
> >> change values?
> >> And then why not remove them completly?
> >>
> >> Thanks
> >>
> >> Jacques
> >>
> >> From: <[hidden email]>
> >> > Author: hansbak
> >> > Date: Sun Oct 16 09:00:07 2011
> >> > New Revision: 1184785
> >> >
> >> > URL: http://svn.apache.org/viewvc?rev=1184785&view=rev
> >> > Log:
> >> > enable the update of the feature category description and remove translation because it is demo data and would not allow update
> >> >
> >> > Modified:
> >> >    ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
> >> >    ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
> >> >    ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
> >> >    ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
> >> >
> >> > Modified: ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
> >> > URL:
> >> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/config/ProductEntityLabels.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> >> > ==============================================================================
> >> > --- ofbiz/trunk/applications/product/config/ProductEntityLabels.xml (original)
> >> > +++ ofbiz/trunk/applications/product/config/ProductEntityLabels.xml Sun Oct 16 09:00:07 2011
> >> > @@ -1519,7 +1519,8 @@
> >> >         <value xml:lang="zh">æ ‡å‡†</value>
> >> >         <value xml:lang="zh_TW">標准</value>
> >> >     </property>
> >> > -    <property key="ProductFeatureCategory.description.1000">
> >> > +    <!-- this is not seed databut demo and should not be translated, otherwise it cannot be updated in the screens -->
> >> > +    <!-- property key="ProductFeatureCategory.description.1000">
> >> >         <value xml:lang="en">Hazmat Features</value>
> >> >         <value xml:lang="es">Características de Peligroso</value>
> >> >         <value xml:lang="fr">Substances dangeureuse</value>
> >> > @@ -1575,7 +1576,7 @@
> >> >         <value xml:lang="it">Caratteristiche widget</value>
> >> >         <value xml:lang="zh">预留名称特征</value>
> >> >         <value xml:lang="zh_TW">é ç•™åç¨±ç‰¹å¾</value>
> >> > -    </property>
> >> > +    </property-->
> >> >     <property key="ProductFeatureIactnType.description.FEATURE_IACTN_DEPEND">
> >> >         <value xml:lang="de">Abhängiges Merkmal</value>
> >> >         <value xml:lang="en">Dependent Feature</value>
> >> >
> >> > Modified: ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
> >> > URL:
> >> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> >> > ==============================================================================
> >> > --- ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml (original)
> >> > +++ ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml Sun Oct 16 09:00:07 2011
> >> > @@ -1104,8 +1104,7 @@ under the License.
> >> >     <request-map uri="UpdateFeatureCategory">
> >> >         <security https="true" auth="true"/>
> >> >         <event type="service" path="" invoke="updateProductFeatureCategory"/>
> >> > -        <response name="success" type="view" value="EditFeatureCategories"/>
> >> > -        <response name="error" type="view" value="EditFeatureCategories"/>
> >> > +        <response name="success" type="request-redirect-noparam" value="EditFeatureCategories"/>
> >> >     </request-map>
> >> >
> >> >     <request-map uri="EditFeatureCategoryFeatures">
> >> >
> >> > Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
> >> > URL:
> >> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> >> > ==============================================================================
> >> > --- ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml (original)
> >> > +++ ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml Sun Oct 16 09:00:07 2011
> >> > @@ -292,12 +292,12 @@
> >> >
> >> >     <form name="FindProductFeature" type="single" target="EditFeatureCategories"
> >> >         header-row-style="header-row" default-table-style="basic-table">
> >> > -        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="25"></text-find></field>
> >> > +        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="20"
> >> > maxlength="255"></text-find></field>
> >> >         <field name="description" title="${uiLabelMap.CommonDescription}"><text-find size="25"></text-find></field>
> >> >         <field name="searchButton" title="${uiLabelMap.CommonFind}" widget-style="smallSubmit find"><submit/></field>
> >> >     </form>
> >> >
> >> > -    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
> >> > +    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
> >> > target="UpdateFeatureCategory"
> >> >         odd-row-style="alternate-row" default-table-style="basic-table hover-bar" separate-columns="true">
> >> >         <actions>
> >> >             <set field="parameters.noConditionFind" value="Y"/>
> >> > @@ -314,8 +314,8 @@
> >> >                 <parameter param-name="productFeatureCategoryId"/>
> >> >             </hyperlink>
> >> >         </field>
> >> > -        <field name="description" title="${uiLabelMap.CommonDescription}"><display/></field>
> >> > -
> >> > +        <field name="description" title="${uiLabelMap.CommonDescription}"><text/></field>
> >> > + <field name="update" title="${uiLabelMap.CommonUpdate}"><submit/></field>
> >> >     </form>
> >> >
> >> >     <form name="CreateProductFeature" type="single" target="CreateFeatureCategory">
> >> >
> >> > Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
> >> > URL:
> >> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
> >> > ==============================================================================
> >> > --- ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml (original)
> >> > +++ ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml Sun Oct 16 09:00:07 2011
> >> > @@ -215,8 +215,8 @@
> >> >                 <set field="titleProperty" value="PageTitleEditProductFeatureCategories"/>
> >> >                 <set field="headerItem" value="featurecats"/>
> >> >                 <set field="tabButtonItem" value="FeatureCategory"/>
> >> > -                <set field="viewIndex" from-field="parameters.VIEW_INDEX" type="Integer" default-value="0"/>
> >> > -                <set field="viewSize" from-field="parameters.VIEW_SIZE" type="Integer" default-value="10"/>
> >> > +                <set field="viewIndex" from-field="parameters.VIEW_INDEX_1" type="Integer" default-value="0"/>
> >> > +                <set field="viewSize" from-field="parameters.VIEW_SIZE_1" type="Integer" default-value="10"/>
> >> >
> >> >                 <!--script
> >> > location="component://product/webapp/catalog/WEB-INF/actions/feature/EditFeatureCategories.groovy"/-->
> >> >             </actions>
> >> >
> >> >
> >
> > --
> > Ofbiz on twitter: http://twitter.com/apache_ofbiz
> > Alternative ofbiz website: http://www.ofbiz.info
> > http://www.antwebsystems.com : Quality services for competitive rates.
> >

--
Ofbiz on twitter: http://twitter.com/apache_ofbiz
Alternative ofbiz website: http://www.ofbiz.info
http://www.antwebsystems.com : Quality services for competitive rates.

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1184785 - in /ofbiz/trunk/applications/product: config/ProductEntityLabels.xml webapp/catalog/WEB-INF/controller.xml widget/catalog/FeatureForms.xml widget/catalog/FeatureScreens.xml

Jacques Le Roux
Administrator
From: "Hans Bakker" <[hidden email]>
> Ok Jacques,
>
> The other comments you had, sure we will try to use the translation
> feature....about the jquery, not sure if we are better in this. The guy
> who did the tree has left us so if you can correct it, sure that would
> be nice.

OK I will have a look when I will get a chance

Jacques

> Regards,
> Hans
>
> On Mon, 2011-10-17 at 06:21 +0200, Jacques Le Roux wrote:
>> Thanks Hans,
>>
>> After a test to be sure, I know understand the problem and removed the whole block of labels at r1184991
>>
>> Could you please have a look at the 2 other demands I posted on dev ML?
>> * Hardcoded sentences in <default-message> tags
>> * r1130415
>>
>> Jacques
>>
>> From: "Hans Bakker" <[hidden email]>
>> > Jacques,
>> >
>> > feature categories can be added, but not modified which is an
>> > inconsistency i corrected.
>> >
>> > However, when there are translations for records these cannot be
>> > modified because the description is taken from the uilabel file, not
>> > from the database which very confusing....
>> >
>> > I order for people to comment i left the descriptions in the uilablel
>> > file. Sure we can remove the descriptions.
>> >
>> > Regards,
>> > Hans
>> >
>> > On Sun, 2011-10-16 at 23:34 +0200, Jacques Le Roux wrote:
>> >> Why not keeping the translations? It would be only overriden when reloading demo data. What do I miss, how/why does it prevent
>> >> to
>> >> change values?
>> >> And then why not remove them completly?
>> >>
>> >> Thanks
>> >>
>> >> Jacques
>> >>
>> >> From: <[hidden email]>
>> >> > Author: hansbak
>> >> > Date: Sun Oct 16 09:00:07 2011
>> >> > New Revision: 1184785
>> >> >
>> >> > URL: http://svn.apache.org/viewvc?rev=1184785&view=rev
>> >> > Log:
>> >> > enable the update of the feature category description and remove translation because it is demo data and would not allow
>> >> > update
>> >> >
>> >> > Modified:
>> >> >    ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
>> >> >    ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
>> >> >    ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
>> >> >    ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
>> >> >
>> >> > Modified: ofbiz/trunk/applications/product/config/ProductEntityLabels.xml
>> >> > URL:
>> >> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/config/ProductEntityLabels.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
>> >> > ==============================================================================
>> >> > --- ofbiz/trunk/applications/product/config/ProductEntityLabels.xml (original)
>> >> > +++ ofbiz/trunk/applications/product/config/ProductEntityLabels.xml Sun Oct 16 09:00:07 2011
>> >> > @@ -1519,7 +1519,8 @@
>> >> >         <value xml:lang="zh">æ ‡å‡†</value>
>> >> >         <value xml:lang="zh_TW">標准</value>
>> >> >     </property>
>> >> > -    <property key="ProductFeatureCategory.description.1000">
>> >> > +    <!-- this is not seed databut demo and should not be translated, otherwise it cannot be updated in the screens -->
>> >> > +    <!-- property key="ProductFeatureCategory.description.1000">
>> >> >         <value xml:lang="en">Hazmat Features</value>
>> >> >         <value xml:lang="es">Características de Peligroso</value>
>> >> >         <value xml:lang="fr">Substances dangeureuse</value>
>> >> > @@ -1575,7 +1576,7 @@
>> >> >         <value xml:lang="it">Caratteristiche widget</value>
>> >> >         <value xml:lang="zh">预留名称特征</value>
>> >> >         <value xml:lang="zh_TW">é ç•™åç¨±ç‰¹å¾</value>
>> >> > -    </property>
>> >> > +    </property-->
>> >> >     <property key="ProductFeatureIactnType.description.FEATURE_IACTN_DEPEND">
>> >> >         <value xml:lang="de">Abhängiges Merkmal</value>
>> >> >         <value xml:lang="en">Dependent Feature</value>
>> >> >
>> >> > Modified: ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml
>> >> > URL:
>> >> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
>> >> > ==============================================================================
>> >> > --- ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml (original)
>> >> > +++ ofbiz/trunk/applications/product/webapp/catalog/WEB-INF/controller.xml Sun Oct 16 09:00:07 2011
>> >> > @@ -1104,8 +1104,7 @@ under the License.
>> >> >     <request-map uri="UpdateFeatureCategory">
>> >> >         <security https="true" auth="true"/>
>> >> >         <event type="service" path="" invoke="updateProductFeatureCategory"/>
>> >> > -        <response name="success" type="view" value="EditFeatureCategories"/>
>> >> > -        <response name="error" type="view" value="EditFeatureCategories"/>
>> >> > +        <response name="success" type="request-redirect-noparam" value="EditFeatureCategories"/>
>> >> >     </request-map>
>> >> >
>> >> >     <request-map uri="EditFeatureCategoryFeatures">
>> >> >
>> >> > Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml
>> >> > URL:
>> >> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
>> >> > ==============================================================================
>> >> > --- ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml (original)
>> >> > +++ ofbiz/trunk/applications/product/widget/catalog/FeatureForms.xml Sun Oct 16 09:00:07 2011
>> >> > @@ -292,12 +292,12 @@
>> >> >
>> >> >     <form name="FindProductFeature" type="single" target="EditFeatureCategories"
>> >> >         header-row-style="header-row" default-table-style="basic-table">
>> >> > -        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="25"></text-find></field>
>> >> > +        <field name="productFeatureCategoryId" title="${uiLabelMap.CommonId}"><text-find size="20"
>> >> > maxlength="255"></text-find></field>
>> >> >         <field name="description" title="${uiLabelMap.CommonDescription}"><text-find size="25"></text-find></field>
>> >> >         <field name="searchButton" title="${uiLabelMap.CommonFind}" widget-style="smallSubmit find"><submit/></field>
>> >> >     </form>
>> >> >
>> >> > -    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
>> >> > +    <form name="ListProductFeature" type="list" list-name="listIt" paginate-target="EditFeatureCategories"
>> >> > target="UpdateFeatureCategory"
>> >> >         odd-row-style="alternate-row" default-table-style="basic-table hover-bar" separate-columns="true">
>> >> >         <actions>
>> >> >             <set field="parameters.noConditionFind" value="Y"/>
>> >> > @@ -314,8 +314,8 @@
>> >> >                 <parameter param-name="productFeatureCategoryId"/>
>> >> >             </hyperlink>
>> >> >         </field>
>> >> > -        <field name="description" title="${uiLabelMap.CommonDescription}"><display/></field>
>> >> > -
>> >> > +        <field name="description" title="${uiLabelMap.CommonDescription}"><text/></field>
>> >> > + <field name="update" title="${uiLabelMap.CommonUpdate}"><submit/></field>
>> >> >     </form>
>> >> >
>> >> >     <form name="CreateProductFeature" type="single" target="CreateFeatureCategory">
>> >> >
>> >> > Modified: ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml
>> >> > URL:
>> >> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml?rev=1184785&r1=1184784&r2=1184785&view=diff
>> >> > ==============================================================================
>> >> > --- ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml (original)
>> >> > +++ ofbiz/trunk/applications/product/widget/catalog/FeatureScreens.xml Sun Oct 16 09:00:07 2011
>> >> > @@ -215,8 +215,8 @@
>> >> >                 <set field="titleProperty" value="PageTitleEditProductFeatureCategories"/>
>> >> >                 <set field="headerItem" value="featurecats"/>
>> >> >                 <set field="tabButtonItem" value="FeatureCategory"/>
>> >> > -                <set field="viewIndex" from-field="parameters.VIEW_INDEX" type="Integer" default-value="0"/>
>> >> > -                <set field="viewSize" from-field="parameters.VIEW_SIZE" type="Integer" default-value="10"/>
>> >> > +                <set field="viewIndex" from-field="parameters.VIEW_INDEX_1" type="Integer" default-value="0"/>
>> >> > +                <set field="viewSize" from-field="parameters.VIEW_SIZE_1" type="Integer" default-value="10"/>
>> >> >
>> >> >                 <!--script
>> >> > location="component://product/webapp/catalog/WEB-INF/actions/feature/EditFeatureCategories.groovy"/-->
>> >> >             </actions>
>> >> >
>> >> >
>> >
>> > --
>> > Ofbiz on twitter: http://twitter.com/apache_ofbiz
>> > Alternative ofbiz website: http://www.ofbiz.info
>> > http://www.antwebsystems.com : Quality services for competitive rates.
>> >
>
> --
> Ofbiz on twitter: http://twitter.com/apache_ofbiz
> Alternative ofbiz website: http://www.ofbiz.info
> http://www.antwebsystems.com : Quality services for competitive rates.
>