merge brachn 2.0.0-m3

Jenkinsfile

@@ -74,7 +74,7 @@ Maven setupMavenBuild() {
 
 void analyzeWith(Maven mvn) {
 
-  withSonarQubeEnv('sonarcloud.io') {
+  withSonarQubeEnv('sonarcloud.io-scm') {
 
     String mvnArgs = "${env.SONAR_MAVEN_GOAL} " +
       "-Dsonar.host.url=${env.SONAR_HOST_URL} " +

pom.xml

@@ -69,6 +69,7 @@
     <module>scm-annotation-processor</module>
     <module>scm-core</module>
     <module>scm-test</module>
+    <module>scm-ui-components</module>
     <module>scm-plugins</module>
     <module>scm-dao-xml</module>
     <module>scm-ui</module>
@@ -359,6 +360,11 @@
 
     <pluginManagement>
       <plugins>
+        <plugin>
+          <groupId>com.github.sdorra</groupId>
+          <artifactId>buildfrontend-maven-plugin</artifactId>
+          <version>2.1.1</version>
+        </plugin>
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-javadoc-plugin</artifactId>
@@ -768,6 +774,10 @@
     <guava.version>26.0-jre</guava.version>
     <quartz.version>2.2.3</quartz.version>
 
+    <!-- frontend -->
+    <nodejs.version>8.11.4</nodejs.version>
+    <yarn.version>1.9.4</yarn.version>
+
     <!-- build properties -->
     <project.build.javaLevel>1.8</project.build.javaLevel>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>

scm-core/src/main/java/sonia/scm/util/CRLFInjectionException.java

@@ -0,0 +1,8 @@
+package sonia.scm.util;
+
+public class CRLFInjectionException extends IllegalArgumentException{
+
+  public CRLFInjectionException(String message) {
+    super(message);
+  }
+}

scm-core/src/main/java/sonia/scm/util/HttpUtil.java

@@ -344,8 +344,7 @@ public final class HttpUtil
         "parameter \"{}\" contains a character which could be an indicator for a crlf injection",
         parameter);
 
-      throw new IllegalArgumentException(
-        "parameter contains an illegal character");
+      throw new CRLFInjectionException("parameter contains an illegal character");
     }
   }
 

scm-core/src/main/java/sonia/scm/web/VndMediaType.java

@@ -12,6 +12,8 @@ public class VndMediaType {
   private static final String SUBTYPE_PREFIX = "vnd.scmm-";
   public static final String PREFIX = TYPE + "/" + SUBTYPE_PREFIX;
   public static final String SUFFIX = "+json;v=" + VERSION;
+  public static final String PLAIN_TEXT_PREFIX = "text/" + SUBTYPE_PREFIX;
+  public static final String PLAIN_TEXT_SUFFIX = "+plain;v=" + VERSION;
 
   public static final String USER = PREFIX + "user" + SUFFIX;
   public static final String GROUP = PREFIX + "group" + SUFFIX;
@@ -19,7 +21,10 @@ public class VndMediaType {
   public static final String PERMISSION = PREFIX + "permission" + SUFFIX;
   public static final String CHANGESET = PREFIX + "changeset" + SUFFIX;
   public static final String CHANGESET_COLLECTION = PREFIX + "changesetCollection" + SUFFIX;
+  public static final String TAG = PREFIX + "tag" + SUFFIX;
+  public static final String TAG_COLLECTION = PREFIX + "tagCollection" + SUFFIX;
   public static final String BRANCH = PREFIX + "branch" + SUFFIX;
+  public static final String DIFF = PLAIN_TEXT_PREFIX + "diff" + PLAIN_TEXT_SUFFIX;
   public static final String USER_COLLECTION = PREFIX + "userCollection" + SUFFIX;
   public static final String GROUP_COLLECTION = PREFIX + "groupCollection" + SUFFIX;
   public static final String REPOSITORY_COLLECTION = PREFIX + "repositoryCollection" + SUFFIX;

scm-it/src/test/java/sonia/scm/it/RepositoryAccessITCase.java

@@ -1,5 +1,7 @@
 package sonia.scm.it;
 
+import io.restassured.response.ExtractableResponse;
+import io.restassured.response.Response;
 import org.apache.http.HttpStatus;
 import org.junit.Assume;
 import org.junit.Before;
@@ -8,18 +10,22 @@ import org.junit.Test;
 import org.junit.rules.TemporaryFolder;
 import org.junit.runner.RunWith;
 import org.junit.runners.Parameterized;
+import sonia.scm.repository.Changeset;
 import sonia.scm.repository.client.api.ClientCommand;
 import sonia.scm.repository.client.api.RepositoryClient;
+import sonia.scm.web.VndMediaType;
 
 import java.io.File;
 import java.io.IOException;
 import java.util.Collection;
 import java.util.List;
 
-import static org.assertj.core.api.Assertions.assertThat;
 import static java.lang.Thread.sleep;
+import static org.assertj.core.api.Assertions.assertThat;
 import static org.hamcrest.Matchers.equalTo;
 import static org.junit.Assert.assertNotNull;
+import static sonia.scm.it.RestUtil.ADMIN_PASSWORD;
+import static sonia.scm.it.RestUtil.ADMIN_USERNAME;
 import static sonia.scm.it.RestUtil.given;
 import static sonia.scm.it.ScmTypes.availableScmTypes;
 
@@ -74,6 +80,85 @@ public class RepositoryAccessITCase {
     assertNotNull(branchName);
   }
 
+  @Test
+  public void shouldFindTags() throws IOException {
+    RepositoryClient repositoryClient = RepositoryUtil.createRepositoryClient(repositoryType, folder);
+
+    Assume.assumeTrue("There are no tags for " + repositoryType, repositoryClient.isCommandSupported(ClientCommand.TAG));
+
+    Changeset changeset = RepositoryUtil.createAndCommitFile(repositoryClient, ADMIN_USERNAME, "a.txt", "a");
+
+    String tagName = "v1.0";
+    String repositoryUrl = TestData.getDefaultRepositoryUrl(repositoryType);
+    String tagsUrl = given()
+      .when()
+      .get(repositoryUrl)
+      .then()
+      .statusCode(HttpStatus.SC_OK)
+      .extract()
+      .path("_links.tags.href");
+
+    ExtractableResponse<Response> response = given(VndMediaType.TAG_COLLECTION, ADMIN_USERNAME, ADMIN_PASSWORD)
+      .when()
+      .get(tagsUrl)
+      .then()
+      .statusCode(HttpStatus.SC_OK)
+      .extract();
+
+    assertThat(response).isNotNull();
+    assertThat(response.body()).isNotNull();
+    assertThat(response.body().asString())
+      .isNotNull()
+      .isNotBlank();
+
+    RepositoryUtil.addTag(repositoryClient, changeset.getId(), tagName);
+    response = given(VndMediaType.TAG_COLLECTION, ADMIN_USERNAME, ADMIN_PASSWORD)
+      .when()
+      .get(tagsUrl)
+      .then()
+      .statusCode(HttpStatus.SC_OK)
+      .extract();
+
+    assertThat(response).isNotNull();
+    assertThat(response.body()).isNotNull();
+    assertThat(response.body().asString())
+      .isNotNull()
+      .isNotBlank();
+
+    assertThat(response.body().jsonPath().getString("_links.self.href"))
+      .as("assert tags self link")
+      .isNotNull()
+      .contains(repositoryUrl + "/tags/");
+
+    assertThat(response.body().jsonPath().getList("_embedded.tags"))
+      .as("assert tag size")
+      .isNotNull()
+      .size()
+      .isGreaterThan(0);
+
+    assertThat(response.body().jsonPath().getMap("_embedded.tags.find{it.name=='" + tagName + "'}"))
+      .as("assert tag name and revision")
+      .isNotNull()
+      .hasSize(3)
+      .containsEntry("name", tagName)
+      .containsEntry("revision", changeset.getId());
+
+    assertThat(response.body().jsonPath().getString("_embedded.tags.find{it.name=='" + tagName + "'}._links.self.href"))
+      .as("assert single tag self link")
+      .isNotNull()
+      .contains(String.format("%s/tags/%s", repositoryUrl, tagName));
+
+    assertThat(response.body().jsonPath().getString("_embedded.tags.find{it.name=='" + tagName + "'}._links.sources.href"))
+      .as("assert single tag source link")
+      .isNotNull()
+      .contains(String.format("%s/sources/%s", repositoryUrl, changeset.getId()));
+
+    assertThat(response.body().jsonPath().getString("_embedded.tags.find{it.name=='" + tagName + "'}._links.changesets.href"))
+      .as("assert single tag changesets link")
+      .isNotNull()
+      .contains(String.format("%s/changesets/%s", repositoryUrl, changeset.getId()));
+  }
+
   @Test
   public void shouldReadContent() throws IOException, InterruptedException {
     RepositoryClient repositoryClient = RepositoryUtil.createRepositoryClient(repositoryType, folder);
@@ -97,7 +182,8 @@ public class RepositoryAccessITCase {
       .then()
       .statusCode(HttpStatus.SC_OK)
       .extract()
-      .path("files.find{it.name=='a.txt'}._links.self.href");
+      .path("_embedded.files.find{it.name=='a.txt'}._links.self.href");
+
     given()
       .when()
       .get(rootContentUrl)
@@ -111,14 +197,22 @@ public class RepositoryAccessITCase {
       .then()
       .statusCode(HttpStatus.SC_OK)
       .extract()
-      .path("files.find{it.name=='subfolder'}._links.self.href");
-    String subfolderContentUrl= given()
+      .path("_embedded.files.find{it.name=='subfolder'}._links.self.href");
+    String selfOfSubfolderUrl = given()
       .when()
       .get(subfolderSourceUrl)
       .then()
       .statusCode(HttpStatus.SC_OK)
       .extract()
-      .path("files[0]._links.self.href");
+      .path("_links.self.href");
+    assertThat(subfolderSourceUrl).isEqualTo(selfOfSubfolderUrl);
+    String subfolderContentUrl = given()
+      .when()
+      .get(subfolderSourceUrl)
+      .then()
+      .statusCode(HttpStatus.SC_OK)
+      .extract()
+      .path("_embedded.files[0]._links.self.href");
     given()
       .when()
       .get(subfolderContentUrl)
@@ -152,5 +246,40 @@ public class RepositoryAccessITCase {
 
     assertThat(changesets).size().isBetween(2, 3); // svn has an implicit root revision '0' that is extra to the two commits
   }
+
+  @Test
+  public void shouldFindDiffs() throws IOException {
+    RepositoryClient repositoryClient = RepositoryUtil.createRepositoryClient(repositoryType, folder);
+
+    RepositoryUtil.createAndCommitFile(repositoryClient, "scmadmin", "a.txt", "a");
+    RepositoryUtil.createAndCommitFile(repositoryClient, "scmadmin", "b.txt", "b");
+
+    String changesetsUrl = given()
+      .when()
+      .get(TestData.getDefaultRepositoryUrl(repositoryType))
+      .then()
+      .statusCode(HttpStatus.SC_OK)
+      .extract()
+      .path("_links.changesets.href");
+
+    String diffUrl = given()
+      .when()
+      .get(changesetsUrl)
+      .then()
+      .statusCode(HttpStatus.SC_OK)
+      .extract()
+      .path("_embedded.changesets[0]._links.diff.href");
+
+    given()
+      .when()
+      .get(diffUrl)
+      .then()
+      .statusCode(HttpStatus.SC_OK)
+      .extract()
+      .body()
+      .asString()
+      .contains("diff");
+
+  }
 }
 

scm-it/src/test/java/sonia/scm/it/RepositoryUtil.java

@@ -7,6 +7,7 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import sonia.scm.repository.Changeset;
 import sonia.scm.repository.Person;
+import sonia.scm.repository.Tag;
 import sonia.scm.repository.client.api.ClientCommand;
 import sonia.scm.repository.client.api.RepositoryClient;
 import sonia.scm.repository.client.api.RepositoryClientFactory;
@@ -40,11 +41,11 @@ public class RepositoryUtil {
     return name;
   }
 
-  static void createAndCommitFile(RepositoryClient repositoryClient, String username, String fileName, String content) throws IOException {
+  static Changeset createAndCommitFile(RepositoryClient repositoryClient, String username, String fileName, String content) throws IOException {
     File file = new File(repositoryClient.getWorkingCopy(), fileName);
     Files.write(content, file, Charsets.UTF_8);
     addWithParentDirectories(repositoryClient, file);
-    commit(repositoryClient, username, "added " + fileName);
+    return commit(repositoryClient, username, "added " + fileName);
   }
 
   private static String addWithParentDirectories(RepositoryClient repositoryClient, File file) throws IOException {
@@ -69,4 +70,16 @@ public class RepositoryUtil {
     }
     return changeset;
   }
+
+  static Tag addTag(RepositoryClient repositoryClient, String revision, String tagName) throws IOException {
+    if (repositoryClient.isCommandSupported(ClientCommand.TAG)) {
+      Tag tag = repositoryClient.getTagCommand().setRevision(revision).tag(tagName, TestData.USER_SCM_ADMIN);
+      if (repositoryClient.isCommandSupported(ClientCommand.PUSH)) {
+        repositoryClient.getPushCommand().pushTags();
+      }
+      return tag;
+    }
+
+    return null;
+  }
 }

scm-plugins/pom.xml

@@ -139,14 +139,13 @@
       <plugin>
         <groupId>com.github.sdorra</groupId>
         <artifactId>buildfrontend-maven-plugin</artifactId>
-        <version>2.0.1</version>
         <configuration>
           <node>
-            <version>8.11.3</version>
+            <version>${nodejs.version}</version>
           </node>
           <pkgManager>
             <type>YARN</type>
-            <version>1.7.0</version>
+            <version>${yarn.version}</version>
           </pkgManager>
           <failOnMissingPackageJson>false</failOnMissingPackageJson>
           <script>build</script>

scm-plugins/scm-git-plugin/.babelrc

@@ -0,0 +1,6 @@
+{
+  "presets": ["@babel/preset-env", "@babel/preset-react", "@babel/preset-flow"],
+  "plugins": [
+    "@babel/plugin-proposal-class-properties"
+  ]
+}

scm-plugins/scm-git-plugin/.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "@scm-manager/eslint-config"
+}

scm-plugins/scm-git-plugin/.flowconfig

@@ -0,0 +1,8 @@
+[ignore]
+.*/node_modules/module-deps/.*
+
+[include]
+
+[libs]
+
+[options]

scm-plugins/scm-git-plugin/flow-typed/npm/classnames_v2.x.x.js

@@ -0,0 +1,23 @@
+// flow-typed signature: cf86673cc32d185bdab1d2ea90578d37
+// flow-typed version: 614bf49aa8/classnames_v2.x.x/flow_>=v0.25.x
+
+type $npm$classnames$Classes =
+  | string
+  | { [className: string]: * }
+  | false
+  | void
+  | null;
+
+declare module "classnames" {
+  declare module.exports: (
+    ...classes: Array<$npm$classnames$Classes | $npm$classnames$Classes[]>
+  ) => string;
+}
+
+declare module "classnames/bind" {
+  declare module.exports: $Exports<"classnames">;
+}
+
+declare module "classnames/dedupe" {
+  declare module.exports: $Exports<"classnames">;
+}

scm-plugins/scm-git-plugin/flow-typed/npm/moment_v2.3.x.js

@@ -0,0 +1,331 @@
+// flow-typed signature: 23b805356f90ad9384dd88489654e380
+// flow-typed version: e9374c5fe9/moment_v2.3.x/flow_>=v0.25.x
+
+type moment$MomentOptions = {
+  y?: number | string,
+  year?: number | string,
+  years?: number | string,
+  M?: number | string,
+  month?: number | string,
+  months?: number | string,
+  d?: number | string,
+  day?: number | string,
+  days?: number | string,
+  date?: number | string,
+  h?: number | string,
+  hour?: number | string,
+  hours?: number | string,
+  m?: number | string,
+  minute?: number | string,
+  minutes?: number | string,
+  s?: number | string,
+  second?: number | string,
+  seconds?: number | string,
+  ms?: number | string,
+  millisecond?: number | string,
+  milliseconds?: number | string
+};
+
+type moment$MomentObject = {
+  years: number,
+  months: number,
+  date: number,
+  hours: number,
+  minutes: number,
+  seconds: number,
+  milliseconds: number
+};
+
+type moment$MomentCreationData = {
+  input: string,
+  format: string,
+  locale: Object,
+  isUTC: boolean,
+  strict: boolean
+};
+
+type moment$CalendarFormat = string | ((moment: moment$Moment) => string);
+
+type moment$CalendarFormats = {
+  sameDay?: moment$CalendarFormat,
+  nextDay?: moment$CalendarFormat,
+  nextWeek?: moment$CalendarFormat,
+  lastDay?: moment$CalendarFormat,
+  lastWeek?: moment$CalendarFormat,
+  sameElse?: moment$CalendarFormat
+};
+
+declare class moment$LocaleData {
+  months(moment: moment$Moment): string,
+  monthsShort(moment: moment$Moment): string,
+  monthsParse(month: string): number,
+  weekdays(moment: moment$Moment): string,
+  weekdaysShort(moment: moment$Moment): string,
+  weekdaysMin(moment: moment$Moment): string,
+  weekdaysParse(weekDay: string): number,
+  longDateFormat(dateFormat: string): string,
+  isPM(date: string): boolean,
+  meridiem(hours: number, minutes: number, isLower: boolean): string,
+  calendar(
+    key:
+      | "sameDay"
+      | "nextDay"
+      | "lastDay"
+      | "nextWeek"
+      | "prevWeek"
+      | "sameElse",
+    moment: moment$Moment
+  ): string,
+  relativeTime(
+    number: number,
+    withoutSuffix: boolean,
+    key: "s" | "m" | "mm" | "h" | "hh" | "d" | "dd" | "M" | "MM" | "y" | "yy",
+    isFuture: boolean
+  ): string,
+  pastFuture(diff: any, relTime: string): string,
+  ordinal(number: number): string,
+  preparse(str: string): any,
+  postformat(str: string): any,
+  week(moment: moment$Moment): string,
+  invalidDate(): string,
+  firstDayOfWeek(): number,
+  firstDayOfYear(): number
+}
+declare class moment$MomentDuration {
+  humanize(suffix?: boolean): string,
+  milliseconds(): number,
+  asMilliseconds(): number,
+  seconds(): number,
+  asSeconds(): number,
+  minutes(): number,
+  asMinutes(): number,
+  hours(): number,
+  asHours(): number,
+  days(): number,
+  asDays(): number,
+  months(): number,
+  asWeeks(): number,
+  weeks(): number,
+  asMonths(): number,
+  years(): number,
+  asYears(): number,
+  add(value: number | moment$MomentDuration | Object, unit?: string): this,
+  subtract(value: number | moment$MomentDuration | Object, unit?: string): this,
+  as(unit: string): number,
+  get(unit: string): number,
+  toJSON(): string,
+  toISOString(): string,
+  isValid(): boolean
+}
+declare class moment$Moment {
+  static ISO_8601: string,
+  static (
+    string?: string,
+    format?: string | Array<string>,
+    strict?: boolean
+  ): moment$Moment,
+  static (
+    string?: string,
+    format?: string | Array<string>,
+    locale?: string,
+    strict?: boolean
+  ): moment$Moment,
+  static (
+    initDate: ?Object | number | Date | Array<number> | moment$Moment | string
+  ): moment$Moment,
+  static unix(seconds: number): moment$Moment,
+  static utc(): moment$Moment,
+  static utc(number: number | Array<number>): moment$Moment,
+  static utc(
+    str: string,
+    str2?: string | Array<string>,
+    str3?: string
+  ): moment$Moment,
+  static utc(moment: moment$Moment): moment$Moment,
+  static utc(date: Date): moment$Moment,
+  static parseZone(): moment$Moment,
+  static parseZone(rawDate: string): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string | Array<string>
+  ): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string,
+    strict: boolean
+  ): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string,
+    locale: string,
+    strict: boolean
+  ): moment$Moment,
+  isValid(): boolean,
+  invalidAt(): 0 | 1 | 2 | 3 | 4 | 5 | 6,
+  creationData(): moment$MomentCreationData,
+  millisecond(number: number): this,
+  milliseconds(number: number): this,
+  millisecond(): number,
+  milliseconds(): number,
+  second(number: number): this,
+  seconds(number: number): this,
+  second(): number,
+  seconds(): number,
+  minute(number: number): this,
+  minutes(number: number): this,
+  minute(): number,
+  minutes(): number,
+  hour(number: number): this,
+  hours(number: number): this,
+  hour(): number,
+  hours(): number,
+  date(number: number): this,
+  dates(number: number): this,
+  date(): number,
+  dates(): number,
+  day(day: number | string): this,
+  days(day: number | string): this,
+  day(): number,
+  days(): number,
+  weekday(number: number): this,
+  weekday(): number,
+  isoWeekday(number: number): this,
+  isoWeekday(): number,
+  dayOfYear(number: number): this,
+  dayOfYear(): number,
+  week(number: number): this,
+  weeks(number: number): this,
+  week(): number,
+  weeks(): number,
+  isoWeek(number: number): this,
+  isoWeeks(number: number): this,
+  isoWeek(): number,
+  isoWeeks(): number,
+  month(number: number): this,
+  months(number: number): this,
+  month(): number,
+  months(): number,
+  quarter(number: number): this,
+  quarter(): number,
+  year(number: number): this,
+  years(number: number): this,
+  year(): number,
+  years(): number,
+  weekYear(number: number): this,
+  weekYear(): number,
+  isoWeekYear(number: number): this,
+  isoWeekYear(): number,
+  weeksInYear(): number,
+  isoWeeksInYear(): number,
+  get(string: string): number,
+  set(unit: string, value: number): this,
+  set(options: { [unit: string]: number }): this,
+  static max(...dates: Array<moment$Moment>): moment$Moment,
+  static max(dates: Array<moment$Moment>): moment$Moment,
+  static min(...dates: Array<moment$Moment>): moment$Moment,
+  static min(dates: Array<moment$Moment>): moment$Moment,
+  add(
+    value: number | moment$MomentDuration | moment$Moment | Object,
+    unit?: string
+  ): this,
+  subtract(
+    value: number | moment$MomentDuration | moment$Moment | string | Object,
+    unit?: string
+  ): this,
+  startOf(unit: string): this,
+  endOf(unit: string): this,
+  local(): this,
+  utc(): this,
+  utcOffset(
+    offset: number | string,
+    keepLocalTime?: boolean,
+    keepMinutes?: boolean
+  ): this,
+  utcOffset(): number,
+  format(format?: string): string,
+  fromNow(removeSuffix?: boolean): string,
+  from(
+    value: moment$Moment | string | number | Date | Array<number>,
+    removePrefix?: boolean
+  ): string,
+  toNow(removePrefix?: boolean): string,
+  to(
+    value: moment$Moment | string | number | Date | Array<number>,
+    removePrefix?: boolean
+  ): string,
+  calendar(refTime?: any, formats?: moment$CalendarFormats): string,
+  diff(
+    date: moment$Moment | string | number | Date | Array<number>,
+    format?: string,
+    floating?: boolean
+  ): number,
+  valueOf(): number,
+  unix(): number,
+  daysInMonth(): number,
+  toDate(): Date,
+  toArray(): Array<number>,
+  toJSON(): string,
+  toISOString(
+    keepOffset?: boolean
+  ): string,
+  toObject(): moment$MomentObject,
+  isBefore(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSame(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isAfter(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSameOrBefore(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSameOrAfter(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isBetween(
+    fromDate: moment$Moment | string | number | Date | Array<number>,
+    toDate?: ?moment$Moment | string | number | Date | Array<number>,
+    granularity?: ?string,
+    inclusion?: ?string
+  ): boolean,
+  isDST(): boolean,
+  isDSTShifted(): boolean,
+  isLeapYear(): boolean,
+  clone(): moment$Moment,
+  static isMoment(obj: any): boolean,
+  static isDate(obj: any): boolean,
+  static locale(locale: string, localeData?: Object): string,
+  static updateLocale(locale: string, localeData?: ?Object): void,
+  static locale(locales: Array<string>): string,
+  locale(locale: string, customization?: Object | null): moment$Moment,
+  locale(): string,
+  static months(): Array<string>,
+  static monthsShort(): Array<string>,
+  static weekdays(): Array<string>,
+  static weekdaysShort(): Array<string>,
+  static weekdaysMin(): Array<string>,
+  static months(): string,
+  static monthsShort(): string,
+  static weekdays(): string,
+  static weekdaysShort(): string,
+  static weekdaysMin(): string,
+  static localeData(key?: string): moment$LocaleData,
+  static duration(
+    value: number | Object | string,
+    unit?: string
+  ): moment$MomentDuration,
+  static isDuration(obj: any): boolean,
+  static normalizeUnits(unit: string): string,
+  static invalid(object: any): moment$Moment
+}
+
+declare module "moment" {
+  declare module.exports: Class<moment$Moment>;
+}

scm-plugins/scm-git-plugin/flow-typed/npm/react-jss_vx.x.x.js

@@ -0,0 +1,137 @@
+// flow-typed signature: ba35d02d668b0d0a3e04a63a6847974e
+// flow-typed version: <<STUB>>/react-jss_v8.6.1/flow_v0.79.1
+
+/**
+ * This is an autogenerated libdef stub for:
+ *
+ *   'react-jss'
+ *
+ * Fill this stub out by replacing all the `any` types.
+ *
+ * Once filled out, we encourage you to share your work with the
+ * community by sending a pull request to:
+ * https://github.com/flowtype/flow-typed
+ */
+
+declare module 'react-jss' {
+  declare module.exports: any;
+}
+
+/**
+ * We include stubs for each file inside this npm package in case you need to
+ * require those files directly. Feel free to delete any files that aren't
+ * needed.
+ */
+declare module 'react-jss/dist/react-jss' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/dist/react-jss.min' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/compose' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/compose.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/contextTypes' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/createHoc' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/getDisplayName' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/index' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/index.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/injectSheet' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/injectSheet.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/jss' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/JssProvider' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/JssProvider.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/ns' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/propTypes' {
+  declare module.exports: any;
+}
+
+// Filename aliases
+declare module 'react-jss/dist/react-jss.js' {
+  declare module.exports: $Exports<'react-jss/dist/react-jss'>;
+}
+declare module 'react-jss/dist/react-jss.min.js' {
+  declare module.exports: $Exports<'react-jss/dist/react-jss.min'>;
+}
+declare module 'react-jss/lib/compose.js' {
+  declare module.exports: $Exports<'react-jss/lib/compose'>;
+}
+declare module 'react-jss/lib/compose.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/compose.test'>;
+}
+declare module 'react-jss/lib/contextTypes.js' {
+  declare module.exports: $Exports<'react-jss/lib/contextTypes'>;
+}
+declare module 'react-jss/lib/createHoc.js' {
+  declare module.exports: $Exports<'react-jss/lib/createHoc'>;
+}
+declare module 'react-jss/lib/getDisplayName.js' {
+  declare module.exports: $Exports<'react-jss/lib/getDisplayName'>;
+}
+declare module 'react-jss/lib/index.js' {
+  declare module.exports: $Exports<'react-jss/lib/index'>;
+}
+declare module 'react-jss/lib/index.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/index.test'>;
+}
+declare module 'react-jss/lib/injectSheet.js' {
+  declare module.exports: $Exports<'react-jss/lib/injectSheet'>;
+}
+declare module 'react-jss/lib/injectSheet.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/injectSheet.test'>;
+}
+declare module 'react-jss/lib/jss.js' {
+  declare module.exports: $Exports<'react-jss/lib/jss'>;
+}
+declare module 'react-jss/lib/JssProvider.js' {
+  declare module.exports: $Exports<'react-jss/lib/JssProvider'>;
+}
+declare module 'react-jss/lib/JssProvider.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/JssProvider.test'>;
+}
+declare module 'react-jss/lib/ns.js' {
+  declare module.exports: $Exports<'react-jss/lib/ns'>;
+}
+declare module 'react-jss/lib/propTypes.js' {
+  declare module.exports: $Exports<'react-jss/lib/propTypes'>;
+}

scm-plugins/scm-git-plugin/package.json

@@ -1,5 +1,6 @@
 {
   "name": "@scm-manager/scm-git-plugin",
+  "license" : "BSD-3-Clause",
   "main": "src/main/js/index.js",
   "scripts": {
     "build": "ui-bundler plugin"
@@ -8,6 +9,6 @@
     "@scm-manager/ui-extensions": "^0.0.7"
   },
   "devDependencies": {
-    "@scm-manager/ui-bundler": "^0.0.7"
+    "@scm-manager/ui-bundler": "^0.0.13"
   }
 }

scm-plugins/scm-git-plugin/pom.xml

@@ -61,6 +61,33 @@
         </executions>
       </plugin>
 
+      <plugin>
+        <groupId>com.github.sdorra</groupId>
+        <artifactId>buildfrontend-maven-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>link-ui-types</id>
+            <phase>process-sources</phase>
+            <goals>
+              <goal>install-link</goal>
+            </goals>
+            <configuration>
+              <pkg>@scm-manager/ui-types</pkg>
+            </configuration>
+          </execution>
+          <execution>
+            <id>link-ui-components</id>
+            <phase>process-sources</phase>
+            <goals>
+              <goal>install-link</goal>
+            </goals>
+            <configuration>
+              <pkg>@scm-manager/ui-components</pkg>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
+
     </plugins>
   </build>
 

scm-plugins/scm-git-plugin/src/main/js/GitAvatar.js

@@ -1,5 +1,6 @@
 //@flow
-import React from 'react';
+import React from "react";
+import { Image } from "@scm-manager/ui-components";
 
 type Props = {
 };
@@ -7,8 +8,7 @@ type Props = {
 class GitAvatar extends React.Component<Props> {
 
   render() {
-    // TODO we have to use Image from ui-components
-    return <img src="/scm/images/git-logo.png" alt="Git Logo" />;
+    return <Image src="/images/git-logo.png" alt="Git Logo" />;
   }
 
 }

scm-plugins/scm-git-plugin/src/main/js/ProtocolInformation.js

@@ -1,9 +1,9 @@
 //@flow
-import React from 'react';
+import React from "react";
+import type { Repository } from "@scm-manager/ui-types";
 
-// TODO flow types ???
 type Props = {
-  repository: Object
+  repository: Repository
 }
 
 class ProtocolInformation extends React.Component<Props> {

scm-plugins/scm-git-plugin/src/main/js/index.js

@@ -1,6 +1,7 @@
+//@flow
 import { binder } from "@scm-manager/ui-extensions";
-import ProtocolInformation from './ProtocolInformation';
-import GitAvatar from './GitAvatar';
+import ProtocolInformation from "./ProtocolInformation";
+import GitAvatar from "./GitAvatar";
 
 const gitPredicate = (props: Object) => {
   return props.repository && props.repository.type === "git";

scm-plugins/scm-git-plugin/src/test/java/sonia/scm/repository/client/spi/GitPushCommand.java

@@ -37,12 +37,12 @@ package sonia.scm.repository.client.spi;
 import org.eclipse.jgit.api.Git;
 import org.eclipse.jgit.api.errors.GitAPIException;
 import org.eclipse.jgit.transport.CredentialsProvider;
-
 import sonia.scm.repository.client.api.RepositoryClientException;
 
-//~--- JDK imports ------------------------------------------------------------
-
 import java.io.IOException;
+import java.util.function.Supplier;
+
+//~--- JDK imports ------------------------------------------------------------
 
 /**
  *
@@ -73,11 +73,20 @@ public class GitPushCommand implements PushCommand
    * @throws IOException
    */
   @Override
-  public void push() throws IOException
+  public void push() throws IOException {
+    push(() -> git.push().setPushAll());
+  }
+
+  @Override
+  public void pushTags() throws IOException {
+    push(() -> git.push().setPushTags());
+  }
+
+  private void push(Supplier<org.eclipse.jgit.api.PushCommand> commandSupplier) throws RepositoryClientException
   {
     try
     {
-      org.eclipse.jgit.api.PushCommand cmd = git.push().setPushAll();
+      org.eclipse.jgit.api.PushCommand cmd = commandSupplier.get();
 
       if (credentialsProvider != null)
       {

scm-plugins/scm-git-plugin/src/test/java/sonia/scm/repository/client/spi/GitTagCommand.java

@@ -35,22 +35,20 @@ package sonia.scm.repository.client.spi;
 //~--- non-JDK imports --------------------------------------------------------
 
 import com.google.common.base.Strings;
-
 import org.eclipse.jgit.api.Git;
 import org.eclipse.jgit.api.errors.GitAPIException;
 import org.eclipse.jgit.lib.ObjectId;
 import org.eclipse.jgit.lib.Ref;
 import org.eclipse.jgit.revwalk.RevObject;
 import org.eclipse.jgit.revwalk.RevWalk;
-
 import sonia.scm.repository.GitUtil;
 import sonia.scm.repository.Tag;
 import sonia.scm.repository.client.api.RepositoryClientException;
 
-//~--- JDK imports ------------------------------------------------------------
-
 import java.io.IOException;
 
+//~--- JDK imports ------------------------------------------------------------
+
 /**
  *
  * @author Sebastian Sdorra
@@ -119,7 +117,11 @@ public class GitTagCommand implements TagCommand
         ref = git.tag().setObjectId(revObject).call();
       }
 
-      tag = new Tag(request.getName(), ref.getPeeledObjectId().toString());
+      if (ref.isPeeled()) {
+        tag = new Tag(request.getName(), ref.getPeeledObjectId().toString());
+      } else {
+        tag = new Tag(request.getName(), ref.getObjectId().toString());
+      }
 
     }
     catch (GitAPIException ex)

scm-plugins/scm-hg-plugin/.babelrc

@@ -0,0 +1,6 @@
+{
+  "presets": ["@babel/preset-env", "@babel/preset-react", "@babel/preset-flow"],
+  "plugins": [
+    "@babel/plugin-proposal-class-properties"
+  ]
+}

scm-plugins/scm-hg-plugin/.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "@scm-manager/eslint-config"
+}

scm-plugins/scm-hg-plugin/.flowconfig

@@ -0,0 +1,8 @@
+[ignore]
+.*/node_modules/module-deps/.*
+
+[include]
+
+[libs]
+
+[options]

scm-plugins/scm-hg-plugin/flow-typed/npm/classnames_v2.x.x.js

@@ -0,0 +1,23 @@
+// flow-typed signature: cf86673cc32d185bdab1d2ea90578d37
+// flow-typed version: 614bf49aa8/classnames_v2.x.x/flow_>=v0.25.x
+
+type $npm$classnames$Classes =
+  | string
+  | { [className: string]: * }
+  | false
+  | void
+  | null;
+
+declare module "classnames" {
+  declare module.exports: (
+    ...classes: Array<$npm$classnames$Classes | $npm$classnames$Classes[]>
+  ) => string;
+}
+
+declare module "classnames/bind" {
+  declare module.exports: $Exports<"classnames">;
+}
+
+declare module "classnames/dedupe" {
+  declare module.exports: $Exports<"classnames">;
+}

scm-plugins/scm-hg-plugin/flow-typed/npm/moment_v2.3.x.js

@@ -0,0 +1,331 @@
+// flow-typed signature: 23b805356f90ad9384dd88489654e380
+// flow-typed version: e9374c5fe9/moment_v2.3.x/flow_>=v0.25.x
+
+type moment$MomentOptions = {
+  y?: number | string,
+  year?: number | string,
+  years?: number | string,
+  M?: number | string,
+  month?: number | string,
+  months?: number | string,
+  d?: number | string,
+  day?: number | string,
+  days?: number | string,
+  date?: number | string,
+  h?: number | string,
+  hour?: number | string,
+  hours?: number | string,
+  m?: number | string,
+  minute?: number | string,
+  minutes?: number | string,
+  s?: number | string,
+  second?: number | string,
+  seconds?: number | string,
+  ms?: number | string,
+  millisecond?: number | string,
+  milliseconds?: number | string
+};
+
+type moment$MomentObject = {
+  years: number,
+  months: number,
+  date: number,
+  hours: number,
+  minutes: number,
+  seconds: number,
+  milliseconds: number
+};
+
+type moment$MomentCreationData = {
+  input: string,
+  format: string,
+  locale: Object,
+  isUTC: boolean,
+  strict: boolean
+};
+
+type moment$CalendarFormat = string | ((moment: moment$Moment) => string);
+
+type moment$CalendarFormats = {
+  sameDay?: moment$CalendarFormat,
+  nextDay?: moment$CalendarFormat,
+  nextWeek?: moment$CalendarFormat,
+  lastDay?: moment$CalendarFormat,
+  lastWeek?: moment$CalendarFormat,
+  sameElse?: moment$CalendarFormat
+};
+
+declare class moment$LocaleData {
+  months(moment: moment$Moment): string,
+  monthsShort(moment: moment$Moment): string,
+  monthsParse(month: string): number,
+  weekdays(moment: moment$Moment): string,
+  weekdaysShort(moment: moment$Moment): string,
+  weekdaysMin(moment: moment$Moment): string,
+  weekdaysParse(weekDay: string): number,
+  longDateFormat(dateFormat: string): string,
+  isPM(date: string): boolean,
+  meridiem(hours: number, minutes: number, isLower: boolean): string,
+  calendar(
+    key:
+      | "sameDay"
+      | "nextDay"
+      | "lastDay"
+      | "nextWeek"
+      | "prevWeek"
+      | "sameElse",
+    moment: moment$Moment
+  ): string,
+  relativeTime(
+    number: number,
+    withoutSuffix: boolean,
+    key: "s" | "m" | "mm" | "h" | "hh" | "d" | "dd" | "M" | "MM" | "y" | "yy",
+    isFuture: boolean
+  ): string,
+  pastFuture(diff: any, relTime: string): string,
+  ordinal(number: number): string,
+  preparse(str: string): any,
+  postformat(str: string): any,
+  week(moment: moment$Moment): string,
+  invalidDate(): string,
+  firstDayOfWeek(): number,
+  firstDayOfYear(): number
+}
+declare class moment$MomentDuration {
+  humanize(suffix?: boolean): string,
+  milliseconds(): number,
+  asMilliseconds(): number,
+  seconds(): number,
+  asSeconds(): number,
+  minutes(): number,
+  asMinutes(): number,
+  hours(): number,
+  asHours(): number,
+  days(): number,
+  asDays(): number,
+  months(): number,
+  asWeeks(): number,
+  weeks(): number,
+  asMonths(): number,
+  years(): number,
+  asYears(): number,
+  add(value: number | moment$MomentDuration | Object, unit?: string): this,
+  subtract(value: number | moment$MomentDuration | Object, unit?: string): this,
+  as(unit: string): number,
+  get(unit: string): number,
+  toJSON(): string,
+  toISOString(): string,
+  isValid(): boolean
+}
+declare class moment$Moment {
+  static ISO_8601: string,
+  static (
+    string?: string,
+    format?: string | Array<string>,
+    strict?: boolean
+  ): moment$Moment,
+  static (
+    string?: string,
+    format?: string | Array<string>,
+    locale?: string,
+    strict?: boolean
+  ): moment$Moment,
+  static (
+    initDate: ?Object | number | Date | Array<number> | moment$Moment | string
+  ): moment$Moment,
+  static unix(seconds: number): moment$Moment,
+  static utc(): moment$Moment,
+  static utc(number: number | Array<number>): moment$Moment,
+  static utc(
+    str: string,
+    str2?: string | Array<string>,
+    str3?: string
+  ): moment$Moment,
+  static utc(moment: moment$Moment): moment$Moment,
+  static utc(date: Date): moment$Moment,
+  static parseZone(): moment$Moment,
+  static parseZone(rawDate: string): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string | Array<string>
+  ): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string,
+    strict: boolean
+  ): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string,
+    locale: string,
+    strict: boolean
+  ): moment$Moment,
+  isValid(): boolean,
+  invalidAt(): 0 | 1 | 2 | 3 | 4 | 5 | 6,
+  creationData(): moment$MomentCreationData,
+  millisecond(number: number): this,
+  milliseconds(number: number): this,
+  millisecond(): number,
+  milliseconds(): number,
+  second(number: number): this,
+  seconds(number: number): this,
+  second(): number,
+  seconds(): number,
+  minute(number: number): this,
+  minutes(number: number): this,
+  minute(): number,
+  minutes(): number,
+  hour(number: number): this,
+  hours(number: number): this,
+  hour(): number,
+  hours(): number,
+  date(number: number): this,
+  dates(number: number): this,
+  date(): number,
+  dates(): number,
+  day(day: number | string): this,
+  days(day: number | string): this,
+  day(): number,
+  days(): number,
+  weekday(number: number): this,
+  weekday(): number,
+  isoWeekday(number: number): this,
+  isoWeekday(): number,
+  dayOfYear(number: number): this,
+  dayOfYear(): number,
+  week(number: number): this,
+  weeks(number: number): this,
+  week(): number,
+  weeks(): number,
+  isoWeek(number: number): this,
+  isoWeeks(number: number): this,
+  isoWeek(): number,
+  isoWeeks(): number,
+  month(number: number): this,
+  months(number: number): this,
+  month(): number,
+  months(): number,
+  quarter(number: number): this,
+  quarter(): number,
+  year(number: number): this,
+  years(number: number): this,
+  year(): number,
+  years(): number,
+  weekYear(number: number): this,
+  weekYear(): number,
+  isoWeekYear(number: number): this,
+  isoWeekYear(): number,
+  weeksInYear(): number,
+  isoWeeksInYear(): number,
+  get(string: string): number,
+  set(unit: string, value: number): this,
+  set(options: { [unit: string]: number }): this,
+  static max(...dates: Array<moment$Moment>): moment$Moment,
+  static max(dates: Array<moment$Moment>): moment$Moment,
+  static min(...dates: Array<moment$Moment>): moment$Moment,
+  static min(dates: Array<moment$Moment>): moment$Moment,
+  add(
+    value: number | moment$MomentDuration | moment$Moment | Object,
+    unit?: string
+  ): this,
+  subtract(
+    value: number | moment$MomentDuration | moment$Moment | string | Object,
+    unit?: string
+  ): this,
+  startOf(unit: string): this,
+  endOf(unit: string): this,
+  local(): this,
+  utc(): this,
+  utcOffset(
+    offset: number | string,
+    keepLocalTime?: boolean,
+    keepMinutes?: boolean
+  ): this,
+  utcOffset(): number,
+  format(format?: string): string,
+  fromNow(removeSuffix?: boolean): string,
+  from(
+    value: moment$Moment | string | number | Date | Array<number>,
+    removePrefix?: boolean
+  ): string,
+  toNow(removePrefix?: boolean): string,
+  to(
+    value: moment$Moment | string | number | Date | Array<number>,
+    removePrefix?: boolean
+  ): string,
+  calendar(refTime?: any, formats?: moment$CalendarFormats): string,
+  diff(
+    date: moment$Moment | string | number | Date | Array<number>,
+    format?: string,
+    floating?: boolean
+  ): number,
+  valueOf(): number,
+  unix(): number,
+  daysInMonth(): number,
+  toDate(): Date,
+  toArray(): Array<number>,
+  toJSON(): string,
+  toISOString(
+    keepOffset?: boolean
+  ): string,
+  toObject(): moment$MomentObject,
+  isBefore(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSame(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isAfter(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSameOrBefore(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSameOrAfter(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isBetween(
+    fromDate: moment$Moment | string | number | Date | Array<number>,
+    toDate?: ?moment$Moment | string | number | Date | Array<number>,
+    granularity?: ?string,
+    inclusion?: ?string
+  ): boolean,
+  isDST(): boolean,
+  isDSTShifted(): boolean,
+  isLeapYear(): boolean,
+  clone(): moment$Moment,
+  static isMoment(obj: any): boolean,
+  static isDate(obj: any): boolean,
+  static locale(locale: string, localeData?: Object): string,
+  static updateLocale(locale: string, localeData?: ?Object): void,
+  static locale(locales: Array<string>): string,
+  locale(locale: string, customization?: Object | null): moment$Moment,
+  locale(): string,
+  static months(): Array<string>,
+  static monthsShort(): Array<string>,
+  static weekdays(): Array<string>,
+  static weekdaysShort(): Array<string>,
+  static weekdaysMin(): Array<string>,
+  static months(): string,
+  static monthsShort(): string,
+  static weekdays(): string,
+  static weekdaysShort(): string,
+  static weekdaysMin(): string,
+  static localeData(key?: string): moment$LocaleData,
+  static duration(
+    value: number | Object | string,
+    unit?: string
+  ): moment$MomentDuration,
+  static isDuration(obj: any): boolean,
+  static normalizeUnits(unit: string): string,
+  static invalid(object: any): moment$Moment
+}
+
+declare module "moment" {
+  declare module.exports: Class<moment$Moment>;
+}

scm-plugins/scm-hg-plugin/flow-typed/npm/react-jss_vx.x.x.js

@@ -0,0 +1,137 @@
+// flow-typed signature: ba35d02d668b0d0a3e04a63a6847974e
+// flow-typed version: <<STUB>>/react-jss_v8.6.1/flow_v0.79.1
+
+/**
+ * This is an autogenerated libdef stub for:
+ *
+ *   'react-jss'
+ *
+ * Fill this stub out by replacing all the `any` types.
+ *
+ * Once filled out, we encourage you to share your work with the
+ * community by sending a pull request to:
+ * https://github.com/flowtype/flow-typed
+ */
+
+declare module 'react-jss' {
+  declare module.exports: any;
+}
+
+/**
+ * We include stubs for each file inside this npm package in case you need to
+ * require those files directly. Feel free to delete any files that aren't
+ * needed.
+ */
+declare module 'react-jss/dist/react-jss' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/dist/react-jss.min' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/compose' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/compose.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/contextTypes' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/createHoc' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/getDisplayName' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/index' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/index.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/injectSheet' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/injectSheet.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/jss' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/JssProvider' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/JssProvider.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/ns' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/propTypes' {
+  declare module.exports: any;
+}
+
+// Filename aliases
+declare module 'react-jss/dist/react-jss.js' {
+  declare module.exports: $Exports<'react-jss/dist/react-jss'>;
+}
+declare module 'react-jss/dist/react-jss.min.js' {
+  declare module.exports: $Exports<'react-jss/dist/react-jss.min'>;
+}
+declare module 'react-jss/lib/compose.js' {
+  declare module.exports: $Exports<'react-jss/lib/compose'>;
+}
+declare module 'react-jss/lib/compose.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/compose.test'>;
+}
+declare module 'react-jss/lib/contextTypes.js' {
+  declare module.exports: $Exports<'react-jss/lib/contextTypes'>;
+}
+declare module 'react-jss/lib/createHoc.js' {
+  declare module.exports: $Exports<'react-jss/lib/createHoc'>;
+}
+declare module 'react-jss/lib/getDisplayName.js' {
+  declare module.exports: $Exports<'react-jss/lib/getDisplayName'>;
+}
+declare module 'react-jss/lib/index.js' {
+  declare module.exports: $Exports<'react-jss/lib/index'>;
+}
+declare module 'react-jss/lib/index.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/index.test'>;
+}
+declare module 'react-jss/lib/injectSheet.js' {
+  declare module.exports: $Exports<'react-jss/lib/injectSheet'>;
+}
+declare module 'react-jss/lib/injectSheet.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/injectSheet.test'>;
+}
+declare module 'react-jss/lib/jss.js' {
+  declare module.exports: $Exports<'react-jss/lib/jss'>;
+}
+declare module 'react-jss/lib/JssProvider.js' {
+  declare module.exports: $Exports<'react-jss/lib/JssProvider'>;
+}
+declare module 'react-jss/lib/JssProvider.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/JssProvider.test'>;
+}
+declare module 'react-jss/lib/ns.js' {
+  declare module.exports: $Exports<'react-jss/lib/ns'>;
+}
+declare module 'react-jss/lib/propTypes.js' {
+  declare module.exports: $Exports<'react-jss/lib/propTypes'>;
+}

scm-plugins/scm-hg-plugin/package.json

@@ -1,6 +1,7 @@
 {
   "name": "@scm-manager/scm-hg-plugin",
   "main": "src/main/js/index.js",
+  "license" : "BSD-3-Clause",
   "scripts": {
     "build": "ui-bundler plugin"
   },
@@ -8,6 +9,6 @@
     "@scm-manager/ui-extensions": "^0.0.7"
   },
   "devDependencies": {
-    "@scm-manager/ui-bundler": "^0.0.7"
+    "@scm-manager/ui-bundler": "^0.0.13"
   }
 }

scm-plugins/scm-hg-plugin/pom.xml

@@ -67,7 +67,35 @@
           </execution>
         </executions>
       </plugin>
-        
+
+      <plugin>
+        <groupId>com.github.sdorra</groupId>
+        <artifactId>buildfrontend-maven-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>link-ui-types</id>
+            <phase>process-sources</phase>
+            <goals>
+              <goal>install-link</goal>
+            </goals>
+            <configuration>
+              <pkg>@scm-manager/ui-types</pkg>
+            </configuration>
+          </execution>
+          <execution>
+            <id>link-ui-components</id>
+            <phase>process-sources</phase>
+            <goals>
+              <goal>install-link</goal>
+            </goals>
+            <configuration>
+              <pkg>@scm-manager/ui-components</pkg>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
+
+
     </plugins>
   </build>
   

scm-plugins/scm-hg-plugin/src/main/js/HgAvatar.js

@@ -1,5 +1,6 @@
 //@flow
-import React from 'react';
+import React from "react";
+import { Image } from "@scm-manager/ui-components";
 
 type Props = {
 };
@@ -7,8 +8,7 @@ type Props = {
 class HgAvatar extends React.Component<Props> {
 
   render() {
-    // TODO we have to use Image from ui-components
-    return <img src="/scm/images/hg-logo.png" alt="Mercurial Logo" />;
+    return <Image src="/scm/images/hg-logo.png" alt="Mercurial Logo" />;
   }
 
 }

scm-plugins/scm-hg-plugin/src/main/js/ProtocolInformation.js

@@ -1,9 +1,9 @@
 //@flow
-import React from 'react';
+import React from "react";
+import type { Repository } from "@scm-manager/ui-types";
 
-// TODO flow types ???
 type Props = {
-  repository: Object
+  repository: Repository
 }
 
 class ProtocolInformation extends React.Component<Props> {

scm-plugins/scm-hg-plugin/src/main/js/index.js

@@ -1,6 +1,7 @@
+// @flow
 import { binder } from "@scm-manager/ui-extensions";
-import ProtocolInformation from './ProtocolInformation';
-import HgAvatar from './HgAvatar';
+import ProtocolInformation from "./ProtocolInformation";
+import HgAvatar from "./HgAvatar";
 
 const hgPredicate = (props: Object) => {
   return props.repository && props.repository.type === "hg";

scm-plugins/scm-hg-plugin/src/test/java/sonia/scm/repository/client/spi/HgPushCommand.java

@@ -32,9 +32,10 @@ package sonia.scm.repository.client.spi;
 
 import com.aragost.javahg.Repository;
 import com.aragost.javahg.commands.ExecutionException;
-import java.io.IOException;
 import sonia.scm.repository.client.api.RepositoryClientException;
 
+import java.io.IOException;
+
 /**
  * Mercurial implementation of the {@link PushCommand}.
  *
@@ -63,5 +64,10 @@ public class HgPushCommand implements PushCommand
       throw new RepositoryClientException("push to repository failed", ex);
     }
   }
-  
+
+  @Override
+  public void pushTags() throws IOException {
+    push();
+  }
+
 }

scm-plugins/scm-hg-plugin/src/test/java/sonia/scm/repository/client/spi/HgTagCommand.java

@@ -32,7 +32,6 @@ package sonia.scm.repository.client.spi;
 
 import com.aragost.javahg.Repository;
 import com.google.common.base.Strings;
-import java.io.IOException;
 import sonia.scm.repository.Tag;
 
 /**
@@ -51,13 +50,16 @@ public class HgTagCommand implements TagCommand
   }
 
   @Override
-  public Tag tag(TagRequest request) throws IOException
+  public Tag tag(TagRequest request)
   {
     String rev = request.getRevision();
     if ( Strings.isNullOrEmpty(rev) ){
       rev = repository.tip().getNode();
     }
-    com.aragost.javahg.commands.TagCommand.on(repository).rev(rev).execute(request.getName());
+    com.aragost.javahg.commands.TagCommand.on(repository)
+      .rev(rev)
+      .user(request.getUserName())
+      .execute(request.getName());
     return new Tag(request.getName(), rev);
   }
   

scm-plugins/scm-svn-plugin/.babelrc

@@ -0,0 +1,6 @@
+{
+  "presets": ["@babel/preset-env", "@babel/preset-react", "@babel/preset-flow"],
+  "plugins": [
+    "@babel/plugin-proposal-class-properties"
+  ]
+}

scm-plugins/scm-svn-plugin/.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "@scm-manager/eslint-config"
+}

scm-plugins/scm-svn-plugin/.flowconfig

@@ -0,0 +1,8 @@
+[ignore]
+.*/node_modules/module-deps/.*
+
+[include]
+
+[libs]
+
+[options]

scm-plugins/scm-svn-plugin/flow-typed/npm/classnames_v2.x.x.js

@@ -0,0 +1,23 @@
+// flow-typed signature: cf86673cc32d185bdab1d2ea90578d37
+// flow-typed version: 614bf49aa8/classnames_v2.x.x/flow_>=v0.25.x
+
+type $npm$classnames$Classes =
+  | string
+  | { [className: string]: * }
+  | false
+  | void
+  | null;
+
+declare module "classnames" {
+  declare module.exports: (
+    ...classes: Array<$npm$classnames$Classes | $npm$classnames$Classes[]>
+  ) => string;
+}
+
+declare module "classnames/bind" {
+  declare module.exports: $Exports<"classnames">;
+}
+
+declare module "classnames/dedupe" {
+  declare module.exports: $Exports<"classnames">;
+}

scm-plugins/scm-svn-plugin/flow-typed/npm/moment_v2.3.x.js

@@ -0,0 +1,331 @@
+// flow-typed signature: 23b805356f90ad9384dd88489654e380
+// flow-typed version: e9374c5fe9/moment_v2.3.x/flow_>=v0.25.x
+
+type moment$MomentOptions = {
+  y?: number | string,
+  year?: number | string,
+  years?: number | string,
+  M?: number | string,
+  month?: number | string,
+  months?: number | string,
+  d?: number | string,
+  day?: number | string,
+  days?: number | string,
+  date?: number | string,
+  h?: number | string,
+  hour?: number | string,
+  hours?: number | string,
+  m?: number | string,
+  minute?: number | string,
+  minutes?: number | string,
+  s?: number | string,
+  second?: number | string,
+  seconds?: number | string,
+  ms?: number | string,
+  millisecond?: number | string,
+  milliseconds?: number | string
+};
+
+type moment$MomentObject = {
+  years: number,
+  months: number,
+  date: number,
+  hours: number,
+  minutes: number,
+  seconds: number,
+  milliseconds: number
+};
+
+type moment$MomentCreationData = {
+  input: string,
+  format: string,
+  locale: Object,
+  isUTC: boolean,
+  strict: boolean
+};
+
+type moment$CalendarFormat = string | ((moment: moment$Moment) => string);
+
+type moment$CalendarFormats = {
+  sameDay?: moment$CalendarFormat,
+  nextDay?: moment$CalendarFormat,
+  nextWeek?: moment$CalendarFormat,
+  lastDay?: moment$CalendarFormat,
+  lastWeek?: moment$CalendarFormat,
+  sameElse?: moment$CalendarFormat
+};
+
+declare class moment$LocaleData {
+  months(moment: moment$Moment): string,
+  monthsShort(moment: moment$Moment): string,
+  monthsParse(month: string): number,
+  weekdays(moment: moment$Moment): string,
+  weekdaysShort(moment: moment$Moment): string,
+  weekdaysMin(moment: moment$Moment): string,
+  weekdaysParse(weekDay: string): number,
+  longDateFormat(dateFormat: string): string,
+  isPM(date: string): boolean,
+  meridiem(hours: number, minutes: number, isLower: boolean): string,
+  calendar(
+    key:
+      | "sameDay"
+      | "nextDay"
+      | "lastDay"
+      | "nextWeek"
+      | "prevWeek"
+      | "sameElse",
+    moment: moment$Moment
+  ): string,
+  relativeTime(
+    number: number,
+    withoutSuffix: boolean,
+    key: "s" | "m" | "mm" | "h" | "hh" | "d" | "dd" | "M" | "MM" | "y" | "yy",
+    isFuture: boolean
+  ): string,
+  pastFuture(diff: any, relTime: string): string,
+  ordinal(number: number): string,
+  preparse(str: string): any,
+  postformat(str: string): any,
+  week(moment: moment$Moment): string,
+  invalidDate(): string,
+  firstDayOfWeek(): number,
+  firstDayOfYear(): number
+}
+declare class moment$MomentDuration {
+  humanize(suffix?: boolean): string,
+  milliseconds(): number,
+  asMilliseconds(): number,
+  seconds(): number,
+  asSeconds(): number,
+  minutes(): number,
+  asMinutes(): number,
+  hours(): number,
+  asHours(): number,
+  days(): number,
+  asDays(): number,
+  months(): number,
+  asWeeks(): number,
+  weeks(): number,
+  asMonths(): number,
+  years(): number,
+  asYears(): number,
+  add(value: number | moment$MomentDuration | Object, unit?: string): this,
+  subtract(value: number | moment$MomentDuration | Object, unit?: string): this,
+  as(unit: string): number,
+  get(unit: string): number,
+  toJSON(): string,
+  toISOString(): string,
+  isValid(): boolean
+}
+declare class moment$Moment {
+  static ISO_8601: string,
+  static (
+    string?: string,
+    format?: string | Array<string>,
+    strict?: boolean
+  ): moment$Moment,
+  static (
+    string?: string,
+    format?: string | Array<string>,
+    locale?: string,
+    strict?: boolean
+  ): moment$Moment,
+  static (
+    initDate: ?Object | number | Date | Array<number> | moment$Moment | string
+  ): moment$Moment,
+  static unix(seconds: number): moment$Moment,
+  static utc(): moment$Moment,
+  static utc(number: number | Array<number>): moment$Moment,
+  static utc(
+    str: string,
+    str2?: string | Array<string>,
+    str3?: string
+  ): moment$Moment,
+  static utc(moment: moment$Moment): moment$Moment,
+  static utc(date: Date): moment$Moment,
+  static parseZone(): moment$Moment,
+  static parseZone(rawDate: string): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string | Array<string>
+  ): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string,
+    strict: boolean
+  ): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string,
+    locale: string,
+    strict: boolean
+  ): moment$Moment,
+  isValid(): boolean,
+  invalidAt(): 0 | 1 | 2 | 3 | 4 | 5 | 6,
+  creationData(): moment$MomentCreationData,
+  millisecond(number: number): this,
+  milliseconds(number: number): this,
+  millisecond(): number,
+  milliseconds(): number,
+  second(number: number): this,
+  seconds(number: number): this,
+  second(): number,
+  seconds(): number,
+  minute(number: number): this,
+  minutes(number: number): this,
+  minute(): number,
+  minutes(): number,
+  hour(number: number): this,
+  hours(number: number): this,
+  hour(): number,
+  hours(): number,
+  date(number: number): this,
+  dates(number: number): this,
+  date(): number,
+  dates(): number,
+  day(day: number | string): this,
+  days(day: number | string): this,
+  day(): number,
+  days(): number,
+  weekday(number: number): this,
+  weekday(): number,
+  isoWeekday(number: number): this,
+  isoWeekday(): number,
+  dayOfYear(number: number): this,
+  dayOfYear(): number,
+  week(number: number): this,
+  weeks(number: number): this,
+  week(): number,
+  weeks(): number,
+  isoWeek(number: number): this,
+  isoWeeks(number: number): this,
+  isoWeek(): number,
+  isoWeeks(): number,
+  month(number: number): this,
+  months(number: number): this,
+  month(): number,
+  months(): number,
+  quarter(number: number): this,
+  quarter(): number,
+  year(number: number): this,
+  years(number: number): this,
+  year(): number,
+  years(): number,
+  weekYear(number: number): this,
+  weekYear(): number,
+  isoWeekYear(number: number): this,
+  isoWeekYear(): number,
+  weeksInYear(): number,
+  isoWeeksInYear(): number,
+  get(string: string): number,
+  set(unit: string, value: number): this,
+  set(options: { [unit: string]: number }): this,
+  static max(...dates: Array<moment$Moment>): moment$Moment,
+  static max(dates: Array<moment$Moment>): moment$Moment,
+  static min(...dates: Array<moment$Moment>): moment$Moment,
+  static min(dates: Array<moment$Moment>): moment$Moment,
+  add(
+    value: number | moment$MomentDuration | moment$Moment | Object,
+    unit?: string
+  ): this,
+  subtract(
+    value: number | moment$MomentDuration | moment$Moment | string | Object,
+    unit?: string
+  ): this,
+  startOf(unit: string): this,
+  endOf(unit: string): this,
+  local(): this,
+  utc(): this,
+  utcOffset(
+    offset: number | string,
+    keepLocalTime?: boolean,
+    keepMinutes?: boolean
+  ): this,
+  utcOffset(): number,
+  format(format?: string): string,
+  fromNow(removeSuffix?: boolean): string,
+  from(
+    value: moment$Moment | string | number | Date | Array<number>,
+    removePrefix?: boolean
+  ): string,
+  toNow(removePrefix?: boolean): string,
+  to(
+    value: moment$Moment | string | number | Date | Array<number>,
+    removePrefix?: boolean
+  ): string,
+  calendar(refTime?: any, formats?: moment$CalendarFormats): string,
+  diff(
+    date: moment$Moment | string | number | Date | Array<number>,
+    format?: string,
+    floating?: boolean
+  ): number,
+  valueOf(): number,
+  unix(): number,
+  daysInMonth(): number,
+  toDate(): Date,
+  toArray(): Array<number>,
+  toJSON(): string,
+  toISOString(
+    keepOffset?: boolean
+  ): string,
+  toObject(): moment$MomentObject,
+  isBefore(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSame(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isAfter(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSameOrBefore(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSameOrAfter(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isBetween(
+    fromDate: moment$Moment | string | number | Date | Array<number>,
+    toDate?: ?moment$Moment | string | number | Date | Array<number>,
+    granularity?: ?string,
+    inclusion?: ?string
+  ): boolean,
+  isDST(): boolean,
+  isDSTShifted(): boolean,
+  isLeapYear(): boolean,
+  clone(): moment$Moment,
+  static isMoment(obj: any): boolean,
+  static isDate(obj: any): boolean,
+  static locale(locale: string, localeData?: Object): string,
+  static updateLocale(locale: string, localeData?: ?Object): void,
+  static locale(locales: Array<string>): string,
+  locale(locale: string, customization?: Object | null): moment$Moment,
+  locale(): string,
+  static months(): Array<string>,
+  static monthsShort(): Array<string>,
+  static weekdays(): Array<string>,
+  static weekdaysShort(): Array<string>,
+  static weekdaysMin(): Array<string>,
+  static months(): string,
+  static monthsShort(): string,
+  static weekdays(): string,
+  static weekdaysShort(): string,
+  static weekdaysMin(): string,
+  static localeData(key?: string): moment$LocaleData,
+  static duration(
+    value: number | Object | string,
+    unit?: string
+  ): moment$MomentDuration,
+  static isDuration(obj: any): boolean,
+  static normalizeUnits(unit: string): string,
+  static invalid(object: any): moment$Moment
+}
+
+declare module "moment" {
+  declare module.exports: Class<moment$Moment>;
+}

scm-plugins/scm-svn-plugin/flow-typed/npm/react-jss_vx.x.x.js

@@ -0,0 +1,137 @@
+// flow-typed signature: ba35d02d668b0d0a3e04a63a6847974e
+// flow-typed version: <<STUB>>/react-jss_v8.6.1/flow_v0.79.1
+
+/**
+ * This is an autogenerated libdef stub for:
+ *
+ *   'react-jss'
+ *
+ * Fill this stub out by replacing all the `any` types.
+ *
+ * Once filled out, we encourage you to share your work with the
+ * community by sending a pull request to:
+ * https://github.com/flowtype/flow-typed
+ */
+
+declare module 'react-jss' {
+  declare module.exports: any;
+}
+
+/**
+ * We include stubs for each file inside this npm package in case you need to
+ * require those files directly. Feel free to delete any files that aren't
+ * needed.
+ */
+declare module 'react-jss/dist/react-jss' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/dist/react-jss.min' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/compose' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/compose.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/contextTypes' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/createHoc' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/getDisplayName' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/index' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/index.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/injectSheet' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/injectSheet.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/jss' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/JssProvider' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/JssProvider.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/ns' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/propTypes' {
+  declare module.exports: any;
+}
+
+// Filename aliases
+declare module 'react-jss/dist/react-jss.js' {
+  declare module.exports: $Exports<'react-jss/dist/react-jss'>;
+}
+declare module 'react-jss/dist/react-jss.min.js' {
+  declare module.exports: $Exports<'react-jss/dist/react-jss.min'>;
+}
+declare module 'react-jss/lib/compose.js' {
+  declare module.exports: $Exports<'react-jss/lib/compose'>;
+}
+declare module 'react-jss/lib/compose.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/compose.test'>;
+}
+declare module 'react-jss/lib/contextTypes.js' {
+  declare module.exports: $Exports<'react-jss/lib/contextTypes'>;
+}
+declare module 'react-jss/lib/createHoc.js' {
+  declare module.exports: $Exports<'react-jss/lib/createHoc'>;
+}
+declare module 'react-jss/lib/getDisplayName.js' {
+  declare module.exports: $Exports<'react-jss/lib/getDisplayName'>;
+}
+declare module 'react-jss/lib/index.js' {
+  declare module.exports: $Exports<'react-jss/lib/index'>;
+}
+declare module 'react-jss/lib/index.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/index.test'>;
+}
+declare module 'react-jss/lib/injectSheet.js' {
+  declare module.exports: $Exports<'react-jss/lib/injectSheet'>;
+}
+declare module 'react-jss/lib/injectSheet.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/injectSheet.test'>;
+}
+declare module 'react-jss/lib/jss.js' {
+  declare module.exports: $Exports<'react-jss/lib/jss'>;
+}
+declare module 'react-jss/lib/JssProvider.js' {
+  declare module.exports: $Exports<'react-jss/lib/JssProvider'>;
+}
+declare module 'react-jss/lib/JssProvider.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/JssProvider.test'>;
+}
+declare module 'react-jss/lib/ns.js' {
+  declare module.exports: $Exports<'react-jss/lib/ns'>;
+}
+declare module 'react-jss/lib/propTypes.js' {
+  declare module.exports: $Exports<'react-jss/lib/propTypes'>;
+}

scm-plugins/scm-svn-plugin/package.json

@@ -1,6 +1,7 @@
 {
   "name": "@scm-manager/scm-svn-plugin",
   "main": "src/main/js/index.js",
+  "license" : "BSD-3-Clause",
   "scripts": {
     "build": "ui-bundler plugin"
   },
@@ -8,6 +9,6 @@
     "@scm-manager/ui-extensions": "^0.0.7"
   },
   "devDependencies": {
-    "@scm-manager/ui-bundler": "^0.0.7"
+    "@scm-manager/ui-bundler": "^0.0.13"
   }
 }

scm-plugins/scm-svn-plugin/pom.xml

@@ -54,7 +54,35 @@
           </execution>
         </executions>
       </plugin>
-        
+
+      <plugin>
+        <groupId>com.github.sdorra</groupId>
+        <artifactId>buildfrontend-maven-plugin</artifactId>
+        <executions>
+          <execution>
+            <id>link-ui-types</id>
+            <phase>process-sources</phase>
+            <goals>
+              <goal>install-link</goal>
+            </goals>
+            <configuration>
+              <pkg>@scm-manager/ui-types</pkg>
+            </configuration>
+          </execution>
+          <execution>
+            <id>link-ui-components</id>
+            <phase>process-sources</phase>
+            <goals>
+              <goal>install-link</goal>
+            </goals>
+            <configuration>
+              <pkg>@scm-manager/ui-components</pkg>
+            </configuration>
+          </execution>
+        </executions>
+      </plugin>
+
+
     </plugins>
   </build>
 

scm-plugins/scm-svn-plugin/src/main/js/ProtocolInformation.js

@@ -1,9 +1,9 @@
 //@flow
-import React from 'react';
+import React from "react";
+import type { Repository } from "@scm-manager/ui-types";
 
-// TODO flow types ???
 type Props = {
-  repository: Object
+  repository: Repository
 }
 
 class ProtocolInformation extends React.Component<Props> {

scm-plugins/scm-svn-plugin/src/main/js/SvnAvatar.js

@@ -1,5 +1,6 @@
 //@flow
-import React from 'react';
+import React from "react";
+import { Image } from "@scm-manager/ui-components";
 
 type Props = {
 };
@@ -7,8 +8,7 @@ type Props = {
 class SvnAvatar extends React.Component<Props> {
 
   render() {
-    // TODO we have to use Image from ui-components
-    return <img src="/scm/images/svn-logo.gif" alt="Subversion Logo" />;
+    return <Image src="/scm/images/svn-logo.gif" alt="Subversion Logo" />;
   }
 
 }

scm-plugins/scm-svn-plugin/src/main/js/index.js

@@ -1,6 +1,7 @@
+// @flow
 import { binder } from "@scm-manager/ui-extensions";
-import ProtocolInformation from './ProtocolInformation';
-import SvnAvatar from './SvnAvatar';
+import ProtocolInformation from "./ProtocolInformation";
+import SvnAvatar from "./SvnAvatar";
 
 const svnPredicate = (props: Object) => {
   return props.repository && props.repository.type === "svn";

scm-test/src/main/java/sonia/scm/repository/client/api/PushCommandBuilder.java

@@ -36,13 +36,12 @@ package sonia.scm.repository.client.api;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-
 import sonia.scm.repository.client.spi.PushCommand;
 
-//~--- JDK imports ------------------------------------------------------------
-
 import java.io.IOException;
 
+//~--- JDK imports ------------------------------------------------------------
+
 /**
  *
  * @author Sebastian Sdorra
@@ -88,6 +87,14 @@ public final class PushCommandBuilder
     command.push();
   }
 
+  public void pushTags() throws IOException {
+    if (logger.isDebugEnabled()) {
+      logger.debug("push tag changes back to main repository");
+    }
+
+    command.pushTags();
+  }
+
   //~--- fields ---------------------------------------------------------------
 
   /** Field description */

scm-test/src/main/java/sonia/scm/repository/client/api/TagCommandBuilder.java

@@ -36,15 +36,14 @@ package sonia.scm.repository.client.api;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-
 import sonia.scm.repository.Tag;
 import sonia.scm.repository.client.spi.TagCommand;
 import sonia.scm.repository.client.spi.TagRequest;
 
-//~--- JDK imports ------------------------------------------------------------
-
 import java.io.IOException;
 
+//~--- JDK imports ------------------------------------------------------------
+
 /**
  *
  * @author Sebastian Sdorra
@@ -84,9 +83,10 @@ public final class TagCommandBuilder
    *
    * @throws IOException
    */
-  public Tag tag(String name) throws IOException
+  public Tag tag(String name, String username) throws IOException
   {
     request.setName(name);
+    request.setUsername(username);
 
     if (logger.isDebugEnabled())
     {

scm-test/src/main/java/sonia/scm/repository/client/spi/PushCommand.java

@@ -44,11 +44,7 @@ import java.io.IOException;
 public interface PushCommand
 {
 
-  /**
-   * Method description
-   *
-   *
-   * @throws IOException
-   */
-  public void push() throws IOException;
+  void push() throws IOException;
+
+  void pushTags() throws IOException;
 }

scm-test/src/main/java/sonia/scm/repository/client/spi/TagRequest.java

@@ -91,6 +91,7 @@ public final class TagRequest
   {
     this.name = null;
     this.revision = null;
+    this.username = null;
   }
 
   /**
@@ -106,6 +107,7 @@ public final class TagRequest
     return MoreObjects.toStringHelper(this)
                   .add("revision", revision)
                   .add("name", name)
+      .add("username", username)
                   .toString();
     //J+
   }
@@ -134,6 +136,10 @@ public final class TagRequest
     this.revision = revision;
   }
 
+  public void setUsername(String username) {
+    this.username = username;
+  }
+
   //~--- get methods ----------------------------------------------------------
 
   /**
@@ -158,6 +164,10 @@ public final class TagRequest
     return revision;
   }
 
+  public String getUserName() {
+    return username;
+  }
+
   //~--- fields ---------------------------------------------------------------
 
   /** Field description */
@@ -165,4 +175,6 @@ public final class TagRequest
 
   /** Field description */
   private String revision;
+
+  private String username;
 }

scm-ui-components/lerna.json

@@ -0,0 +1,7 @@
+{
+  "packages": [
+    "packages/*"
+  ],
+  "version": "0.0.0",
+  "npmClient": "yarn"
+}

scm-ui-components/package.json

@@ -0,0 +1,12 @@
+{
+  "name": "scm-ui-components",
+  "description": "Lerna root for SCM-Manager UI Components",
+  "private": true,
+  "scripts": {
+    "bootstrap": "lerna bootstrap",
+    "link": "lerna exec -- yarn link"
+  },
+  "devDependencies": {
+    "lerna": "^3.2.1"
+  }
+}

scm-ui-components/packages/ui-components/.babelrc

@@ -0,0 +1,6 @@
+{
+  "presets": ["@babel/preset-env", "@babel/preset-react", "@babel/preset-flow"],
+  "plugins": [
+    "@babel/plugin-proposal-class-properties"
+  ]
+}

scm-ui-components/packages/ui-components/.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "@scm-manager/eslint-config"
+}

scm-ui-components/packages/ui-components/.flowconfig

@@ -0,0 +1,8 @@
+[ignore]
+.*/node_modules/module-deps/.*
+
+[include]
+
+[libs]
+
+[options]

scm-ui-components/packages/ui-components/flow-typed/npm/classnames_v2.x.x.js

@@ -0,0 +1,23 @@
+// flow-typed signature: cf86673cc32d185bdab1d2ea90578d37
+// flow-typed version: 614bf49aa8/classnames_v2.x.x/flow_>=v0.25.x
+
+type $npm$classnames$Classes =
+  | string
+  | { [className: string]: * }
+  | false
+  | void
+  | null;
+
+declare module "classnames" {
+  declare module.exports: (
+    ...classes: Array<$npm$classnames$Classes | $npm$classnames$Classes[]>
+  ) => string;
+}
+
+declare module "classnames/bind" {
+  declare module.exports: $Exports<"classnames">;
+}
+
+declare module "classnames/dedupe" {
+  declare module.exports: $Exports<"classnames">;
+}

scm-ui-components/packages/ui-components/flow-typed/npm/moment_v2.3.x.js

@@ -0,0 +1,331 @@
+// flow-typed signature: 23b805356f90ad9384dd88489654e380
+// flow-typed version: e9374c5fe9/moment_v2.3.x/flow_>=v0.25.x
+
+type moment$MomentOptions = {
+  y?: number | string,
+  year?: number | string,
+  years?: number | string,
+  M?: number | string,
+  month?: number | string,
+  months?: number | string,
+  d?: number | string,
+  day?: number | string,
+  days?: number | string,
+  date?: number | string,
+  h?: number | string,
+  hour?: number | string,
+  hours?: number | string,
+  m?: number | string,
+  minute?: number | string,
+  minutes?: number | string,
+  s?: number | string,
+  second?: number | string,
+  seconds?: number | string,
+  ms?: number | string,
+  millisecond?: number | string,
+  milliseconds?: number | string
+};
+
+type moment$MomentObject = {
+  years: number,
+  months: number,
+  date: number,
+  hours: number,
+  minutes: number,
+  seconds: number,
+  milliseconds: number
+};
+
+type moment$MomentCreationData = {
+  input: string,
+  format: string,
+  locale: Object,
+  isUTC: boolean,
+  strict: boolean
+};
+
+type moment$CalendarFormat = string | ((moment: moment$Moment) => string);
+
+type moment$CalendarFormats = {
+  sameDay?: moment$CalendarFormat,
+  nextDay?: moment$CalendarFormat,
+  nextWeek?: moment$CalendarFormat,
+  lastDay?: moment$CalendarFormat,
+  lastWeek?: moment$CalendarFormat,
+  sameElse?: moment$CalendarFormat
+};
+
+declare class moment$LocaleData {
+  months(moment: moment$Moment): string,
+  monthsShort(moment: moment$Moment): string,
+  monthsParse(month: string): number,
+  weekdays(moment: moment$Moment): string,
+  weekdaysShort(moment: moment$Moment): string,
+  weekdaysMin(moment: moment$Moment): string,
+  weekdaysParse(weekDay: string): number,
+  longDateFormat(dateFormat: string): string,
+  isPM(date: string): boolean,
+  meridiem(hours: number, minutes: number, isLower: boolean): string,
+  calendar(
+    key:
+      | "sameDay"
+      | "nextDay"
+      | "lastDay"
+      | "nextWeek"
+      | "prevWeek"
+      | "sameElse",
+    moment: moment$Moment
+  ): string,
+  relativeTime(
+    number: number,
+    withoutSuffix: boolean,
+    key: "s" | "m" | "mm" | "h" | "hh" | "d" | "dd" | "M" | "MM" | "y" | "yy",
+    isFuture: boolean
+  ): string,
+  pastFuture(diff: any, relTime: string): string,
+  ordinal(number: number): string,
+  preparse(str: string): any,
+  postformat(str: string): any,
+  week(moment: moment$Moment): string,
+  invalidDate(): string,
+  firstDayOfWeek(): number,
+  firstDayOfYear(): number
+}
+declare class moment$MomentDuration {
+  humanize(suffix?: boolean): string,
+  milliseconds(): number,
+  asMilliseconds(): number,
+  seconds(): number,
+  asSeconds(): number,
+  minutes(): number,
+  asMinutes(): number,
+  hours(): number,
+  asHours(): number,
+  days(): number,
+  asDays(): number,
+  months(): number,
+  asWeeks(): number,
+  weeks(): number,
+  asMonths(): number,
+  years(): number,
+  asYears(): number,
+  add(value: number | moment$MomentDuration | Object, unit?: string): this,
+  subtract(value: number | moment$MomentDuration | Object, unit?: string): this,
+  as(unit: string): number,
+  get(unit: string): number,
+  toJSON(): string,
+  toISOString(): string,
+  isValid(): boolean
+}
+declare class moment$Moment {
+  static ISO_8601: string,
+  static (
+    string?: string,
+    format?: string | Array<string>,
+    strict?: boolean
+  ): moment$Moment,
+  static (
+    string?: string,
+    format?: string | Array<string>,
+    locale?: string,
+    strict?: boolean
+  ): moment$Moment,
+  static (
+    initDate: ?Object | number | Date | Array<number> | moment$Moment | string
+  ): moment$Moment,
+  static unix(seconds: number): moment$Moment,
+  static utc(): moment$Moment,
+  static utc(number: number | Array<number>): moment$Moment,
+  static utc(
+    str: string,
+    str2?: string | Array<string>,
+    str3?: string
+  ): moment$Moment,
+  static utc(moment: moment$Moment): moment$Moment,
+  static utc(date: Date): moment$Moment,
+  static parseZone(): moment$Moment,
+  static parseZone(rawDate: string): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string | Array<string>
+  ): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string,
+    strict: boolean
+  ): moment$Moment,
+  static parseZone(
+    rawDate: string,
+    format: string,
+    locale: string,
+    strict: boolean
+  ): moment$Moment,
+  isValid(): boolean,
+  invalidAt(): 0 | 1 | 2 | 3 | 4 | 5 | 6,
+  creationData(): moment$MomentCreationData,
+  millisecond(number: number): this,
+  milliseconds(number: number): this,
+  millisecond(): number,
+  milliseconds(): number,
+  second(number: number): this,
+  seconds(number: number): this,
+  second(): number,
+  seconds(): number,
+  minute(number: number): this,
+  minutes(number: number): this,
+  minute(): number,
+  minutes(): number,
+  hour(number: number): this,
+  hours(number: number): this,
+  hour(): number,
+  hours(): number,
+  date(number: number): this,
+  dates(number: number): this,
+  date(): number,
+  dates(): number,
+  day(day: number | string): this,
+  days(day: number | string): this,
+  day(): number,
+  days(): number,
+  weekday(number: number): this,
+  weekday(): number,
+  isoWeekday(number: number): this,
+  isoWeekday(): number,
+  dayOfYear(number: number): this,
+  dayOfYear(): number,
+  week(number: number): this,
+  weeks(number: number): this,
+  week(): number,
+  weeks(): number,
+  isoWeek(number: number): this,
+  isoWeeks(number: number): this,
+  isoWeek(): number,
+  isoWeeks(): number,
+  month(number: number): this,
+  months(number: number): this,
+  month(): number,
+  months(): number,
+  quarter(number: number): this,
+  quarter(): number,
+  year(number: number): this,
+  years(number: number): this,
+  year(): number,
+  years(): number,
+  weekYear(number: number): this,
+  weekYear(): number,
+  isoWeekYear(number: number): this,
+  isoWeekYear(): number,
+  weeksInYear(): number,
+  isoWeeksInYear(): number,
+  get(string: string): number,
+  set(unit: string, value: number): this,
+  set(options: { [unit: string]: number }): this,
+  static max(...dates: Array<moment$Moment>): moment$Moment,
+  static max(dates: Array<moment$Moment>): moment$Moment,
+  static min(...dates: Array<moment$Moment>): moment$Moment,
+  static min(dates: Array<moment$Moment>): moment$Moment,
+  add(
+    value: number | moment$MomentDuration | moment$Moment | Object,
+    unit?: string
+  ): this,
+  subtract(
+    value: number | moment$MomentDuration | moment$Moment | string | Object,
+    unit?: string
+  ): this,
+  startOf(unit: string): this,
+  endOf(unit: string): this,
+  local(): this,
+  utc(): this,
+  utcOffset(
+    offset: number | string,
+    keepLocalTime?: boolean,
+    keepMinutes?: boolean
+  ): this,
+  utcOffset(): number,
+  format(format?: string): string,
+  fromNow(removeSuffix?: boolean): string,
+  from(
+    value: moment$Moment | string | number | Date | Array<number>,
+    removePrefix?: boolean
+  ): string,
+  toNow(removePrefix?: boolean): string,
+  to(
+    value: moment$Moment | string | number | Date | Array<number>,
+    removePrefix?: boolean
+  ): string,
+  calendar(refTime?: any, formats?: moment$CalendarFormats): string,
+  diff(
+    date: moment$Moment | string | number | Date | Array<number>,
+    format?: string,
+    floating?: boolean
+  ): number,
+  valueOf(): number,
+  unix(): number,
+  daysInMonth(): number,
+  toDate(): Date,
+  toArray(): Array<number>,
+  toJSON(): string,
+  toISOString(
+    keepOffset?: boolean
+  ): string,
+  toObject(): moment$MomentObject,
+  isBefore(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSame(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isAfter(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSameOrBefore(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isSameOrAfter(
+    date?: moment$Moment | string | number | Date | Array<number>,
+    units?: ?string
+  ): boolean,
+  isBetween(
+    fromDate: moment$Moment | string | number | Date | Array<number>,
+    toDate?: ?moment$Moment | string | number | Date | Array<number>,
+    granularity?: ?string,
+    inclusion?: ?string
+  ): boolean,
+  isDST(): boolean,
+  isDSTShifted(): boolean,
+  isLeapYear(): boolean,
+  clone(): moment$Moment,
+  static isMoment(obj: any): boolean,
+  static isDate(obj: any): boolean,
+  static locale(locale: string, localeData?: Object): string,
+  static updateLocale(locale: string, localeData?: ?Object): void,
+  static locale(locales: Array<string>): string,
+  locale(locale: string, customization?: Object | null): moment$Moment,
+  locale(): string,
+  static months(): Array<string>,
+  static monthsShort(): Array<string>,
+  static weekdays(): Array<string>,
+  static weekdaysShort(): Array<string>,
+  static weekdaysMin(): Array<string>,
+  static months(): string,
+  static monthsShort(): string,
+  static weekdays(): string,
+  static weekdaysShort(): string,
+  static weekdaysMin(): string,
+  static localeData(key?: string): moment$LocaleData,
+  static duration(
+    value: number | Object | string,
+    unit?: string
+  ): moment$MomentDuration,
+  static isDuration(obj: any): boolean,
+  static normalizeUnits(unit: string): string,
+  static invalid(object: any): moment$Moment
+}
+
+declare module "moment" {
+  declare module.exports: Class<moment$Moment>;
+}

scm-ui-components/packages/ui-components/flow-typed/npm/react-jss_vx.x.x.js

@@ -0,0 +1,137 @@
+// flow-typed signature: ba35d02d668b0d0a3e04a63a6847974e
+// flow-typed version: <<STUB>>/react-jss_v8.6.1/flow_v0.79.1
+
+/**
+ * This is an autogenerated libdef stub for:
+ *
+ *   'react-jss'
+ *
+ * Fill this stub out by replacing all the `any` types.
+ *
+ * Once filled out, we encourage you to share your work with the
+ * community by sending a pull request to:
+ * https://github.com/flowtype/flow-typed
+ */
+
+declare module 'react-jss' {
+  declare module.exports: any;
+}
+
+/**
+ * We include stubs for each file inside this npm package in case you need to
+ * require those files directly. Feel free to delete any files that aren't
+ * needed.
+ */
+declare module 'react-jss/dist/react-jss' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/dist/react-jss.min' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/compose' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/compose.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/contextTypes' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/createHoc' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/getDisplayName' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/index' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/index.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/injectSheet' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/injectSheet.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/jss' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/JssProvider' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/JssProvider.test' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/ns' {
+  declare module.exports: any;
+}
+
+declare module 'react-jss/lib/propTypes' {
+  declare module.exports: any;
+}
+
+// Filename aliases
+declare module 'react-jss/dist/react-jss.js' {
+  declare module.exports: $Exports<'react-jss/dist/react-jss'>;
+}
+declare module 'react-jss/dist/react-jss.min.js' {
+  declare module.exports: $Exports<'react-jss/dist/react-jss.min'>;
+}
+declare module 'react-jss/lib/compose.js' {
+  declare module.exports: $Exports<'react-jss/lib/compose'>;
+}
+declare module 'react-jss/lib/compose.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/compose.test'>;
+}
+declare module 'react-jss/lib/contextTypes.js' {
+  declare module.exports: $Exports<'react-jss/lib/contextTypes'>;
+}
+declare module 'react-jss/lib/createHoc.js' {
+  declare module.exports: $Exports<'react-jss/lib/createHoc'>;
+}
+declare module 'react-jss/lib/getDisplayName.js' {
+  declare module.exports: $Exports<'react-jss/lib/getDisplayName'>;
+}
+declare module 'react-jss/lib/index.js' {
+  declare module.exports: $Exports<'react-jss/lib/index'>;
+}
+declare module 'react-jss/lib/index.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/index.test'>;
+}
+declare module 'react-jss/lib/injectSheet.js' {
+  declare module.exports: $Exports<'react-jss/lib/injectSheet'>;
+}
+declare module 'react-jss/lib/injectSheet.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/injectSheet.test'>;
+}
+declare module 'react-jss/lib/jss.js' {
+  declare module.exports: $Exports<'react-jss/lib/jss'>;
+}
+declare module 'react-jss/lib/JssProvider.js' {
+  declare module.exports: $Exports<'react-jss/lib/JssProvider'>;
+}
+declare module 'react-jss/lib/JssProvider.test.js' {
+  declare module.exports: $Exports<'react-jss/lib/JssProvider.test'>;
+}
+declare module 'react-jss/lib/ns.js' {
+  declare module.exports: $Exports<'react-jss/lib/ns'>;
+}
+declare module 'react-jss/lib/propTypes.js' {
+  declare module.exports: $Exports<'react-jss/lib/propTypes'>;
+}

scm-ui-components/packages/ui-components/package.json

@@ -0,0 +1,46 @@
+{
+  "name": "@scm-manager/ui-components",
+  "version": "0.0.1",
+  "description": "UI Components for SCM-Manager and its plugins",
+  "main": "src/index.js",
+  "files": ["src"],
+  "repository": "https://bitbucket.org/sdorra/scm-manager",
+  "author": "Sebastian Sdorra <sebastian.sdorra@cloudogu.com>",
+  "license" : "BSD-3-Clause",
+  "scripts": {
+    "update-index": "create-index -r src",
+    "eslint-fix": "eslint src --fix"
+  },
+  "devDependencies": {
+    "@scm-manager/ui-bundler": "^0.0.13",
+    "create-index": "^2.3.0",
+    "enzyme": "^3.5.0",
+    "enzyme-adapter-react-16": "^1.3.1",
+    "flow-bin": "^0.79.1",
+    "flow-typed": "^2.5.1",
+    "jest": "^23.5.0",
+    "raf": "^3.4.0"
+  },
+  "dependencies": {
+    "classnames": "^2.2.6",
+    "moment": "^2.22.2",
+    "react": "^16.4.2",
+    "react-dom": "^16.4.2",
+    "react-i18next": "^7.11.0",
+    "react-jss": "^8.6.1",
+    "react-router-dom": "^4.3.1",
+    "@scm-manager/ui-types": "0.0.1"
+  },
+  "browserify": {
+    "transform": [
+      ["browserify-css"],
+      [
+        "babelify",
+        {
+          "plugins": ["@babel/plugin-proposal-class-properties"],
+          "presets": ["@babel/preset-env", "@babel/preset-flow", "@babel/preset-react"]
+        }
+      ]
+    ]
+  }
+}

scm-ui-components/packages/ui-components/src/Image.js

@@ -1,6 +1,6 @@
 //@flow
 import React from "react";
-import { withContextPath } from "../urls";
+import { withContextPath } from "./urls";
 
 type Props = {
   src: string,

scm-ui-components/packages/ui-components/src/Paginator.js

@@ -1,12 +1,12 @@
 //@flow
 import React from "react";
 import { translate } from "react-i18next";
-import type { PagedCollection } from "../types/Collection";
+import type { PagedCollection } from "@scm-manager/ui-types";
 import { Button } from "./buttons";
 
 type Props = {
   collection: PagedCollection,
-  onPageChange: string => void,
+  onPageChange?: string => void,
   t: string => string
 };
 
@@ -17,8 +17,10 @@ class Paginator extends React.Component<Props> {
 
   createAction = (linkType: string) => () => {
     const { collection, onPageChange } = this.props;
-    const link = collection._links[linkType].href;
-    onPageChange(link);
+    if (onPageChange) {
+      const link = collection._links[linkType].href;
+      onPageChange(link);
+    }
   };
 
   renderFirstButton() {
@@ -94,7 +96,8 @@ class Paginator extends React.Component<Props> {
     if (page + 1 < pageTotal) {
       links.push(this.renderPageButton(page + 1, "next"));
     }
-    if(page+2 < pageTotal) //if there exists pages between next and last
+    if (page + 2 < pageTotal)
+      //if there exists pages between next and last
       links.push(this.seperator());
     if (page < pageTotal) {
       links.push(this.renderLastButton());

scm-ui-components/packages/ui-components/src/Paginator.test.js

@@ -1,8 +1,8 @@
 // @flow
 import React from "react";
 import { mount, shallow } from "enzyme";
-import "../tests/enzyme";
-import "../tests/i18n";
+import "./tests/enzyme";
+import "./tests/i18n";
 
 import Paginator from "./Paginator";
 

scm-ui-components/packages/ui-components/src/buttons/index.js

@@ -0,0 +1,10 @@
+// @create-index
+
+export { default as AddButton } from "./AddButton.js";
+export { default as Button } from "./Button.js";
+export { default as CreateButton } from "./CreateButton.js";
+export { default as DeleteButton } from "./DeleteButton.js";
+export { default as EditButton } from "./EditButton.js";
+export { default as RemoveEntryOfTableButton } from "./RemoveEntryOfTableButton.js";
+export { default as SubmitButton } from "./SubmitButton.js";
+

scm-ui-components/packages/ui-components/src/forms/Select.js

@@ -41,6 +41,7 @@ class Select extends React.Component<Props> {
   render() {
     const { options, value, loading } = this.props;
     const loadingClass = loading ? "is-loading" : "";
+
     return (
       <div className="field">
         {this.renderLabel()}

scm-ui-components/packages/ui-components/src/forms/index.js

@@ -0,0 +1,8 @@
+// @create-index
+
+export { default as AddEntryToTableField } from "./AddEntryToTableField.js";
+export { default as Checkbox } from "./Checkbox.js";
+export { default as InputField } from "./InputField.js";
+export { default as Select } from "./Select.js";
+export { default as Textarea } from "./Textarea.js";
+

scm-ui-components/packages/ui-components/src/index.js

@@ -0,0 +1,25 @@
+// @create-index
+
+import * as validation from "./validation.js";
+import * as urls from "./urls";
+
+export { validation, urls };
+
+export { default as DateFromNow } from "./DateFromNow.js";
+export { default as ErrorNotification } from "./ErrorNotification.js";
+export { default as ErrorPage } from "./ErrorPage.js";
+export { default as Image } from "./Image.js";
+export { default as Loading } from "./Loading.js";
+export { default as Logo } from "./Logo.js";
+export { default as MailLink } from "./MailLink.js";
+export { default as Notification } from "./Notification.js";
+export { default as Paginator } from "./Paginator.js";
+export { default as ProtectedRoute } from "./ProtectedRoute.js";
+
+export { apiClient, NOT_FOUND_ERROR, UNAUTHORIZED_ERROR } from "./apiclient.js";
+
+export * from "./buttons";
+export * from "./forms";
+export * from "./layout";
+export * from "./modals";
+export * from "./navigation";

scm-ui-components/packages/ui-components/src/layout/Footer.js

@@ -1,6 +1,6 @@
 //@flow
 import React from "react";
-import type { Me } from "../../types/Me";
+import type { Me } from "@scm-manager/ui-types";
 
 type Props = {
   me?: Me

scm-ui-components/packages/ui-components/src/layout/index.js

@@ -0,0 +1,8 @@
+// @create-index
+
+export { default as Footer } from "./Footer.js";
+export { default as Header } from "./Header.js";
+export { default as Page } from "./Page.js";
+export { default as Subtitle } from "./Subtitle.js";
+export { default as Title } from "./Title.js";
+