opendev/gerrit
Code Issues Proposed changes

Revert "Fix plugin .url() calculation when loaded from bundle"

Browse Source 
This reverts commit b325113980.

Reason for revert: This never passed the Gerrit-CI validation
                   and actually broke the build.

Change-Id: Ib30ff6a4cae7c8927fa212dd5ae8b87c0cd0d9ef
This commit is contained in:
Luca Milanesio
2018-12-29 01:03:32 +00:00
parent d0ee0c15f2
commit 3d27e0ce8a
2 changed files with 5 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-js-api-interface_test.html
View File

@@ -441,11 +441,11 @@ limitations under the License.
test('installing preloaded plugin', () => {
let plugin;
window.ASSETS_PATH = 'http://blips.com/chitz';
window.ASSETS_PATH = 'http://blips.com/chitz/';
Gerrit.install(p => { plugin = p; }, '0.1', 'preloaded:foo');
assert.strictEqual(plugin.getPluginName(), 'foo');
assert.strictEqual(plugin.url('/some/thing.html'),
'http://blips.com/chitz/plugins/foo/some/thing.html');
'http://blips.com/plugins/foo/some/thing.html');
delete window.ASSETS_PATH;
});

11
polygerrit-ui/app/elements/shared/gr-js-api-interface/gr-public-js-api.js
View File

@@ -217,14 +217,9 @@
};
Plugin.prototype.url = function(opt_path) {
const relPath = '/plugins/' + this._name + (opt_path || '/');
if (window.location.origin === this._url.origin) {
// Plugin loaded from the same origin as gr-app, getBaseUrl in effect.
return this._url.origin + Gerrit.BaseUrlBehavior.getBaseUrl() + relPath;
} else {
// Plugin loaded from assets bundle, expect assets placed along with it.
return this._url.href.split('/plugins/' + this._name)[0] + relPath;
}
const base = Gerrit.BaseUrlBehavior.getBaseUrl();
return this._url.origin + base + '/plugins/' +
this._name + (opt_path || '/');
};
Plugin.prototype.screenUrl = function(opt_screenName) {