Author: fmui
Date: Wed Dec 17 11:04:39 2014
New Revision: 1646195
URL: http://svn.apache.org/r1646195
Log:
minor improvements
Modified:
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-api/src/main/java/org/apache/chemistry/opencmis/client/api/CmisObject.java
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/browser/ObjectServiceImpl.java
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/SessionImpl.java
chemistry/opencmis/trunk/chemistry-opencmis-commons/chemistry-opencmis-commons-impl/src/main/java/org/apache/chemistry/opencmis/commons/impl/Constants.java
Modified:
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-api/src/main/java/org/apache/chemistry/opencmis/client/api/CmisObject.java
URL:
http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-api/src/main/java/org/apache/chemistry/opencmis/client/api/CmisObject.java?rev=1646195&r1=1646194&r2=1646195&view=diff
==============================================================================
---
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-api/src/main/java/org/apache/chemistry/opencmis/client/api/CmisObject.java
(original)
+++
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-api/src/main/java/org/apache/chemistry/opencmis/client/api/CmisObject.java
Wed Dec 17 11:04:39 2014
@@ -140,7 +140,7 @@ public interface CmisObject extends Obje
* @param properties
* the properties to update
* @param refresh
- * {@code true} if this object should be refresh after the
+ * {@code true} if this object should be refreshed after the
* update, {@code false} if not
*
* @return the object ID of the updated object
@@ -172,7 +172,7 @@ public interface CmisObject extends Obje
* @param newName
* the new name, not {@code null} or empty
* @param refresh
- * {@code true} if this object should be refresh after the
+ * {@code true} if this object should be refreshed after the
* update, {@code false} if not
*
* @return the object ID of the updated object
Modified:
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/browser/ObjectServiceImpl.java
URL:
http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/browser/ObjectServiceImpl.java?rev=1646195&r1=1646194&r2=1646195&view=diff
==============================================================================
---
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/browser/ObjectServiceImpl.java
(original)
+++
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-bindings/src/main/java/org/apache/chemistry/opencmis/client/bindings/spi/browser/ObjectServiceImpl.java
Wed Dec 17 11:04:39 2014
@@ -381,7 +381,7 @@ public class ObjectServiceImpl extends A
Properties properties, ExtensionsData extension) {
// we need an object ID
if ((objectId == null) || (objectId.getValue() == null) ||
(objectId.getValue().length() == 0)) {
- throw new CmisInvalidArgumentException("Object id must be set!");
+ throw new CmisInvalidArgumentException("Object ID must be set!");
}
// build URL
@@ -447,7 +447,7 @@ public class ObjectServiceImpl extends A
ExtensionsData extension) {
// we need an object id
if ((objectId == null) || (objectId.getValue() == null) ||
(objectId.getValue().length() == 0)) {
- throw new CmisInvalidArgumentException("Object id must be set!");
+ throw new CmisInvalidArgumentException("Object ID must be set!");
}
// build URL
@@ -528,7 +528,7 @@ public class ObjectServiceImpl extends A
Holder<String> changeToken, ContentStream contentStream,
ExtensionsData extension) {
// we need an object id
if ((objectId == null) || (objectId.getValue() == null) ||
(objectId.getValue().length() == 0)) {
- throw new CmisInvalidArgumentException("Object id must be set!");
+ throw new CmisInvalidArgumentException("Object ID must be set!");
}
// build URL
@@ -564,7 +564,7 @@ public class ObjectServiceImpl extends A
ContentStream contentStream, boolean isLastChunk, ExtensionsData
extension) {
// we need an object id
if ((objectId == null) || (objectId.getValue() == null) ||
(objectId.getValue().length() == 0)) {
- throw new CmisInvalidArgumentException("Object id must be set!");
+ throw new CmisInvalidArgumentException("Object ID must be set!");
}
// build URL
@@ -600,7 +600,7 @@ public class ObjectServiceImpl extends A
ExtensionsData extension) {
// we need an object id
if ((objectId == null) || (objectId.getValue() == null) ||
(objectId.getValue().length() == 0)) {
- throw new CmisInvalidArgumentException("Object id must be set!");
+ throw new CmisInvalidArgumentException("Object ID must be set!");
}
// build URL
Modified:
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/SessionImpl.java
URL:
http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/SessionImpl.java?rev=1646195&r1=1646194&r2=1646195&view=diff
==============================================================================
---
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/SessionImpl.java
(original)
+++
chemistry/opencmis/trunk/chemistry-opencmis-client/chemistry-opencmis-client-impl/src/main/java/org/apache/chemistry/opencmis/client/runtime/SessionImpl.java
Wed Dec 17 11:04:39 2014
@@ -958,7 +958,7 @@ public class SessionImpl implements Sess
public ItemIterable<CmisObject> queryObjects(String typeId, String where,
final boolean searchAllVersions,
OperationContext context) {
if (typeId == null || typeId.trim().length() == 0) {
- throw new IllegalArgumentException("Type id must be set!");
+ throw new IllegalArgumentException("Type ID must be set!");
}
if (context == null) {
@@ -1042,7 +1042,7 @@ public class SessionImpl implements Sess
try {
binding = CmisBindingHelper.createBinding(parameters,
authenticationProvider, typeDefCache);
- /* get initial repository id from session parameter */
+ /* get initial repository ID from session parameter */
String repositoryId =
parameters.get(SessionParameter.REPOSITORY_ID);
if (repositoryId == null) {
throw new IllegalStateException("Repository ID is not set!");
@@ -1241,7 +1241,7 @@ public class SessionImpl implements Sess
public ItemIterable<Relationship> getRelationships(ObjectId objectId,
final boolean includeSubRelationshipTypes,
final RelationshipDirection relationshipDirection, ObjectType
type, OperationContext context) {
if ((objectId == null) || (objectId.getId() == null)) {
- throw new IllegalArgumentException("Invalid object id!");
+ throw new IllegalArgumentException("Invalid object ID!");
}
if (context == null) {
throw new IllegalArgumentException("Operation context must be
set!");
@@ -1347,7 +1347,7 @@ public class SessionImpl implements Sess
public void delete(ObjectId objectId, boolean allVersions) {
if ((objectId == null) || (objectId.getId() == null)) {
- throw new IllegalArgumentException("Invalid object id!");
+ throw new IllegalArgumentException("Invalid object ID!");
}
getBinding().getObjectService().deleteObject(getRepositoryId(),
objectId.getId(), allVersions, null);
@@ -1362,7 +1362,7 @@ public class SessionImpl implements Sess
public ContentStream getContentStream(ObjectId docId, String streamId,
BigInteger offset, BigInteger length) {
if ((docId == null) || (docId.getId() == null)) {
- throw new IllegalArgumentException("Invalid document id!");
+ throw new IllegalArgumentException("Invalid document ID!");
}
// get the stream
@@ -1382,7 +1382,7 @@ public class SessionImpl implements Sess
public Acl getAcl(ObjectId objectId, boolean onlyBasicPermissions) {
if ((objectId == null) || (objectId.getId() == null)) {
- throw new IllegalArgumentException("Invalid object id!");
+ throw new IllegalArgumentException("Invalid object ID!");
}
String id = objectId.getId();
@@ -1392,7 +1392,7 @@ public class SessionImpl implements Sess
public Acl applyAcl(ObjectId objectId, List<Ace> addAces, List<Ace>
removeAces, AclPropagation aclPropagation) {
if ((objectId == null) || (objectId.getId() == null)) {
- throw new IllegalArgumentException("Invalid object id!");
+ throw new IllegalArgumentException("Invalid object ID!");
}
ObjectFactory of = getObjectFactory();
@@ -1403,7 +1403,7 @@ public class SessionImpl implements Sess
public Acl setAcl(ObjectId objectId, List<Ace> aces) {
if ((objectId == null) || (objectId.getId() == null)) {
- throw new IllegalArgumentException("Invalid object id!");
+ throw new IllegalArgumentException("Invalid object ID!");
}
if (aces == null) {
aces = Collections.emptyList();
@@ -1423,7 +1423,7 @@ public class SessionImpl implements Sess
public void applyPolicy(ObjectId objectId, ObjectId... policyIds) {
if ((objectId == null) || (objectId.getId() == null)) {
- throw new IllegalArgumentException("Invalid object id!");
+ throw new IllegalArgumentException("Invalid object ID!");
}
if ((policyIds == null) || (policyIds.length == 0)) {
@@ -1446,7 +1446,7 @@ public class SessionImpl implements Sess
public void removePolicy(ObjectId objectId, ObjectId... policyIds) {
if ((objectId == null) || (objectId.getId() == null)) {
- throw new IllegalArgumentException("Invalid object id!");
+ throw new IllegalArgumentException("Invalid object ID!");
}
if ((policyIds == null) || (policyIds.length == 0)) {
Modified:
chemistry/opencmis/trunk/chemistry-opencmis-commons/chemistry-opencmis-commons-impl/src/main/java/org/apache/chemistry/opencmis/commons/impl/Constants.java
URL:
http://svn.apache.org/viewvc/chemistry/opencmis/trunk/chemistry-opencmis-commons/chemistry-opencmis-commons-impl/src/main/java/org/apache/chemistry/opencmis/commons/impl/Constants.java?rev=1646195&r1=1646194&r2=1646195&view=diff
==============================================================================
---
chemistry/opencmis/trunk/chemistry-opencmis-commons/chemistry-opencmis-commons-impl/src/main/java/org/apache/chemistry/opencmis/commons/impl/Constants.java
(original)
+++
chemistry/opencmis/trunk/chemistry-opencmis-commons/chemistry-opencmis-commons-impl/src/main/java/org/apache/chemistry/opencmis/commons/impl/Constants.java
Wed Dec 17 11:04:39 2014
@@ -199,7 +199,6 @@ public final class Constants {
public static final String PARAM_RENDITION_FILTER = "renditionFilter";
public static final String PARAM_REPOSITORY_ID = "repositoryId";
public static final String PARAM_RETURN_VERSION = "returnVersion";
- public static final String PARAM_ROPERTY_DEFINITIONS =
"includePropertyDefinitions";
public static final String PARAM_SKIP_COUNT = "skipCount";
public static final String PARAM_SOURCE_FOLDER_ID = "sourceFolderId";
public static final String PARAM_TARGET_FOLDER_ID = "targetFolderId";
