Skip to content

Commit

Permalink
[uiApp] remove unused templateName config option
Browse files Browse the repository at this point in the history
  • Loading branch information
spalger committed Mar 30, 2018
1 parent 3a30d0b commit ec5e41c
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 16 deletions.
9 changes: 0 additions & 9 deletions src/ui/ui_apps/__tests__/ui_app.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ function createStubUiAppSpec(extraParams) {
linkToLastSubUrl: true,
hidden: false,
listed: false,
templateName: 'ui_app_test',
uses: [
'visTypes',
'chromeNavControls',
Expand Down Expand Up @@ -73,10 +72,6 @@ describe('ui apps / UiApp', () => {
expect(app.getPluginId()).to.be(undefined);
});

it('has a default template name of ui_app', () => {
expect(app.getTemplateName()).to.be('ui_app');
});

it('is not hidden', () => {
expect(app.isHidden()).to.be(false);
});
Expand Down Expand Up @@ -126,10 +121,6 @@ describe('ui apps / UiApp', () => {
expect(app.getPluginId()).to.be(undefined);
});

it('uses the specs template', () => {
expect(app.getTemplateName()).to.be(spec.templateName);
});

it('is not hidden', () => {
expect(app.isHidden()).to.be(false);
});
Expand Down
6 changes: 0 additions & 6 deletions src/ui/ui_apps/ui_app.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ export class UiApp {
hidden,
linkToLastSubUrl,
listed,
templateName = 'ui_app',
injectVars,
url = `/app/${id}`,
uses = []
Expand All @@ -32,7 +31,6 @@ export class UiApp {
this._linkToLastSubUrl = linkToLastSubUrl;
this._hidden = hidden;
this._listed = listed;
this._templateName = templateName;
this._url = url;
this._injectedVarsProvider = injectVars;
this._pluginId = pluginId;
Expand Down Expand Up @@ -79,10 +77,6 @@ export class UiApp {
return plugin ? plugin.id : undefined;
}

getTemplateName() {
return this._templateName;
}

isHidden() {
return !!this._hidden;
}
Expand Down
2 changes: 1 addition & 1 deletion src/ui/ui_render/ui_render_mixin.js
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,7 @@ export function uiRenderMixin(kbnServer, server, config) {
const request = reply.request;
const translations = await request.getUiTranslations();

return reply.view(app.getTemplateName(), {
return reply.view('ui_app', {
app,
kibanaPayload: await getKibanaPayload({
app,
Expand Down

0 comments on commit ec5e41c

Please sign in to comment.