Remove Redcarpet markdown engine

This engine was replaced with CommonMarker in 11.4, it was deprecated
since then.

Gemfile

@@ -116,7 +116,6 @@ gem 'html-pipeline', '~> 2.8'
 gem 'deckar01-task_list', '2.2.0'
 gem 'gitlab-markup', '~> 1.6.5'
 gem 'github-markup', '~> 1.7.0', require: 'github/markup'
-gem 'redcarpet', '~> 3.4'
 gem 'commonmarker', '~> 0.17'
 gem 'RedCloth', '~> 4.3.2'
 gem 'rdoc', '~> 6.0'

Gemfile.lock

@@ -682,7 +682,6 @@ GEM
     recaptcha (3.0.0)
       json
     recursive-open-struct (1.1.0)
-    redcarpet (3.4.0)
     redis (3.3.5)
     redis-actionpack (5.0.2)
       actionpack (>= 4.0, < 6)
@@ -1118,7 +1117,6 @@ DEPENDENCIES
   rdoc (~> 6.0)
   re2 (~> 1.1.1)
   recaptcha (~> 3.0)
-  redcarpet (~> 3.4)
   redis (~> 3.2)
   redis-namespace (~> 1.6.0)
   redis-rails (~> 5.0.2)

app/assets/javascripts/behaviors/preview_markdown.js

@@ -28,16 +28,13 @@ MarkdownPreview.prototype.ajaxCache = {};
 MarkdownPreview.prototype.showPreview = function($form) {
   var mdText;
-  var markdownVersion;
-  var url;
   var preview = $form.find('.js-md-preview');
+  var url = preview.data('url');
   if (preview.hasClass('md-preview-loading')) {
     return;
   }
   mdText = $form.find('textarea.markdown-area').val();
-  markdownVersion = $form.attr('data-markdown-version');
-  url = this.versionedPreviewPath(preview.data('url'), markdownVersion);
   if (mdText.trim().length === 0) {
     preview.text(this.emptyMessage);
@@ -67,16 +64,6 @@ MarkdownPreview.prototype.showPreview = function($form) {
   }
 };
-MarkdownPreview.prototype.versionedPreviewPath = function(markdownPreviewPath, markdownVersion) {
-  if (typeof markdownVersion === 'undefined') {
-    return markdownPreviewPath;
-  }
-
-  return `${markdownPreviewPath}${
-    markdownPreviewPath.indexOf('?') === -1 ? '?' : '&'
-  }markdown_version=${markdownVersion}`;
-};
-
 MarkdownPreview.prototype.fetchMarkdownPreview = function(text, url, success) {
   if (!url) {
     return;

app/assets/javascripts/issue_show/components/app.vue

@@ -108,11 +108,6 @@ export default {
       type: String,
       required: true,
     },
-    markdownVersion: {
-      type: Number,
-      required: false,
-      default: 0,
-    },
     projectPath: {
       type: String,
       required: true,
@@ -313,7 +308,6 @@ export default {
         :issuable-templates="issuableTemplates"
         :markdown-docs-path="markdownDocsPath"
         :markdown-preview-path="markdownPreviewPath"
-        :markdown-version="markdownVersion"
         :project-path="projectPath"
         :project-namespace="projectNamespace"
         :show-delete-button="showDeleteButton"

app/assets/javascripts/issue_show/components/fields/description.vue

@@ -20,11 +20,6 @@ export default {
       type: String,
       required: true,
     },
-    markdownVersion: {
-      type: Number,
-      required: false,
-      default: 0,
-    },
     canAttachFile: {
       type: Boolean,
       required: false,
@@ -48,7 +43,6 @@ export default {
     <markdown-field
       :markdown-preview-path="markdownPreviewPath"
       :markdown-docs-path="markdownDocsPath"
-      :markdown-version="markdownVersion"
       :can-attach-file="canAttachFile"
       :enable-autocomplete="enableAutocomplete"
     >

app/assets/javascripts/issue_show/components/form.vue

@@ -39,11 +39,6 @@ export default {
       type: String,
       required: true,
     },
-    markdownVersion: {
-      type: Number,
-      required: false,
-      default: 0,
-    },
     projectPath: {
       type: String,
       required: true,
@@ -101,7 +96,6 @@ export default {
       :form-state="formState"
       :markdown-preview-path="markdownPreviewPath"
       :markdown-docs-path="markdownDocsPath"
-      :markdown-version="markdownVersion"
       :can-attach-file="canAttachFile"
       :enable-autocomplete="enableAutocomplete"
     />

app/assets/javascripts/notes.js

@@ -1239,15 +1239,13 @@ export default class Notes {
     var postUrl = $originalContentEl.data('postUrl');
     var targetId = $originalContentEl.data('targetId');
     var targetType = $originalContentEl.data('targetType');
-    var markdownVersion = $originalContentEl.data('markdownVersion');
     this.glForm = new GLForm($editForm.find('form'), this.enableGFM);
     $editForm
       .find('form')
       .attr('action', `${postUrl}?html=true`)
-      .attr('data-remote', 'true')
-      .attr('data-markdown-version', markdownVersion);
+      .attr('data-remote', 'true');
     $editForm.find('.js-form-target-id').val(targetId);
     $editForm.find('.js-form-target-type').val(targetType);
     $editForm

app/assets/javascripts/notes/components/comment_form.vue

@@ -39,11 +39,6 @@ export default {
       type: String,
       required: true,
     },
-    markdownVersion: {
-      type: Number,
-      required: false,
-      default: 0,
-    },
   },
   data() {
     return {
@@ -342,7 +337,6 @@ Please check your network connection and try again.`;
               :markdown-preview-path="markdownPreviewPath"
               :markdown-docs-path="markdownDocsPath"
               :quick-actions-docs-path="quickActionsDocsPath"
-              :markdown-version="markdownVersion"
               :add-spacing-classes="false"
             >
               <textarea

app/assets/javascripts/notes/components/note_body.vue

@@ -111,7 +111,6 @@ export default {
       :line="line"
       :note="note"
       :help-page-path="helpPagePath"
-      :markdown-version="note.cached_markdown_version"
       @handleFormUpdate="handleFormUpdate"
       @cancelForm="formCancelHandler"
     />

app/assets/javascripts/notes/components/note_form.vue

@@ -26,11 +26,6 @@ export default {
       required: false,
       default: '',
     },
-    markdownVersion: {
-      type: Number,
-      required: false,
-      default: 0,
-    },
     saveButtonTitle: {
       type: String,
       required: false,
@@ -202,7 +197,6 @@ export default {
       <markdown-field
         :markdown-preview-path="markdownPreviewPath"
         :markdown-docs-path="markdownDocsPath"
-        :markdown-version="markdownVersion"
         :quick-actions-docs-path="quickActionsDocsPath"
         :line="line"
         :note="discussionNote"

app/assets/javascripts/notes/components/notes_app.vue

@@ -44,11 +44,6 @@ export default {
       required: false,
       default: true,
     },
-    markdownVersion: {
-      type: Number,
-      required: false,
-      default: 0,
-    },
     helpPagePath: {
       type: String,
       required: false,
@@ -216,10 +211,6 @@ export default {
       </template>
     </ul>
-    <comment-form
-      v-if="!commentsDisabled"
-      :noteable-type="noteableType"
-      :markdown-version="markdownVersion"
-    />
+    <comment-form v-if="!commentsDisabled" :noteable-type="noteableType" />
   </div>
 </template>

app/assets/javascripts/notes/index.js

@@ -18,7 +18,6 @@ document.addEventListener('DOMContentLoaded', () => {
       const notesDataset = document.getElementById('js-vue-notes').dataset;
       const parsedUserData = JSON.parse(notesDataset.currentUserData);
       const noteableData = JSON.parse(notesDataset.noteableData);
-      const markdownVersion = parseInt(notesDataset.markdownVersion, 10);
       let currentUserData = {};
       noteableData.noteableType = notesDataset.noteableType;
@@ -37,7 +36,6 @@ document.addEventListener('DOMContentLoaded', () => {
       return {
         noteableData,
         currentUserData,
-        markdownVersion,
         notesData: JSON.parse(notesDataset.notesData),
       };
     },
@@ -47,7 +45,6 @@ document.addEventListener('DOMContentLoaded', () => {
           noteableData: this.noteableData,
           notesData: this.notesData,
           userData: this.currentUserData,
-          markdownVersion: this.markdownVersion,
         },
       });
     },

app/assets/javascripts/vue_shared/components/markdown/field.vue

@@ -27,11 +27,6 @@ export default {
       type: String,
       required: true,
     },
-    markdownVersion: {
-      type: Number,
-      required: false,
-      default: 0,
-    },
     addSpacingClasses: {
       type: Boolean,
       required: false,
@@ -158,7 +153,7 @@ export default {
         this.markdownPreviewLoading = true;
         this.markdownPreview = __('Loading…');
         this.$http
-          .post(this.versionedPreviewPath(), { text })
+          .post(this.markdownPreviewPath, { text })
           .then(resp => resp.json())
           .then(data => this.renderMarkdown(data))
           .catch(() => new Flash(__('Error loading markdown preview')));
@@ -186,13 +181,6 @@ export default {
         .then(() => $(this.$refs['markdown-preview']).renderGFM())
         .catch(() => new Flash(__('Error rendering markdown preview')));
     },
-
-    versionedPreviewPath() {
-      const { markdownPreviewPath, markdownVersion } = this;
-      return `${markdownPreviewPath}${
-        markdownPreviewPath.indexOf('?') === -1 ? '?' : '&'
-      }markdown_version=${markdownVersion}`;
-    },
   },
 };
 </script>

app/controllers/concerns/preview_markdown.rb

@@ -16,8 +16,6 @@ module PreviewMarkdown
       else {}
       end
-    markdown_params[:markdown_engine] = result[:markdown_engine]
-
     render json: {
       body: view_context.markdown(result[:text], markdown_params),
       references: {

app/helpers/issuables_helper.rb

@@ -268,7 +268,6 @@ module IssuablesHelper
       issuableRef: issuable.to_reference,
       markdownPreviewPath: preview_markdown_path(parent),
       markdownDocsPath: help_page_path('user/markdown'),
-      markdownVersion: issuable.cached_markdown_version,
       lockVersion: issuable.lock_version,
       issuableTemplates: issuable_templates(issuable),
       initialTitleHtml: markdown_field(issuable, :title),

app/helpers/markup_helper.rb

@@ -116,7 +116,6 @@ module MarkupHelper
   def markup(file_name, text, context = {})
     context[:project] ||= @project
-    context[:markdown_engine] ||= :redcarpet unless commonmark_for_repositories_enabled?
     html = context.delete(:rendered) || markup_unsafe(file_name, text, context)
     prepare_for_rendering(html, context)
   end
@@ -132,7 +131,6 @@ module MarkupHelper
       page_slug: wiki_page.slug,
       issuable_state_filter_enabled: true
     )
-    context[:markdown_engine] ||= :redcarpet unless commonmark_for_repositories_enabled?
     html =
       case wiki_page.format
@@ -187,10 +185,6 @@ module MarkupHelper
     end
   end
-  def commonmark_for_repositories_enabled?
-    Feature.enabled?(:commonmark_for_repositories, default_enabled: true)
-  end
-
   private
   # Return +text+, truncated to +max_chars+ characters, excluding any HTML

app/helpers/notes_helper.rb

@@ -171,7 +171,6 @@ module NotesHelper
       registerPath: new_session_path(:user, redirect_to_referer: 'yes', anchor: 'register-pane'),
       newSessionPath: new_session_path(:user, redirect_to_referer: 'yes'),
       markdownDocsPath: help_page_path('user/markdown'),
-      markdownVersion: issuable.cached_markdown_version,
       quickActionsDocsPath: help_page_path('user/project/quick_actions'),
       closePath: close_issuable_path(issuable),
       reopenPath: reopen_issuable_path(issuable),

app/helpers/projects_helper.rb

@@ -265,10 +265,6 @@ module ProjectsHelper
     link_to 'BFG', 'https://rtyley.github.io/bfg-repo-cleaner/', target: '_blank', rel: 'noopener noreferrer'
   end
-  def legacy_render_context(params)
-    params[:legacy_render] ? { markdown_engine: :redcarpet } : {}
-  end
-
   def explore_projects_tab?
     current_page?(explore_projects_path) ||
       current_page?(trending_explore_projects_path) ||

app/models/concerns/cache_markdown_field.rb

@@ -13,7 +13,6 @@ module CacheMarkdownField
   extend ActiveSupport::Concern
   # Increment this number every time the renderer changes its output
-  CACHE_REDCARPET_VERSION         = 3
   CACHE_COMMONMARK_VERSION_START  = 10
   CACHE_COMMONMARK_VERSION        = 14
@@ -42,18 +41,6 @@ module CacheMarkdownField
     end
   end
-  class MarkdownEngine
-    def self.from_version(version = nil)
-      return :common_mark if version.nil? || version == 0
-
-      if version < CacheMarkdownField::CACHE_COMMONMARK_VERSION_START
-        :redcarpet
-      else
-        :common_mark
-      end
-    end
-  end
-
   def skip_project_check?
     false
   end
@@ -71,7 +58,7 @@ module CacheMarkdownField
     # Banzai is less strict about authors, so don't always have an author key
     context[:author] = self.author if self.respond_to?(:author)
-    context[:markdown_engine] = MarkdownEngine.from_version(latest_cached_markdown_version)
+    context[:markdown_engine] = :common_mark
     context
   end
@@ -128,17 +115,7 @@ module CacheMarkdownField
   end
   def latest_cached_markdown_version
-    return CacheMarkdownField::CACHE_COMMONMARK_VERSION unless cached_markdown_version
-
-    if legacy_markdown?
-      CacheMarkdownField::CACHE_REDCARPET_VERSION
-    else
-      CacheMarkdownField::CACHE_COMMONMARK_VERSION
-    end
-  end
-
-  def legacy_markdown?
-    cached_markdown_version && cached_markdown_version.between?(1, CacheMarkdownField::CACHE_COMMONMARK_VERSION_START - 1)
+    CacheMarkdownField::CACHE_COMMONMARK_VERSION
   end
   included do

app/models/repository.rb

@@ -614,7 +614,6 @@ class Repository
     return unless readme
     context = { project: project }
-    context[:markdown_engine] = :redcarpet unless MarkupHelper.commonmark_for_repositories_enabled?
     MarkupHelper.markup_unsafe(readme.name, readme.data, context)
   end