From aa102e3c4ab6104f4e9a6338b1675b05fb6da9be Mon Sep 17 00:00:00 2001
From: jNullj <15849761+jNullj@users.noreply.github.com>
Date: Sat, 12 Oct 2024 16:55:50 +0300
Subject: [PATCH 1/2] Refactor version rendering in f-droid, nexus, and
 wordpress services

---
 services/f-droid/f-droid.service.js             | 12 ++----------
 services/nexus/nexus.service.js                 | 12 ++----------
 services/wordpress/wordpress-version.service.js | 12 ++----------
 3 files changed, 6 insertions(+), 30 deletions(-)

diff --git a/services/f-droid/f-droid.service.js b/services/f-droid/f-droid.service.js
index 02f5ab3ee9872..bc2d36717d1fe 100644
--- a/services/f-droid/f-droid.service.js
+++ b/services/f-droid/f-droid.service.js
@@ -3,8 +3,7 @@ import {
   optionalNonNegativeInteger,
   nonNegativeInteger,
 } from '../validators.js'
-import { addv } from '../text-formatters.js'
-import { version as versionColor } from '../color-formatters.js'
+import { renderVersionBadge } from '../version.js'
 import { BaseJsonService, NotFound, pathParam, queryParam } from '../index.js'
 
 const schema = Joi.object({
@@ -46,13 +45,6 @@ export default class FDroid extends BaseJsonService {
 
   static defaultBadgeData = { label: 'f-droid' }
 
-  static render({ version }) {
-    return {
-      message: addv(version),
-      color: versionColor(version),
-    }
-  }
-
   async fetch({ appId }) {
     const url = `https://f-droid.org/api/v1/packages/${appId}`
     return this._requestJson({
@@ -83,6 +75,6 @@ export default class FDroid extends BaseJsonService {
     const json = await this.fetch({ appId })
     const suggested = includePre === undefined
     const { version } = this.transform({ json, suggested })
-    return this.constructor.render({ version })
+    return renderVersionBadge({ version })
   }
 }
diff --git a/services/nexus/nexus.service.js b/services/nexus/nexus.service.js
index bcb0a2a07f79b..a6bd0cea0cc62 100644
--- a/services/nexus/nexus.service.js
+++ b/services/nexus/nexus.service.js
@@ -1,6 +1,5 @@
 import Joi from 'joi'
-import { version as versionColor } from '../color-formatters.js'
-import { addv } from '../text-formatters.js'
+import { renderVersionBadge } from '../version.js'
 import {
   optionalUrl,
   optionalDottedVersionNClausesWithOptionalSuffix,
@@ -143,13 +142,6 @@ export default class Nexus extends BaseJsonService {
     label: 'nexus',
   }
 
-  static render({ version }) {
-    return {
-      message: addv(version),
-      color: versionColor(version),
-    }
-  }
-
   addQueryParamsToQueryString({ searchParams, queryOpt }) {
     // Users specify query options with 'key=value' pairs, using a
     // colon delimiter between pairs ([:k1=v1[:k2=v2[...]]]).
@@ -321,6 +313,6 @@ export default class Nexus extends BaseJsonService {
     })
 
     const { version } = this.transform({ repo, json, actualNexusVersion })
-    return this.constructor.render({ version })
+    return renderVersionBadge({ version })
   }
 }
diff --git a/services/wordpress/wordpress-version.service.js b/services/wordpress/wordpress-version.service.js
index 8e7432bf16393..3fb9273bbfa58 100644
--- a/services/wordpress/wordpress-version.service.js
+++ b/services/wordpress/wordpress-version.service.js
@@ -1,6 +1,5 @@
 import { pathParams } from '../index.js'
-import { addv } from '../text-formatters.js'
-import { version as versionColor } from '../color-formatters.js'
+import { renderVersionBadge } from '../version.js'
 import { description, BaseWordpress } from './wordpress-base.js'
 
 function VersionForExtensionType(extensionType) {
@@ -43,19 +42,12 @@ function VersionForExtensionType(extensionType) {
 
     static defaultBadgeData = { label: extensionType }
 
-    static render({ version }) {
-      return {
-        message: addv(version),
-        color: versionColor(version),
-      }
-    }
-
     async handle({ slug }) {
       const { version } = await this.fetch({
         extensionType,
         slug,
       })
-      return this.constructor.render({ version })
+      return renderVersionBadge({ version })
     }
   }
 }

From d1d60db12ac35ff81b1071392661ed6294efb3c4 Mon Sep 17 00:00:00 2001
From: jNullj <15849761+jNullj@users.noreply.github.com>
Date: Sat, 12 Oct 2024 17:04:17 +0300
Subject: [PATCH 2/2] add missing label in nexus test

---
 services/nexus/nexus.spec.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/services/nexus/nexus.spec.js b/services/nexus/nexus.spec.js
index 1f4114bfdc61a..f0ce13c258506 100644
--- a/services/nexus/nexus.spec.js
+++ b/services/nexus/nexus.spec.js
@@ -154,6 +154,7 @@ describe('Nexus', function () {
           },
         ),
       ).to.deep.equal({
+        label: undefined,
         message: 'v2.3.4',
         color: 'blue',
       })