Download & Extend
Term merge » Issues

Port to Drupal 6

Posted by dberror on March 8, 2009 at 12:36am
Project:Term merge
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:bleen18
Status:needs work

Issue Summary

what about version for Drupal 6.x?

Login or register to post comments

Comments

#1

Posted by Stephan Vanhaverbeke on March 31, 2009 at 2:49am

I'm also looking for this kind of functionality on Drupal 6.
How about it?

#2

Posted by eli on April 8, 2009 at 11:33pm
Status:active» postponed

We only use Drupal 5 right now, so I'm afraid this is pretty far down my To Do list. However, if you want to take a crack at porting it, I'd be glad to help.

#3

Posted by asb on April 16, 2009 at 9:02am

Hi,

I'm also looking for a D6 version. Currently, the "Taxonomy Manager" module might be an option (thus this is a great module it's a bit less comfortable than the "Term Merge" link).

Greetings, -asb

#4

Posted by eli on April 27, 2009 at 1:31pm
Assigned to:Anonymous» eli
Status:postponed» active

Ok, I did a quick & dirty Drupal 6 version. Please let me know if you encounter any bugs.

#5

Posted by eli on April 27, 2009 at 2:03pm
Version:5.x-1.1» 6.x-1.x-dev

#6

Posted by giorgio79 on July 8, 2009 at 5:51am

I get "page not found" when clicking on merge term link in term edit page.

#7

Posted by giorgio79 on August 28, 2009 at 5:26am

This might help
http://drupal.org/project/deadwood

d5 to d6 converter :)

#8

Posted by Gabriel Radic on November 16, 2009 at 6:06pm

I tried it on my installation of D6, got a nice

Fatal error: Call to undefined function _taxonomy_access_get_nodes_for_term() in /usr/www/users/danb007/beta/sites/all/modules/taxonomy_access/taxonomy_access.module on line 253

No luck.

#9

Posted by bleen18 on February 2, 2010 at 7:15pm

I've been doing some work with this module for D6 ...

The attached path includes the following:

  • ~ 85% is simple housekeeping (http://drupal.org/coding-standards).
  • I removed the term_merge_merge function and called drupal_get_form directly from the hook_menu page callback
  • I converted term_merge_merge_form_submit to use taxanomy_save_term
  • I removed bug where synonyms might end up duplicated
  • I made the success message a bit more informative ... but left in "Have a nice day!"

I'm sure I'll have some more updates to this, and I'll probably want to update this module for D7 at some point ... looking for a D6/D7 maintainer? Please let me know.

AttachmentSize
term_merge_d6.patch 10 KB

#10

Posted by bleen18 on March 16, 2010 at 11:08pm
Assigned to:eli» bleen18
Status:active» needs review

I've recently been made a comaintainer for this module and I'd love to get at least one or two more people to take a look at this patch (#9) before I commit it. Any takers?

#11

Posted by Cyberwolf on April 20, 2010 at 1:30pm

Subscribing.

#12

Posted by bleen18 on April 20, 2010 at 4:00pm

testers? anyone? anyone?

#13

Posted by grobot on June 23, 2010 at 4:01am
Status:needs review» needs work

Some thoughts on D6-6.x-1.x-dev after a brief (not extensive) test -

  • The 'merge' link appears at the top of my term edit form. Seems out of place being above the title.
  • The 'merge' link text is "Merge this term into another another" which should probably be "Merge this term with another"
  • Merging terms says "The terms have been merged. Have a great day.". This should just say "The terms have been merged".*
  • Might want to test in conjunction with Taxonomy Image module. When merging terms, I see,
    • Not deleted from the files table because it is use on -1 other terms.
    • image removed.
  • Should use the admin/* path so as to remain in admin theme when viewing the merge term page.
  • Did you mean to offer a "Merge term" link on the term delete page?

* IMO introducing out-of-context chattiness detracts from the message in Drupal's admittedly already busy UI. Keep it simple and businesslike, no need to confuse the UI with irrelevant personal messages.

#14

Posted by nylin on May 12, 2011 at 2:45pm
Status:needs work» closed (duplicate)

Marking this as closed (won't fix) until there is someone willing to continue the work of the 6.x release.

#15

Posted by asb on May 12, 2011 at 3:05pm
Status:closed (duplicate)» closed (won't fix)

#16

Posted by nylin on May 12, 2011 at 3:14pm

Woops :)

#17

Posted by yarroha on December 8, 2011 at 8:21pm
Status:closed (won't fix)» needs work

Hi all,
As D6-6.x-1.x-dev doesn't work I've completely rewrote the module for my needs.
Maybe somebody will also need it.
You can find the code on GitHub: https://github.com/yarroha/term_merge

nobody click here