Done at r1427819

Jacques

From: "Jacques Le Roux" <jacques.le.r...@les7arts.com>
> From: "Adrian Crum" <adrian.c...@sandglass-software.com>
>> This is still misleading. The getServerTimestamp service returns a Long, 
>> not a Timestamp.
> 
> I can rename to getServerTimestampAsLong if you prefer?
> 
>> Also, it would be nice if the service returned something useful, like a 
>> java.sql.Timestamp or a java.lang.String - so the service can be reused. 
>> It appears this service is being tailored to a specific use case.
> 
> Yes it's is. I found easier to return a Long because it can be easily 
> transformed in javascript Data
> 
> I could create also a getServerTimestamp wich returns Timestamp of course, 
> agreed?
> 
> Jacques
> 
>> -Adrian
>> 
>> On 12/31/2012 7:50 PM, jler...@apache.org wrote:
>>> Author: jleroux
>>> Date: Mon Dec 31 19:50:54 2012
>>> New Revision: 1427233
>>>
>>> URL: http://svn.apache.org/viewvc?rev=1427233&view=rev
>>> Log:
>>> Complete the server hour on webtools jobs list, with the help of Adrian and 
>>> Atul
>>>
>>> Remarks:
>>> * Finally date.format-1.2.3-min.js is quite useful
>>> * I renamed the getServerHour service to getServerTimestamp. It now returns 
>>> a Long, easier to parse to a date in javascript
>>> * I renamed WindowClockJs.ftl to ServerHour24HourFormatJs.ftl
>>>
>>> Added:
>>>      
>>> ofbiz/trunk/framework/common/webcommon/includes/ServerHour24HourFormatJs.ftl
>>>        - copied, changed from r1427046, 
>>> ofbiz/trunk/framework/common/webcommon/includes/WindowClockJs.ftl
>>> Removed:
>>>      ofbiz/trunk/framework/common/webcommon/includes/WindowClockJs.ftl
>>> Modified:
>>>      ofbiz/trunk/framework/common/script/org/ofbiz/common/CommonServices.xml
>>>      ofbiz/trunk/framework/common/servicedef/services.xml
>>>      ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml
>>>      ofbiz/trunk/framework/images/webapp/images/miscAjaxFunctions.js
>>>      ofbiz/trunk/framework/webtools/widget/ServiceScreens.xml
>>>
>>> Modified: 
>>> ofbiz/trunk/framework/common/script/org/ofbiz/common/CommonServices.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/script/org/ofbiz/common/CommonServices.xml?rev=1427233&r1=1427232&r2=1427233&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/framework/common/script/org/ofbiz/common/CommonServices.xml 
>>> (original)
>>> +++ ofbiz/trunk/framework/common/script/org/ofbiz/common/CommonServices.xml 
>>> Mon Dec 31 19:50:54 2012
>>> @@ -347,8 +347,9 @@ under the License.
>>>           <field-to-result field="exist"/>
>>>       </simple-method>
>>>   
>>> -    <simple-method method-name="getServerHour" login-required="false">
>>> -        <set field="nowDateString" 
>>> value="${groovy:org.ofbiz.base.util.UtilDateTime.nowDateString('yyyy-MM-dd 
>>> HH:mm:ss')}"/>
>>> -        <field-to-result field="nowDateString"/>
>>> +    <simple-method method-name="getServerTimestamp" login-required="false">
>>> +        <now field="serverTimestamp" type="java.lang.Long"/>
>>> +        <field-to-result field="serverTimestamp"/>
>>>       </simple-method>
>>> +
>>>   </simple-methods>
>>>
>>> Modified: ofbiz/trunk/framework/common/servicedef/services.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/servicedef/services.xml?rev=1427233&r1=1427232&r2=1427233&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/framework/common/servicedef/services.xml (original)
>>> +++ ofbiz/trunk/framework/common/servicedef/services.xml Mon Dec 31 
>>> 19:50:54 2012
>>> @@ -781,9 +781,9 @@ under the License.
>>>           <attribute name="geoList" mode="OUT" type="java.util.List"/>
>>>       </service>
>>>       
>>> -    <service name="getServerHour" engine="simple" auth="false"
>>> -        
>>> location="component://common/script/org/ofbiz/common/CommonServices.xml" 
>>> invoke="getServerHour">
>>> -        <attribute name="nowDateString" mode="OUT" type="String"/>
>>> +    <service name="getServerTimestamp" engine="simple" auth="false"
>>> +        
>>> location="component://common/script/org/ofbiz/common/CommonServices.xml" 
>>> invoke="getServerTimestamp">
>>> +        <attribute name="serverTimestamp" mode="OUT" type="Long"/>
>>>       </service>
>>>       
>>>       <service name="createJsLanguageFileMapping" engine="java" 
>>> auth="false" use-transaction="false"
>>>
>>> Modified: 
>>> ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml?rev=1427233&r1=1427232&r2=1427233&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml 
>>> (original)
>>> +++ ofbiz/trunk/framework/common/webcommon/WEB-INF/common-controller.xml 
>>> Mon Dec 31 19:50:54 2012
>>> @@ -253,9 +253,9 @@ under the License.
>>>           <response name="error" type="request" value="json"/>
>>>       </request-map>
>>>       
>>> -    <request-map uri="getServerHour">
>>> +    <request-map uri="getServerTimestamp">
>>>           <security https="true" auth="true"/>
>>> -        <event type="service" invoke="getServerHour"/>
>>> +        <event type="service" invoke="getServerTimestamp"/>
>>>           <response name="success" type="request" value="json"/>
>>>           <response name="error" type="request" value="json"/>
>>>       </request-map>
>>>
>>> Copied: 
>>> ofbiz/trunk/framework/common/webcommon/includes/ServerHour24HourFormatJs.ftl
>>>  (from r1427046, 
>>> ofbiz/trunk/framework/common/webcommon/includes/WindowClockJs.ftl)
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/common/webcommon/includes/ServerHour24HourFormatJs.ftl?p2=ofbiz/trunk/framework/common/webcommon/includes/ServerHour24HourFormatJs.ftl&p1=ofbiz/trunk/framework/common/webcommon/includes/WindowClockJs.ftl&r1=1427046&r2=1427233&rev=1427233&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/framework/common/webcommon/includes/WindowClockJs.ftl 
>>> (original)
>>> +++ 
>>> ofbiz/trunk/framework/common/webcommon/includes/ServerHour24HourFormatJs.ftl
>>>  Mon Dec 31 19:50:54 2012
>>> @@ -19,8 +19,17 @@ under the License.
>>>   <script type="text/javascript">
>>>   jQuery(document).ready(function() {
>>>     window.setInterval(function(){clock()}, 1000);
>>> +  var serverTimestamp = 0;
>>> +  var date
>>>     function clock() {
>>> -    jQuery("#${clockField}").text("${uiLabelMap.CommonServerHour}: "  + 
>>> getServerHour("getServerHour"));
>>> +    if (jQuery("#${clockField}").text() === 
>>> "${uiLabelMap.CommonServerHour}:") {
>>> +      serverTimestamp = getServerTimestamp("getServerTimestamp");
>>> +      date = new Date(serverTimestamp);
>>> +      jQuery("#${clockField}").text("${uiLabelMap.CommonServerHour}: "  + 
>>> dateFormat(date, "yyyy-mm-dd HH:MM:ss"));
>>> +    } else {
>>> +      date.setSeconds(date.getSeconds() + 1);
>>> +      jQuery("#${clockField}").text("${uiLabelMap.CommonServerHour}: "  + 
>>> dateFormat(date, "yyyy-mm-dd HH:MM:ss"));
>>> +    }
>>>     }
>>>   })
>>>   </script>
>>> \ No newline at end of file
>>>
>>> Modified: ofbiz/trunk/framework/images/webapp/images/miscAjaxFunctions.js
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/images/webapp/images/miscAjaxFunctions.js?rev=1427233&r1=1427232&r2=1427233&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/framework/images/webapp/images/miscAjaxFunctions.js 
>>> (original)
>>> +++ ofbiz/trunk/framework/images/webapp/images/miscAjaxFunctions.js Mon Dec 
>>> 31 19:50:54 2012
>>> @@ -144,7 +144,7 @@ function checkUomConversion(request, par
>>>       return data['exist'];
>>>   }
>>>   
>>> -function getServerHour(request){
>>> +function getServerTimestamp(request){
>>>       data = getServiceResult(request);
>>> -    return data['nowDateString'];
>>> +    return data['serverTimestamp'];
>>>   }
>>> \ No newline at end of file
>>>
>>> Modified: ofbiz/trunk/framework/webtools/widget/ServiceScreens.xml
>>> URL: 
>>> http://svn.apache.org/viewvc/ofbiz/trunk/framework/webtools/widget/ServiceScreens.xml?rev=1427233&r1=1427232&r2=1427233&view=diff
>>> ==============================================================================
>>> --- ofbiz/trunk/framework/webtools/widget/ServiceScreens.xml (original)
>>> +++ ofbiz/trunk/framework/webtools/widget/ServiceScreens.xml Mon Dec 31 
>>> 19:50:54 2012
>>> @@ -49,7 +49,7 @@ under the License.
>>>                   <set field="tabButtonItem" value="findJob"/>
>>>                   <set field="jobCtx" from-field="parameters"/>
>>>                   <set field="dummy" value="${groovy: if 
>>> ('SERVICE_PENDING'.equals(jobCtx.statusId)) jobCtx.jobId = ''}"/> <!--  To 
>>> more easily see current pending jobs and possibly cancel some others -->
>>> -                <!-- fields for WindowClockJs.ftl -->
>>> +                <!-- fields for ServerHour24HourFormatJs.ftl -->
>>>                   <set field="clockField" value="FindJobs_clock_title"/>
>>>               </actions>
>>>               <widgets>
>>> @@ -57,7 +57,7 @@ under the License.
>>>                       <decorator-section name="body">
>>>                           <section>
>>>                               <widgets>
>>> -                                <platform-specific><html><html-template 
>>> location="component://common/webcommon/includes/WindowClockJs.ftl"/></html></platform-specific>
>>> +                                <platform-specific><html><html-template 
>>> location="component://common/webcommon/includes/ServerHour24HourFormatJs.ftl"/></html></platform-specific>
>>>                                   <decorator-screen 
>>> name="FindScreenDecorator" 
>>> location="component://common/widget/CommonScreens.xml">
>>>                                       <decorator-section 
>>> name="search-options">
>>>                                           <include-form name="FindJobs" 
>>> location="component://webtools/widget/ServiceForms.xml"/>
>>>
>>>
>>
>

Reply via email to