As suggested https://www.drupal.org/node/2271491#comment-8998409 by cyborg_572, duplicate tool should have additional filters by "same parents" and by "same description" which would make it even more flexible for the needs of Term Merge users.

I myself like the idea. cyborg_572 provided a couple of patches on this issue. I am attaching them to this ticket.

In the nearest future I'll review the patches and will test them. Additionally, I'd like to update Term Merge simpletests to cover this new feature too. Once I got it done, I'll provide a new patch that will incorporate all of it.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bucefal91’s picture

I've processed this issue and eventually coded it in a bit more flexible way than it was originally thought.

I defined cTools plugin type - duplicate suggestion. It's something that determines if terms are duplicates of each other. Then on the duplicate term the end user can specify which duplicate suggestions to enable when searching for terms. Term Merge module supplies the 3 duplicate suggestions: by name, by description and by parent.

I've also updated the simpletests of the module to cover this new functionality. I tested it myself and see no issues.

I leave the issue as "needs work", because I still need to document the new cTools plugin type through advanced help or something else similar. Though apart from them insufficient documentation, it's all good.

bucefal91’s picture

Status: Needs work » Needs review
FileSize
25.08 KB

Here's the same patch, but now enriched with appropriate advanced help documentation that covers new cTools plugin type.

  • bucefal91 committed 8af62a4 on 7.x-1.x
    Issue #2310719 by bucefal91: adding the advanced help for the freshly...
  • bucefal91 committed a8abd96 on 7.x-1.x
    Issue #2310719 by cyborg_572, bucefal91: Adding more options as for what...
bucefal91’s picture

Status: Needs review » Fixed

I have committed my last patch into the module.

Status: Fixed » Closed (fixed)

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