Anyone is planning the port?

Comments

fanta’s picture

anyone idea of how hard/easy would this be ?

or any alternative module that does similar in v6

Crell’s picture

Yes, 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.)

rmiddle’s picture

Subscribe need to look at the code for possible 6 upgrade.

Thanks
Robert

gagarine’s picture

suscribe,
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...

sam6’s picture

Subscribe

edward.peters’s picture

I am keen on this port, but not a developer. Grateful if others are able to do it!

gagarine’s picture

ouf... 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.

brahms’s picture

As 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.

brahms’s picture

Status: Active » Needs review

@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

dan_aka_jack’s picture

subscribe

cloudmine’s picture

Subscribe

doublejosh’s picture

subscribe.

Babalu’s picture

subscribing

Babalu’s picture

patch 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

Crell’s picture

The 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!

Crell’s picture

Status: Needs review » Fixed
brahms’s picture

@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. :-)

Crell’s picture

Well 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.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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