From f4dd9cabb21e26fdc21f8413be822ea7c64251f8 Mon Sep 17 00:00:00 2001
From: Julian Lam <julian@nodebb.org>
Date: Tue, 13 Aug 2019 11:33:05 -0400
Subject: [PATCH] fix: resolved breaking changes from sitemap v4 upgrade

---
 src/sitemap.js | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/src/sitemap.js b/src/sitemap.js
index 192a1f49fe..c980fc1d22 100644
--- a/src/sitemap.js
+++ b/src/sitemap.js
@@ -1,7 +1,7 @@
 'use strict';
 
 var async = require('async');
-var sm = require('sitemap');
+const { Sitemap } = require('sitemap');
 var nconf = require('nconf');
 
 var db = require('./database');
@@ -45,7 +45,7 @@ sitemap.getPages = function (callback) {
 		sitemap.maps.pages &&
 		Date.now() < parseInt(sitemap.maps.pages.cacheSetTimestamp, 10) + parseInt(sitemap.maps.pages.cacheResetPeriod, 10)
 	) {
-		return sitemap.maps.pages.toXML(callback);
+		return callback(null, sitemap.maps.pages.toXML());
 	}
 
 	var urls = [{
@@ -70,13 +70,13 @@ sitemap.getPages = function (callback) {
 		if (err) {
 			return callback(err);
 		}
-		sitemap.maps.pages = sm.createSitemap({
+		sitemap.maps.pages = new Sitemap({
 			hostname: nconf.get('url'),
 			cacheTime: 1000 * 60 * 60 * 24,	// Cached for 24 hours
 			urls: data.urls,
 		});
 
-		sitemap.maps.pages.toXML(callback);
+		callback(null, sitemap.maps.pages.toXML());
 	});
 };
 
@@ -85,7 +85,7 @@ sitemap.getCategories = function (callback) {
 		sitemap.maps.categories &&
 		Date.now() < parseInt(sitemap.maps.categories.cacheSetTimestamp, 10) + parseInt(sitemap.maps.categories.cacheResetPeriod, 10)
 	) {
-		return sitemap.maps.categories.toXML(callback);
+		return callback(null, sitemap.maps.categories.toXML());
 	}
 
 	var categoryUrls = [];
@@ -104,13 +104,13 @@ sitemap.getCategories = function (callback) {
 			}
 		});
 
-		sitemap.maps.categories = sm.createSitemap({
+		sitemap.maps.categories = new Sitemap({
 			hostname: nconf.get('url'),
 			cacheTime: 1000 * 60 * 60 * 24,	// Cached for 24 hours
 			urls: categoryUrls,
 		});
 
-		sitemap.maps.categories.toXML(callback);
+		callback(null, sitemap.maps.categories.toXML());
 	});
 };
 
@@ -127,7 +127,7 @@ sitemap.getTopicPage = function (page, callback) {
 		sitemap.maps.topics[page - 1] &&
 		Date.now() < parseInt(sitemap.maps.topics[page - 1].cacheSetTimestamp, 10) + parseInt(sitemap.maps.topics[page - 1].cacheResetPeriod, 10)
 	) {
-		return sitemap.maps.topics[page - 1].toXML(callback);
+		return callback(null, sitemap.maps.topics[page - 1].toXML());
 	}
 
 	var topicUrls = [];
@@ -158,13 +158,13 @@ sitemap.getTopicPage = function (page, callback) {
 			}
 		});
 
-		sitemap.maps.topics[page - 1] = sm.createSitemap({
+		sitemap.maps.topics[page - 1] = new Sitemap({
 			hostname: nconf.get('url'),
 			cacheTime: 1000 * 60 * 60,	// Cached for 1 hour
 			urls: topicUrls,
 		});
 
-		sitemap.maps.topics[page - 1].toXML(callback);
+		callback(null, sitemap.maps.topics[page - 1].toXML());
 	});
 };