Re: svn commit: r731508 - in /ofbiz/trunk: framework/common/webcommon/WEB-INF/actions/includes/ framework/common/webcommon/portal/ framework/common/widget/ framework/example/webapp/ framework/example/widget/example/ specialpurpose/myportal/data/ specialpur...

[Jacopo Cappellato-3]

Hi Hans,

instead of using the DESC word, that is SQL specific code, it is  
better to use the + symbol.

Cheers,

Jacopo


On Jan 5, 2009, at 10:48 AM, [hidden email] wrote:

> Author: hansbak
> Date: Mon Jan  5 01:48:15 2009
> New Revision: 731508
>
> URL: http://svn.apache.org/viewvc?rev=731508&view=rev
> Log:
> updates according the comments from bruno
>
> Added:
>    ofbiz/trunk/framework/example/webapp/appheader.ftl   (with props)
> Removed:
>    ofbiz/trunk/framework/common/webcommon/portal/appheader.ftl
> Modified:
>    ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/includes/
> ListPortalPages.groovy
>    ofbiz/trunk/framework/common/widget/PortalPageForms.xml
>    ofbiz/trunk/framework/common/widget/PortalPageScreens.xml
>    ofbiz/trunk/framework/example/widget/example/CommonScreens.xml
>    ofbiz/trunk/specialpurpose/myportal/data/MyPortalTypeData.xml
>    ofbiz/trunk/specialpurpose/myportal/widget/MyPortalScreens.xml
>
> Modified: ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/
> includes/ListPortalPages.groovy
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/includes/ListPortalPages.groovy?rev=731508&r1=731507&r2=731508&view=diff
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/includes/
> ListPortalPages.groovy (original)
> +++ ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/includes/
> ListPortalPages.groovy Mon Jan  5 01:48:15 2009
> @@ -36,7 +36,7 @@
>     ],EntityOperator.AND);
> sortField = parameters.sortField;
> if (UtilValidate.isEmpty(sortField)) {
> - sortField = "portalPageName";
> + sortField = "parentPortalPageId DESC";
> }
> orderBy = [sortField];
> portalPages = delegator.findList("PortalPage", ppCond, null,  
> orderBy, null, false);

[Jacopo Cappellato-4]

sorry,

I meant the - symbol.

Jacopo

On Feb 5, 2009, at 4:24 PM, Jacopo Cappellato wrote:

> Hi Hans,
>
> instead of using the DESC word, that is SQL specific code, it is  
> better to use the + symbol.
>
> Cheers,
>
> Jacopo
>
>
> On Jan 5, 2009, at 10:48 AM, [hidden email] wrote:
>
>> Author: hansbak
>> Date: Mon Jan  5 01:48:15 2009
>> New Revision: 731508
>>
>> URL: http://svn.apache.org/viewvc?rev=731508&view=rev
>> Log:
>> updates according the comments from bruno
>>
>> Added:
>>   ofbiz/trunk/framework/example/webapp/appheader.ftl   (with props)
>> Removed:
>>   ofbiz/trunk/framework/common/webcommon/portal/appheader.ftl
>> Modified:
>>   ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/includes/
>> ListPortalPages.groovy
>>   ofbiz/trunk/framework/common/widget/PortalPageForms.xml
>>   ofbiz/trunk/framework/common/widget/PortalPageScreens.xml
>>   ofbiz/trunk/framework/example/widget/example/CommonScreens.xml
>>   ofbiz/trunk/specialpurpose/myportal/data/MyPortalTypeData.xml
>>   ofbiz/trunk/specialpurpose/myportal/widget/MyPortalScreens.xml
>>
>> Modified: ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/
>> includes/ListPortalPages.groovy
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/includes/ListPortalPages.groovy?rev=731508&r1=731507&r2=731508&view=diff
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =
>> =====================================================================
>> --- ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/includes/
>> ListPortalPages.groovy (original)
>> +++ ofbiz/trunk/framework/common/webcommon/WEB-INF/actions/includes/
>> ListPortalPages.groovy Mon Jan  5 01:48:15 2009
>> @@ -36,7 +36,7 @@
>>    ],EntityOperator.AND);
>> sortField = parameters.sortField;
>> if (UtilValidate.isEmpty(sortField)) {
>> - sortField = "portalPageName";
>> + sortField = "parentPortalPageId DESC";
>> }
>> orderBy = [sortField];
>> portalPages = delegator.findList("PortalPage", ppCond, null,  
>> orderBy, null, false);
>