svn commit: r578275 - /ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh

Author: jleroux
Date: Fri Sep 21 14:06:32 2007
New Revision: 578275

URL: http://svn.apache.org/viewvc?rev=578275&view=rev
Log:
Applied fix from trunk for revision: 572174

Modified:
    ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh

Modified: ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh
URL: http://svn.apache.org/viewvc/ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh?rev=578275&r1=578274&r2=578275&view=diff
==============================================================================
--- ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh (original)
+++ ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh Fri Sep 21 14:06:32 2007
@@ -78,7 +78,7 @@
             runOnFailure.setAccessible(false);
 
             //extract actions
-            actions = curRuleClass.getDeclaredField("actions");            
+            actions = curRuleClass.getDeclaredField("actionsAndSets");            
             actions.setAccessible(true);
             actionsVal = actions.get(curRule);
             if(actionsVal != null){

[David E Jones-2]

This breaks this script in the branch because the variable referred to only exists in the trunk. It was a bug fix in the trunk, but a fix for a bug that ONLY existed in the trunk.

This should be reverted, and in general: when it doubt, leave it out!

-David


[hidden email] wrote:

> Author: jleroux
> Date: Fri Sep 21 14:06:32 2007
> New Revision: 578275
>
> URL: http://svn.apache.org/viewvc?rev=578275&view=rev
> Log:
> Applied fix from trunk for revision: 572174
>
> Modified:
>     ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh
>
> Modified: ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh
> URL: http://svn.apache.org/viewvc/ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh?rev=578275&r1=578274&r2=578275&view=diff
> ==============================================================================
> --- ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh (original)
> +++ ofbiz/branches/release4.0/framework/webtools/webapp/webtools/WEB-INF/actions/service/availableservices.bsh Fri Sep 21 14:06:32 2007
> @@ -78,7 +78,7 @@
>              runOnFailure.setAccessible(false);
>  
>              //extract actions
> -            actions = curRuleClass.getDeclaredField("actions");            
> +            actions = curRuleClass.getDeclaredField("actionsAndSets");            
>              actions.setAccessible(true);
>              actionsVal = actions.get(curRule);
>              if(actionsVal != null){
>
>