When trying to add a second parent to a term, the action overrides the existing parent. This was working properly in taxonomy_manager-7.x-1.0-rc2

CommentFileSizeAuthor
#6 taxonomy_manager-multiple_parents-2021649-5.patch2.23 KBAnonymous (not verified)
#5 taxonomy_manager-multiple_parents-2021649-4.patch3.13 KBAnonymous (not verified)
#4 taxonomy_manager-multiple_parents-2021649-4.patch3.13 KBAnonymous (not verified)
#3 taxonomy_manager-multiple_parents-2021649-3.patch2.2 KBAnonymous (not verified)
#1 taxonomy_manager-multiple_parents-2021649-1.patch2.01 KBAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Active » Needs review
FileSize
taxonomy_manager-multiple_parents-2021649-1.patch2.01 KB

patch attached

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

Bump. Can I get some eyes on this please? This issue also exists in 7.x-1.x.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
FileSize
taxonomy_manager-multiple_parents-2021649-3.patch2.2 KB

ignore this patch.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
FileSize
taxonomy_manager-multiple_parents-2021649-4.patch3.13 KB

Updated patch. Prevents user from adding a parent that already exists on the term.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
FileSize
taxonomy_manager-multiple_parents-2021649-4.patch3.13 KB

Updated patch is attached. Updated comments per coding style.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Issue summary: View changes
FileSize
taxonomy_manager-multiple_parents-2021649-5.patch2.23 KB

Updated patch. When adding a parent to a term, the root entry in the hierarchy table should be updated. The previous patch allowed a term to have the root and another term as its parent at once.

adrien.felipe’s picture

adrien.felipe CreditAttribution: adrien.felipe commented

While I still can't add a term to have root as parent and another term from the Taxonomy Manager module, #6 works for me to avoid loosing root as parent when saving the term.

Thansk a lot for the patch.

dillix’s picture

dillix CreditAttribution: dillix commented
Status: Needs review » Reviewed & tested by the community

#6 also works for me

VladimirAus’s picture

VladimirAus
Primary language English
Location Brisbane 🇦🇺
CreditAttribution: VladimirAus at Tomato Elephant Studio commented
Status: Reviewed & tested by the community » Fixed

  • VladimirAus committed 6c9b93f on 7.x-1.x 
    Issue #2021649 by dillix, adrien.felipe, VladimirAus: Unable to save...

Status: Fixed » Closed (fixed)

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