Posted by dasjo on November 8, 2010 at 10:23am
11 followers
| Project: | Extractor |
| Version: | 6.x-1.0-alpha7 |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
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?
Comments
#1
subscribing
#2
subscribing
#3
subscribing
#4
see the Rules Autotag module for basic drupal 7 term extraction
#5
+1
#6
+1
#7
sub
#8
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.
#9
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...
#10
Simple coder upgrade patch attached. Completely untested (but that won't be the case for long!).
#11
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...
#12
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.
#13
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...).
#14
Patch applies nicely.