Skip to content

Commit 7312e8a

Browse files
author
Jacob Schatz
committed
Merge branch 'projects-r-s' into 'master'
refactor for projects-r*, s* See merge request gitlab-org/gitlab-ce!16351
2 parents 00aaf50 + c166960 commit 7312e8a

File tree

9 files changed

+93
-38
lines changed

9 files changed

+93
-38
lines changed

app/assets/javascripts/dispatcher.js

+26-38
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,24 @@
11
/* eslint-disable func-names, space-before-function-paren, no-var, prefer-arrow-callback, wrap-iife, no-shadow, consistent-return, one-var, one-var-declaration-per-line, camelcase, default-case, no-new, quotes, no-duplicate-case, no-case-declarations, no-fallthrough, max-len */
22
import Milestone from './milestone';
3-
import NotificationsForm from './notifications_form';
43
import notificationsDropdown from './notifications_dropdown';
54
import LineHighlighter from './line_highlighter';
65
import MergeRequest from './merge_request';
76
import Sidebar from './right_sidebar';
87
import Flash from './flash';
9-
import SecretValues from './behaviors/secret_values';
108
import UserCallout from './user_callout';
119
import BlobViewer from './blob/viewer/index';
1210
import GfmAutoComplete from './gfm_auto_complete';
1311
import Star from './star';
14-
import TreeView from './tree';
1512
import ZenMode from './zen_mode';
16-
import initSettingsPanels from './settings_panels';
1713
import PerformanceBar from './performance_bar';
1814
import initNotes from './init_notes';
1915
import initIssuableSidebar from './init_issuable_sidebar';
20-
import { ajaxGet, convertPermissionToBoolean } from './lib/utils/common_utils';
16+
import { convertPermissionToBoolean } from './lib/utils/common_utils';
2117
import GlFieldErrors from './gl_field_errors';
22-
import GLForm from './gl_form';
2318
import Shortcuts from './shortcuts';
24-
import ShortcutsNavigation from './shortcuts_navigation';
2519
import ShortcutsIssuable from './shortcuts_issuable';
2620
import Diff from './diff';
2721
import SearchAutocomplete from './search_autocomplete';
28-
import Activities from './activities';
2922

3023
(function() {
3124
var Dispatcher;
@@ -233,15 +226,21 @@ import Activities from './activities';
233226
.catch(fail);
234227
break;
235228
case 'projects:snippets:show':
236-
initNotes();
237-
new ZenMode();
229+
import('./pages/projects/snippets/show')
230+
.then(callDefault)
231+
.catch(fail);
238232
break;
239233
case 'projects:snippets:new':
240-
case 'projects:snippets:edit':
241234
case 'projects:snippets:create':
235+
import('./pages/projects/snippets/new')
236+
.then(callDefault)
237+
.catch(fail);
238+
break;
239+
case 'projects:snippets:edit':
242240
case 'projects:snippets:update':
243-
new GLForm($('.snippet-form'), true);
244-
new ZenMode();
241+
import('./pages/projects/snippets/edit')
242+
.then(callDefault)
243+
.catch(fail);
245244
break;
246245
case 'snippets:new':
247246
import('./pages/snippets/new')
@@ -264,8 +263,9 @@ import Activities from './activities';
264263
.catch(fail);
265264
break;
266265
case 'projects:releases:edit':
267-
new ZenMode();
268-
new GLForm($('.release-form'), true);
266+
import('./pages/projects/releases/edit')
267+
.then(callDefault)
268+
.catch(fail);
269269
break;
270270
case 'projects:merge_requests:show':
271271
new Diff();
@@ -309,19 +309,10 @@ import Activities from './activities';
309309
shortcut_handler = true;
310310
break;
311311
case 'projects:show':
312-
shortcut_handler = new ShortcutsNavigation();
313-
new NotificationsForm();
314-
new UserCallout({
315-
setCalloutPerProject: true,
316-
className: 'js-autodevops-banner',
317-
});
318-
319-
if ($('#tree-slider').length) new TreeView();
320-
if ($('.blob-viewer').length) new BlobViewer();
321-
if ($('.project-show-activity').length) new Activities();
322-
$('#tree-slider').waitForImages(function() {
323-
ajaxGet(document.querySelector('.js-tree-content').dataset.logsPath);
324-
});
312+
import('./pages/projects/show')
313+
.then(callDefault)
314+
.catch(fail);
315+
shortcut_handler = true;
325316
break;
326317
case 'projects:edit':
327318
import('./pages/projects/edit')
@@ -476,18 +467,15 @@ import Activities from './activities';
476467
.catch(fail);
477468
break;
478469
case 'projects:settings:repository:show':
479-
// Initialize expandable settings panels
480-
initSettingsPanels();
470+
import('./pages/projects/settings/repository/show')
471+
.then(callDefault)
472+
.catch(fail);
481473
break;
482474
case 'projects:settings:ci_cd:show':
483-
// Initialize expandable settings panels
484-
initSettingsPanels();
485-
486-
const runnerToken = document.querySelector('.js-secret-runner-token');
487-
if (runnerToken) {
488-
const runnerTokenSecretValue = new SecretValues(runnerToken);
489-
runnerTokenSecretValue.init();
490-
}
475+
import('./pages/projects/settings/ci_cd/show')
476+
.then(callDefault)
477+
.catch(fail);
478+
break;
491479
case 'groups:settings:ci_cd:show':
492480
import('./pages/groups/settings/ci_cd/show')
493481
.then(callDefault)
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
import ZenMode from '~/zen_mode';
2+
import GLForm from '~/gl_form';
3+
4+
export default function ($formEl) {
5+
new ZenMode(); // eslint-disable-line no-new
6+
new GLForm($formEl, true); // eslint-disable-line no-new
7+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import initForm from '~/pages/projects/init_form';
2+
3+
export default initForm($('.release-form'));
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
import initSettingsPanels from '~/settings_panels';
2+
import SecretValues from '~/behaviors/secret_values';
3+
4+
export default function () {
5+
// Initialize expandable settings panels
6+
initSettingsPanels();
7+
const runnerToken = document.querySelector('.js-secret-runner-token');
8+
if (runnerToken) {
9+
const runnerTokenSecretValue = new SecretValues(runnerToken);
10+
runnerTokenSecretValue.init();
11+
}
12+
13+
const secretVariableTable = document.querySelector('.js-secret-variable-table');
14+
if (secretVariableTable) {
15+
const secretVariableTableValues = new SecretValues(secretVariableTable);
16+
secretVariableTableValues.init();
17+
}
18+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import initSettingsPanels from '~/settings_panels';
2+
3+
export default initSettingsPanels;
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
import ShortcutsNavigation from '~/shortcuts_navigation';
2+
import NotificationsForm from '~/notifications_form';
3+
import UserCallout from '~/user_callout';
4+
import TreeView from '~/tree';
5+
import BlobViewer from '~/blob/viewer/index';
6+
import Activities from '~/activities';
7+
import { ajaxGet } from '~/lib/utils/common_utils';
8+
9+
export default () => {
10+
new ShortcutsNavigation(); // eslint-disable-line no-new
11+
new NotificationsForm(); // eslint-disable-line no-new
12+
new UserCallout({ // eslint-disable-line no-new
13+
setCalloutPerProject: true,
14+
className: 'js-autodevops-banner',
15+
});
16+
17+
if ($('#tree-slider').length) new TreeView(); // eslint-disable-line no-new
18+
if ($('.blob-viewer').length) new BlobViewer(); // eslint-disable-line no-new
19+
if ($('.project-show-activity').length) new Activities(); // eslint-disable-line no-new
20+
$('#tree-slider').waitForImages(() => {
21+
ajaxGet(document.querySelector('.js-tree-content').dataset.logsPath);
22+
});
23+
};
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import initForm from '~/pages/projects/init_form';
2+
3+
export default initForm($('.snippet-form'));
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
import initForm from '~/pages/projects/init_form';
2+
3+
export default initForm($('.snippet-form'));
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
import initNotes from '~/init_notes';
2+
import ZenMode from '~/zen_mode';
3+
4+
export default function () {
5+
initNotes();
6+
new ZenMode(); // eslint-disable-line no-new
7+
}

0 commit comments

Comments
 (0)