Child issue of #2873705: Add change records to all @deprecated code.

Add Change record to @deprecated for Term.php on line 18

CommentFileSizeAuthor
#12 2873742-12.patch582 bytesheddn
#10 add_change_record_to-2873742-10.patch582 bytesada hernandez
#5 depreciated-see-2873742.patch588 bytesdrupalmatts

Comments

jhood created an issue. See original summary.

jhood’s picture

jhood
he/him
Primary language English
Location Ohio, United States
commented

I was unable to find a change request for this and commented on the ticket where the change was made here https://www.drupal.org/node/2746671#comment-12060606

If anyone wants to pick this one up, feel free!

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Title: Add Change record to @deprecated for Term.php » Add Change record to @deprecated for Term
drupalmatts’s picture

drupalmatts commented

Ill add the @see for this.

drupalmatts’s picture

drupalmatts commented
StatusFileSize
new depreciated-see-2873742.patch588 bytes

Adding patch file for:

core/modules/taxonomy/src/Plugin/migrate/source/Term.php

tachiinii’s picture

tachiinii commented
Status: Active » Reviewed & tested by the community

Check #5 @see link appears in Term class.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Needs work

The @see is linking back to this issue, which is very circular. We need to add the change record for the original issue before we can do anything here.

heddn’s picture

heddn
Primary language English
Location Illinois
commented

re #2: then let's create one. See https://www.drupal.org/contributor-tasks/draft-change-record for instructions.

ada hernandez’s picture

ada hernandez commented
Assigned: Unassigned » ada hernandez
ada hernandez’s picture

ada hernandez commented
Assigned: ada hernandez » Unassigned
Status: Needs work » Needs review
StatusFileSize
new add_change_record_to-2873742-10.patch582 bytes

For this @see the issue is https://www.drupal.org/node/2746671 because in this was added.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Needs review » Needs work
Related issues: +#2746671: CCK field data not available for D7 taxonomy term migrations

There's still no change record for #2746671: CCK field data not available for D7 taxonomy term migrations - one needs to be added, and the @see in this issue pointed to it.

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Status: Needs work » Needs review
StatusFileSize
new 2873742-12.patch582 bytes

Added a CR. No interdiff.

mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Status: Needs review » Reviewed & tested by the community

The change record is published.

  • Gábor Hojtsy committed e1688b8 on 8.3.x 
    Issue #2873742 by heddn, DrupalMattS, Adita, jhood, mikeryan: Add Change...
gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed

Superb, thanks all.

  • Gábor Hojtsy committed 7292b48 on 8.4.x 
    Issue #2873742 by heddn, DrupalMattS, Adita, jhood, mikeryan: Add Change...
xjm’s picture

xjm
she/her
Primary language English
commented
Issue summary: View changes
Parent issue: #2873705: Add change records to all @deprecated code »

Removing the issue relationship from fixed issues to keep the parent issue usable. Thanks!

xjm’s picture

xjm
she/her
Primary language English
commented

Status: Fixed » Closed (fixed)

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