2d0aee4e68
After recent refactorings status images popup started to fetch branches info whenever a repo page was opened, resulting in additional HTTP requests. Furthermore, because of a way we load branches, it could result in builds view displaying very old builds, because in API V2 we essentially download last build for each branch for branches request. This commit fixes the situation in 2 ways: 1. We wait with downloading branhes till the popup is open 2. We use a V3 requests to download branches and we don't put that data into the store
62 lines
1.8 KiB
JavaScript
62 lines
1.8 KiB
JavaScript
import Ember from 'ember';
|
|
import { format as formatStatusImage } from 'travis/utils/status-image-formats';
|
|
import Config from 'travis/config/environment';
|
|
|
|
export default Ember.Component.extend({
|
|
popup: Ember.inject.service(),
|
|
auth: Ember.inject.service(),
|
|
popupNameBinding: 'popup.popupName',
|
|
|
|
id: 'status-images',
|
|
attributeBindings: ['id'],
|
|
classNames: ['popup', 'status-images'],
|
|
formats: ['Image URL', 'Markdown', 'Textile', 'Rdoc', 'AsciiDoc', 'RST', 'Pod', 'CCTray'],
|
|
|
|
branches: function() {
|
|
let repoId = this.get('repo.id'),
|
|
popupName = this.get('popupName');
|
|
|
|
if(popupName === 'status-images') {
|
|
let array = Ember.ArrayProxy.create({ content: [] }),
|
|
apiEndpoint = Config.apiEndpoint,
|
|
options = {};
|
|
|
|
array.set('isLoaded', false);
|
|
|
|
if (this.get('auth.signedIn')) {
|
|
options.headers = {
|
|
Authorization: "token " + (this.auth.token())
|
|
};
|
|
}
|
|
|
|
$.ajax(apiEndpoint + "/v3/repo/" + repoId + "/branches?limit=100", options).then(function(response) {
|
|
if(response.branches.length) {
|
|
array.pushObjects(response.branches.map((branch) => { return branch.name; }));
|
|
} else {
|
|
array.pushObject('master');
|
|
}
|
|
|
|
array.set('isLoaded', true);
|
|
});
|
|
|
|
return array;
|
|
} else {
|
|
// if status images popup is not open, don't fetch any branches
|
|
return [];
|
|
}
|
|
}.property('popupName', 'repo'),
|
|
|
|
actions: {
|
|
close() {
|
|
return this.get('popup').close();
|
|
}
|
|
},
|
|
|
|
statusString: function() {
|
|
let format = this.get('format') || this.get('formats.firstObject'),
|
|
branch = this.get('branch') || 'master';
|
|
|
|
return formatStatusImage(format, this.get('repo.slug'), branch);
|
|
}.property('format', 'repo.slug', 'branch')
|
|
});
|