Refactored CustomRedirectsAPI constructor

refs https://github.com/TryGhost/Toolbox/issues/139

- The pattern we use accross the codebase is a single "options" object passed into a constructor instead of passing multiple parametes. Fixed the broken pattern in CustomRedirectsAPI constructor

core/server/services/redirects/api.js

@@ -138,12 +138,12 @@ class CustomRedirectsAPI {
     /**
      * @param {object} config
      * @param {string} config.basePath
-     *
-     * @param {IRedirectManager} redirectManager
+     * @param {IRedirectManager} config.redirectManager
      */
-    constructor(config, redirectManager) {
+    constructor({basePath, redirectManager}) {
         /** @private */
-        this.config = config;
+        this.basePath = basePath;
+
         /** @private */
         this.redirectManager = redirectManager;
     }
@@ -187,7 +187,7 @@ class CustomRedirectsAPI {
      * @returns {string}
      */
     createRedirectsFilePath(ext) {
-        return path.join(this.config.basePath, `redirects${ext}`);
+        return path.join(this.basePath, `redirects${ext}`);
     }
 
     /**

core/server/services/redirects/index.js

@@ -16,8 +16,9 @@ let customRedirectsAPI;
 module.exports = {
     init() {
         customRedirectsAPI = new CustomRedirectsAPI({
-            basePath: config.getContentPath('data')
-        }, redirectManager);
+            basePath: config.getContentPath('data'),
+            redirectManager
+        });
 
         return customRedirectsAPI.init();
     },