From 70c8c22c52ae0d429321f58b8a787ef4b157af3f Mon Sep 17 00:00:00 2001
From: HampusM <hampus@hampusmat.com>
Date: Thu, 19 Aug 2021 15:18:03 +0200
Subject: Moved the implementations of getting all branches or tags to their
 respective classes

---
 packages/server/src/git/branch.ts     | 12 ++++++++++++
 packages/server/src/git/repository.ts |  6 ++----
 packages/server/src/git/tag.ts        | 12 ++++++++++++
 3 files changed, 26 insertions(+), 4 deletions(-)

(limited to 'packages')

diff --git a/packages/server/src/git/branch.ts b/packages/server/src/git/branch.ts
index 85f83be..0819f3c 100644
--- a/packages/server/src/git/branch.ts
+++ b/packages/server/src/git/branch.ts
@@ -61,4 +61,16 @@ export class Branch extends Reference {
 			.then(() => true)
 			.catch(() => false);
 	}
+
+	/**
+	 * Returns all of a repository's branches
+	 *
+	 * @param owner - An instance of a repository
+	 * @returns An array of branch instances
+	 */
+	public static async getAll(owner: Repository): Promise<Branch[]> {
+		const references = await owner.ng_repository.getReferences();
+
+		return references.filter(ref => ref.isBranch()).map(ref => new Branch(owner, ref));
+	}
 }
\ No newline at end of file
diff --git a/packages/server/src/git/repository.ts b/packages/server/src/git/repository.ts
index 53245be..199985c 100644
--- a/packages/server/src/git/repository.ts
+++ b/packages/server/src/git/repository.ts
@@ -146,8 +146,7 @@ export class Repository {
 	 * @returns An array of branch instances
 	 */
 	public async branches(): Promise<Branch[]> {
-		const references = await this.ng_repository.getReferences();
-		return references.filter(ref => ref.isBranch()).map(ref => new Branch(this, ref));
+		return Branch.getAll(this);
 	}
 
 	/**
@@ -156,8 +155,7 @@ export class Repository {
 	 * @returns An array of tag instances
 	 */
 	public async tags(): Promise<Tag[]> {
-		const references = await this.ng_repository.getReferences();
-		return references.filter(ref => ref.isTag()).map(ref => new Tag(this, ref));
+		return Tag.getAll(this);
 	}
 
 	/**
diff --git a/packages/server/src/git/tag.ts b/packages/server/src/git/tag.ts
index 6c61a65..7d8c5dc 100644
--- a/packages/server/src/git/tag.ts
+++ b/packages/server/src/git/tag.ts
@@ -96,4 +96,16 @@ export class Tag extends Reference {
 
 		return new Tag(owner, reference);
 	}
+
+	/**
+	 * Returns all of a repository's tags
+	 *
+	 * @param owner - An instance of a repository
+	 * @returns An array of tag instances
+	 */
+	public static async getAll(owner: Repository): Promise<Tag[]> {
+		const references = await owner.ng_repository.getReferences();
+
+		return references.filter(ref => ref.isTag()).map(ref => new Tag(owner, ref));
+	}
 }
\ No newline at end of file
-- 
cgit v1.2.3-18-g5258