Skip to content

Commit

Permalink
Update Sonar issue identifiers: 'squid' becomes 'java'
Browse files Browse the repository at this point in the history
  • Loading branch information
tommydeboer committed Feb 12, 2020
1 parent 5f5a319 commit d939af3
Show file tree
Hide file tree
Showing 328 changed files with 348 additions and 464 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ protected List<String> getExcludedAttrs() {
return attrs;
}

@SuppressWarnings("squid:S2699") // Tests should include assertions
@SuppressWarnings("java:S2699") // Tests should include assertions
@Test
protected void testSystemEntity() {
internalTestAttributes(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@

@AutoValue
@AutoGson(autoValueClass = AutoValue_CopyEntityRequestV2.class)
@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
@SuppressWarnings("java:S1610") // Abstract classes without fields should be converted to interfaces
public abstract class CopyEntityRequestV2 {
@NotNull
public abstract String getNewEntityName();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,7 @@

@AutoValue
@AutoGson(autoValueClass = AutoValue_EntityCollectionBatchRequestV2.class)
@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
@SuppressWarnings("java:S1610") // Abstract classes without fields should be converted to interfaces
public abstract class EntityCollectionBatchRequestV2 {
@NotEmpty(message = "Please provide at least one entity in the entities property.")
@Size(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@

@AutoValue
@AutoGson(autoValueClass = AutoValue_EntityCollectionDeleteRequestV2.class)
@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
@SuppressWarnings("java:S1610") // Abstract classes without fields should be converted to interfaces
public abstract class EntityCollectionDeleteRequestV2 {
@NotEmpty(message = "Please provide at least one entity in the entityIds property.")
@Size(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,7 @@

@AutoValue
@AutoGson(autoValueClass = AutoValue_ResourcesResponseV2.class)
@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
@SuppressWarnings("java:S1610") // Abstract classes without fields should be converted to interfaces
public abstract class ResourcesResponseV2 {
@NotNull
public abstract String getHref();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -303,7 +303,7 @@ public AttributeResponseV2 retrieveEntityAttributeMetaPost(
* @param response HttpServletResponse
* @return EntityCollectionCreateResponseBodyV2
*/
@SuppressWarnings("squid:S2259") // getEntities is guaranteed to be not empty
@SuppressWarnings("java:S2259") // getEntities is guaranteed to be not empty
@Transactional
@PostMapping(value = "/{entityTypeId}", produces = APPLICATION_JSON_VALUE)
public EntityCollectionBatchCreateResponseBodyV2 createEntities(
Expand Down Expand Up @@ -428,7 +428,7 @@ public String copyEntity(
* @param request EntityCollectionCreateRequestV2
* @param response HttpServletResponse
*/
@SuppressWarnings("squid:S2259") // getEntities is guaranteed to be not empty
@SuppressWarnings("java:S2259") // getEntities is guaranteed to be not empty
@Transactional
@PutMapping("/{entityTypeId}")
public synchronized void updateEntities(
Expand Down Expand Up @@ -481,7 +481,7 @@ public Map<String, String> getL10nStrings() {
* @param request EntityCollectionBatchRequestV2
* @param response HttpServletResponse
*/
@SuppressWarnings("squid:S2259") // getEntities is guaranteed to be not empty
@SuppressWarnings("java:S2259") // getEntities is guaranteed to be not empty
@Transactional
@PutMapping("/{entityTypeId}/{attributeName}")
@ResponseStatus(OK)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ int getSize() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
abstract static class Builder {

Expand Down Expand Up @@ -72,7 +72,7 @@ static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
import org.molgenis.data.meta.model.Attribute;
import org.molgenis.util.exception.BadRequestException;

@SuppressWarnings("squid:MaximumInheritanceDepth")
@SuppressWarnings("java:MaximumInheritanceDepth")
public class UnsupportedAttributeTypeException extends BadRequestException {
private static final String ERROR_CODE = "DAPI01";

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

@AutoValue
@AutoGson(autoValueClass = AutoValue_AddGroupMemberCommand.class)
@SuppressWarnings("squid:S1610")
@SuppressWarnings("java:S1610")
public abstract class AddGroupMemberCommand {
public abstract String getUsername();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

@AutoValue
@AutoGson(autoValueClass = AutoValue_GroupCommand.class)
@SuppressWarnings("squid:S1610")
@SuppressWarnings("java:S1610")
public abstract class GroupCommand {
public abstract String getName();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
import org.molgenis.data.security.auth.RoleMembership;

@AutoValue
@SuppressWarnings("squid:S1610")
@SuppressWarnings("java:S1610")
public abstract class GroupMemberResponse {
public abstract UserResponse getUser();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
import org.molgenis.data.security.auth.Group;

@AutoValue
@SuppressWarnings("squid:S1610")
@SuppressWarnings("java:S1610")
public abstract class GroupResponse {
public abstract String getName();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@
public class IdentitiesApiController {
public static final String USER = "/user";

@SuppressWarnings("squid:S1075") // URIs should not be hardcoded
@SuppressWarnings("java:S1075") // URIs should not be hardcoded
private static final String SECURITY_API_PATH = ApiNamespace.API_PATH + "/identities";

static final String GROUP_END_POINT = SECURITY_API_PATH + "/group";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@
import org.molgenis.data.security.auth.Role;

@AutoValue
@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
@SuppressWarnings("java:S1610") // Abstract classes without fields should be converted to interfaces
public abstract class RoleResponse {
public abstract String getRoleName();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

@AutoValue
@AutoGson(autoValueClass = AutoValue_UpdateGroupMemberCommand.class)
@SuppressWarnings("squid:S1610")
@SuppressWarnings("java:S1610")
public abstract class UpdateGroupMemberCommand {
public abstract String getRoleName();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

@AutoValue
@AutoGson(autoValueClass = AutoValue_UpdateIncludeCommand.class)
@SuppressWarnings("squid:S1610")
@SuppressWarnings("java:S1610")
public abstract class UpdateIncludeCommand {

public abstract String getRole();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,7 @@
import org.molgenis.data.security.auth.User;

@AutoValue
@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
@SuppressWarnings("java:S1610") // Abstract classes without fields should be converted to interfaces
public abstract class UserResponse {
public abstract String getId();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ abstract class EntityTypeCollection {
abstract @Nullable @CheckForNull String getEntityId();

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
abstract static class Builder {

Expand Down Expand Up @@ -59,7 +59,7 @@ static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ public void updateEntityType(EntityType entityType, Map<String, Object> entityTy
}

@SuppressWarnings({
"squid:S1192"
"java:S1192"
}) // ATTRIBUTES constant in this class is not related to the one in this switch
private void updateEntityType(EntityType entityType, Entry<String, Object> entry) {
switch (entry.getKey()) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

import org.molgenis.util.exception.BadRequestException;

@SuppressWarnings("squid:MaximumInheritanceDepth")
@SuppressWarnings("java:MaximumInheritanceDepth")
public class EmptyAttributesException extends BadRequestException {
private static final String ERROR_CODE = "MAPI02";

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

import org.molgenis.util.exception.BadRequestException;

@SuppressWarnings("squid:MaximumInheritanceDepth")
@SuppressWarnings("java:MaximumInheritanceDepth")
public class IdModificationException extends BadRequestException {

private static final String ERROR_CODE = "MAPI03";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

import org.molgenis.util.exception.BadRequestException;

@SuppressWarnings("squid:MaximumInheritanceDepth")
@SuppressWarnings("java:MaximumInheritanceDepth")
public class InvalidKeyException extends BadRequestException {
private static final String ERROR_CODE = "MAPI04";
private final String target;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

import org.molgenis.util.exception.BadRequestException;

@SuppressWarnings("squid:MaximumInheritanceDepth")
@SuppressWarnings("java:MaximumInheritanceDepth")
public class ReadOnlyFieldException extends BadRequestException {
private static final String ERROR_CODE = "MAPI07";
private final String field;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
import org.molgenis.data.meta.model.EntityType;
import org.molgenis.util.exception.BadRequestException;

@SuppressWarnings("squid:MaximumInheritanceDepth")
@SuppressWarnings("java:MaximumInheritanceDepth")
public class UnknownLookupAttributesException extends BadRequestException {

private static final String ERROR_CODE = "MAPI05";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

import org.molgenis.util.exception.BadRequestException;

@SuppressWarnings("squid:MaximumInheritanceDepth")
@SuppressWarnings("java:MaximumInheritanceDepth")
public class UnsupportedFieldException extends BadRequestException {
private static final String ERROR_CODE = "MAPI06";
private final String field;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to Integererfaces
"java:S1610") // Abstract classes without fields should be converted to Integererfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ public static Builder builder() {
}

// Abstract classes without fields should be converted to interfaces
@SuppressWarnings("squid:S1610")
@SuppressWarnings("java:S1610")
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ public static Builder builder() {
}

@SuppressWarnings(
"squid:S1610") // Abstract classes without fields should be converted to interfaces
"java:S1610") // Abstract classes without fields should be converted to interfaces
@AutoValue.Builder
public abstract static class Builder {
public abstract Builder setLinks(LinksResponse newLinks);
Expand Down
Loading

0 comments on commit d939af3

Please sign in to comment.