[SOLVED] How can I assign a labels trough Article filters

[BobSquarePants]

Hi everyone,

I've tried by myself to make a Article filters that will assign a label when the title or content match a keyword, but definitely JavaScript make not sens at all for me..

I'm sure it will be something close to -->

var whitelist = [
  'test subject', 'blabla'
];
function filterMessage() {
  if (whitelist.some(i => msg.title.indexOf(i) != -1)) {
  msg.assignLabel(findLabelId('MyLabel'))
  } 
}

But I can't figure it out. thanks

[M1chael]

I did almost the same thing and see that you missed msg. part before findLabelId. But in my case it also doesn't work. No errors and console.log debugging confirms all my assumptions, but no labels assigned.

[martinrotter]

I checked your situation and found a potential bug in the application which prevents your script from working. Your script is correct. I am trying to make it reasonably work.

Will post "semi-fix" with workaround soon + full fix later.

[martinrotter]

OK, I added a semi-fix to allow it to work with "workaround" (see below), f0bb361.

Workaround is to use latest RSS Guard with the above fix (see development builds) and place line:

msg.customId = msg.url;

to be the first line inside filterMessage() function. Like this (note I also changed name of the label for my testing):

var kernel = [
  'Linux kernel'
];

var year = '-' + new Date().getFullYear() + '-';

var contents_blacklist = [
  'WordPress', 'Discourse', 'MediaWiki', 'IBM', 'Samsung', 'D-link', 'Mikrotik', 'Advancecomp', 'php', 'scada', 'Tenda', 'Dell', 'PLC', 'BlueZ', 'Bluetooth', 'cms', 'firmware', '** REJECT ** '
];

var title_blacklist = [
  'firmware', 'android', 'windows', 'azure', 'cms', 'stock_management_system', 'teleport', 'doctor-appointment-system'
];

var title_whitelist = [
  year
];

function filterMessage() {
  msg.customId = msg.url;

  if (title_whitelist.some(i => msg.title.toUpperCase().indexOf(i.toUpperCase()) != -1)) {
    if (title_blacklist.some(i => msg.title.toUpperCase().indexOf(i.toUpperCase()) != -1) || contents_blacklist.some(i => msg.contents.toUpperCase().indexOf(i.toUpperCase()) != -1)) {
      return MessageObject.Ignore;
    } else {
      if (kernel.some(i => msg.contents.toUpperCase().indexOf(i.toUpperCase()) != -1)) {
         msg.assignLabel(msg.findLabelId('Test test'));
      }
      return MessageObject.Accept;
    }
  } else {
    return MessageObject.Ignore;
  }
}

This is only temporary workaround but will also work after I push my full fix. Also note that this workaround will only reliably work for feeds whose entries DO HAVE UNIQUE URLs, which in your case seems to be true.

[M1chael]

Wow! Extremely fast fixing! You are incredible! Thanks a lot!

I switched to the AppImage from development builds and will report back when another liable to be labeled article comes to my feed.

[M1chael]

Yeah! Seems to be worked. Little minor bug though still there: after fetching articles RSSGuard shown 25 Labels, but only 10 of them were named. After clicking on the Labels or the group name digits became equal. Nothing critical, just FYI. Thanks again for fast fix!

[martinrotter]

Yes, that seems like a bug. Will check.

EDIT: Checked it and can reproduce. It is result of "THE" bug we workarounded above and will be likely fixed when I manage to fix the base problem here.

To outline what the problem is: we have kind of chicken-egg problem:

1. RSS Guard downloads articles.
2. Your article filter run and if your article filter assigns labels, they are saved to database now (this is the problematic behavior because the messages itself are not saved in DB so far).
3. Articles are saved to database.

[martinrotter]

Hi everyone,

I've tried by myself to make a Article filters that will assign a label when the title or content match a keyword, but definitely JavaScript make not sens at all for me..

I'm sure it will be something close to -->

var whitelist = [
  'test subject', 'blabla'
];
function filterMessage() {
  if (whitelist.some(i => msg.title.indexOf(i) != -1)) {
  msg.assignLabel(findLabelId('MyLabel'))
  } 
}

But I can't figure it out. thanks

Your script is missing fundamental part, return statement indicating whether message (modified) is actually accepted by the filter or not.

Fixed:

var whitelist = [
  'qt', 'abc'
];
function filterMessage() {
  if (whitelist.some(i => msg.title.indexOf(i) != -1)) {
    console.log("inside");
    msg.assignLabel(msg.findLabelId('Test Test'))
  }

return MessageObject.Accept;
}

[BobSquarePants]

Thank you @martinrotter ! it worked !!!

of topic, would it possible to create a plugin for RSSguard (or a fork) that support Python for the filter instead of javascript ?

[martinrotter]

yes, it is possible, but i will not do it, someone else can do it of course, rssguard is open-source free software

[BobSquarePants]

Indeed, and thanks to have created RSSguard as a FOSS software ❤️
Maybe I'll do it in the future then :)