SONAR-12719 move Changelog to ws-commons.proto

server/sonar-webserver-webapi/src/main/java/org/sonar/server/issue/ws/ChangelogAction.java

@@ -46,8 +46,8 @@ import org.sonar.db.user.UserDto;
 import org.sonar.server.issue.AvatarResolver;
 import org.sonar.server.issue.IssueFinder;
 import org.sonar.server.user.UserSession;
+import org.sonarqube.ws.Common.Changelog;
 import org.sonarqube.ws.Issues.ChangelogWsResponse;
-import org.sonarqube.ws.Issues.ChangelogWsResponse.Changelog;
 
 import static com.google.common.base.Preconditions.checkState;
 import static com.google.common.base.Strings.emptyToNull;

server/sonar-webserver-webapi/src/test/java/org/sonar/server/issue/ws/ChangelogActionTest.java

@@ -42,8 +42,8 @@ import org.sonar.server.issue.IssueFinder;
 import org.sonar.server.tester.UserSessionRule;
 import org.sonar.server.ws.TestRequest;
 import org.sonar.server.ws.WsActionTester;
+import org.sonarqube.ws.Common.Changelog.Diff;
 import org.sonarqube.ws.Issues.ChangelogWsResponse;
-import org.sonarqube.ws.Issues.ChangelogWsResponse.Changelog.Diff;
 
 import static java.util.Optional.ofNullable;
 import static org.assertj.core.api.Assertions.assertThat;

sonar-ws/src/main/protobuf/ws-commons.proto

@@ -105,6 +105,23 @@ message Location {
   optional string msg = 3;
 }
 
+message Changelog {
+  optional string user = 1;
+  optional string userName = 2;
+  // Email is no more returned since 6.3
+  optional string deprecatedEmail = 3;
+  optional string creationDate = 4;
+  repeated Diff diffs = 5;
+  optional string avatar = 6;
+  optional bool isUserActive = 7;
+
+  message Diff {
+    optional string key = 1;
+    optional string newValue = 2;
+    optional string oldValue = 3;
+  }
+}
+
 message Metric {
   optional string key = 1;
   optional string name = 2;

sonar-ws/src/main/protobuf/ws-issues.proto

@@ -226,24 +226,7 @@ message Component {
 
 // Response of GET api/issues/changelog
 message ChangelogWsResponse {
-  repeated Changelog changelog = 1;
-
-  message Changelog {
-    optional string user = 1;
-    optional string userName = 2;
-    // Email is no more returned since 6.3
-    optional string deprecatedEmail = 3;
-    optional string creationDate = 4;
-    repeated Diff diffs = 5;
-    optional string avatar = 6;
-    optional bool isUserActive = 7;
-
-    message Diff {
-      optional string key = 1;
-      optional string newValue = 2;
-      optional string oldValue = 3;
-    }
-  }
+  repeated sonarqube.ws.commons.Changelog changelog = 1;
 }
 
 // Response of POST api/issues/bulk_change