Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#2 | 2310719-extra-duplicate-filters-2.patch | 25.08 KB | bucefal91 |
#1 | 2310719-extra-duplicate-filters-1.patch | 17.36 KB | bucefal91 |
2271491-scaling-2.patch | 11.25 KB | bucefal91 | |
interdiff-2271491-1-2.txt | 1.92 KB | bucefal91 |
Comments
Comment #1
bucefal91 CreditAttribution: bucefal91 commentedI'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.
Comment #2
bucefal91 CreditAttribution: bucefal91 commentedHere's the same patch, but now enriched with appropriate advanced help documentation that covers new cTools plugin type.
Comment #4
bucefal91 CreditAttribution: bucefal91 commentedI have committed my last patch into the module.