zorba-coders team mailing list archive

[Dennis Knochenwefel <dennis.knochenwefel@xxxxxxxxxx>]

Dennis Knochenwefel has proposed merging lp:~zorba-coders/zorba/staticcontextentityresolver-ignores-deny-access into lp:zorba.

Requested reviews:
  Chris Hillery (ceejatec)
Related bugs:
  Bug #984759 in Zorba: "StaticContextEntityResolver ignores deny_access "
  https://bugs.launchpad.net/zorba/+bug/984759

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/staticcontextentityresolver-ignores-deny-access/+merge/102506

fix for bug #984759 (StaticContextEntityResolver ignores deny_access)
-- 
https://code.launchpad.net/~zorba-coders/zorba/staticcontextentityresolver-ignores-deny-access/+merge/102506
Your team Zorba Coders is subscribed to branch lp:zorba.

=== modified file 'src/types/schema/schema.cpp'
--- src/types/schema/schema.cpp	2012-04-16 20:56:43 +0000
+++ src/types/schema/schema.cpp	2012-04-18 12:38:20 +0000
@@ -255,15 +255,17 @@
         else
           return NULL;          
       }
-// avoiding the warning that e is not used
-#ifdef DO_TRACE
       catch (ZorbaException const& e) {
         TRACE("!!! ZorbaException: " << e );
-#else
-      catch (ZorbaException const& ) {
-#endif
-        //don't throw let Xerces resolve it
-        return NULL;
+        if ( e.diagnostic() == zerr::ZXQP0029_URI_ACCESS_DENIED )
+        {
+          throw;
+        }
+        else
+        {
+          //don't throw let Xerces resolve it
+          return NULL;
+        }
       }
     }
   }