From 9ae88819c202a6ce4a36b56506f508a5603e8eab Mon Sep 17 00:00:00 2001 From: Chocobozzz Date: Wed, 24 Jul 2019 11:17:42 +0200 Subject: Change plugin models names --- server/controllers/api/plugins.ts | 4 ++-- server/lib/plugins/plugin-manager.ts | 14 +++++++------- server/tests/api/server/plugins.ts | 4 ++-- server/typings/plugins/index.d.ts | 2 +- server/typings/plugins/plugin-library.model.ts | 4 ++-- server/typings/plugins/register-options.model.ts | 19 ------------------- .../typings/plugins/register-server-option.model.ts | 19 +++++++++++++++++++ 7 files changed, 33 insertions(+), 33 deletions(-) delete mode 100644 server/typings/plugins/register-options.model.ts create mode 100644 server/typings/plugins/register-server-option.model.ts (limited to 'server') diff --git a/server/controllers/api/plugins.ts b/server/controllers/api/plugins.ts index de58a7350..86384ee27 100644 --- a/server/controllers/api/plugins.ts +++ b/server/controllers/api/plugins.ts @@ -25,7 +25,7 @@ import { ManagePlugin } from '../../../shared/models/plugins/manage-plugin.model import { logger } from '../../helpers/logger' import { listAvailablePluginsFromIndex } from '../../lib/plugins/plugin-index' import { PeertubePluginIndexList } from '../../../shared/models/plugins/peertube-plugin-index-list.model' -import { RegisteredSettings } from '../../../shared/models/plugins/register-setting.model' +import { RegisteredServerSettings } from '../../../shared/models/plugins/register-server-setting.model' const pluginRouter = express.Router() @@ -164,7 +164,7 @@ async function uninstallPlugin (req: express.Request, res: express.Response) { function getPluginRegisteredSettings (req: express.Request, res: express.Response) { const settings = PluginManager.Instance.getRegisteredSettings(req.params.npmName) - const json: RegisteredSettings = { settings } + const json: RegisteredServerSettings = { settings } return res.json(json) } diff --git a/server/lib/plugins/plugin-manager.ts b/server/lib/plugins/plugin-manager.ts index a87d02c56..381a89473 100644 --- a/server/lib/plugins/plugin-manager.ts +++ b/server/lib/plugins/plugin-manager.ts @@ -9,15 +9,15 @@ import { PLUGIN_GLOBAL_CSS_PATH } from '../../initializers/constants' import { PluginType } from '../../../shared/models/plugins/plugin.type' import { installNpmPlugin, installNpmPluginFromDisk, removeNpmPlugin } from './yarn' import { outputFile, readJSON } from 'fs-extra' -import { RegisterSettingOptions } from '../../../shared/models/plugins/register-setting.model' -import { RegisterHookOptions } from '../../../shared/models/plugins/register-hook.model' import { PluginSettingsManager } from '../../../shared/models/plugins/plugin-settings-manager.model' import { PluginStorageManager } from '../../../shared/models/plugins/plugin-storage-manager.model' import { ServerHook, ServerHookName, serverHookObject } from '../../../shared/models/plugins/server-hook.model' import { getHookType, internalRunHook } from '../../../shared/core-utils/plugins/hooks' -import { RegisterOptions } from '../../typings/plugins/register-options.model' +import { RegisterServerOptions } from '../../typings/plugins/register-server-option.model' import { PluginLibrary } from '../../typings/plugins' import { ClientHtml } from '../client-html' +import { RegisterServerHookOptions } from '../../../shared/models/plugins/register-server-hook.model' +import { RegisterServerSettingOptions } from '../../../shared/models/plugins/register-server-setting.model' export interface RegisteredPlugin { npmName: string @@ -51,7 +51,7 @@ export class PluginManager implements ServerHook { private static instance: PluginManager private registeredPlugins: { [ name: string ]: RegisteredPlugin } = {} - private settings: { [ name: string ]: RegisterSettingOptions[] } = {} + private settings: { [ name: string ]: RegisterServerSettingOptions[] } = {} private hooks: { [ name: string ]: HookInformationValue[] } = {} private constructor () { @@ -391,8 +391,8 @@ export class PluginManager implements ServerHook { // ###################### Generate register helpers ###################### - private getRegisterHelpers (npmName: string, plugin: PluginModel): RegisterOptions { - const registerHook = (options: RegisterHookOptions) => { + private getRegisterHelpers (npmName: string, plugin: PluginModel): RegisterServerOptions { + const registerHook = (options: RegisterServerHookOptions) => { if (serverHookObject[options.target] !== true) { logger.warn('Unknown hook %s of plugin %s. Skipping.', options.target, npmName) return @@ -408,7 +408,7 @@ export class PluginManager implements ServerHook { }) } - const registerSetting = (options: RegisterSettingOptions) => { + const registerSetting = (options: RegisterServerSettingOptions) => { if (!this.settings[npmName]) this.settings[npmName] = [] this.settings[npmName].push(options) diff --git a/server/tests/api/server/plugins.ts b/server/tests/api/server/plugins.ts index b3d003f45..f8b2d78c9 100644 --- a/server/tests/api/server/plugins.ts +++ b/server/tests/api/server/plugins.ts @@ -23,10 +23,10 @@ import { import { PluginType } from '../../../../shared/models/plugins/plugin.type' import { PeerTubePluginIndex } from '../../../../shared/models/plugins/peertube-plugin-index.model' import { ServerConfig } from '../../../../shared/models/server' -import { RegisteredSettings } from '../../../../shared/models/plugins/register-setting.model' import { PeerTubePlugin } from '../../../../shared/models/plugins/peertube-plugin.model' import { User } from '../../../../shared/models/users' import { PluginPackageJson } from '../../../../shared/models/plugins/plugin-package-json.model' +import { RegisteredServerSettings } from '../../../../shared/models/plugins/register-server-setting.model' const expect = chai.expect @@ -217,7 +217,7 @@ describe('Test plugins', function () { npmName: 'peertube-plugin-hello-world' }) - const settings = (res.body as RegisteredSettings).settings + const settings = (res.body as RegisteredServerSettings).settings expect(settings).to.have.length.at.least(1) diff --git a/server/typings/plugins/index.d.ts b/server/typings/plugins/index.d.ts index bddb3089e..9570579ef 100644 --- a/server/typings/plugins/index.d.ts +++ b/server/typings/plugins/index.d.ts @@ -1,2 +1,2 @@ -export * from './register-options.model' +export * from './register-server-option.model' export * from './plugin-library.model' diff --git a/server/typings/plugins/plugin-library.model.ts b/server/typings/plugins/plugin-library.model.ts index fd90a3b46..5b517ee9f 100644 --- a/server/typings/plugins/plugin-library.model.ts +++ b/server/typings/plugins/plugin-library.model.ts @@ -1,7 +1,7 @@ -import { RegisterOptions } from './register-options.model' +import { RegisterServerOptions } from './register-server-option.model' export interface PluginLibrary { - register: (options: RegisterOptions) => Promise + register: (options: RegisterServerOptions) => Promise unregister: () => Promise } diff --git a/server/typings/plugins/register-options.model.ts b/server/typings/plugins/register-options.model.ts deleted file mode 100644 index 007ef5e1b..000000000 --- a/server/typings/plugins/register-options.model.ts +++ /dev/null @@ -1,19 +0,0 @@ -import { logger } from '../../helpers/logger' -import { RegisterHookOptions } from '../../../shared/models/plugins/register-hook.model' -import { RegisterSettingOptions } from '../../../shared/models/plugins/register-setting.model' -import { PluginSettingsManager } from '../../../shared/models/plugins/plugin-settings-manager.model' -import { PluginStorageManager } from '../../../shared/models/plugins/plugin-storage-manager.model' - -export type RegisterOptions = { - registerHook: (options: RegisterHookOptions) => void - - registerSetting: (options: RegisterSettingOptions) => void - - settingsManager: PluginSettingsManager - - storageManager: PluginStorageManager - - peertubeHelpers: { - logger: typeof logger - } -} diff --git a/server/typings/plugins/register-server-option.model.ts b/server/typings/plugins/register-server-option.model.ts new file mode 100644 index 000000000..91a06a7c5 --- /dev/null +++ b/server/typings/plugins/register-server-option.model.ts @@ -0,0 +1,19 @@ +import { logger } from '../../helpers/logger' +import { PluginSettingsManager } from '../../../shared/models/plugins/plugin-settings-manager.model' +import { PluginStorageManager } from '../../../shared/models/plugins/plugin-storage-manager.model' +import { RegisterServerHookOptions } from '../../../shared/models/plugins/register-server-hook.model' +import { RegisterServerSettingOptions } from '../../../shared/models/plugins/register-server-setting.model' + +export type RegisterServerOptions = { + registerHook: (options: RegisterServerHookOptions) => void + + registerSetting: (options: RegisterServerSettingOptions) => void + + settingsManager: PluginSettingsManager + + storageManager: PluginStorageManager + + peertubeHelpers: { + logger: typeof logger + } +} -- cgit v1.2.3