On Jan 25, 2010, at 9:46 PM, Sergiu Dumitriu wrote:
On 01/25/2010 09:08 PM, vmassol (SVN) wrote:
Author: vmassol
Date: 2010-01-25 21:08:08 +0100 (Mon, 25 Jan 2010)
New Revision: 26377
Added:
platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/test/AbstractBridgedComponentTestCase.java
Modified:
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/store/XWikiHibernateBaseStore.java
platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/store/XWikiHibernateStoreTest.java
Log:
XWIKI-2758: PostgreSQL exceptions unusable
Forgot to give credit.
Yes. I was applying my own fix and then applied Frits' suggestion later on and I
forgot. Fixed in the new commit.
Modified:
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/store/XWikiHibernateBaseStore.java
===================================================================
---
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/store/XWikiHibernateBaseStore.java 2010-01-25
18:38:18 UTC (rev 26376)
+++
platform/core/trunk/xwiki-core/src/main/java/com/xpn/xwiki/store/XWikiHibernateBaseStore.java 2010-01-25
20:08:08 UTC (rev 26377)
You should cleanup this commented code:
Yes thanks for catching it.
+/*
+ if (t.getCause() instanceof BatchUpdateException
+&& ((BatchUpdateException) t.getCause()).getNextException() != null) {
+ final BatchUpdateException bue = (BatchUpdateException) t.getCause();
+ return bue.getNextException().getMessage();
+ }
+
+ return t.getMessage();
+ */
+ return sb.toString();
+ }
Are you sure this was supposed to be part of this commit?
Yes it is (I refactored the test to move it to JUnit 4.x and JMock 2.x before adding a new
unit test).
Thanks
-Vincent
> Added:
platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/test/AbstractBridgedComponentTestCase.java
> ===================================================================
> ---
platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/test/AbstractBridgedComponentTestCase.java
(rev 0)
> +++
platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/test/AbstractBridgedComponentTestCase.java 2010-01-25
20:08:08 UTC (rev 26377)