You can use instead of drigg_rss_candidate_tags the AutoKeyword Class (http://www.phpclasses.org/autokeyword) as an option to those that would like to have the tagging work similar to Yahoo Terms (http://www.drupal.org/project/yahoo_terms) but without it.

Some of the reasons I think this more modular approach could be better than using just one embebed method is that it would be interesting to have options on this stuff, i.e user could choose from diferent term extraction modules.

So, maybe your "smart terms" could be an independent module.

How a local module like "smart terms" or "autokeyword" would be better than using Yahoo Terms:

1. This post talks about it: http://vernonkesner.com/141/automatic-tag-clouds-for-your-site.php
2. It doesn't have the http://developer.yahoo.com/faq/#limits that Yahoo Web Services impose.
3. It doesn't depends on the moods of an external company to work.
4. A program running on your server is surely faster than a call to any external server.

I think these are the main reasons for having an independent Drupal module for it.

CommentFileSizeAuthor
#5 rssAutoTag.patch5.28 KBbeata57

Comments

mercmobily’s picture

mercmobily commented

Hi,

I am not sure I have time to write a generic wrapper module for Autokeyword, although it would probably be a nice addition to Drupal in general.

Sorry... I want to give priority to Drigg in general, and there are many crucial features still missing. Since we _have_ an RSS feeder, this feature seems less urgent than a lot of other ones... :-(

Merc.

beata57’s picture

beata57 commented

I think i got a semi working version of this hacked together. I have to do some more work on it, and let it run against some feeds. If everything looks good i will post a patch up some time this weekend.

Beata

mercmobily’s picture

mercmobily commented

Hi,

Beata: OK thanks.
Please make sure you create a patch that works against the DRUPAL-5 branch of the module!

Thanks,

Merc.

lopolencastredealmeida’s picture

lopolencastredealmeida commented

Thanks Beata.

I'm starting also to create a module based on Yahoo Terms but using this class. Probably next tuesday I'll post it here to tests. It won't be a patch but a new module.

The patch stuff in Drigg is still needed :D

beata57’s picture

beata57 commented
Assigned: Unassigned » beata57
StatusFileSize
new rssAutoTag.patch5.28 KB

Here is my first patch, let me know how it works. You have to have the class.autokeyword.php file in the drigg directory. I have had it running on my site for a couple days and it seems to be working fine.

Beata

mercmobily’s picture

mercmobily commented
Status: Active » Needs review
lopolencastredealmeida’s picture

lopolencastredealmeida commented

Thanks :)

Silly question but why did you used "iso-8859-1" instead of the code used by the feed itself?
If the feed is UTF or even iso-8859-15 it won't work.

I think that it should...

  1. Check the charset of the feed
  2. Convert the feed to the sites charset (iconv)
  3. Set the proper charset for autokeyword
  4. Get the keywords

.. but I could be damn wrong.

Shouldn't this ...

$keyword = new autokeyword1($params,

... be ...

$keyword = new autokeyword($params,

I think the class name is wrong autokeyword and not autokeyword1

mercmobily’s picture

mercmobily commented

Hi,

I will look into this once I apply the patch.
Now that I've launched Free Software Magazine's new web site (http://www.freesoftwaremagazine.com), I have some extra time...

(It's been a little insane)

Bye,

Merc.

lopolencastredealmeida’s picture

lopolencastredealmeida commented

Real pretty :)

lopolencastredealmeida’s picture

lopolencastredealmeida commented

Still about my comment at http://drupal.org/node/225191#comment-758556

That was just in case you are using the auto-feeder.

If you are using the normal Drigg procedure where is the user that writes the article then:

  1. Check the charset of the site
  2. Set the proper charset for autokeyword
  3. Get the keywords
  4. User should be able to use the suggestion but be able to add or remove keywords

What do you think?

Basicaly, the feeder should check if it is auto feed or user feed and proceed in accordance.

liupascal’s picture

liupascal commented

Hello everyone,
I'd like to know if the patch beata made was working as expected or not ? (code review status ?)

Can it be safely applied, without any side effect ?

Thanks

Cheers,
Pascal

mercmobily’s picture

mercmobily commented

Hi,

We have been concentrating on bug reports.
This will be one of the first things we'll work on once the last bug is gone (and there's only 1 left :-D ).

I am not sure about the patch... I haven't even seen it!

Merc.

philbar’s picture

philbar commented
Status: Needs review » Needs work

Needs some work:

http://drupal.org/node/225191#comment-758700

philbar’s picture

philbar commented
Title: External module for Keyword/Terms Extraction » Use class.autokeyword.php for Keyword Extraction
mercmobily’s picture

mercmobily commented

Hi,

People, I don't even understand what this feature is about.
an somebody please explain?

Merc.

philbar’s picture

philbar commented

I don't use RSS at the moment, so this feature doesn't affect me.

From what I gather, it improves the creation of a list of "candidate tags". If a story that comes from RSS has one of these keywords, it will be added as a "tag" to the story.

  $form['drigg_rss_candidate_tags'] = array(
    '#type' => 'textarea',
    '#title' => t('Comma-separated list of possible tags'),
    '#description' => t('A list of "candidate tags". If a story that comes from RSS has one of these keywords, it will be added as a "tag" to the story. This needs to be a straight list, no enters etc. <string>NO SPACES!</strong>'),
    '#default_value' => variable_get('drigg_rss_candidate_tags', ''),
  );
mercmobily’s picture

mercmobily commented

Hi,

My goodness, I had forgotten I added that feature...
OK, it's a little clearer now, thank you for that. Will look into it!

Merc.