You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by mr...@apache.org on 2013/10/29 09:19:07 UTC
svn commit: r1536604 - in /jackrabbit/oak/trunk/oak-mk/src:
main/java/org/apache/jackrabbit/mk/core/MicroKernelImpl.java
test/java/org/apache/jackrabbit/mk/MicroKernelImplTest.java
Author: mreutegg
Date: Tue Oct 29 08:19:06 2013
New Revision: 1536604
URL: http://svn.apache.org/r1536604
Log:
OAK-1122: Empty branch commit returns head revision on trunk
Modified:
jackrabbit/oak/trunk/oak-mk/src/main/java/org/apache/jackrabbit/mk/core/MicroKernelImpl.java
jackrabbit/oak/trunk/oak-mk/src/test/java/org/apache/jackrabbit/mk/MicroKernelImplTest.java
Modified: jackrabbit/oak/trunk/oak-mk/src/main/java/org/apache/jackrabbit/mk/core/MicroKernelImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-mk/src/main/java/org/apache/jackrabbit/mk/core/MicroKernelImpl.java?rev=1536604&r1=1536603&r2=1536604&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-mk/src/main/java/org/apache/jackrabbit/mk/core/MicroKernelImpl.java (original)
+++ jackrabbit/oak/trunk/oak-mk/src/main/java/org/apache/jackrabbit/mk/core/MicroKernelImpl.java Tue Oct 29 08:19:06 2013
@@ -411,7 +411,7 @@ public class MicroKernelImpl implements
throw new IllegalArgumentException("absolute path expected: " + path);
}
if (jsonDiff == null || jsonDiff.length() == 0) {
- return getHeadRevision();
+ return revisionId != null ? revisionId : getHeadRevision();
}
Id revId = revisionId == null ? getHeadRevisionId() : Id.fromString(revisionId);
Modified: jackrabbit/oak/trunk/oak-mk/src/test/java/org/apache/jackrabbit/mk/MicroKernelImplTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-mk/src/test/java/org/apache/jackrabbit/mk/MicroKernelImplTest.java?rev=1536604&r1=1536603&r2=1536604&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-mk/src/test/java/org/apache/jackrabbit/mk/MicroKernelImplTest.java (original)
+++ jackrabbit/oak/trunk/oak-mk/src/test/java/org/apache/jackrabbit/mk/MicroKernelImplTest.java Tue Oct 29 08:19:06 2013
@@ -401,4 +401,16 @@ public class MicroKernelImplTest {
assertTrue(trunkNode.contains("\"q\":43"));
}
+ @Test // OAK-1122
+ public void emptyCommit() {
+ String rev = mk.getHeadRevision();
+ assertEquals("empty commit must return current head revision",
+ rev, mk.commit("/", "", rev, null));
+ // now the same on a branch
+ rev = mk.branch(rev);
+ // commit something to branch
+ rev = mk.commit("/", "+\"x\":{}", rev, null);
+ assertEquals("empty branch commit must return current head of branch revision",
+ rev, mk.commit("/", "", rev, null));
+ }
}