# HG changeset patch
# User Kim Alvefur <zash@zash.se>
# Date 1630275118 -7200
# Node ID ae565e49289a70ac3d26154965a306bcfd69f8db
# Parent  21a9b3f2a728bf6e29a30475dda186c0f50f6c9a
mod_external_services: Factor out public function returning current services

This way you get the _prepared_ services and don't have to do that mapping
yourself.

diff -r 21a9b3f2a728 -r ae565e49289a plugins/mod_external_services.lua
--- a/plugins/mod_external_services.lua	Mon Aug 30 20:19:09 2021 +0200
+++ b/plugins/mod_external_services.lua	Mon Aug 30 00:11:58 2021 +0200
@@ -122,6 +122,15 @@
 	end;
 }
 
+function get_services()
+	local extras = module:get_host_items("external_service");
+	local services = ( configured_services + extras ) / prepare;
+
+	setmetatable(services, services_mt);
+
+	return services;
+end
+
 local function handle_services(event)
 	local origin, stanza = event.origin, event.stanza;
 	local action = stanza.tags[1];
@@ -134,8 +143,7 @@
 	end
 
 	local reply = st.reply(stanza):tag("services", { xmlns = action.attr.xmlns });
-	local extras = module:get_host_items("external_service");
-	local services = ( configured_services + extras ) / prepare;
+	local services = get_services();
 
 	local requested_type = action.attr.type;
 	if requested_type then
@@ -144,8 +152,6 @@
 		end);
 	end
 
-	setmetatable(services, services_mt);
-
 	module:fire_event("external_service/services", {
 			origin = origin;
 			stanza = stanza;
@@ -181,8 +187,7 @@
 	end
 
 	local reply = st.reply(stanza):tag("credentials", { xmlns = action.attr.xmlns });
-	local extras = module:get_host_items("external_service");
-	local services = ( configured_services + extras ) / prepare;
+	local services = get_services();
 	services:filter(function (item)
 		return item.restricted;
 	end)
@@ -198,8 +203,6 @@
 			tonumber(service.attr.port) or 0));
 	end
 
-	setmetatable(services, services_mt);
-
 	module:fire_event("external_service/credentials", {
 			origin = origin;
 			stanza = stanza;