Closed (fixed)
Project:
Suggested Terms
Version:
5.x-1.x-dev
Component:
Code
Priority:
Critical
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
17 Feb 2008 at 13:11 UTC
Updated:
14 Jul 2012 at 23:15 UTC
Jump to comment: Most recent file
Comments
Comment #1
fanta commentedanyone idea of how hard/easy would this be ?
or any alternative module that does similar in v6
Comment #2
Crell commentedYes, I do intend to have a 6.x port of this module. I'm afraid I don't have a schedule to do so right now as none of my sites need it yet (my blog still runs Drupal 5 for now :-) ). I would, however, gratefully accept patches. (Roll them against the 5.x-dev branch, as HEAD is long out of date and unused.)
Comment #3
rmiddle commentedSubscribe need to look at the code for possible 6 upgrade.
Thanks
Robert
Comment #4
gagarine commentedsuscribe,
I already port favorite node to D6 http://drupal.org/node/269377 . I need this module but it's not in my top list right now... but perhaps i can do the port in 1-2 month. Anyway if other poeple post a patch I can review/test the code...
Comment #5
sam6 commentedSubscribe
Comment #6
edward.peters commentedI am keen on this port, but not a developer. Grateful if others are able to do it!
Comment #7
gagarine commentedouf... I receive right now a very big work.. so I'm not sure i have time for this in the 1-2 month... sorry. But I take a look, if it is realy easy i do a first port.
Comment #8
brahms commentedAs I am also interested in a port of this really useful module to Drupal 6, I tried to port it today. It seems not to afford to much changes to get it to work. Here is a patch for the actual 5.x-1.x-dev release (dated with June 26, 2008 - 02:12). To apply the patch file suggestedterms-5.x-1.x-dev_drupal-6.x.patch please download and unzip the latest 5.x-1.x-dev release suggestedterms-5.x-1.x-dev.tar.gz, copy the patch file suggestedterms-5.x-1.x-dev_drupal-6.x.patch into the modules directory suggestedterms and apply the patch with command
patch -p0 < suggestedterms-5.x-1.x-dev_drupal-6.x.patch. The patch modifies the 2 files suggestedterms.module and suggestedterms.info. I just did a short test where the patch seemed to work fine.@Crell: if you think that the patch is ok it would be fine if you would provide a 6.x dev release of the ported module. Many thanks in advance.
Comment #9
brahms commented@Crell: I just changed the issue's state from active to patch (code needs review) to let you know about the patch from comment#8
Comment #10
dan_aka_jack commentedsubscribe
Comment #11
cloudmine commentedSubscribe
Comment #12
doublejosh commentedsubscribe.
Comment #13
Babalu commentedsubscribing
Comment #14
Babalu commentedpatch don't work:
patch -p0 < suggestedterms-5.x-1.x-dev_drupal-6.x.patch
patching file suggestedterms.module
patching file suggestedterms.info
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- saving rejects to file suggestedterms.info.rej
Comment #15
Crell commentedThe patch failure is because the patch was against a downloaded dev version instead of a CVS checkout, so the .info file didn't apply properly. No biggie, I fixed it. Patch applied and appears to be working properly. Yay! I'll role a new release node shortly.
Thanks, brahms!
Comment #16
Crell commentedComment #17
brahms commented@Crell
My pleasure! In return I thank you for roling the official release of suggested terms and much more for all the awesome ideas and work you contribute for Drupal's evolution in topics like the database API and the field API. As I spent much time developing (non web based) applications with Oracle and DB2 databases for data storage in the past I'm looking forward (and nearly can't wait) to see those exciting ideas work. :-)
Comment #18
Crell commentedWell we still need someone to actually try and write an Oracle driver using the new API. AFAIK, no one is, so I'm not exactly sure what will happen. :-) If you want to help with that, drop by #Drupal or the database group on g.d.o so we can talk. (I'm still planning to overhaul Schema API, I hope, but you can at least get started with the query builders.) Let's move this out of this thread, though.
Comment #19
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.