svn commit: r956016 - in /ode/trunk: bpel-compiler/src/main/java/org/apache/ode/bpel/compiler/bom/Process.java bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java

[rr]

Author: rr
Date: Fri Jun 18 14:37:17 2010
New Revision: 956016

URL: http://svn.apache.org/viewvc?rev=956016&view=rev
Log:
Warn about selection errors during BPEL properties extraction

Modified:
    
ode/trunk/bpel-compiler/src/main/java/org/apache/ode/bpel/compiler/bom/Process.java
    
ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java

Modified: 
ode/trunk/bpel-compiler/src/main/java/org/apache/ode/bpel/compiler/bom/Process.java
URL: 
http://svn.apache.org/viewvc/ode/trunk/bpel-compiler/src/main/java/org/apache/ode/bpel/compiler/bom/Process.java?rev=956016&r1=956015&r2=956016&view=diff
==============================================================================
--- 
ode/trunk/bpel-compiler/src/main/java/org/apache/ode/bpel/compiler/bom/Process.java
 (original)
+++ 
ode/trunk/bpel-compiler/src/main/java/org/apache/ode/bpel/compiler/bom/Process.java
 Fri Jun 18 14:37:17 2010
@@ -72,7 +72,7 @@ public class Process extends Scope {
      * @return BPEL source URL.
      */
     public String getSource() {
-        return "todo";
+        return getName();
     }
 
     /**

Modified: 
ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java
URL: 
http://svn.apache.org/viewvc/ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java?rev=956016&r1=956015&r2=956016&view=diff
==============================================================================
--- 
ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java
 (original)
+++ 
ode/trunk/bpel-runtime/src/main/java/org/apache/ode/bpel/engine/BpelRuntimeContextImpl.java
 Fri Jun 18 14:37:17 2010
@@ -1158,6 +1158,11 @@ public class BpelRuntimeContextImpl impl
         _bpelProcess.saveEvent(event, _dao, scopeNames);
     }
 
+    public static String debugInfoToString(org.apache.ode.bpel.o.DebugInfo 
debugInfo) {
+       if (debugInfo == null) return "";
+       else return " at " + debugInfo.sourceURI + ":" + debugInfo.startLine;
+    }
+    
     /**
      * We record all values of properties of a 'MessageType' variable for
      * efficient lookup.
@@ -1176,9 +1181,9 @@ public class BpelRuntimeContextImpl impl
                     } catch (FaultException e) {
                         // This will fail as we're basically trying to extract 
properties on all
                         // received messages for optimization purposes.
-                        if (__log.isDebugEnabled())
-                            __log.debug("Couldn't extract property '" + 
property.toString()
-                                    + "' in property pre-extraction: " + 
e.toString());
+                        if (__log.isWarnEnabled())
+                            __log.warn("Couldn't extract property '" + 
property.toString()
+                                    + "' and variable " + variable.declaration 
+ debugInfoToString(variable.declaration.debugInfo) + " in property 
pre-extraction: " + e.toString());
                     }
                 }
             }