when patch relationship with empty list, class BraveRepositoryFilter throw exception.

[jianglibo]

version: 3.0.1

Look at this code snippet:

		if (ids != null) {
			Iterator<Object> iterator = ids.iterator();
			pathBuilder.append(iterator.next());
			while (iterator.hasNext()) {
				pathBuilder.append(",");
				pathBuilder.append(iterator.next());
			}
			pathBuilder.append("/");
		}

pathBuilder.append(iterator.next()); will cause java.util.NoSuchElementException.

	private String getComponentName(RepositoryRequestSpec request) {
		ResourceField relationshipField = request.getRelationshipField();
		StringBuilder pathBuilder = new StringBuilder();
		String method = request.getMethod().toString();
		pathBuilder.append(COMPONENT_NAME);
		pathBuilder.append(COMPONENT_NAME_SEPARATOR);
		pathBuilder.append(method);
		pathBuilder.append(COMPONENT_NAME_SEPARATOR);
		pathBuilder.append("/");
		
		if (relationshipField == null) {
			pathBuilder.append(request.getQueryAdapter().getResourceInformation().getResourceType());
		}
		else {
			pathBuilder.append(relationshipField.getParentResourceInformation().getResourceType());
		}
		pathBuilder.append("/");
		
		Iterable<Object> ids = request.getIds();
		if (ids != null) {
			Iterator<Object> iterator = ids.iterator();
			pathBuilder.append(iterator.next());
			while (iterator.hasNext()) {
				pathBuilder.append(",");
				pathBuilder.append(iterator.next());
			}
			pathBuilder.append("/");
		}
		if (relationshipField != null) {
			pathBuilder.append(relationshipField.getJsonName());
			pathBuilder.append("/");
		}
		return pathBuilder.toString();
	}

[chb0github]

Agreed - this is not well implemented. The original author of the code has left the project and it's for reasons like this. You're are welcome to offer a PR.