You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by do...@apache.org on 2009/10/20 19:05:07 UTC

svn commit: r827734 - in /ofbiz/branches/release09.04: ./ applications/order/webapp/ordermgr/order/orderlist.ftl

Author: doogie
Date: Tue Oct 20 17:05:07 2009
New Revision: 827734

URL: http://svn.apache.org/viewvc?rev=827734&view=rev
Log:
Applied fix from trunk for revision: 827730

Modified:
    ofbiz/branches/release09.04/   (props changed)
    ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl

Propchange: ofbiz/branches/release09.04/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Tue Oct 20 17:05:07 2009
@@ -1 +1 @@
-/ofbiz/trunk:765933,766011,766015,766293,766307,766316,766325,766462,766522,766800,767060,767072,767093,767098-767099,767102,767123,767125,767127,767279,767287,767671,767688,767694,767822,767845,768358,768490,768550,768675,768686,768705,768811,768815,768960,769030,769500,770272,770308,770997,771073,772401,772464-772465,773076,773557,773628,773659,773697,774014,774632,774661,774995,775292,775667,776227,776594,776620,776922,777004,777020,777768,777792,777893,777947,778078,778094,778107,778273,778278,778280,778364,778374,778402,778576,778594,778628,779020,779477,779496,779639,779834,779856,779866,779873,780111,780138,780180,780199,780203,780906,780945,781201,781534,781549,781669,781680,781694,782663,783257,783266,783833,783913,783917,785123,785764,785967,786778,787126,787435-787436,787442,787520,788965,788983,788987,789329,789337,789506,789548,796769,799185,800461,800846,801023,802346,804364,805307,806127,806377,808786-808787,808792,809141,810370,810438,810465,810807,810809,810
 814,810832,810836,810878,810917,811020,811280,811297,811419,811528,811708,811714,811716,811793,811838,811860,811865,811870,812159,812182,812192,812456,812540,813126,813131,813283,813672,813702,814168,814205,814251,814349,814531,814576,814681,814731,815158,815165,815350,815687,815977,816255,816863,818030,818049,818150,818494,818500,818716,818976,819275-819276,819282,819337,821263,821270,822659,823877-823878,823883,823888,823892,824511,825181-825182,826253
+/ofbiz/trunk:765933,766011,766015,766293,766307,766316,766325,766462,766522,766800,767060,767072,767093,767098-767099,767102,767123,767125,767127,767279,767287,767671,767688,767694,767822,767845,768358,768490,768550,768675,768686,768705,768811,768815,768960,769030,769500,770272,770308,770997,771073,772401,772464-772465,773076,773557,773628,773659,773697,774014,774632,774661,774995,775292,775667,776227,776594,776620,776922,777004,777020,777768,777792,777893,777947,778078,778094,778107,778273,778278,778280,778364,778374,778402,778576,778594,778628,779020,779477,779496,779639,779834,779856,779866,779873,780111,780138,780180,780199,780203,780906,780945,781201,781534,781549,781669,781680,781694,782663,783257,783266,783833,783913,783917,785123,785764,785967,786778,787126,787435-787436,787442,787520,788965,788983,788987,789329,789337,789506,789548,796769,799185,800461,800846,801023,802346,804364,805307,806127,806377,808786-808787,808792,809141,810370,810438,810465,810807,810809,810
 814,810832,810836,810878,810917,811020,811280,811297,811419,811528,811708,811714,811716,811793,811838,811860,811865,811870,812159,812182,812192,812456,812540,813126,813131,813283,813672,813702,814168,814205,814251,814349,814531,814576,814681,814731,815158,815165,815350,815687,815977,816255,816863,818030,818049,818150,818494,818500,818716,818976,819275-819276,819282,819337,821263,821270,822659,823877-823878,823883,823888,823892,824511,825181-825182,826253,827730

Modified: ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl
URL: http://svn.apache.org/viewvc/ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl?rev=827734&r1=827733&r2=827734&view=diff
==============================================================================
--- ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl (original)
+++ ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl Tue Oct 20 17:05:07 2009
@@ -172,9 +172,13 @@
                 <#assign billToPartyNameResult = dispatcher.runSync("getPartyNameForDate", Static["org.ofbiz.base.util.UtilMisc"].toMap("partyId", billToParty.partyId, "compareDate", orderHeader.orderDate, "userLogin", userLogin))/>
                 <#assign billTo = billToPartyNameResult.fullName?default("[${uiLabelMap.OrderPartyNameNotFound}]")/>
                 <#-- <#assign billTo = Static["org.ofbiz.party.party.PartyHelper"].getPartyName(billToParty, true)?if_exists> -->
+            <#else>
+              <#assign billTo = ''/>
             </#if>
             <#if billFromParty?has_content>
               <#assign billFrom = Static["org.ofbiz.party.party.PartyHelper"].getPartyName(billFromParty, true)?if_exists>
+            <#else>
+              <#assign billFrom = ''/>
             </#if>
             <tr>
               <td>${orderHeader.orderDate.toString()}</td>



Re: svn commit: r827734 - in /ofbiz/branches/release09.04: ./ applications/order/webapp/ordermgr/order/orderlist.ftl

Posted by Ashish Vijaywargiya <vi...@gmail.com>.
Hello Adam,

Thanks for your comment.
Please see my comments inline:

--
Ashish

On Fri, Oct 23, 2009 at 2:39 AM, Adam Heath <do...@brainfood.com> wrote:

> Ashish Vijaywargiya wrote:
> > Is this log message verbose for the person reading the history of commit
> > logs?
> > Please try to use mergefromtrunk.sh  file that will do everything for you
> > automatically or do copy paste of log message from trunk here for easy
> > reference for the person reading log.
> >
> > Here is the command that can be used on terminal window.
> > ./mergefromtrunk.sh 827730
>
> I did 2 backports.  release4.0 had a conflict, and I had to do it by
> hand.  I looked at what the script did, and did it all manually.
>
> However, for release09.04, which you are commenting on, I used the
> script completely, so I don't know what you are complaining about.
>


If you might have used the mergefromtrunk.sh script then you log message
might have looked like as shown below:
If you will see the script then it also reads the log message from the trunk
revision from which you are merging code to the release branch.

--------------------------
Applied fix from trunk for revision: 827730
Freemarker does not reset values each time thru a loop, so reset billTo
and billFrom to empty.
--------------------------

It appears that there is some problem with the script.

Re: svn commit: r827734 - in /ofbiz/branches/release09.04: ./ applications/order/webapp/ordermgr/order/orderlist.ftl

Posted by Adam Heath <do...@brainfood.com>.
Ashish Vijaywargiya wrote:
> Is this log message verbose for the person reading the history of commit
> logs?
> Please try to use mergefromtrunk.sh  file that will do everything for you
> automatically or do copy paste of log message from trunk here for easy
> reference for the person reading log.
> 
> Here is the command that can be used on terminal window.
> ./mergefromtrunk.sh 827730

I did 2 backports.  release4.0 had a conflict, and I had to do it by
hand.  I looked at what the script did, and did it all manually.

However, for release09.04, which you are commenting on, I used the
script completely, so I don't know what you are complaining about.


Re: svn commit: r827734 - in /ofbiz/branches/release09.04: ./ applications/order/webapp/ordermgr/order/orderlist.ftl

Posted by Ashish Vijaywargiya <vi...@gmail.com>.
Is this log message verbose for the person reading the history of commit
logs?
Please try to use mergefromtrunk.sh  file that will do everything for you
automatically or do copy paste of log message from trunk here for easy
reference for the person reading log.

Here is the command that can be used on terminal window.
./mergefromtrunk.sh 827730

--
Ashish

On Tue, Oct 20, 2009 at 10:35 PM, <do...@apache.org> wrote:

> Author: doogie
> Date: Tue Oct 20 17:05:07 2009
> New Revision: 827734
>
> URL: http://svn.apache.org/viewvc?rev=827734&view=rev
> Log:
> Applied fix from trunk for revision: 827730
>
> Modified:
>    ofbiz/branches/release09.04/   (props changed)
>
>  ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl
>
> Propchange: ofbiz/branches/release09.04/
>
> ------------------------------------------------------------------------------
> --- svn:mergeinfo (original)
> +++ svn:mergeinfo Tue Oct 20 17:05:07 2009
> @@ -1 +1 @@
>
> -/ofbiz/trunk:765933,766011,766015,766293,766307,766316,766325,766462,766522,766800,767060,767072,767093,767098-767099,767102,767123,767125,767127,767279,767287,767671,767688,767694,767822,767845,768358,768490,768550,768675,768686,768705,768811,768815,768960,769030,769500,770272,770308,770997,771073,772401,772464-772465,773076,773557,773628,773659,773697,774014,774632,774661,774995,775292,775667,776227,776594,776620,776922,777004,777020,777768,777792,777893,777947,778078,778094,778107,778273,778278,778280,778364,778374,778402,778576,778594,778628,779020,779477,779496,779639,779834,779856,779866,779873,780111,780138,780180,780199,780203,780906,780945,781201,781534,781549,781669,781680,781694,782663,783257,783266,783833,783913,783917,785123,785764,785967,786778,787126,787435-787436,787442,787520,788965,788983,788987,789329,789337,789506,789548,796769,799185,800461,800846,801023,802346,804364,805307,806127,806377,808786-808787,808792,809141,810370,810438,810465,810807,810809,810
>
>  814,810832,810836,810878,810917,811020,811280,811297,811419,811528,811708,811714,811716,811793,811838,811860,811865,811870,812159,812182,812192,812456,812540,813126,813131,813283,813672,813702,814168,814205,814251,814349,814531,814576,814681,814731,815158,815165,815350,815687,815977,816255,816863,818030,818049,818150,818494,818500,818716,818976,819275-819276,819282,819337,821263,821270,822659,823877-823878,823883,823888,823892,824511,825181-825182,826253
>
> +/ofbiz/trunk:765933,766011,766015,766293,766307,766316,766325,766462,766522,766800,767060,767072,767093,767098-767099,767102,767123,767125,767127,767279,767287,767671,767688,767694,767822,767845,768358,768490,768550,768675,768686,768705,768811,768815,768960,769030,769500,770272,770308,770997,771073,772401,772464-772465,773076,773557,773628,773659,773697,774014,774632,774661,774995,775292,775667,776227,776594,776620,776922,777004,777020,777768,777792,777893,777947,778078,778094,778107,778273,778278,778280,778364,778374,778402,778576,778594,778628,779020,779477,779496,779639,779834,779856,779866,779873,780111,780138,780180,780199,780203,780906,780945,781201,781534,781549,781669,781680,781694,782663,783257,783266,783833,783913,783917,785123,785764,785967,786778,787126,787435-787436,787442,787520,788965,788983,788987,789329,789337,789506,789548,796769,799185,800461,800846,801023,802346,804364,805307,806127,806377,808786-808787,808792,809141,810370,810438,810465,810807,810809,810
>
>  814,810832,810836,810878,810917,811020,811280,811297,811419,811528,811708,811714,811716,811793,811838,811860,811865,811870,812159,812182,812192,812456,812540,813126,813131,813283,813672,813702,814168,814205,814251,814349,814531,814576,814681,814731,815158,815165,815350,815687,815977,816255,816863,818030,818049,818150,818494,818500,818716,818976,819275-819276,819282,819337,821263,821270,822659,823877-823878,823883,823888,823892,824511,825181-825182,826253,827730
>
> Modified:
> ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl
> URL:
> http://svn.apache.org/viewvc/ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl?rev=827734&r1=827733&r2=827734&view=diff
>
> ==============================================================================
> ---
> ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl
> (original)
> +++
> ofbiz/branches/release09.04/applications/order/webapp/ordermgr/order/orderlist.ftl
> Tue Oct 20 17:05:07 2009
> @@ -172,9 +172,13 @@
>                 <#assign billToPartyNameResult =
> dispatcher.runSync("getPartyNameForDate",
> Static["org.ofbiz.base.util.UtilMisc"].toMap("partyId", billToParty.partyId,
> "compareDate", orderHeader.orderDate, "userLogin", userLogin))/>
>                 <#assign billTo =
> billToPartyNameResult.fullName?default("[${uiLabelMap.OrderPartyNameNotFound}]")/>
>                 <#-- <#assign billTo =
> Static["org.ofbiz.party.party.PartyHelper"].getPartyName(billToParty,
> true)?if_exists> -->
> +            <#else>
> +              <#assign billTo = ''/>
>             </#if>
>             <#if billFromParty?has_content>
>               <#assign billFrom =
> Static["org.ofbiz.party.party.PartyHelper"].getPartyName(billFromParty,
> true)?if_exists>
> +            <#else>
> +              <#assign billFrom = ''/>
>             </#if>
>             <tr>
>               <td>${orderHeader.orderDate.toString()}</td>
>
>
>