Delete terms related to the selected vocabularies.

CommentFileSizeAuthor
#4 2755629-4.patch1.01 KBharivenuv
#2 2755629.patch1.02 KBimalabya
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

malavya created an issue. See original summary.

imalabya’s picture

imalabya
he/him
Primary language English
Location Bangalore
CreditAttribution: imalabya at Valuebound commented
Status: Active » Needs review
FileSize
2755629.patch1.02 KB

Added a patch to delete the terms for selected vocabs.

mahaveer003’s picture

mahaveer003 CreditAttribution: mahaveer003 as a volunteer and at Valuebound commented
Status: Needs review » Reviewed & tested by the community

@malabya Thanks for patch

Working fine for me

+RTBC

harivenuv’s picture

harivenuv
Location India
CreditAttribution: harivenuv at Zyxware Technologies commented
FileSize
2755629-4.patch1.01 KB

hi malavya,

Thank for the patch. Some simple issues there in PHP 5.4 version array declaration is changed to short array declaration. Drupal 8: PHP 5.5.9 or higher so we can practice short array declaration. :)

So kindly check my patch.

imalabya’s picture

imalabya
he/him
Primary language English
Location Bangalore
CreditAttribution: imalabya at Valuebound commented
Status: Reviewed & tested by the community » Needs work

@harivenu_zyxware Can you please attach an interdiff?

imalabya’s picture

imalabya
he/him
Primary language English
Location Bangalore
CreditAttribution: imalabya at Valuebound commented
Status: Needs work » Reviewed & tested by the community

Made the changes in the commit.

imalabya’s picture

imalabya
he/him
Primary language English
Location Bangalore
CreditAttribution: imalabya at Valuebound commented
Status: Reviewed & tested by the community » Fixed

Changes pushed to 8.x branch. Not sure why the commit is not showing up. Maybe will show up in sometime.

imalabya’s picture

imalabya
he/him
Primary language English
Location Bangalore
CreditAttribution: imalabya at Valuebound commented

Status: Fixed » Closed (fixed)

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