You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by Jacques Le Roux <ja...@les7arts.com> on 2016/09/01 14:17:04 UTC
Re: svn commit: r1758774 - in /ofbiz/trunk:
applications/product/src/main/java/org/apache/ofbiz/shipment/thirdparty/dhl
/ framework/widget/src/main/java/org/apache/ofbiz/widget/renderer/macro/
specialpurpose/ecommerce/widget/ specialpurpose/ecommerce/widget...
Thanks, reverted at revision: 1758779
Jacques
Le 01/09/2016 � 15:57, Jacopo Cappellato a �crit :
> Hi Jacques,
>
> you have committed some unwanted/unrelated changes in this commit.
>
> Jacopo
>
> On Thu, Sep 1, 2016 at 3:34 PM, <jl...@apache.org> wrote:
>
>> Author: jleroux
>> Date: Thu Sep 1 13:34:17 2016
>> New Revision: 1758774
>>
>> URL: http://svn.apache.org/viewvc?rev=1758774&view=rev
>> Log:
>> A patch from Pierre Smits for "Maximise the utilisation of the Label
>> CommonStatus in various ecommerce forms" https://issues.apache.org/
>> jira/browse/OFBIZ-8102
>>
>> There are forms in the ecommerce component referencing the status. This
>> issue is about applying the CommonLabel label where it is possible in these
>> forms.
>>
>> Modified:
>> ofbiz/trunk/applications/product/src/main/java/org/
>> apache/ofbiz/shipment/thirdparty/dhl/DhlServices.java
>> ofbiz/trunk/framework/widget/src/main/java/org/apache/
>> ofbiz/widget/renderer/macro/MacroFormRenderer.java
>> ofbiz/trunk/specialpurpose/ecommerce/widget/ForumForms.xml
>> ofbiz/trunk/specialpurpose/ecommerce/widget/blog/BlogForms.xml
>>
>> Modified: ofbiz/trunk/applications/product/src/main/java/org/
>> apache/ofbiz/shipment/thirdparty/dhl/DhlServices.java
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/
>> product/src/main/java/org/apache/ofbiz/shipment/
>> thirdparty/dhl/DhlServices.java?rev=1758774&r1=1758773&
>> r2=1758774&view=diff
>> ============================================================
>> ==================
>> --- ofbiz/trunk/applications/product/src/main/java/org/
>> apache/ofbiz/shipment/thirdparty/dhl/DhlServices.java (original)
>> +++ ofbiz/trunk/applications/product/src/main/java/org/
>> apache/ofbiz/shipment/thirdparty/dhl/DhlServices.java Thu Sep 1 13:34:17
>> 2016
>> @@ -809,20 +809,9 @@ public class DhlServices {
>> return handleDhlShipmentConfirmResponse(responseString,
>> shipmentRouteSegment, shipmentPackageRouteSegs, locale);
>> } catch (GenericEntityException e) {
>> Debug.logError(e, module);
>> - if (shipmentConfirmResponseString != null) {
>> - Debug.logError("Got XML ShipmentConfirmRespose: " +
>> shipmentConfirmResponseString, module);
>> - return ServiceUtil.returnError(UtilMisc.toList(
>> - UtilProperties.getMessage(resourceError,
>> - "FacilityShipmentFedexRateTempl
>> ateReadingError",
>> - UtilMisc.toMap("errorString",
>> e.toString()), locale),
>> - UtilProperties.getMessage(resourceError,
>> - "FacilityShipmentFedexShipmentC
>> onfirmResponse",
>> - UtilMisc.toMap("
>> shipmentConfirmResponseString", shipmentConfirmResponseString), locale)));
>> - } else {
>> - return ServiceUtil.returnError(UtilProperties.getMessage(
>> resourceError,
>> - "FacilityShipmentFedexRateTemplateReadingError",
>> - UtilMisc.toMap("errorString", e.toString()),
>> locale));
>> - }
>> + return ServiceUtil.returnError(UtilProperties.getMessage(
>> resourceError,
>> + "FacilityShipmentFedexRateTemplateReadingError",
>> + UtilMisc.toMap("errorString", e.toString()), locale));
>> } catch (GenericServiceException e) {
>> Debug.logError(e, module);
>> return ServiceUtil.returnError(UtilProperties.getMessage(
>> resourceError,
>>
>> Modified: ofbiz/trunk/framework/widget/src/main/java/org/apache/
>> ofbiz/widget/renderer/macro/MacroFormRenderer.java
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/
>> src/main/java/org/apache/ofbiz/widget/renderer/macro/
>> MacroFormRenderer.java?rev=1758774&r1=1758773&r2=1758774&view=diff
>> ============================================================
>> ==================
>> --- ofbiz/trunk/framework/widget/src/main/java/org/apache/
>> ofbiz/widget/renderer/macro/MacroFormRenderer.java (original)
>> +++ ofbiz/trunk/framework/widget/src/main/java/org/apache/
>> ofbiz/widget/renderer/macro/MacroFormRenderer.java Thu Sep 1 13:34:17
>> 2016
>> @@ -22,6 +22,7 @@ import java.io.IOException;
>> import java.io.Reader;
>> import java.io.StringReader;
>> import java.io.StringWriter;
>> +import java.net.URLEncoder;
>> import java.rmi.server.UID;
>> import java.sql.Timestamp;
>> import java.util.HashSet;
>> @@ -2866,6 +2867,7 @@ public final class MacroFormRenderer imp
>> String newQueryString = sb.toString();
>> String urlPath = UtilHttp.removeQueryStringFromTarget(
>> paginateTarget);
>> linkUrl = rh.makeLink(this.request, this.response,
>> urlPath.concat(newQueryString));
>> + linkUrl = URLEncoder.encode(linkUrl, "UTF-8");
>> }
>> StringWriter sr = new StringWriter();
>> sr.append("<@renderSortField ");
>> @@ -2873,7 +2875,7 @@ public final class MacroFormRenderer imp
>> sr.append(sortFieldStyle);
>> sr.append("\" title=\"");
>> sr.append(titleText);
>> - sr.append("\" linkUrl=\"");
>> + sr.append("\" linkUrl=r\"");
>> sr.append(linkUrl);
>> sr.append("\" ajaxEnabled=");
>> sr.append(Boolean.toString(ajaxEnabled));
>>
>> Modified: ofbiz/trunk/specialpurpose/ecommerce/widget/ForumForms.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/
>> ecommerce/widget/ForumForms.xml?rev=1758774&r1=1758773&r2=
>> 1758774&view=diff
>> ============================================================
>> ==================
>> --- ofbiz/trunk/specialpurpose/ecommerce/widget/ForumForms.xml (original)
>> +++ ofbiz/trunk/specialpurpose/ecommerce/widget/ForumForms.xml Thu Sep 1
>> 13:34:17 2016
>> @@ -254,7 +254,7 @@ under the License.
>> </field>
>>
>> <field name="textData" title="${uiLabelMap.ContentText}"
>> id-name="textData"><textarea rows="20" cols="60"/></field>
>> - <field name="statusId" >
>> + <field name="statusId" title="${uiLabelMap.CommonStatus}">
>> <drop-down allow-empty="false" no-current-selected-key="
>> SUBSCRIBE_BRONZE">
>> <list-options list-name="statusList" key-name="statusId"/>
>> </drop-down>
>> @@ -359,7 +359,7 @@ under the License.
>> <entity-options entity-name="CharacterSet"
>> key-field-name="characterSetId"/>
>> </drop-down>
>> </field>
>> - <field name="statusId">
>> + <field name="statusId" title="${uiLabelMap.CommonStatus}">
>> <drop-down allow-empty="true">
>> <entity-options entity-name="StatusItem"
>> key-field-name="statusId">
>> <entity-constraint name="statusTypeId"
>> value="CONTENT_STATUS"/>
>> @@ -397,7 +397,7 @@ under the License.
>> <entity-options entity-name="DataTemplateType"
>> key-field-name="dataTemplateTypeId"/>
>> </drop-down>
>> </field>
>> - <field name="statusId">
>> + <field name="statusId" title="${uiLabelMap.CommonStatus}">
>> <drop-down allow-empty="true">
>> <entity-options entity-name="StatusItem"
>> key-field-name="statusId">
>> <entity-constraint name="statusTypeId"
>> value="CONTENT_STATUS"/>
>> @@ -580,7 +580,7 @@ under the License.
>> <field name="privilegeEnumId" >
>> <ignored />
>> </field>
>> - <field name="statusId" >
>> + <field name="statusId" title="${uiLabelMap.CommonStatus}">
>> <hidden value="CTNT_PUBLISHED"/>
>> </field>
>> <field name="contentAssocTypeId" >
>>
>> Modified: ofbiz/trunk/specialpurpose/ecommerce/widget/blog/BlogForms.xml
>> URL: http://svn.apache.org/viewvc/ofbiz/trunk/specialpurpose/
>> ecommerce/widget/blog/BlogForms.xml?rev=1758774&r1=
>> 1758773&r2=1758774&view=diff
>> ============================================================
>> ==================
>> --- ofbiz/trunk/specialpurpose/ecommerce/widget/blog/BlogForms.xml
>> (original)
>> +++ ofbiz/trunk/specialpurpose/ecommerce/widget/blog/BlogForms.xml Thu
>> Sep 1 13:34:17 2016
>> @@ -98,7 +98,7 @@
>> <field name="textData" title="${uiLabelMap.EcommerceRespond}"
>> map-name="rsp" widget-style="inputBox dojo-ResizableTextArea">
>> <textarea cols="60" rows="12"/>
>> </field>
>> - <field name="statusId">
>> + <field name="statusId" title="${uiLabelMap.CommonStatus}">
>> <radio no-current-selected-key="CTNT_INITIAL_DRAFT">
>> <option key="CTNT_PUBLISHED" description="${uiLabelMap.
>> ContentBlogPost}"/>
>> <option key="CTNT_INITIAL_DRAFT"
>> description="${uiLabelMap.ContentBlogPreview}"/>
>> @@ -165,7 +165,7 @@
>> <field name="textData" title="${uiLabelMap.EcommerceRespond}"
>> map-name="electronicText" widget-style="inputBox dojo-ResizableTextArea">
>> <textarea cols="60" rows="12"/>
>> </field>
>> - <field name="statusId">
>> + <field name="statusId" title="${uiLabelMap.CommonStatus}">
>> <radio no-current-selected-key="CTNT_INITIAL_DRAFT">
>> <option key="CTNT_PUBLISHED" description="${uiLabelMap.
>> ContentBlogPost}"/>
>> <option key="CTNT_INITIAL_DRAFT"
>> description="${uiLabelMap.ContentBlogPreview}"/>
>>
>>
>>