repository-provider 28.3.8 → 28.3.11

package/package.json

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

package/src/base-object.mjs

@@ -88,6 +88,22 @@ export class BaseObject {
     return this.name;
   }
 
+  /**
+   * Beautified name use for human displaying only.
+   * @return {string} human readable name
+   */
+  get displayName() {
+    return this.name;
+  }
+
+  /**
+   * Complete name in the hierachy.
+   * @return {string}
+   */
+  get fullName() {
+    return this.name;
+  }
+
   /**
    * By default cannot be written to.
    * @return {boolean} false

package/src/base-provider.mjs

@@ -405,14 +405,6 @@ export class BaseProvider extends BaseObject {
     return this.constructor.name;
   }
 
-  /**
-   * @return {string} name
-   */
-  get fullName()
-  {
-    return this.name;
-  }
-
   /**
    * We are our own provider.
    * @return {BaseProvider} this

package/src/named-object.mjs

@@ -29,15 +29,6 @@ export class NamedObject extends BaseObject {
       this.provider.equals(other.provider)
     );
   }
-
-  /**
-   * Beautified name use for human displaying only.
-   * @return {string} human readable name
-   */
-  get displayName() {
-    return this.name;
-  }
-
   /**
    * @return {string} name with owner name
    */

package/src/owned-object.mjs

@@ -5,9 +5,9 @@ import { NamedObject } from "./named-object.mjs";
  */
 export class OwnedObject extends NamedObject {
   /**
-   * Method name to be called to register one instance in the owner. 
+   * Method name to be called to register one instance in the owner.
    * sample: Application => _addApplication
-   * @return {string} 
+   * @return {string}
    */
   static get registerInstanceMethodName() {
     return "_add" + this.name;
@@ -21,4 +21,13 @@ export class OwnedObject extends NamedObject {
 
     owner[this.constructor.registerInstanceMethodName](this);
   }
+
+  /**
+   * Check for equality.
+   * @param {OwnedObject} other
+   * @return {boolean} true if receiver and owner are equal
+   */
+  equals(other) {
+    return super.equals(other) && this.owner.equals(other.owner);
+  }
 }

package/src/ref.mjs

@@ -25,15 +25,6 @@ export class Ref extends OwnedObject {
     };
   }
 
-  /**
-   * Check for equality.
-   * @param {Ref} other
-   * @return {boolean} true if name and repository are equal
-   */
-  equals(other) {
-    return super.equals(other) && this.repository.equals(other.repository);
-  }
-
   get refType() {
     return "unknown";
   }

package/src/repository-owner.mjs

@@ -153,11 +153,16 @@ export function RepositoryOwner(base) {
       let repository = this._repositories.get(normalizedName);
       if (repository === undefined) {
         repository = new this.repositoryClass(this, name, options);
-        this._repositories.set(normalizedName, repository);
       }
       return repository;
     }
 
+    _addRepository(repository)
+    {
+      const normalizedName = this.normalizeRepositoryName(repository.name, true);
+      this._repositories.set(normalizedName, repository);
+    }
+
     /**
      * Delete a repository.
      * @param {string} name

package/src/repository.mjs

@@ -1,6 +1,6 @@
 import { matcher } from "matching-iterator";
 import { optionJSON } from "./attribute.mjs";
-import { NamedObject } from "./named-object.mjs";
+import { OwnedObject } from "./owned-object.mjs";
 import { Hook } from "./hook.mjs";
 import { Tag } from "./tag.mjs";
 import { Branch } from "./branch.mjs";
@@ -24,7 +24,12 @@ import { BaseProvider } from "./base-provider.mjs";
  * @property {Map<string,PullRequest>} pullRequests
  * @property {Map<string,Milestone>} milestones
  */
-export class Repository extends NamedObject {
+export class Repository extends OwnedObject {
+
+  static get registerInstanceMethodName() {
+    return "_addRepository";
+  }
+
   /**
    * options
    */
@@ -62,8 +67,7 @@ export class Repository extends NamedObject {
   }
 
   constructor(owner, name, options) {
-    super(owner.normalizeRepositoryName(name, false), options, {
-      owner: { value: owner },
+    super(owner, owner.normalizeRepositoryName(name, false), options, {
       _branches: { value: new Map() },
       _tags: { value: new Map() },
       _pullRequests: { value: new Map() },