repository-provider 28.3.15 → 28.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "repository-provider",
-  "version": "28.3.15",
+  "version": "28.4.0",
   "publishConfig": {
     "access": "public"
   },

package/src/multi-group-provider.mjs

@@ -105,13 +105,10 @@ export class MultiGroupProvider extends BaseProvider {
    * @return {RepositoryGroup}
    */
   addRepositoryGroup(name, options) {
-    const normalizedName = this.normalizeGroupName(name, true);
+    return this.#repositoryGroups.get(this.normalizeGroupName(name, true)) || new this.repositoryGroupClass(this, name, options);
+  }
 
-    let repositoryGroup = this.#repositoryGroups.get(normalizedName);
-    if (repositoryGroup === undefined) {
-      repositoryGroup = new this.repositoryGroupClass(this, name, options);
-      this.#repositoryGroups.set(normalizedName, repositoryGroup);
-    }
-    return repositoryGroup;
+  _addRepositoryGroup(repositoryGroup) {
+    this.#repositoryGroups.set(this.normalizeGroupName(repositoryGroup.name, true), repositoryGroup);
   }
 }

package/src/repository-group.mjs

@@ -1,5 +1,5 @@
 import { RepositoryOwner } from "./repository-owner.mjs";
-import { NamedObject } from "./named-object.mjs";
+import { OwnedObject } from "./owned-object.mjs";
 import { BaseProvider } from "./base-provider.mjs";
 
 /**
@@ -16,7 +16,12 @@ import { BaseProvider } from "./base-provider.mjs";
  * @property {string} name
  */
 
-export class RepositoryGroup extends RepositoryOwner(NamedObject) {
+export class RepositoryGroup extends RepositoryOwner(OwnedObject) {
+
+  static get registerInstanceMethodName() {
+    return "_addRepositoryGroup";
+  }
+
   static get attributes() {
     return {
       ...super.attributes,
@@ -43,15 +48,9 @@ export class RepositoryGroup extends RepositoryOwner(NamedObject) {
     return {};
   }
 
-  constructor(provider, name, options) {
-    super(name, options, {
-      provider: { value: provider }
-    });
-  }
-
-  get owner()
+  get provider()
   {
-    return this.provider;
+    return this.owner;
   }
 
   get isAdmin() {
@@ -59,10 +58,10 @@ export class RepositoryGroup extends RepositoryOwner(NamedObject) {
   }
 
   get areRepositoryNamesCaseSensitive() {
-    return this.provider.areRepositoryNamesCaseSensitive;
+    return this.owner.areRepositoryNamesCaseSensitive;
   }
 
   get areRepositoryGroupNamesCaseSensitive() {
-    return this.provider.areRepositoryGroupNamesCaseSensitive;
+    return this.owner.areRepositoryGroupNamesCaseSensitive;
   }
 }

package/src/repository-owner.mjs

@@ -146,12 +146,7 @@ export function RepositoryOwner(base) {
      * @return {Promise<Repository>} newly created repository
      */
     addRepository(name, options) {
-      const normalizedName = this.normalizeRepositoryName(name, true);
-      let repository = this.#repositories.get(normalizedName);
-      if (repository === undefined) {
-        repository = new this.repositoryClass(this, name, options);
-      }
-      return repository;
+      return this.#repositories.get(this.normalizeRepositoryName(name, true)) || new this.repositoryClass(this, name, options);
     }
 
     _addRepository(repository)

package/src/repository.mjs

@@ -242,12 +242,7 @@ export class Repository extends OwnedObject {
    */
   async branch(name) {
     if(name === this.defaultBranchName) {
-      const branch = this._branches.get(name);
-      if(branch) {
-        return branch;
-      }
-      
-      return this.addBranch(name);
+      return this._branches.get(name) || this.addBranch(name);
     }
 
     await this.initializeBranches();
@@ -284,12 +279,7 @@ export class Repository extends OwnedObject {
    * @return {Branch} newly created branch
    */
   addBranch(name, options) {
-    let branch = this._branches.get(name);
-    if (branch === undefined) {
-      branch = new this.branchClass(this, name, options);
-    }
-
-    return branch;
+    return this._branches.get(name) || new this.branchClass(this, name, options);
   }
 
   _addBranch(branch) {
@@ -335,12 +325,7 @@ export class Repository extends OwnedObject {
    * @return {Tag} newly created tag
    */
   addTag(name, options) {
-    let tag = this._tags.get(name);
-    if (tag === undefined) {
-      tag = new this.tagClass(this, name, options);
-    }
-
-    return tag;
+    return this._tags.get(name) || new this.tagClass(this, name, options);
   }
 
   _addTag(tag) {