Closed (won't fix)
Project:
Subscriptions
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
8 Apr 2009 at 19:20 UTC
Updated:
27 Sep 2019 at 14:06 UTC
Jump to comment: Most recent
Comments
Comment #1
salvisThere's currently no option to do that, but it could be implemented. Right now, it's strictly in time order, which is the one order that pretty much everyone can immediately relate to...
How would by-content-type be an improvement?
Comment #2
somebodysysop commentedA user made the request. They have 3 content types: Feeds, Stories and Events. It would be more readable and useful (according to him) if the digests were sorted by the content types so that the user could focus on the most important section(s) first. Of course, they're getting lots of these daily, so that's why I think it sort of makes sense.
If I were to attempt some sort of patch, where would you recommend that I look first?
Comment #3
salvisSorry for the long delay...
In subscriptions_mail_cron() look for
$user_mails is an array of the notifications for this one $uid. Sort them as you like.
However, I'm afraid you don't have the content type that you want to sort by. So look above, where the $mails array is filled, and add the content type to each mail.
Comment #4
salvisIt might make sense to implement a hook here, so that a custom module could provide sorting. Care to submit a patch?
Comment #5
somebodysysop commentedThanks for the follow-up. I'll take a look.
Comment #6
peterx commentedPerhaps an option to use a view so we can modify the view without touching the code in the module.
Comment #7
salvisWe'd need a patch for D7 first, then someone could backport.
Comment #8
salvisWe'd need a patch for D7 first, then someone could backport.
Comment #9
salvis