}
@Override
- public IssueHandler.Context setDescription(@Nullable String description) {
- updater.setDescription(issue, description);
+ public IssueHandler.Context setMessage(@Nullable String s) {
+ updater.setMessage(issue, s);
return this;
}
}
private boolean isSameMessage(DefaultIssue newIssue, IssueDto pastIssue) {
- return Objects.equal(newIssue.description(), pastIssue.getDescription());
+ return Objects.equal(newIssue.message(), pastIssue.getMessage());
}
private boolean isSameKey(DefaultIssue newIssue, IssueDto pastIssue) {
}
private DefaultIssue newDefaultIssue(String message, Integer line, RuleKey ruleKey, String checksum) {
- return new DefaultIssue().setDescription(message).setLine(line).setRuleKey(ruleKey).setChecksum(checksum).setStatus(Issue.STATUS_OPEN);
+ return new DefaultIssue().setMessage(message).setLine(line).setRuleKey(ruleKey).setChecksum(checksum).setStatus(Issue.STATUS_OPEN);
}
private IssueDto newReferenceIssue(String message, Integer lineId, int ruleId, String lineChecksum) {
referenceIssue.setId(id);
referenceIssue.setKee(Long.toString(id));
referenceIssue.setLine(lineId);
- referenceIssue.setDescription(message);
+ referenceIssue.setMessage(message);
referenceIssue.setRuleId(ruleId);
referenceIssue.setChecksum(lineChecksum);
referenceIssue.setResolution(null);
.ruleKey(RuleKey.of(violation.getRule().getRepositoryKey(), violation.getRule().getKey()))
.effortToFix(violation.getCost())
.line(violation.getLineId())
- .description(violation.getMessage())
+ .message(violation.getMessage())
.severity(violation.getSeverity() != null ? violation.getSeverity().name() : Severity.MAJOR)
.build();
}
put(jsonIssue, "key", issue.key());
put(jsonIssue, "component", issue.componentKey());
put(jsonIssue, "line", issue.line());
- put(jsonIssue, "description", issue.description());
+ put(jsonIssue, "message", issue.message());
put(jsonIssue, "severity", issue.severity());
put(jsonIssue, "rule", issue.ruleKey());
put(jsonIssue, "status", issue.status());
put(jsonIssue, "resolution", issue.resolution());
put(jsonIssue, "isNew", issue.isNew());
+ put(jsonIssue, "reporter", issue.reporter());
put(jsonIssue, "assignee", issue.assignee());
+ put(jsonIssue, "effortToFix", issue.effortToFix());
put(jsonIssue, "creationDate", issue.creationDate());
put(jsonIssue, "updateDate", issue.updateDate());
put(jsonIssue, "closeDate", issue.closeDate());
DefaultIssue issue = new DefaultIssue()
.setKey("200")
.setComponentKey("Action.java")
- .setDescription("SystemPrintln")
+ .setMessage("SystemPrintln")
.setSeverity("MINOR")
.setStatus(Issue.STATUS_CLOSED)
.setResolution(Issue.RESOLUTION_FALSE_POSITIVE)
.setLine(1)
+ .setEffortToFix(3.14)
+ .setReporter("julien")
.setAssignee("simon")
.setRuleKey(RuleKey.of("squid", "AvoidCycle"))
.setCreationDate(DateUtils.parseDate("2013-04-24"))
JSONArray issues = (JSONArray) json.get("issues");
assertThat(issues).hasSize(1);
JSONObject jsonIssue = (JSONObject) issues.get(0);
- assertThat(jsonIssue.values()).hasSize(13);
+ assertThat(jsonIssue.values()).hasSize(15);
assertThat(jsonIssue.get("key")).isEqualTo("200");
assertThat(jsonIssue.get("component")).isEqualTo("Action.java");
assertThat(jsonIssue.get("line")).isEqualTo(1);
- assertThat(jsonIssue.get("description")).isEqualTo("SystemPrintln");
+ assertThat(jsonIssue.get("message")).isEqualTo("SystemPrintln");
assertThat(jsonIssue.get("severity")).isEqualTo("MINOR");
assertThat(jsonIssue.get("rule")).isEqualTo("squid:AvoidCycle");
assertThat(jsonIssue.get("status")).isEqualTo("CLOSED");
assertThat(jsonIssue.get("resolution")).isEqualTo("FALSE-POSITIVE");
assertThat(jsonIssue.get("assignee")).isEqualTo("simon");
+ assertThat(jsonIssue.get("effortToFix")).isEqualTo(3.14);
+ assertThat(jsonIssue.get("reporter")).isEqualTo("julien");
assertThat(jsonIssue.get("isNew")).isEqualTo(false);
assertThat((String) jsonIssue.get("creationDate")).contains("2013-04-24T00:00");
assertThat((String) jsonIssue.get("updateDate")).contains("2013-04-25T00:00");
private RuleKey ruleKey;
private String severity;
private boolean manualSeverity = false;
- private String description;
+ private String message;
private Integer line;
private Double effortToFix;
private String status;
}
@CheckForNull
- public String description() {
- return description;
+ public String message() {
+ return message;
}
- public DefaultIssue setDescription(@Nullable String s) {
- this.description = StringUtils.abbreviate(StringUtils.trim(s), DESCRIPTION_MAX_SIZE);
+ public DefaultIssue setMessage(@Nullable String s) {
+ this.message = StringUtils.abbreviate(StringUtils.trim(s), MESSAGE_MAX_SIZE);
return this;
}
private String componentKey;
private RuleKey ruleKey;
private Integer line;
- private String description;
+ private String message;
private String severity;
private Double effortToFix;
- private boolean manual = false;
private Date createdDate;
+ private String reporter;
private Map<String, String> attributes;
public DefaultIssueBuilder() {
}
@Override
- public Issuable.IssueBuilder description(String s) {
- this.description = s;
+ public Issuable.IssueBuilder message(@Nullable String s) {
+ this.message = s;
return this;
}
}
@Override
- public Issuable.IssueBuilder manual(boolean b) {
- this.manual = b;
+ public Issuable.IssueBuilder reporter(@Nullable String s) {
+ this.reporter = s;
return this;
}
issue.setUpdateDate(date);
issue.setComponentKey(componentKey);
issue.setRuleKey(ruleKey);
- issue.setDescription(description);
+ issue.setMessage(message);
issue.setSeverity(Objects.firstNonNull(severity, Severity.MAJOR));
issue.setEffortToFix(effortToFix);
issue.setLine(line);
- issue.setManualSeverity(manual);
+ issue.setManualSeverity(false);
+ issue.setReporter(reporter);
issue.setAttributes(attributes);
issue.setNew(true);
issue.setAlive(true);
issue.setAuthorLogin(authorLogin);
}
- public void setDescription(DefaultIssue issue, @Nullable String description) {
- issue.setDescription(description);
+ public void setMessage(DefaultIssue issue, @Nullable String s) {
+ issue.setMessage(s);
}
public void addComment(DefaultIssue issue, String text, IssueChangeContext context) {
private Integer ruleId;
private String severity;
private boolean manualSeverity;
- private String description;
+ private String message;
private Integer line;
private Double effortToFix;
private String status;
}
@CheckForNull
- public String getDescription() {
- return description;
+ public String getMessage() {
+ return message;
}
- public IssueDto setDescription(@Nullable String s) {
- this.description = s;
+ public IssueDto setMessage(@Nullable String s) {
+ this.message = s;
return this;
}
return new IssueDto()
.setKee(issue.key())
.setLine(issue.line())
- .setDescription(issue.description())
+ .setMessage(issue.message())
.setEffortToFix(issue.effortToFix())
.setResolution(issue.resolution())
.setStatus(issue.status())
issue.setKey(kee);
issue.setStatus(status);
issue.setResolution(resolution);
- issue.setDescription(description);
+ issue.setMessage(message);
issue.setEffortToFix(effortToFix);
issue.setLine(line);
issue.setSeverity(severity);
i.action_plan_key as actionPlanKey,
i.severity as severity,
i.manual_severity as manualSeverity,
- i.description as description,
+ i.message as message,
i.line as line,
i.effort_to_fix as effortToFix,
i.status as status,
<insert id="insert" parameterType="Issue" useGeneratedKeys="false" keyProperty="id">
INSERT INTO issues (kee, resource_id, rule_id, action_plan_key, severity, manual_severity,
- description, line, effort_to_fix, status,
+ message, line, effort_to_fix, status,
resolution, checksum, reporter, assignee, author_login, attributes, issue_creation_date, issue_update_date,
issue_close_date, created_at, updated_at)
VALUES (#{kee}, #{resourceId}, #{ruleId}, #{actionPlanKey}, #{severity}, #{manualSeverity},
- #{description}, #{line}, #{effortToFix}, #{status},
+ #{message}, #{line}, #{effortToFix}, #{status},
#{resolution}, #{checksum}, #{reporter}, #{assignee}, #{authorLogin}, #{attributes}, #{issueCreationDate},
#{issueUpdateDate}, #{issueCloseDate}, #{createdAt}, #{updatedAt})
</insert>
select issues_seq.NEXTVAL from DUAL
</selectKey>
INSERT INTO issues (id, kee, resource_id, rule_id, action_plan_key, severity, manual_severity,
- description, line, effort_to_fix, status,
+ message, line, effort_to_fix, status,
resolution, checksum, reporter, assignee, author_login, attributes, issue_creation_date, issue_update_date,
issue_close_date, created_at, updated_at)
VALUES (#{id}, #{kee}, #{resourceId}, #{ruleId}, #{actionPlanKey}, #{severity}, #{manualSeverity},
- #{description}, #{line}, #{effortToFix}, #{status},
+ #{message}, #{line}, #{effortToFix}, #{status},
#{resolution}, #{checksum}, #{reporter}, #{assignee}, #{authorLogin}, #{attributes}, #{issueCreationDate},
#{issueUpdateDate}, #{issueCloseDate}, #{createdAt}, #{updatedAt})
</insert>
action_plan_key=#{actionPlanKey},
severity=#{severity},
manual_severity=#{manualSeverity},
- description=#{description},
+ message=#{message},
line=#{line},
effort_to_fix=#{effortToFix},
status=#{status},
"RULE_ID" INTEGER,
"SEVERITY" VARCHAR(10),
"MANUAL_SEVERITY" BOOLEAN NOT NULL,
- "DESCRIPTION" VARCHAR(4000),
+ "MESSAGE" VARCHAR(4000),
"LINE" INTEGER,
"EFFORT_TO_FIX" DOUBLE,
"STATUS" VARCHAR(20),
String componentKey = "org.apache.struts:struts-core:Action.java";
DefaultIssue issue = (DefaultIssue) new DefaultIssueBuilder()
.componentKey(componentKey)
- .description("the desc")
+ .message("the message")
.line(123)
.effortToFix(10000.0)
.ruleKey(RuleKey.of("squid", "NullDereference"))
assertThat(issue.key()).isNotNull();
assertThat(issue.effortToFix()).isEqualTo(10000.0);
assertThat(issue.componentKey()).isEqualTo(componentKey);
- assertThat(issue.description()).isEqualTo("the desc");
+ assertThat(issue.message()).isEqualTo("the message");
assertThat(issue.line()).isEqualTo(123);
assertThat(issue.ruleKey().repository()).isEqualTo("squid");
assertThat(issue.ruleKey().rule()).isEqualTo("NullDereference");
}
@Test
- public void description_should_be_abbreviated_if_too_long() {
- issue.setDescription(StringUtils.repeat("a", 5000));
- assertThat(issue.description()).hasSize(4000);
+ public void message_should_be_abbreviated_if_too_long() {
+ issue.setMessage(StringUtils.repeat("a", 5000));
+ assertThat(issue.message()).hasSize(4000);
}
@Test
- public void description_should_be_trimmed() {
- issue.setDescription(" foo ");
- assertThat(issue.description()).isEqualTo("foo");
+ public void message_should_be_trimmed() {
+ issue.setMessage(" foo ");
+ assertThat(issue.message()).isEqualTo("foo");
}
@Test
- public void description_could_be_null() {
- issue.setDescription(null);
- assertThat(issue.description()).isNull();
+ public void message_could_be_null() {
+ issue.setMessage(null);
+ assertThat(issue.message()).isNull();
}
}
assertThat(issue.getRuleId()).isEqualTo(500);
assertThat(issue.getSeverity()).isEqualTo("BLOCKER");
assertThat(issue.isManualSeverity()).isFalse();
- assertThat(issue.getDescription()).isNull();
+ assertThat(issue.getMessage()).isNull();
assertThat(issue.getLine()).isEqualTo(200);
assertThat(issue.getEffortToFix()).isEqualTo(4.2);
assertThat(issue.getStatus()).isEqualTo("OPEN");
.setEffortToFix(15.0)
.setLine(6)
.setSeverity("BLOCKER")
- .setDescription("message")
+ .setMessage("message")
.setManualSeverity(true)
.setReporter("arthur")
.setAssignee("perceval")
assertThat(issue.effortToFix()).isEqualTo(15.0);
assertThat(issue.line()).isEqualTo(6);
assertThat(issue.severity()).isEqualTo("BLOCKER");
- assertThat(issue.description()).isEqualTo("message");
+ assertThat(issue.message()).isEqualTo("message");
assertThat(issue.manualSeverity()).isTrue();
assertThat(issue.reporter()).isEqualTo("arthur");
assertThat(issue.assignee()).isEqualTo("perceval");
dto.setActionPlanKey("current_sprint");
dto.setAttributes("JIRA=FOO-1234");
dto.setChecksum("123456789");
- dto.setDescription("the description");
+ dto.setMessage("the message");
dto.setIssueCreationDate(DateUtils.parseDate("2013-05-18"));
dto.setIssueUpdateDate(DateUtils.parseDate("2013-05-19"));
dto.setActionPlanKey("current_sprint");
dto.setAttributes("JIRA=FOO-1234");
dto.setChecksum("123456789");
- dto.setDescription("the description");
+ dto.setMessage("the message");
dto.setIssueCreationDate(DateUtils.parseDate("2013-05-18"));
dto.setIssueUpdateDate(DateUtils.parseDate("2013-05-19"));
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
status="OPEN"
resolution="[null]"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
status="CLOSED"
resolution="FIXED"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
status="CLOSED"
resolution="FIXED"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="501"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
action_plan_key="ABC"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
action_plan_key="ABC"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
action_plan_key="DEF"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="501"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="[null]"
status="OPEN"
rule_id="501"
severity="MAJOR"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="120"
effort_to_fix="[null]"
status="CLOSED"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="501"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
action_plan_key="ABC"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
action_plan_key="ABC"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
action_plan_key="[null]"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="RESOLVED"
rule_id="501"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="CLOSED"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="501"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="501"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="[null]"
status="OPEN"
rule_id="501"
severity="MAJOR"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="120"
effort_to_fix="[null]"
status="OPEN"
rule_id="501"
severity="MAJOR"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="120"
effort_to_fix="[null]"
status="CLOSED"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="501"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="4.2"
status="OPEN"
rule_id="200"
severity="BLOCKER"
manual_severity="[false]"
- description="the description"
+ message="the message"
line="500"
effort_to_fix="3.14"
status="RESOLVED"
rule_id="200"
severity="BLOCKER"
manual_severity="[false]"
- description="the description"
+ message="the message"
line="500"
effort_to_fix="3.14"
status="RESOLVED"
rule_id="200"
severity="INFO"
manual_severity="[false]"
- description="old"
+ message="old"
line="[null]"
effort_to_fix="[null]"
status="OPEN"
author_login="[null]"
checksum="[null]"
effort_to_fix="[null]"
- description="[null]"
+ message="[null]"
line="5000"
resource_id="100"
rule_id="200"
author_login="simon"
checksum="FFFFF"
effort_to_fix="[null]"
- description="[null]"
+ message="[null]"
line="5000"
resource_id="100"
rule_id="200"
author_login="simon"
checksum="FFFFF"
effort_to_fix="[null]"
- description="[null]"
+ message="[null]"
line="3000"
resource_id="100"
rule_id="200"
rule_id="500"
severity="BLOCKER"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="200"
effort_to_fix="[null]"
status="OPEN"
rule_id="501"
severity="MAJOR"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
line="120"
effort_to_fix="[null]"
status="CLOSED"
<issues id="1" kee="ABCDE" resource_id="1" status="CLOSED" resolution="[null]" line="200" severity="BLOCKER"
reporter="perceval" assignee="arthur" rule_id="500"
manual_severity="[false]"
- description="[null]"
+ message="[null]"
issue_creation_date="2013-04-16"
issue_update_date="2013-04-16"
issue_close_date="2013-04-16"
<issues id="1" kee="ABCDE" resource_id="1" status="CLOSED" resolution="[null]" line="200" severity="BLOCKER"
reporter="perceval" assignee="arthur" rule_id="500"
- manual_severity="[false]" description="[null]" action_plan_key="[null]"
+ manual_severity="[false]"
+ message="[null]"
+ action_plan_key="[null]"
created_at="[null]"
updated_at="[null]"
issue_creation_date="2013-04-16"
<issues id="2" kee="ABCDF" resource_id="1" status="CLOSED" resolution="[null]" line="200" severity="BLOCKER"
reporter="perceval" assignee="arthur" rule_id="500"
- manual_severity="[false]" description="[null]" action_plan_key="[null]"
+ manual_severity="[false]"
+ message="[null]"
+ action_plan_key="[null]"
created_at="[null]"
updated_at="[null]"
issue_creation_date="2013-04-16"
IssueBuilder line(@Nullable Integer line);
- IssueBuilder description(String description);
+ IssueBuilder message(@Nullable String message);
IssueBuilder severity(String severity);
- IssueBuilder effortToFix(@Nullable Double d);
+ IssueBuilder reporter(String reporter);
- IssueBuilder manual(boolean b);
+ IssueBuilder effortToFix(@Nullable Double d);
IssueBuilder attribute(String key, @Nullable String value);
*/
public interface Issue extends Serializable {
- int DESCRIPTION_MAX_SIZE = 4000;
+ int MESSAGE_MAX_SIZE = 4000;
String STATUS_OPEN = "OPEN";
String STATUS_REOPENED = "REOPENED";
String severity();
- String description();
+ @CheckForNull
+ String message();
@CheckForNull
Integer line();
// TODO rename isClosed()
boolean isAlive();
- Context setLine(@Nullable Integer line);
+ Context setLine(@Nullable Integer i);
- Context setDescription(@Nullable String description);
+ Context setMessage(@Nullable String s);
- Context setSeverity(String severity);
+ Context setSeverity(String s);
- Context setEffortToFix(@Nullable Double effortToFix);
+ Context setEffortToFix(@Nullable Double d);
- Context setAuthorLogin(@Nullable String login);
+ Context setAuthorLogin(@Nullable String s);
Context setAttribute(String key, @Nullable String value);
public static final int MAX_ISSUE_KEYS = 500;
public static enum Sort {
- CREATION_DATE, UPDATE_DATE, CLOSE_DATE, ASSIGNEE
+ CREATION_DATE, UPDATE_DATE, CLOSE_DATE, ASSIGNEE, SEVERITY
}
private final Collection<String> issueKeys;
DefaultIssueBuilder builder = new DefaultIssueBuilder().componentKey(componentKey);
String line = parameters.get("line");
builder.line(line != null ? Integer.parseInt(line) : null);
- builder.description(parameters.get("description"));
+ builder.message(parameters.get("message"));
builder.severity(parameters.get("severity"));
String effortToFix = parameters.get("effortToFix");
builder.effortToFix(effortToFix != null ? Double.parseDouble(effortToFix) : null);
// TODO verify existence of rule
builder.ruleKey(RuleKey.parse(parameters.get("rule")));
- builder.manual(true);
Issue issue = builder.build();
return issueService.create((DefaultIssue) issue, UserSession.get());
}
hash[:actionPlan] = issue.actionPlanKey if issue.actionPlanKey
hash[:resolution] = issue.resolution if issue.resolution
hash[:severity] = issue.severity if issue.severity
- hash[:desc] = issue.description if issue.description
+ hash[:message] = issue.message if issue.message
hash[:line] = issue.line.to_i if issue.line
hash[:effortToFix] = issue.effortToFix.to_f if issue.effortToFix
hash[:reporter] = issue.reporter if issue.reporter
t.column :rule_id, :integer, :null => true
t.column :severity, :string, :null => true, :limit => 10
t.column :manual_severity, :boolean, :null => false
- t.column :description, :string, :null => true, :limit => 4000
+ t.column :message, :string, :null => true, :limit => 4000
t.column :line, :integer, :null => true
t.column :effort_to_fix, :decimal, :null => true, :precision => 30, :scale => 20
t.column :status, :string , :null => true, :limit => 20
return JsonUtils.getString(json, "severity");
}
+ /**
+ * TODO to be removed
+ */
@CheckForNull
public String description() {
- return JsonUtils.getString(json, "desc");
+ return JsonUtils.getString(json, "message");
+ }
+
+ @CheckForNull
+ public String message() {
+ return JsonUtils.getString(json, "message");
}
@CheckForNull
assertThat(first.resolution()).isEqualTo("FIXED");
assertThat(first.status()).isEqualTo("OPEN");
assertThat(first.assignee()).isEqualTo("karadoc");
- assertThat(first.description()).isEqualTo("the desc");
+ assertThat(first.message()).isEqualTo("the message");
assertThat(first.effortToFix()).isEqualTo(4.2);
assertThat(first.userLogin()).isEqualTo("perceval");
assertThat(first.creationDate()).isNotNull();
"status": "OPEN",
"assignee": "karadoc",
"effortToFix": 4.2,
- "desc": "the desc",
+ "message": "the message",
"title": "the title",
"reporter": "perceval",
"creationDate": "2013-05-18T12:45:34+0200",