diff options
Diffstat (limited to 'server')
-rw-r--r-- | server/controllers/api/plugins.ts | 4 | ||||
-rw-r--r-- | server/lib/plugins/plugin-manager.ts | 14 | ||||
-rw-r--r-- | server/tests/api/server/plugins.ts | 4 | ||||
-rw-r--r-- | server/typings/plugins/index.d.ts | 2 | ||||
-rw-r--r-- | server/typings/plugins/plugin-library.model.ts | 4 | ||||
-rw-r--r-- | server/typings/plugins/register-server-option.model.ts (renamed from server/typings/plugins/register-options.model.ts) | 10 |
6 files changed, 19 insertions, 19 deletions
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 | |||
25 | import { logger } from '../../helpers/logger' | 25 | import { logger } from '../../helpers/logger' |
26 | import { listAvailablePluginsFromIndex } from '../../lib/plugins/plugin-index' | 26 | import { listAvailablePluginsFromIndex } from '../../lib/plugins/plugin-index' |
27 | import { PeertubePluginIndexList } from '../../../shared/models/plugins/peertube-plugin-index-list.model' | 27 | import { PeertubePluginIndexList } from '../../../shared/models/plugins/peertube-plugin-index-list.model' |
28 | import { RegisteredSettings } from '../../../shared/models/plugins/register-setting.model' | 28 | import { RegisteredServerSettings } from '../../../shared/models/plugins/register-server-setting.model' |
29 | 29 | ||
30 | const pluginRouter = express.Router() | 30 | const pluginRouter = express.Router() |
31 | 31 | ||
@@ -164,7 +164,7 @@ async function uninstallPlugin (req: express.Request, res: express.Response) { | |||
164 | function getPluginRegisteredSettings (req: express.Request, res: express.Response) { | 164 | function getPluginRegisteredSettings (req: express.Request, res: express.Response) { |
165 | const settings = PluginManager.Instance.getRegisteredSettings(req.params.npmName) | 165 | const settings = PluginManager.Instance.getRegisteredSettings(req.params.npmName) |
166 | 166 | ||
167 | const json: RegisteredSettings = { settings } | 167 | const json: RegisteredServerSettings = { settings } |
168 | 168 | ||
169 | return res.json(json) | 169 | return res.json(json) |
170 | } | 170 | } |
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' | |||
9 | import { PluginType } from '../../../shared/models/plugins/plugin.type' | 9 | import { PluginType } from '../../../shared/models/plugins/plugin.type' |
10 | import { installNpmPlugin, installNpmPluginFromDisk, removeNpmPlugin } from './yarn' | 10 | import { installNpmPlugin, installNpmPluginFromDisk, removeNpmPlugin } from './yarn' |
11 | import { outputFile, readJSON } from 'fs-extra' | 11 | import { outputFile, readJSON } from 'fs-extra' |
12 | import { RegisterSettingOptions } from '../../../shared/models/plugins/register-setting.model' | ||
13 | import { RegisterHookOptions } from '../../../shared/models/plugins/register-hook.model' | ||
14 | import { PluginSettingsManager } from '../../../shared/models/plugins/plugin-settings-manager.model' | 12 | import { PluginSettingsManager } from '../../../shared/models/plugins/plugin-settings-manager.model' |
15 | import { PluginStorageManager } from '../../../shared/models/plugins/plugin-storage-manager.model' | 13 | import { PluginStorageManager } from '../../../shared/models/plugins/plugin-storage-manager.model' |
16 | import { ServerHook, ServerHookName, serverHookObject } from '../../../shared/models/plugins/server-hook.model' | 14 | import { ServerHook, ServerHookName, serverHookObject } from '../../../shared/models/plugins/server-hook.model' |
17 | import { getHookType, internalRunHook } from '../../../shared/core-utils/plugins/hooks' | 15 | import { getHookType, internalRunHook } from '../../../shared/core-utils/plugins/hooks' |
18 | import { RegisterOptions } from '../../typings/plugins/register-options.model' | 16 | import { RegisterServerOptions } from '../../typings/plugins/register-server-option.model' |
19 | import { PluginLibrary } from '../../typings/plugins' | 17 | import { PluginLibrary } from '../../typings/plugins' |
20 | import { ClientHtml } from '../client-html' | 18 | import { ClientHtml } from '../client-html' |
19 | import { RegisterServerHookOptions } from '../../../shared/models/plugins/register-server-hook.model' | ||
20 | import { RegisterServerSettingOptions } from '../../../shared/models/plugins/register-server-setting.model' | ||
21 | 21 | ||
22 | export interface RegisteredPlugin { | 22 | export interface RegisteredPlugin { |
23 | npmName: string | 23 | npmName: string |
@@ -51,7 +51,7 @@ export class PluginManager implements ServerHook { | |||
51 | private static instance: PluginManager | 51 | private static instance: PluginManager |
52 | 52 | ||
53 | private registeredPlugins: { [ name: string ]: RegisteredPlugin } = {} | 53 | private registeredPlugins: { [ name: string ]: RegisteredPlugin } = {} |
54 | private settings: { [ name: string ]: RegisterSettingOptions[] } = {} | 54 | private settings: { [ name: string ]: RegisterServerSettingOptions[] } = {} |
55 | private hooks: { [ name: string ]: HookInformationValue[] } = {} | 55 | private hooks: { [ name: string ]: HookInformationValue[] } = {} |
56 | 56 | ||
57 | private constructor () { | 57 | private constructor () { |
@@ -391,8 +391,8 @@ export class PluginManager implements ServerHook { | |||
391 | 391 | ||
392 | // ###################### Generate register helpers ###################### | 392 | // ###################### Generate register helpers ###################### |
393 | 393 | ||
394 | private getRegisterHelpers (npmName: string, plugin: PluginModel): RegisterOptions { | 394 | private getRegisterHelpers (npmName: string, plugin: PluginModel): RegisterServerOptions { |
395 | const registerHook = (options: RegisterHookOptions) => { | 395 | const registerHook = (options: RegisterServerHookOptions) => { |
396 | if (serverHookObject[options.target] !== true) { | 396 | if (serverHookObject[options.target] !== true) { |
397 | logger.warn('Unknown hook %s of plugin %s. Skipping.', options.target, npmName) | 397 | logger.warn('Unknown hook %s of plugin %s. Skipping.', options.target, npmName) |
398 | return | 398 | return |
@@ -408,7 +408,7 @@ export class PluginManager implements ServerHook { | |||
408 | }) | 408 | }) |
409 | } | 409 | } |
410 | 410 | ||
411 | const registerSetting = (options: RegisterSettingOptions) => { | 411 | const registerSetting = (options: RegisterServerSettingOptions) => { |
412 | if (!this.settings[npmName]) this.settings[npmName] = [] | 412 | if (!this.settings[npmName]) this.settings[npmName] = [] |
413 | 413 | ||
414 | this.settings[npmName].push(options) | 414 | 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 { | |||
23 | import { PluginType } from '../../../../shared/models/plugins/plugin.type' | 23 | import { PluginType } from '../../../../shared/models/plugins/plugin.type' |
24 | import { PeerTubePluginIndex } from '../../../../shared/models/plugins/peertube-plugin-index.model' | 24 | import { PeerTubePluginIndex } from '../../../../shared/models/plugins/peertube-plugin-index.model' |
25 | import { ServerConfig } from '../../../../shared/models/server' | 25 | import { ServerConfig } from '../../../../shared/models/server' |
26 | import { RegisteredSettings } from '../../../../shared/models/plugins/register-setting.model' | ||
27 | import { PeerTubePlugin } from '../../../../shared/models/plugins/peertube-plugin.model' | 26 | import { PeerTubePlugin } from '../../../../shared/models/plugins/peertube-plugin.model' |
28 | import { User } from '../../../../shared/models/users' | 27 | import { User } from '../../../../shared/models/users' |
29 | import { PluginPackageJson } from '../../../../shared/models/plugins/plugin-package-json.model' | 28 | import { PluginPackageJson } from '../../../../shared/models/plugins/plugin-package-json.model' |
29 | import { RegisteredServerSettings } from '../../../../shared/models/plugins/register-server-setting.model' | ||
30 | 30 | ||
31 | const expect = chai.expect | 31 | const expect = chai.expect |
32 | 32 | ||
@@ -217,7 +217,7 @@ describe('Test plugins', function () { | |||
217 | npmName: 'peertube-plugin-hello-world' | 217 | npmName: 'peertube-plugin-hello-world' |
218 | }) | 218 | }) |
219 | 219 | ||
220 | const settings = (res.body as RegisteredSettings).settings | 220 | const settings = (res.body as RegisteredServerSettings).settings |
221 | 221 | ||
222 | expect(settings).to.have.length.at.least(1) | 222 | expect(settings).to.have.length.at.least(1) |
223 | 223 | ||
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 @@ | |||
1 | export * from './register-options.model' | 1 | export * from './register-server-option.model' |
2 | export * from './plugin-library.model' | 2 | 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 @@ | |||
1 | import { RegisterOptions } from './register-options.model' | 1 | import { RegisterServerOptions } from './register-server-option.model' |
2 | 2 | ||
3 | export interface PluginLibrary { | 3 | export interface PluginLibrary { |
4 | register: (options: RegisterOptions) => Promise<any> | 4 | register: (options: RegisterServerOptions) => Promise<any> |
5 | 5 | ||
6 | unregister: () => Promise<any> | 6 | unregister: () => Promise<any> |
7 | } | 7 | } |
diff --git a/server/typings/plugins/register-options.model.ts b/server/typings/plugins/register-server-option.model.ts index 007ef5e1b..91a06a7c5 100644 --- a/server/typings/plugins/register-options.model.ts +++ b/server/typings/plugins/register-server-option.model.ts | |||
@@ -1,13 +1,13 @@ | |||
1 | import { logger } from '../../helpers/logger' | 1 | import { logger } from '../../helpers/logger' |
2 | import { RegisterHookOptions } from '../../../shared/models/plugins/register-hook.model' | ||
3 | import { RegisterSettingOptions } from '../../../shared/models/plugins/register-setting.model' | ||
4 | import { PluginSettingsManager } from '../../../shared/models/plugins/plugin-settings-manager.model' | 2 | import { PluginSettingsManager } from '../../../shared/models/plugins/plugin-settings-manager.model' |
5 | import { PluginStorageManager } from '../../../shared/models/plugins/plugin-storage-manager.model' | 3 | import { PluginStorageManager } from '../../../shared/models/plugins/plugin-storage-manager.model' |
4 | import { RegisterServerHookOptions } from '../../../shared/models/plugins/register-server-hook.model' | ||
5 | import { RegisterServerSettingOptions } from '../../../shared/models/plugins/register-server-setting.model' | ||
6 | 6 | ||
7 | export type RegisterOptions = { | 7 | export type RegisterServerOptions = { |
8 | registerHook: (options: RegisterHookOptions) => void | 8 | registerHook: (options: RegisterServerHookOptions) => void |
9 | 9 | ||
10 | registerSetting: (options: RegisterSettingOptions) => void | 10 | registerSetting: (options: RegisterServerSettingOptions) => void |
11 | 11 | ||
12 | settingsManager: PluginSettingsManager | 12 | settingsManager: PluginSettingsManager |
13 | 13 | ||