s&d sends too many e-mails

Is this still an issue, Simone?

I'm using WeeklyScheduler on S&D 0.9.0 which is working fine. "Triggered the last time" field is updated correctly.

I think I tracked down the issue. It only occurs when there is no sort order specified on the collector's collection. This results in S&D being unable to filter out the content what was already sent out. I think the default should be that it filters out by creation date. But currently no filter will be applied if the sort order is unspecified, or set to something like Title.

The problem is in the get_items_for_topic function of collective.dancing.collector.Collector:

    def get_items_for_topic(topic, cue):
        query_args = {}
        if cue is not None and topic.hasSortCriterion():
            sort_criterion = topic.getSortCriterion()
            fname = str(sort_criterion.field)
            query_factory = sort_criteria.get(
                fname, sort_criteria.get('default'))
            query_args[fname] = query_factory(cue)
        return topic.queryCatalog(full_objects=True, **query_args)

I think it should be changed to check what the sort_criteria is. Because it doesn't make sense to add a query filter of max date on the title field.

This caused me many hours of hair pulling. Maybe I missed this critical bit of info somewhere in the documentation.

I applied the sort rule by created date on collector,but the filter still do not work.