Problem/Motivation

This was found while working on #2855175: [META] Many documentation / handbook URLs redirect to D7 content.

The URL for the change record in the deprecation messages added in #2914251: Move forum related logic from taxonomy migrations to new forum migrations is https://www.drupal.org/project/drupal/issues/000000.

Steps to reproduce

Proposed resolution

Change https://www.drupal.org/project/drupal/issues/000000 to https://www.drupal.org/node/3387830

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Release notes snippet

CommentFileSizeAuthor
#5 3387831-5-removed-trailing-space.patch1.36 KBthakurnishant_06
#3 3387831-3-update-issue-link.patch1.36 KBvbouchet

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone CreditAttribution: quietone at PreviousNext commented
Issue summary: View changes
vbouchet’s picture

vbouchet
Location Toulouse
CreditAttribution: vbouchet at Acquia commented
Status: Active » Needs review
StatusFileSize
new 3387831-3-update-issue-link.patch1.36 KB
quietone’s picture

quietone CreditAttribution: quietone at PreviousNext commented
Status: Needs review » Needs work

@vbouchet, thanks for the fix.

+++ b/core/modules/taxonomy/src/Plugin/migrate/process/ForumVocabulary.php
@@ -45,7 +45,7 @@ class ForumVocabulary extends ProcessPluginBase {
+    @trigger_error(__CLASS__ . 'is deprecated in drupal:10.1.0 and is removed from drupal:11.0.0. Use \Drupal\forum\Plugin\migrate\process\ForumVocabulary instead. See https://www.drupal.org/node/3387830 ', E_USER_DEPRECATED);

The trailing space after the URL needs to be removed as well. Thanks.

thakurnishant_06’s picture

thakurnishant_06 CreditAttribution: thakurnishant_06 as a volunteer and at SV Infotech for SV Infotech commented
StatusFileSize
new 3387831-5-removed-trailing-space.patch1.36 KB

Hello @quietone
Added patch to remove the trailing space after url. kindly review the changes.
Thank you.

thakurnishant_06’s picture

thakurnishant_06 CreditAttribution: thakurnishant_06 as a volunteer and at SV Infotech for SV Infotech commented
Status: Needs work » Needs review
smustgrave’s picture

smustgrave CreditAttribution: smustgrave at Mobomo commented
Status: Needs review » Reviewed & tested by the community

Change looks good but think this would of been an excellent ticket to leave for a new user (novice tag) just FYI. But what is done is done.

Thanks!

  • quietone committed 5eaab206 on 11.x 
    Issue #3387831 by thakurnishant_06, vbouchet, smustgrave: Fix change...
quietone’s picture

quietone CreditAttribution: quietone at PreviousNext commented
Status: Reviewed & tested by the community » Fixed

Thanks for getting this done quickly!

Committed 5eaab20 and pushed to 11.x. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.