Done at revision: 918476

Jacques

From: "Jacques Le Roux" <jacques.le.r...@les7arts.com>
Hi devs,

I'm using these changes in a custom application based from the trunk at r917244 with almost no other modifications (custom theme and
locales.available in general.properrties and postgres in entityengine.xml) but 
a specific component in hot-deploy and I don't have
issues connecting to

https://localhost:8443/ap/control/main nor
https://localhost:8443/ar/control/main

And no issues using them forward. So I wonder now if this was actually not 
another issue which has been resolved since.

If nobody see a problem with that, I will commit these changes again. If you 
have the time to test it, please do.

Jacques

From: "Adam Heath" <doo...@brainfood.com>
Erwan de FERRIERES wrote:
Le 23/02/2010 17:46, Adam Heath a écrit :
Erwan de FERRIERES wrote:
Le 22/02/2010 17:34, doo...@apache.org a écrit :
Author: doogie
Date: Mon Feb 22 16:34:47 2010
New Revision: 912645

URL: http://svn.apache.org/viewvc?rev=912645&view=rev
Log:
Make use of FSE.expand(), instead of expandString, to provide support
for direct setting of values in widgets.  This closes OFBIZ-3486.

Modified:

ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java



ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenAction.java



Modified:
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java


URL:
http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java?rev=912645&r1=912644&r2=912645&view=diff


==============================================================================


---
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java

(original)
+++
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java

Mon Feb 22 16:34:47 2010
@@ -136,12 +136,12 @@
                   newValue = this.fromField.get(context);
                   if (Debug.verboseOn()) Debug.logVerbose("In screen
getting value for field from [" + this.fromField.getOriginalName() +
"]: " + newValue, module);
               } else if (!this.valueExdr.isEmpty()) {
-                newValue = this.valueExdr.expandString(context);
+                newValue = this.valueExdr.expand(context);
               }

               // If newValue is still empty, use the default value
               if (ObjectType.isEmpty(newValue)&&
!this.defaultExdr.isEmpty()) {
-                newValue = this.defaultExdr.expandString(context);
+                newValue = this.defaultExdr.expand(context);
               }

               if (UtilValidate.isNotEmpty(this.type)) {

Modified:
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenAction.java


URL:
http://svn.apache.org/viewvc/ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenAction.java?rev=912645&r1=912644&r2=912645&view=diff


==============================================================================


---
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenAction.java

(original)
+++
ofbiz/trunk/framework/widget/src/org/ofbiz/widget/screen/ModelScreenAction.java

Mon Feb 22 16:34:47 2010
@@ -157,7 +157,7 @@
                       newValue =
getInMemoryPersistedFromField(session, context);
                       if (Debug.verboseOn()) Debug.logVerbose("In user
getting value for field from [" + this.fromField.getOriginalName() +
"]: " + newValue, module);
                   } else if (!this.valueExdr.isEmpty()) {
-                    newValue = this.valueExdr.expandString(context);
+                    newValue = this.valueExdr.expand(context);
                   }
               } else if (this.fromScope != null&&
this.fromScope.equals("application")) {
                   if (!this.fromField.isEmpty()) {
@@ -165,20 +165,20 @@
                       newValue =
getInMemoryPersistedFromField(servletContext, context);
                       if (Debug.verboseOn()) Debug.logVerbose("In
application getting value for field from [" +
this.fromField.getOriginalName() + "]: " + newValue, module);
                   } else if (!this.valueExdr.isEmpty()) {
-                    newValue = this.valueExdr.expandString(context);
+                    newValue = this.valueExdr.expand(context);
                   }
               } else {
                   if (!this.fromField.isEmpty()) {
                       newValue = this.fromField.get(context);
                       if (Debug.verboseOn()) Debug.logVerbose("In
screen getting value for field from [" +
this.fromField.getOriginalName() + "]: " + newValue, module);
                   } else if (!this.valueExdr.isEmpty()) {
-                    newValue = this.valueExdr.expandString(context);
+                    newValue = this.valueExdr.expand(context);
                   }
               }

               // If newValue is still empty, use the default value
               if (ObjectType.isEmpty(newValue)&&
!this.defaultExdr.isEmpty()) {
-                newValue = this.defaultExdr.expandString(context);
+                newValue = this.defaultExdr.expand(context);
               }

               if (UtilValidate.isNotEmpty(this.type)) {




Since this commit, it's impossible to connect to AP and AF
applications !

The test cases worked for this, which is bad since they didn't detect
a problem.

Please give me an *exact* series of steps to reproduce this.  I don't
know anything about those apps.

Same for me ! I just tried to connect

Steps to reproduce :

update your OFBiz instance to any version after the 912645, compile and
just try to login to
https://localhost:8443/ap/control/main or
https://localhost:8443/ar/control/main

The page will not be displayed, and the only thing you will see is this
message
 ERROR rendering error page [/error/error.jsp], but here is the error
text: org.ofbiz.widget.screen.ScreenRenderException: Error rendering
screen [component://accounting/widget/ar/CommonScreens.xml#main]:
org.ofbiz.base.util.GeneralException: Error running Groovy script at
location
[component://accounting/webapp/accounting/WEB-INF/actions/invoice/InvoiceReport.groovy]
(The current transaction is marked for rollback, not beginning a new
transaction and aborting current operation; the rollbackOnly was caused
by: Service [getPartyAccountingPreferences] threw an unexpected
exception/errororg.ofbiz.service.ServiceValidationException: The
following required parameter is missing:
[getPartyAccountingPreferences.organizationPartyId] (The following
required parameter is missing:
[getPartyAccountingPreferences.organizationPartyId])) (Error running
Groovy script at location
[component://accounting/webapp/accounting/WEB-INF/actions/invoice/InvoiceReport.groovy]
(The current transaction is marked for rollback, not beginning a new
transaction and aborting current operation; the rollbackOnly was caused
by: Service [getPartyAccountingPreferences] threw an unexpected
exception/errororg.ofbiz.service.ServiceValidationException: The
following required parameter is missing:
[getPartyAccountingPreferences.organizationPartyId] (The following
required parameter is missing:
[getPartyAccountingPreferences.organizationPartyId])))

I reverted this change in 915650, until I can look into it more.





Reply via email to