any plans on updating the extractor api for drupal 7?

from the recent managing news blog post (http://developmentseed.org/blog/2010/sep/30/managing-news-11-released) i understand that with MN several related modules are being upgraded yet. how about extractor?

CommentFileSizeAuthor
#13 extractor_d7-965314-13.patch14.56 KBgeerlingguy
#12 extractor_d7-965314-12.patch14.03 KBgeerlingguy
#11 extractor_d7-965314-11.patch13.17 KBgeerlingguy
#10 extractor_d7-965314-10.patch15.88 KBgeerlingguy

Comments

nearthesoul’s picture

nearthesoul commented

subscribing

TimelessDomain’s picture

TimelessDomain commented

subscribing

Ooxo’s picture

Ooxo commented

subscribing

dasjo’s picture

dasjo
he/him
Primary language German
Location Zurich
commented

see the Rules Autotag module for basic drupal 7 term extraction

mrsinguyen’s picture

mrsinguyen commented

+1

sylvanos’s picture

sylvanos commented

+1

mefisto75’s picture

mefisto75 commented

sub

mgifford’s picture

mgifford
he/him
Primary language English
commented

I do recommend that folks work who want this try to do an upgrade via Coder to see if that can get us closer to a dev version for testing. Great way for non-maintainers to get involved.

geerlingguy’s picture

geerlingguy commented

I'm planning on taking on at least a partial port pretty soon, as I'm rebuilding a site I originally built in ManagingNews in Drupal 7, but from scratch, and the only missing component is geo tagging, for which the Extractor module is essential. I won't need anything for taxonomy/interface, so I'm going to focus on getting the bare API working for my site...

geerlingguy’s picture

geerlingguy commented
Status: Active » Needs review
StatusFileSize
new extractor_d7-965314-10.patch15.88 KB

Simple coder upgrade patch attached. Completely untested (but that won't be the case for long!).

geerlingguy’s picture

geerlingguy commented
StatusFileSize
new extractor_d7-965314-11.patch13.17 KB

Teaches me to copy and paste... the last patch had HTML characters in it. Attached is a patch that actually applies cleanly, and is straight from the coder module's output, rather than a copy/paste from coder's UI...

geerlingguy’s picture

geerlingguy commented
StatusFileSize
new extractor_d7-965314-12.patch14.03 KB

Okay, this patch is tested and working, at least on the API level, on a D7 site I'm building. There may be an error or two in the form_alters, but I've converted the relevant database queries that needed adjustment, and I've split out D6's hook_taxonomy() into the D7 taxonomy hooks.

There was some other cleanup I had to do here or there to get the core of the module working.

I haven't tested Placemaker (though I probably will pretty soon), and I'm not using Feeds, so I haven't tested (nor will I be able to test, since I don't use feeds much) the Feeds integration.

geerlingguy’s picture

geerlingguy commented
StatusFileSize
new extractor_d7-965314-13.patch14.56 KB

Had some time to test Yahoo Placemaker integration, which is working well enough. Attached patch is fairly well-tested, but throws a couple warnings when adding or deleting a vocabulary (I didn't want to touch some of that form submit code, since it looks like some of it is required for feeds module integration...).

mgifford’s picture

mgifford
he/him
Primary language English
commented

Patch applies nicely.

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented
Issue tags: +port to d7

...would be nice if this was rolled into a 7.x dev.