Re: svn commit: r676607 - /ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy

[Scott Gray]

Hi Vikas

nowTimestamp should already be defined in the context, where were you
seeing the problem?

Thanks
Scott

2008/7/15  <[hidden email]>:

> Author: mor
> Date: Mon Jul 14 07:42:15 2008
> New Revision: 676607
>
> URL: http://svn.apache.org/viewvc?rev=676607&view=rev
> Log:
> Fixed a small bug - I think introduced through bsh -> groovy conversion
>
> Modified:
>    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy
>
> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy?rev=676607&r1=676606&r2=676607&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy (original)
> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy Mon Jul 14 07:42:15 2008
> @@ -37,6 +37,7 @@
>  returnId = parameters.returnId;
>  quote = null;
>  quoteId = parameters.quoteId;
> +nowTimestamp = UtilDateTime.nowTimestamp();
>  fromPartyId = parameters.fromPartyId;
>
>  if (!orderHeader && orderId) {
>
>
>

[Vikas Mayur-2]

Hi Scott,

I double check it after reverting my changes and the got the same error on Order submit.(tested only from ecommerce).

ScreenFactory.java:128:INFO ] Got 4 screens in 0.117s from: file:/Users/vikas/work/projects/ofbiz/applications/order/widget/ordermgr/OrderPrintScreens.xml

2008-07-15 09:54:34,815 (default-invoker-Thread-7) [        ModelScreen.java:398:ERROR] Error rendering screen [component://order/widget/ordermgr/OrderPrintScreens.xml#CompanyLogo]: groovy.lang.MissingPropertyException: No such property: nowTimestamp for class: CompanyHeader. Rolling back transaction.

2008-07-15 09:54:34,818 (default-invoker-Thread-7) [    TransactionUtil.java:311:WARN ] 

---- exception report ----------------------------------------------------------

[TransactionUtil.setRollbackOnly] Calling transaction setRollbackOnly; this stack trace shows where this is happening:

Exception: java.lang.Exception

Message: Error rendering screen [component://order/widget/ordermgr/OrderPrintScreens.xml#CompanyLogo]: groovy.lang.MissingPropertyException: No such property: nowTimestamp for class: CompanyHeader

---- stack trace ---------------------------------------------------------------

java.lang.Exception: Error rendering screen [component://order/widget/ordermgr/OrderPrintScreens.xml#CompanyLogo]: groovy.lang.MissingPropertyException: No such property: nowTimestamp for class: CompanyHeader

For complete log please see the attached file.

BTW, I do not see nowTimestamp defined in groovy context.

Thanks,

Vikas

On Tue, Jul 15, 2008 at 1:25 AM, Scott Gray <[hidden email]> wrote:

Hi Vikas

nowTimestamp should already be defined in the context, where were you
seeing the problem?

Thanks
Scott

2008/7/15  <[hidden email]>:

> Author: mor
> Date: Mon Jul 14 07:42:15 2008
> New Revision: 676607
>
> URL: http://svn.apache.org/viewvc?rev=676607&view=rev
> Log:
> Fixed a small bug - I think introduced through bsh -> groovy conversion
>
> Modified:
>    ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy
>
> Modified: ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy
> URL: http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy?rev=676607&r1=676606&r2=676607&view=diff
> ==============================================================================
> --- ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy (original)
> +++ ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy Mon Jul 14 07:42:15 2008
> @@ -37,6 +37,7 @@
>  returnId = parameters.returnId;
>  quote = null;
>  quoteId = parameters.quoteId;
> +nowTimestamp = UtilDateTime.nowTimestamp();
>  fromPartyId = parameters.fromPartyId;
>
>  if (!orderHeader && orderId) {
>
>
>

[Scott Gray]

Hi Vikas

Thanks for the details, the problem is that nowTimestamp is only
available automatically when the screen context is populated via
ScreenRenderer.populateContextForRequest() but the email services
don't use that method.

I'll commit a change later on so that nowTimestamp is always available.

Thanks for your help

Regards
Scott

2008/7/15 Vikas Mayur <[hidden email]>:

> Hi Scott,
> I double check it after reverting my changes and the got the same error on
> Order submit.(tested only from ecommerce).
> ScreenFactory.java:128:INFO ] Got 4 screens in 0.117s from:
> file:/Users/vikas/work/projects/ofbiz/applications/order/widget/ordermgr/OrderPrintScreens.xml
> 2008-07-15 09:54:34,815 (default-invoker-Thread-7) [
>  ModelScreen.java:398:ERROR] Error rendering screen
> [component://order/widget/ordermgr/OrderPrintScreens.xml#CompanyLogo]:
> groovy.lang.MissingPropertyException: No such property: nowTimestamp for
> class: CompanyHeader. Rolling back transaction.
> 2008-07-15 09:54:34,818 (default-invoker-Thread-7) [
>  TransactionUtil.java:311:WARN ]
> ---- exception report
> ----------------------------------------------------------
> [TransactionUtil.setRollbackOnly] Calling transaction setRollbackOnly; this
> stack trace shows where this is happening:
> Exception: java.lang.Exception
> Message: Error rendering screen
> [component://order/widget/ordermgr/OrderPrintScreens.xml#CompanyLogo]:
> groovy.lang.MissingPropertyException: No such property: nowTimestamp for
> class: CompanyHeader
> ---- stack trace
> ---------------------------------------------------------------
> java.lang.Exception: Error rendering screen
> [component://order/widget/ordermgr/OrderPrintScreens.xml#CompanyLogo]:
> groovy.lang.MissingPropertyException: No such property: nowTimestamp for
> class: CompanyHeader
>
> For complete log please see the attached file.
> BTW, I do not see nowTimestamp defined in groovy context.
>
> Thanks,
> Vikas
> On Tue, Jul 15, 2008 at 1:25 AM, Scott Gray <[hidden email]> wrote:
>>
>> Hi Vikas
>>
>> nowTimestamp should already be defined in the context, where were you
>> seeing the problem?
>>
>> Thanks
>> Scott
>>
>> 2008/7/15  <[hidden email]>:
>> > Author: mor
>> > Date: Mon Jul 14 07:42:15 2008
>> > New Revision: 676607
>> >
>> > URL: http://svn.apache.org/viewvc?rev=676607&view=rev
>> > Log:
>> > Fixed a small bug - I think introduced through bsh -> groovy conversion
>> >
>> > Modified:
>> >
>> >  ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy
>> >
>> > Modified:
>> > ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy
>> > URL:
>> > http://svn.apache.org/viewvc/ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy?rev=676607&r1=676606&r2=676607&view=diff
>> >
>> > ==============================================================================
>> > ---
>> > ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy
>> > (original)
>> > +++
>> > ofbiz/trunk/applications/order/webapp/ordermgr/WEB-INF/actions/order/CompanyHeader.groovy
>> > Mon Jul 14 07:42:15 2008
>> > @@ -37,6 +37,7 @@
>> >  returnId = parameters.returnId;
>> >  quote = null;
>> >  quoteId = parameters.quoteId;
>> > +nowTimestamp = UtilDateTime.nowTimestamp();
>> >  fromPartyId = parameters.fromPartyId;
>> >
>> >  if (!orderHeader && orderId) {
>> >
>> >
>> >
>
>