Re: svn commit: r1854306 - /ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml

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

Re: svn commit: r1854306 - /ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml

Deepak Dixit-5
Hi Jacques,

I think instead of adding space in uiLabel it would be food if we add space
in the template

Thanks & Regards
--
Deepak Dixit
ofbiz.apache.org


On Mon, Feb 25, 2019 at 4:03 PM <[hidden email]> wrote:

> Author: jleroux
> Date: Mon Feb 25 10:33:53 2019
> New Revision: 1854306
>
> URL: http://svn.apache.org/viewvc?rev=1854306&view=rev
> Log:
> Improved: no functional change
>
> A missing space before CommonBranch label made the word trunkbranch
> instead of
> trunk branch.
>
> I'll backport this
>
> Modified:
>     ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>
> Modified:
> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> URL:
> http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml?rev=1854306&r1=1854305&r2=1854306&view=diff
>
> ==============================================================================
> --- ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> (original)
> +++ ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> Mon Feb 25 10:33:53 2019
> @@ -1389,9 +1389,9 @@
>          <value xml:lang="zh-TW">å›žå ±</value>
>      </property>
>      <property key="CommonBranch">
> -        <value xml:lang="en">branch</value>
> -        <value xml:lang="es">rama</value>
> -        <value xml:lang="fr">branche</value>
> +        <value xml:lang="en"> branch</value>
> +        <value xml:lang="es"> rama</value>
> +        <value xml:lang="fr"> branche</value>
>      </property>
>      <property key="CommonBuiltOn">
>          <value xml:lang="en">built on</value>
>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1854306 - /ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml

Jacques Le Roux
Administrator
Actually I changed it there because it does not work in build.gradle where it's already

gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" + System.lineSeparator()

svnFooterFile << '${uiLabelMap.CommonBranch} : ' + "${info.entry.url.text()}" + System.lineSeparator()

Jacques

Le 12/03/2019 à 10:33, Jacques Le Roux a écrit :

>
> Agreed Deepak,
>
> Makes sense, I 'll change that
>
> Thanks
>
> Jacques
>
> Le 12/03/2019 à 10:07, Deepak Dixit a écrit :
>> Hi Jacques,
>>
>> I think instead of adding space in uiLabel it would be food if we add space in the template
>>
>> Thanks & Regards
>> --
>> Deepak Dixit
>> ofbiz.apache.org <http://ofbiz.apache.org>
>>
>>
>> On Mon, Feb 25, 2019 at 4:03 PM <[hidden email] <mailto:[hidden email]>> wrote:
>>
>>     Author: jleroux
>>     Date: Mon Feb 25 10:33:53 2019
>>     New Revision: 1854306
>>
>>     URL: http://svn.apache.org/viewvc?rev=1854306&view=rev
>>     Log:
>>     Improved: no functional change
>>
>>     A missing space before CommonBranch label made the word trunkbranch instead of
>>     trunk branch.
>>
>>     I'll backport this
>>
>>     Modified:
>>     ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>>
>>     Modified: ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>>     URL:
>>     http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml?rev=1854306&r1=1854305&r2=1854306&view=diff
>>     ==============================================================================
>>     --- ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml (original)
>>     +++ ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml Mon Feb 25 10:33:53 2019
>>     @@ -1389,9 +1389,9 @@
>>              <value xml:lang="zh-TW">回報</value>
>>          </property>
>>          <property key="CommonBranch">
>>     -        <value xml:lang="en">branch</value>
>>     -        <value xml:lang="es">rama</value>
>>     -        <value xml:lang="fr">branche</value>
>>     +        <value xml:lang="en"> branch</value>
>>     +        <value xml:lang="es"> rama</value>
>>     +        <value xml:lang="fr"> branche</value>
>>          </property>
>>          <property key="CommonBuiltOn">
>>              <value xml:lang="en">built on</value>
>>
>>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1854306 - /ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml

Deepak Dixit-4
Hi Jacques,

I tested the following patch and it works
=====================

Index: build.gradle

===================================================================

--- build.gradle (revision 1855081)

+++ build.gradle (working copy)

@@ -638,7 +638,7 @@

         revision = revisionOutput.toString()

         gitFooterFile.delete()

         gitFooterFile.createNewFile()

-        gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" +
System.lineSeparator()

+        gitFooterFile << ' ${uiLabelMap.CommonBranch} : ' + "${branch}" +
System.lineSeparator()

         gitFooterFile << '${uiLabelMap.CommonRevision} : ' + "${revision}"
+ System.lineSeparator()

         gitFooterFile << '${uiLabelMap.CommonBuiltOn} : ' + "${timestamp}"
+ System.lineSeparator()

         gitFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
"${org.gradle.internal.jvm.Jvm.current()}"

@@ -663,7 +663,7 @@

         def info = new XmlParser().parseText(svnOutput.toString())

         svnFooterFile.delete()

         svnFooterFile.createNewFile()

-        svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
"${info.entry.url.text()}" + System.lineSeparator()

+        svnFooterFile << ' ${uiLabelMap.CommonBranch} : ' +
"${info.entry.url.text()}" + System.lineSeparator()

         svnFooterFile << '${uiLabelMap.CommonRevision} : ' +
"${info.entry.commit.@revision}" + System.lineSeparator()

         svnFooterFile << '${uiLabelMap.CommonBuiltOn} : ' + "${timestamp}"
+ System.lineSeparator()

         svnFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
"${org.gradle.internal.jvm.Jvm.current()}"

Index: framework/common/config/CommonUiLabels.xml

===================================================================

--- framework/common/config/CommonUiLabels.xml (revision 1855081)

+++ framework/common/config/CommonUiLabels.xml (working copy)

@@ -1389,9 +1389,9 @@

         <value xml:lang="zh-TW">回報</value>

     </property>

     <property key="CommonBranch">

-        <value xml:lang="en"> branch</value>

-        <value xml:lang="es"> rama</value>

-        <value xml:lang="fr"> branche</value>

+        <value xml:lang="en">branch</value>

+        <value xml:lang="es">rama</value>

+        <value xml:lang="fr">branche</value>

     </property>

     <property key="CommonBuiltOn">

         <value xml:lang="en">built on</value>


=====================


Kind Regards,
Deepak Dixit

On Wed, Mar 13, 2019 at 2:16 AM Jacques Le Roux <
[hidden email]> wrote:

> Actually I changed it there because it does not work in build.gradle where
> it's already
>
> gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" +
> System.lineSeparator()
>
> svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
> "${info.entry.url.text()}" + System.lineSeparator()
>
> Jacques
>
> Le 12/03/2019 à 10:33, Jacques Le Roux a écrit :
> >
> > Agreed Deepak,
> >
> > Makes sense, I 'll change that
> >
> > Thanks
> >
> > Jacques
> >
> > Le 12/03/2019 à 10:07, Deepak Dixit a écrit :
> >> Hi Jacques,
> >>
> >> I think instead of adding space in uiLabel it would be food if we add
> space in the template
> >>
> >> Thanks & Regards
> >> --
> >> Deepak Dixit
> >> ofbiz.apache.org <http://ofbiz.apache.org>
> >>
> >>
> >> On Mon, Feb 25, 2019 at 4:03 PM <[hidden email] <mailto:
> [hidden email]>> wrote:
> >>
> >>     Author: jleroux
> >>     Date: Mon Feb 25 10:33:53 2019
> >>     New Revision: 1854306
> >>
> >>     URL: http://svn.apache.org/viewvc?rev=1854306&view=rev
> >>     Log:
> >>     Improved: no functional change
> >>
> >>     A missing space before CommonBranch label made the word trunkbranch
> instead of
> >>     trunk branch.
> >>
> >>     I'll backport this
> >>
> >>     Modified:
> >>
>  ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> >>
> >>     Modified:
> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> >>     URL:
> >>
> http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml?rev=1854306&r1=1854305&r2=1854306&view=diff
> >>
>  ==============================================================================
> >>     ---
> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> (original)
> >>     +++
> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml Mon
> Feb 25 10:33:53 2019
> >>     @@ -1389,9 +1389,9 @@
> >>              <value xml:lang="zh-TW">å›žå ±</value>
> >>          </property>
> >>          <property key="CommonBranch">
> >>     -        <value xml:lang="en">branch</value>
> >>     -        <value xml:lang="es">rama</value>
> >>     -        <value xml:lang="fr">branche</value>
> >>     +        <value xml:lang="en"> branch</value>
> >>     +        <value xml:lang="es"> rama</value>
> >>     +        <value xml:lang="fr"> branche</value>
> >>          </property>
> >>          <property key="CommonBuiltOn">
> >>              <value xml:lang="en">built on</value>
> >>
> >>
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1854306 - /ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml

Jacques Le Roux
Administrator
Thanks Deepak,

Unfortunately it's difficult to use it as us and I have not created a Jira for that.

Could you please commit it?

TIA

Jacques

Le 13/03/2019 à 06:42, Deepak Dixit a écrit :

> Hi Jacques,
>
> I tested the following patch and it works
> =====================
>
> Index: build.gradle
>
> ===================================================================
>
> --- build.gradle (revision 1855081)
>
> +++ build.gradle (working copy)
>
> @@ -638,7 +638,7 @@
>
>           revision = revisionOutput.toString()
>
>           gitFooterFile.delete()
>
>           gitFooterFile.createNewFile()
>
> -        gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" +
> System.lineSeparator()
>
> +        gitFooterFile << ' ${uiLabelMap.CommonBranch} : ' + "${branch}" +
> System.lineSeparator()
>
>           gitFooterFile << '${uiLabelMap.CommonRevision} : ' + "${revision}"
> + System.lineSeparator()
>
>           gitFooterFile << '${uiLabelMap.CommonBuiltOn} : ' + "${timestamp}"
> + System.lineSeparator()
>
>           gitFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
> "${org.gradle.internal.jvm.Jvm.current()}"
>
> @@ -663,7 +663,7 @@
>
>           def info = new XmlParser().parseText(svnOutput.toString())
>
>           svnFooterFile.delete()
>
>           svnFooterFile.createNewFile()
>
> -        svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
> "${info.entry.url.text()}" + System.lineSeparator()
>
> +        svnFooterFile << ' ${uiLabelMap.CommonBranch} : ' +
> "${info.entry.url.text()}" + System.lineSeparator()
>
>           svnFooterFile << '${uiLabelMap.CommonRevision} : ' +
> "${info.entry.commit.@revision}" + System.lineSeparator()
>
>           svnFooterFile << '${uiLabelMap.CommonBuiltOn} : ' + "${timestamp}"
> + System.lineSeparator()
>
>           svnFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
> "${org.gradle.internal.jvm.Jvm.current()}"
>
> Index: framework/common/config/CommonUiLabels.xml
>
> ===================================================================
>
> --- framework/common/config/CommonUiLabels.xml (revision 1855081)
>
> +++ framework/common/config/CommonUiLabels.xml (working copy)
>
> @@ -1389,9 +1389,9 @@
>
>           <value xml:lang="zh-TW">回報</value>
>
>       </property>
>
>       <property key="CommonBranch">
>
> -        <value xml:lang="en"> branch</value>
>
> -        <value xml:lang="es"> rama</value>
>
> -        <value xml:lang="fr"> branche</value>
>
> +        <value xml:lang="en">branch</value>
>
> +        <value xml:lang="es">rama</value>
>
> +        <value xml:lang="fr">branche</value>
>
>       </property>
>
>       <property key="CommonBuiltOn">
>
>           <value xml:lang="en">built on</value>
>
>
> =====================
>
>
> Kind Regards,
> Deepak Dixit
>
> On Wed, Mar 13, 2019 at 2:16 AM Jacques Le Roux <
> [hidden email]> wrote:
>
>> Actually I changed it there because it does not work in build.gradle where
>> it's already
>>
>> gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" +
>> System.lineSeparator()
>>
>> svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
>> "${info.entry.url.text()}" + System.lineSeparator()
>>
>> Jacques
>>
>> Le 12/03/2019 à 10:33, Jacques Le Roux a écrit :
>>> Agreed Deepak,
>>>
>>> Makes sense, I 'll change that
>>>
>>> Thanks
>>>
>>> Jacques
>>>
>>> Le 12/03/2019 à 10:07, Deepak Dixit a écrit :
>>>> Hi Jacques,
>>>>
>>>> I think instead of adding space in uiLabel it would be food if we add
>> space in the template
>>>> Thanks & Regards
>>>> --
>>>> Deepak Dixit
>>>> ofbiz.apache.org <http://ofbiz.apache.org>
>>>>
>>>>
>>>> On Mon, Feb 25, 2019 at 4:03 PM <[hidden email] <mailto:
>> [hidden email]>> wrote:
>>>>      Author: jleroux
>>>>      Date: Mon Feb 25 10:33:53 2019
>>>>      New Revision: 1854306
>>>>
>>>>      URL: http://svn.apache.org/viewvc?rev=1854306&view=rev
>>>>      Log:
>>>>      Improved: no functional change
>>>>
>>>>      A missing space before CommonBranch label made the word trunkbranch
>> instead of
>>>>      trunk branch.
>>>>
>>>>      I'll backport this
>>>>
>>>>      Modified:
>>>>
>>   ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>>>>      Modified:
>> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>>>>      URL:
>>>>
>> http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml?rev=1854306&r1=1854305&r2=1854306&view=diff
>>   ==============================================================================
>>>>      ---
>> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>> (original)
>>>>      +++
>> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml Mon
>> Feb 25 10:33:53 2019
>>>>      @@ -1389,9 +1389,9 @@
>>>>               <value xml:lang="zh-TW">å›žå ±</value>
>>>>           </property>
>>>>           <property key="CommonBranch">
>>>>      -        <value xml:lang="en">branch</value>
>>>>      -        <value xml:lang="es">rama</value>
>>>>      -        <value xml:lang="fr">branche</value>
>>>>      +        <value xml:lang="en"> branch</value>
>>>>      +        <value xml:lang="es"> rama</value>
>>>>      +        <value xml:lang="fr"> branche</value>
>>>>           </property>
>>>>           <property key="CommonBuiltOn">
>>>>               <value xml:lang="en">built on</value>
>>>>
>>>>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1854306 - /ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml

Deepak Dixit-5
Sure Jacques, Will do
Thanks & Regards
--
Deepak Dixit
ofbiz.apache.org


On Thu, Mar 14, 2019 at 2:16 PM Jacques Le Roux <
[hidden email]> wrote:

> Thanks Deepak,
>
> Unfortunately it's difficult to use it as us and I have not created a Jira
> for that.
>
> Could you please commit it?
>
> TIA
>
> Jacques
>
> Le 13/03/2019 à 06:42, Deepak Dixit a écrit :
> > Hi Jacques,
> >
> > I tested the following patch and it works
> > =====================
> >
> > Index: build.gradle
> >
> > ===================================================================
> >
> > --- build.gradle (revision 1855081)
> >
> > +++ build.gradle (working copy)
> >
> > @@ -638,7 +638,7 @@
> >
> >           revision = revisionOutput.toString()
> >
> >           gitFooterFile.delete()
> >
> >           gitFooterFile.createNewFile()
> >
> > -        gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" +
> > System.lineSeparator()
> >
> > +        gitFooterFile << ' ${uiLabelMap.CommonBranch} : ' + "${branch}"
> +
> > System.lineSeparator()
> >
> >           gitFooterFile << '${uiLabelMap.CommonRevision} : ' +
> "${revision}"
> > + System.lineSeparator()
> >
> >           gitFooterFile << '${uiLabelMap.CommonBuiltOn} : ' +
> "${timestamp}"
> > + System.lineSeparator()
> >
> >           gitFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
> > "${org.gradle.internal.jvm.Jvm.current()}"
> >
> > @@ -663,7 +663,7 @@
> >
> >           def info = new XmlParser().parseText(svnOutput.toString())
> >
> >           svnFooterFile.delete()
> >
> >           svnFooterFile.createNewFile()
> >
> > -        svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
> > "${info.entry.url.text()}" + System.lineSeparator()
> >
> > +        svnFooterFile << ' ${uiLabelMap.CommonBranch} : ' +
> > "${info.entry.url.text()}" + System.lineSeparator()
> >
> >           svnFooterFile << '${uiLabelMap.CommonRevision} : ' +
> > "${info.entry.commit.@revision}" + System.lineSeparator()
> >
> >           svnFooterFile << '${uiLabelMap.CommonBuiltOn} : ' +
> "${timestamp}"
> > + System.lineSeparator()
> >
> >           svnFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
> > "${org.gradle.internal.jvm.Jvm.current()}"
> >
> > Index: framework/common/config/CommonUiLabels.xml
> >
> > ===================================================================
> >
> > --- framework/common/config/CommonUiLabels.xml (revision 1855081)
> >
> > +++ framework/common/config/CommonUiLabels.xml (working copy)
> >
> > @@ -1389,9 +1389,9 @@
> >
> >           <value xml:lang="zh-TW">回報</value>
> >
> >       </property>
> >
> >       <property key="CommonBranch">
> >
> > -        <value xml:lang="en"> branch</value>
> >
> > -        <value xml:lang="es"> rama</value>
> >
> > -        <value xml:lang="fr"> branche</value>
> >
> > +        <value xml:lang="en">branch</value>
> >
> > +        <value xml:lang="es">rama</value>
> >
> > +        <value xml:lang="fr">branche</value>
> >
> >       </property>
> >
> >       <property key="CommonBuiltOn">
> >
> >           <value xml:lang="en">built on</value>
> >
> >
> > =====================
> >
> >
> > Kind Regards,
> > Deepak Dixit
> >
> > On Wed, Mar 13, 2019 at 2:16 AM Jacques Le Roux <
> > [hidden email]> wrote:
> >
> >> Actually I changed it there because it does not work in build.gradle
> where
> >> it's already
> >>
> >> gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" +
> >> System.lineSeparator()
> >>
> >> svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
> >> "${info.entry.url.text()}" + System.lineSeparator()
> >>
> >> Jacques
> >>
> >> Le 12/03/2019 à 10:33, Jacques Le Roux a écrit :
> >>> Agreed Deepak,
> >>>
> >>> Makes sense, I 'll change that
> >>>
> >>> Thanks
> >>>
> >>> Jacques
> >>>
> >>> Le 12/03/2019 à 10:07, Deepak Dixit a écrit :
> >>>> Hi Jacques,
> >>>>
> >>>> I think instead of adding space in uiLabel it would be food if we add
> >> space in the template
> >>>> Thanks & Regards
> >>>> --
> >>>> Deepak Dixit
> >>>> ofbiz.apache.org <http://ofbiz.apache.org>
> >>>>
> >>>>
> >>>> On Mon, Feb 25, 2019 at 4:03 PM <[hidden email] <mailto:
> >> [hidden email]>> wrote:
> >>>>      Author: jleroux
> >>>>      Date: Mon Feb 25 10:33:53 2019
> >>>>      New Revision: 1854306
> >>>>
> >>>>      URL: http://svn.apache.org/viewvc?rev=1854306&view=rev
> >>>>      Log:
> >>>>      Improved: no functional change
> >>>>
> >>>>      A missing space before CommonBranch label made the word
> trunkbranch
> >> instead of
> >>>>      trunk branch.
> >>>>
> >>>>      I'll backport this
> >>>>
> >>>>      Modified:
> >>>>
> >>   ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> >>>>      Modified:
> >> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> >>>>      URL:
> >>>>
> >>
> http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml?rev=1854306&r1=1854305&r2=1854306&view=diff
> >>
>  ==============================================================================
> >>>>      ---
> >> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> >> (original)
> >>>>      +++
> >> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
> Mon
> >> Feb 25 10:33:53 2019
> >>>>      @@ -1389,9 +1389,9 @@
> >>>>               <value xml:lang="zh-TW">å›žå ±</value>
> >>>>           </property>
> >>>>           <property key="CommonBranch">
> >>>>      -        <value xml:lang="en">branch</value>
> >>>>      -        <value xml:lang="es">rama</value>
> >>>>      -        <value xml:lang="fr">branche</value>
> >>>>      +        <value xml:lang="en"> branch</value>
> >>>>      +        <value xml:lang="es"> rama</value>
> >>>>      +        <value xml:lang="fr"> branche</value>
> >>>>           </property>
> >>>>           <property key="CommonBuiltOn">
> >>>>               <value xml:lang="en">built on</value>
> >>>>
> >>>>
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1854306 - /ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml

Deepak Dixit-5
Done, Committed slightly modified version of the suggested patch.

Thanks & Regards
--
Deepak Dixit
ofbiz.apache.org


On Thu, Mar 14, 2019 at 2:41 PM Deepak Dixit <[hidden email]> wrote:

> Sure Jacques, Will do
> Thanks & Regards
> --
> Deepak Dixit
> ofbiz.apache.org
>
>
> On Thu, Mar 14, 2019 at 2:16 PM Jacques Le Roux <
> [hidden email]> wrote:
>
>> Thanks Deepak,
>>
>> Unfortunately it's difficult to use it as us and I have not created a
>> Jira for that.
>>
>> Could you please commit it?
>>
>> TIA
>>
>> Jacques
>>
>> Le 13/03/2019 à 06:42, Deepak Dixit a écrit :
>> > Hi Jacques,
>> >
>> > I tested the following patch and it works
>> > =====================
>> >
>> > Index: build.gradle
>> >
>> > ===================================================================
>> >
>> > --- build.gradle (revision 1855081)
>> >
>> > +++ build.gradle (working copy)
>> >
>> > @@ -638,7 +638,7 @@
>> >
>> >           revision = revisionOutput.toString()
>> >
>> >           gitFooterFile.delete()
>> >
>> >           gitFooterFile.createNewFile()
>> >
>> > -        gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}"
>> +
>> > System.lineSeparator()
>> >
>> > +        gitFooterFile << ' ${uiLabelMap.CommonBranch} : ' +
>> "${branch}" +
>> > System.lineSeparator()
>> >
>> >           gitFooterFile << '${uiLabelMap.CommonRevision} : ' +
>> "${revision}"
>> > + System.lineSeparator()
>> >
>> >           gitFooterFile << '${uiLabelMap.CommonBuiltOn} : ' +
>> "${timestamp}"
>> > + System.lineSeparator()
>> >
>> >           gitFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
>> > "${org.gradle.internal.jvm.Jvm.current()}"
>> >
>> > @@ -663,7 +663,7 @@
>> >
>> >           def info = new XmlParser().parseText(svnOutput.toString())
>> >
>> >           svnFooterFile.delete()
>> >
>> >           svnFooterFile.createNewFile()
>> >
>> > -        svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
>> > "${info.entry.url.text()}" + System.lineSeparator()
>> >
>> > +        svnFooterFile << ' ${uiLabelMap.CommonBranch} : ' +
>> > "${info.entry.url.text()}" + System.lineSeparator()
>> >
>> >           svnFooterFile << '${uiLabelMap.CommonRevision} : ' +
>> > "${info.entry.commit.@revision}" + System.lineSeparator()
>> >
>> >           svnFooterFile << '${uiLabelMap.CommonBuiltOn} : ' +
>> "${timestamp}"
>> > + System.lineSeparator()
>> >
>> >           svnFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
>> > "${org.gradle.internal.jvm.Jvm.current()}"
>> >
>> > Index: framework/common/config/CommonUiLabels.xml
>> >
>> > ===================================================================
>> >
>> > --- framework/common/config/CommonUiLabels.xml (revision 1855081)
>> >
>> > +++ framework/common/config/CommonUiLabels.xml (working copy)
>> >
>> > @@ -1389,9 +1389,9 @@
>> >
>> >           <value xml:lang="zh-TW">回報</value>
>> >
>> >       </property>
>> >
>> >       <property key="CommonBranch">
>> >
>> > -        <value xml:lang="en"> branch</value>
>> >
>> > -        <value xml:lang="es"> rama</value>
>> >
>> > -        <value xml:lang="fr"> branche</value>
>> >
>> > +        <value xml:lang="en">branch</value>
>> >
>> > +        <value xml:lang="es">rama</value>
>> >
>> > +        <value xml:lang="fr">branche</value>
>> >
>> >       </property>
>> >
>> >       <property key="CommonBuiltOn">
>> >
>> >           <value xml:lang="en">built on</value>
>> >
>> >
>> > =====================
>> >
>> >
>> > Kind Regards,
>> > Deepak Dixit
>> >
>> > On Wed, Mar 13, 2019 at 2:16 AM Jacques Le Roux <
>> > [hidden email]> wrote:
>> >
>> >> Actually I changed it there because it does not work in build.gradle
>> where
>> >> it's already
>> >>
>> >> gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" +
>> >> System.lineSeparator()
>> >>
>> >> svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
>> >> "${info.entry.url.text()}" + System.lineSeparator()
>> >>
>> >> Jacques
>> >>
>> >> Le 12/03/2019 à 10:33, Jacques Le Roux a écrit :
>> >>> Agreed Deepak,
>> >>>
>> >>> Makes sense, I 'll change that
>> >>>
>> >>> Thanks
>> >>>
>> >>> Jacques
>> >>>
>> >>> Le 12/03/2019 à 10:07, Deepak Dixit a écrit :
>> >>>> Hi Jacques,
>> >>>>
>> >>>> I think instead of adding space in uiLabel it would be food if we add
>> >> space in the template
>> >>>> Thanks & Regards
>> >>>> --
>> >>>> Deepak Dixit
>> >>>> ofbiz.apache.org <http://ofbiz.apache.org>
>> >>>>
>> >>>>
>> >>>> On Mon, Feb 25, 2019 at 4:03 PM <[hidden email] <mailto:
>> >> [hidden email]>> wrote:
>> >>>>      Author: jleroux
>> >>>>      Date: Mon Feb 25 10:33:53 2019
>> >>>>      New Revision: 1854306
>> >>>>
>> >>>>      URL: http://svn.apache.org/viewvc?rev=1854306&view=rev
>> >>>>      Log:
>> >>>>      Improved: no functional change
>> >>>>
>> >>>>      A missing space before CommonBranch label made the word
>> trunkbranch
>> >> instead of
>> >>>>      trunk branch.
>> >>>>
>> >>>>      I'll backport this
>> >>>>
>> >>>>      Modified:
>> >>>>
>> >>
>>  ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>> >>>>      Modified:
>> >> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>> >>>>      URL:
>> >>>>
>> >>
>> http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml?rev=1854306&r1=1854305&r2=1854306&view=diff
>> >>
>>  ==============================================================================
>> >>>>      ---
>> >> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>> >> (original)
>> >>>>      +++
>> >> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>> Mon
>> >> Feb 25 10:33:53 2019
>> >>>>      @@ -1389,9 +1389,9 @@
>> >>>>               <value xml:lang="zh-TW">å›žå ±</value>
>> >>>>           </property>
>> >>>>           <property key="CommonBranch">
>> >>>>      -        <value xml:lang="en">branch</value>
>> >>>>      -        <value xml:lang="es">rama</value>
>> >>>>      -        <value xml:lang="fr">branche</value>
>> >>>>      +        <value xml:lang="en"> branch</value>
>> >>>>      +        <value xml:lang="es"> rama</value>
>> >>>>      +        <value xml:lang="fr"> branche</value>
>> >>>>           </property>
>> >>>>           <property key="CommonBuiltOn">
>> >>>>               <value xml:lang="en">built on</value>
>> >>>>
>> >>>>
>>
>
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r1854306 - /ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml

Jacques Le Roux
Administrator
Thanks Deepak!

Le 14/03/2019 à 10:38, Deepak Dixit a écrit :

> Done, Committed slightly modified version of the suggested patch.
>
> Thanks & Regards
> --
> Deepak Dixit
> ofbiz.apache.org
>
>
> On Thu, Mar 14, 2019 at 2:41 PM Deepak Dixit <[hidden email]> wrote:
>
>> Sure Jacques, Will do
>> Thanks & Regards
>> --
>> Deepak Dixit
>> ofbiz.apache.org
>>
>>
>> On Thu, Mar 14, 2019 at 2:16 PM Jacques Le Roux <
>> [hidden email]> wrote:
>>
>>> Thanks Deepak,
>>>
>>> Unfortunately it's difficult to use it as us and I have not created a
>>> Jira for that.
>>>
>>> Could you please commit it?
>>>
>>> TIA
>>>
>>> Jacques
>>>
>>> Le 13/03/2019 à 06:42, Deepak Dixit a écrit :
>>>> Hi Jacques,
>>>>
>>>> I tested the following patch and it works
>>>> =====================
>>>>
>>>> Index: build.gradle
>>>>
>>>> ===================================================================
>>>>
>>>> --- build.gradle (revision 1855081)
>>>>
>>>> +++ build.gradle (working copy)
>>>>
>>>> @@ -638,7 +638,7 @@
>>>>
>>>>            revision = revisionOutput.toString()
>>>>
>>>>            gitFooterFile.delete()
>>>>
>>>>            gitFooterFile.createNewFile()
>>>>
>>>> -        gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}"
>>> +
>>>> System.lineSeparator()
>>>>
>>>> +        gitFooterFile << ' ${uiLabelMap.CommonBranch} : ' +
>>> "${branch}" +
>>>> System.lineSeparator()
>>>>
>>>>            gitFooterFile << '${uiLabelMap.CommonRevision} : ' +
>>> "${revision}"
>>>> + System.lineSeparator()
>>>>
>>>>            gitFooterFile << '${uiLabelMap.CommonBuiltOn} : ' +
>>> "${timestamp}"
>>>> + System.lineSeparator()
>>>>
>>>>            gitFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
>>>> "${org.gradle.internal.jvm.Jvm.current()}"
>>>>
>>>> @@ -663,7 +663,7 @@
>>>>
>>>>            def info = new XmlParser().parseText(svnOutput.toString())
>>>>
>>>>            svnFooterFile.delete()
>>>>
>>>>            svnFooterFile.createNewFile()
>>>>
>>>> -        svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
>>>> "${info.entry.url.text()}" + System.lineSeparator()
>>>>
>>>> +        svnFooterFile << ' ${uiLabelMap.CommonBranch} : ' +
>>>> "${info.entry.url.text()}" + System.lineSeparator()
>>>>
>>>>            svnFooterFile << '${uiLabelMap.CommonRevision} : ' +
>>>> "${info.entry.commit.@revision}" + System.lineSeparator()
>>>>
>>>>            svnFooterFile << '${uiLabelMap.CommonBuiltOn} : ' +
>>> "${timestamp}"
>>>> + System.lineSeparator()
>>>>
>>>>            svnFooterFile << '${uiLabelMap.CommonJavaVersion} : ' +
>>>> "${org.gradle.internal.jvm.Jvm.current()}"
>>>>
>>>> Index: framework/common/config/CommonUiLabels.xml
>>>>
>>>> ===================================================================
>>>>
>>>> --- framework/common/config/CommonUiLabels.xml (revision 1855081)
>>>>
>>>> +++ framework/common/config/CommonUiLabels.xml (working copy)
>>>>
>>>> @@ -1389,9 +1389,9 @@
>>>>
>>>>            <value xml:lang="zh-TW">回報</value>
>>>>
>>>>        </property>
>>>>
>>>>        <property key="CommonBranch">
>>>>
>>>> -        <value xml:lang="en"> branch</value>
>>>>
>>>> -        <value xml:lang="es"> rama</value>
>>>>
>>>> -        <value xml:lang="fr"> branche</value>
>>>>
>>>> +        <value xml:lang="en">branch</value>
>>>>
>>>> +        <value xml:lang="es">rama</value>
>>>>
>>>> +        <value xml:lang="fr">branche</value>
>>>>
>>>>        </property>
>>>>
>>>>        <property key="CommonBuiltOn">
>>>>
>>>>            <value xml:lang="en">built on</value>
>>>>
>>>>
>>>> =====================
>>>>
>>>>
>>>> Kind Regards,
>>>> Deepak Dixit
>>>>
>>>> On Wed, Mar 13, 2019 at 2:16 AM Jacques Le Roux <
>>>> [hidden email]> wrote:
>>>>
>>>>> Actually I changed it there because it does not work in build.gradle
>>> where
>>>>> it's already
>>>>>
>>>>> gitFooterFile << '${uiLabelMap.CommonBranch} : ' + "${branch}" +
>>>>> System.lineSeparator()
>>>>>
>>>>> svnFooterFile << '${uiLabelMap.CommonBranch} : ' +
>>>>> "${info.entry.url.text()}" + System.lineSeparator()
>>>>>
>>>>> Jacques
>>>>>
>>>>> Le 12/03/2019 à 10:33, Jacques Le Roux a écrit :
>>>>>> Agreed Deepak,
>>>>>>
>>>>>> Makes sense, I 'll change that
>>>>>>
>>>>>> Thanks
>>>>>>
>>>>>> Jacques
>>>>>>
>>>>>> Le 12/03/2019 à 10:07, Deepak Dixit a écrit :
>>>>>>> Hi Jacques,
>>>>>>>
>>>>>>> I think instead of adding space in uiLabel it would be food if we add
>>>>> space in the template
>>>>>>> Thanks & Regards
>>>>>>> --
>>>>>>> Deepak Dixit
>>>>>>> ofbiz.apache.org <http://ofbiz.apache.org>
>>>>>>>
>>>>>>>
>>>>>>> On Mon, Feb 25, 2019 at 4:03 PM <[hidden email] <mailto:
>>>>> [hidden email]>> wrote:
>>>>>>>       Author: jleroux
>>>>>>>       Date: Mon Feb 25 10:33:53 2019
>>>>>>>       New Revision: 1854306
>>>>>>>
>>>>>>>       URL: http://svn.apache.org/viewvc?rev=1854306&view=rev
>>>>>>>       Log:
>>>>>>>       Improved: no functional change
>>>>>>>
>>>>>>>       A missing space before CommonBranch label made the word
>>> trunkbranch
>>>>> instead of
>>>>>>>       trunk branch.
>>>>>>>
>>>>>>>       I'll backport this
>>>>>>>
>>>>>>>       Modified:
>>>>>>>
>>>   ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>>>>>>>       Modified:
>>>>> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>>>>>>>       URL:
>>>>>>>
>>> http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml?rev=1854306&r1=1854305&r2=1854306&view=diff
>>>   ==============================================================================
>>>>>>>       ---
>>>>> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>>>>> (original)
>>>>>>>       +++
>>>>> ofbiz/ofbiz-framework/trunk/framework/common/config/CommonUiLabels.xml
>>> Mon
>>>>> Feb 25 10:33:53 2019
>>>>>>>       @@ -1389,9 +1389,9 @@
>>>>>>>                <value xml:lang="zh-TW">å›žå ±</value>
>>>>>>>            </property>
>>>>>>>            <property key="CommonBranch">
>>>>>>>       -        <value xml:lang="en">branch</value>
>>>>>>>       -        <value xml:lang="es">rama</value>
>>>>>>>       -        <value xml:lang="fr">branche</value>
>>>>>>>       +        <value xml:lang="en"> branch</value>
>>>>>>>       +        <value xml:lang="es"> rama</value>
>>>>>>>       +        <value xml:lang="fr"> branche</value>
>>>>>>>            </property>
>>>>>>>            <property key="CommonBuiltOn">
>>>>>>>                <value xml:lang="en">built on</value>
>>>>>>>
>>>>>>>