From 5025412fc4ab16cc7049a38d43fdc2e4095a1f87 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Mon, 31 Oct 2022 09:09:32 +0000 Subject: [PATCH] Add latest changes from gitlab-org/gitlab@master --- .../behaviors/markdown/render_gfm.js | 2 +- .../markdown/render_sandboxed_mermaid.js | 127 +++++----- .../components/related_issues_block.vue | 14 +- .../components/related_issues_root.vue | 10 +- .../webhooks/components/form_url_app.vue | 15 +- .../components/form_url_mask_item.vue | 20 +- app/assets/javascripts/webhooks/constants.js | 2 + app/assets/javascripts/webhooks/index.js | 7 +- app/helpers/hooks_helper.rb | 2 +- .../clusters/applications/patch_service.rb | 32 --- .../clusters/applications/update_service.rb | 17 -- app/workers/cluster_patch_app_worker.rb | 9 +- ..._add_temp_index_for_user_details_fields.rb | 22 ++ ...2459_queue_backfill_user_details_fields.rb | 16 ++ db/schema_migrations/20221018232820 | 1 + db/schema_migrations/20221019002459 | 1 + db/structure.sql | 2 + doc/.markdownlint/require_helper.js | 14 ++ doc/.markdownlint/rules/tabs_blank_lines.js | 26 ++ doc/.markdownlint/rules/tabs_title_markup.js | 31 +++ doc/.markdownlint/rules/tabs_title_text.js | 23 ++ doc/.markdownlint/rules/tabs_wrapper_tags.js | 21 ++ lib/api/entities/pull_mirror.rb | 16 +- .../backfill_user_details_fields.rb | 61 +++++ .../projects/menus/deployments_menu.rb | 2 +- locale/gitlab.pot | 6 +- package.json | 2 +- scripts/lint-doc.sh | 2 +- spec/factories/project_hooks.rb | 4 + .../markdown/render_sandboxed_mermaid_spec.js | 147 +++--------- .../components/related_issues_root_spec.js | 8 +- .../webhooks/components/form_url_app_spec.js | 38 ++- .../components/form_url_mask_item_spec.js | 37 ++- spec/helpers/hooks_helper_spec.rb | 35 ++- .../backfill_user_details_fields_spec.rb | 222 ++++++++++++++++++ ...queue_backfill_user_details_fields_spec.rb | 24 ++ .../applications/patch_service_spec.rb | 80 ------- .../applications/update_service_spec.rb | 91 ------- .../navbar_structure_context.rb | 2 +- .../has_web_hook_shared_examples.rb | 2 +- yarn.lock | 8 +- 41 files changed, 729 insertions(+), 472 deletions(-) delete mode 100644 app/services/clusters/applications/patch_service.rb delete mode 100644 app/services/clusters/applications/update_service.rb create mode 100644 db/post_migrate/20221018232820_add_temp_index_for_user_details_fields.rb create mode 100644 db/post_migrate/20221019002459_queue_backfill_user_details_fields.rb create mode 100644 db/schema_migrations/20221018232820 create mode 100644 db/schema_migrations/20221019002459 create mode 100644 doc/.markdownlint/require_helper.js create mode 100644 doc/.markdownlint/rules/tabs_blank_lines.js create mode 100644 doc/.markdownlint/rules/tabs_title_markup.js create mode 100644 doc/.markdownlint/rules/tabs_title_text.js create mode 100644 doc/.markdownlint/rules/tabs_wrapper_tags.js create mode 100644 lib/gitlab/background_migration/backfill_user_details_fields.rb create mode 100644 spec/lib/gitlab/background_migration/backfill_user_details_fields_spec.rb create mode 100644 spec/migrations/queue_backfill_user_details_fields_spec.rb delete mode 100644 spec/services/clusters/applications/patch_service_spec.rb delete mode 100644 spec/services/clusters/applications/update_service_spec.rb diff --git a/app/assets/javascripts/behaviors/markdown/render_gfm.js b/app/assets/javascripts/behaviors/markdown/render_gfm.js index a08cf48c327..ee5c0fe5ef3 100644 --- a/app/assets/javascripts/behaviors/markdown/render_gfm.js +++ b/app/assets/javascripts/behaviors/markdown/render_gfm.js @@ -15,7 +15,7 @@ $.fn.renderGFM = function renderGFM() { syntaxHighlight(this.find('.js-syntax-highlight').get()); renderKroki(this.find('.js-render-kroki[hidden]').get()); renderMath(this.find('.js-render-math')); - renderSandboxedMermaid(this.find('.js-render-mermaid').get()); + renderSandboxedMermaid(this.find('.js-render-mermaid')); renderJSONTable( Array.from(this.find('[lang="json"][data-lang-params="table"]').get()).map((e) => e.parentNode), ); diff --git a/app/assets/javascripts/behaviors/markdown/render_sandboxed_mermaid.js b/app/assets/javascripts/behaviors/markdown/render_sandboxed_mermaid.js index 031b03e0d59..077e96b2fee 100644 --- a/app/assets/javascripts/behaviors/markdown/render_sandboxed_mermaid.js +++ b/app/assets/javascripts/behaviors/markdown/render_sandboxed_mermaid.js @@ -1,4 +1,5 @@ -import { countBy } from 'lodash'; +import $ from 'jquery'; +import { once, countBy } from 'lodash'; import { __ } from '~/locale'; import { getBaseURL, @@ -7,8 +8,7 @@ import { joinPaths, } from '~/lib/utils/url_utility'; import { darkModeEnabled } from '~/lib/utils/color_utils'; -import { setAttributes, isElementVisible } from '~/lib/utils/dom_utils'; -import { createAlert, VARIANT_WARNING } from '~/flash'; +import { setAttributes } from '~/lib/utils/dom_utils'; import { unrestrictedPages } from './constants'; // Renders diagrams and flowcharts from text using Mermaid in any element with the @@ -27,30 +27,17 @@ import { unrestrictedPages } from './constants'; const SANDBOX_FRAME_PATH = '/-/sandbox/mermaid'; // This is an arbitrary number; Can be iterated upon when suitable. -export const MAX_CHAR_LIMIT = 2000; +const MAX_CHAR_LIMIT = 2000; // Max # of mermaid blocks that can be rendered in a page. -export const MAX_MERMAID_BLOCK_LIMIT = 50; +const MAX_MERMAID_BLOCK_LIMIT = 50; // Max # of `&` allowed in Chaining of links syntax const MAX_CHAINING_OF_LINKS_LIMIT = 30; - export const BUFFER_IFRAME_HEIGHT = 10; export const SANDBOX_ATTRIBUTES = 'allow-scripts allow-popups'; - -const ALERT_CONTAINER_CLASS = 'mermaid-alert-container'; -export const LAZY_ALERT_SHOWN_CLASS = 'lazy-alert-shown'; - // Keep a map of mermaid blocks we've already rendered. const elsProcessingMap = new WeakMap(); let renderedMermaidBlocks = 0; -/** - * Determines whether a given Mermaid diagram is visible. - * - * @param {Element} el The Mermaid DOM node - * @returns - */ -const isVisibleMermaid = (el) => el.closest('details') === null && isElementVisible(el); - function shouldLazyLoadMermaidBlock(source) { /** * If source contains `&`, which means that it might @@ -117,8 +104,8 @@ function renderMermaidEl(el, source) { ); } -function renderMermaids(els) { - if (!els.length) return; +function renderMermaids($els) { + if (!$els.length) return; const pageName = document.querySelector('body').dataset.page; @@ -127,7 +114,7 @@ function renderMermaids(els) { let renderedChars = 0; - els.forEach((el) => { + $els.each((i, el) => { // Skipping all the elements which we've already queued in requestIdleCallback if (elsProcessingMap.has(el)) { return; @@ -146,29 +133,33 @@ function renderMermaids(els) { renderedMermaidBlocks >= MAX_MERMAID_BLOCK_LIMIT || shouldLazyLoadMermaidBlock(source)) ) { - const parent = el.parentNode; + const html = ` + + `; - if (!parent.classList.contains(LAZY_ALERT_SHOWN_CLASS)) { - const alertContainer = document.createElement('div'); - alertContainer.classList.add(ALERT_CONTAINER_CLASS); - alertContainer.classList.add('gl-mb-5'); - parent.after(alertContainer); - createAlert({ - message: __( - 'Warning: Displaying this diagram might cause performance issues on this page.', - ), - variant: VARIANT_WARNING, - parent: parent.parentNode, - containerSelector: `.${ALERT_CONTAINER_CLASS}`, - primaryButton: { - text: __('Display'), - clickHandler: () => { - alertContainer.remove(); - renderMermaidEl(el, source); - }, - }, - }); - parent.classList.add(LAZY_ALERT_SHOWN_CLASS); + const $parent = $(el).parent(); + + if (!$parent.hasClass('lazy-alert-shown')) { + $parent.after(html); + $parent + .siblings() + .find('.js-warning-text') + .text( + __('Warning: Displaying this diagram might cause performance issues on this page.'), + ); + $parent.addClass('lazy-alert-shown'); } return; @@ -185,33 +176,37 @@ function renderMermaids(els) { }); } -export default function renderMermaid(els) { - if (!els.length) return; +const hookLazyRenderMermaidEvent = once(() => { + $(document.body).on('click', '.js-lazy-render-mermaid', function eventHandler() { + const parent = $(this).closest('.js-lazy-render-mermaid-container'); + const pre = parent.prev(); - const visibleMermaids = []; - const hiddenMermaids = []; + const el = pre.find('.js-render-mermaid'); - for (const el of els) { - if (isVisibleMermaid(el)) { - visibleMermaids.push(el); - } else { - hiddenMermaids.push(el); - } - } + parent.remove(); + + // sandbox update + const element = el.get(0); + const { source } = fixElementSource(element); + + renderMermaidEl(element, source); + }); +}); + +export default function renderMermaid($els) { + if (!$els.length) return; + + const visibleMermaids = $els.filter(function filter() { + return $(this).closest('details').length === 0 && $(this).is(':visible'); + }); renderMermaids(visibleMermaids); - hiddenMermaids.forEach((el) => { - el.closest('details').addEventListener( - 'toggle', - ({ target: details }) => { - if (details.open) { - renderMermaids([...details.querySelectorAll('.js-render-mermaid')]); - } - }, - { - once: true, - }, - ); + $els.closest('details').one('toggle', function toggle() { + if (this.open) { + renderMermaids($(this).find('.js-render-mermaid')); + } }); + + hookLazyRenderMermaidEvent(); } diff --git a/app/assets/javascripts/related_issues/components/related_issues_block.vue b/app/assets/javascripts/related_issues/components/related_issues_block.vue index 1ab41ee2f0a..c3726ebc14a 100644 --- a/app/assets/javascripts/related_issues/components/related_issues_block.vue +++ b/app/assets/javascripts/related_issues/components/related_issues_block.vue @@ -95,6 +95,16 @@ export default { required: false, default: true, }, + hasError: { + type: Boolean, + required: false, + default: false, + }, + itemAddFailureMessage: { + type: String, + required: false, + default: '', + }, }, data() { return { @@ -233,7 +243,7 @@ export default {