Multiple messaging endpoints support. [#1101806]

This is probably related to #329412: enable muliple Sending Methods; also - there is a patch in messaging that is necessary for this to work #309461: Multiple Send methods.

This patch aims to add support to notifications methods to allow a user object to pass multiple messaging endpoints, such that multiple subscriptions are added at that point in time. Clearly the notifications framework supports multiple subscriptions to different messaging endpoints. The thought here is to take it a step further and allow a user to add *default* multiple selections and have notifications add those default as subscriptions on a notification add.

Comment	File	Size	Author
#13	multiple_endpoint_support-1101806-13.D6.patch	5.47 KB	christianchristensen
#12	multiple_endpoint_support-1101806-12.D6.patch	5.45 KB	christianchristensen
#11	multiple_endpoint_support-1101806-11.D6.patch	6.79 KB	travist
#10	multiple_endpoint_support-1101806-10.D6.patch	9.94 KB	travist
#9	multiple_endpoint_support-1101806-9.D6.patch	9.94 KB	travist
#8	multiple_endpoint_support-1101806-8.D6.patch	7.91 KB	travist
#7	multiple_endpoint_support-1101806-7.D6.patch	4.73 KB	pdrake
#6	multiple_endpoint_support-1101806-6.D6.patch	4.72 KB	pdrake
#4	multiple_endpoint_support-1101806-4.make_.patch	4.7 KB	travist
#3	multiple_endpoint_support-1101806-2.make_.patch	5.53 KB	christianchristensen
#2	multiple_endpoint_support-1101806-2.make_.patch	5.51 KB	christianchristensen
#1	multiple_endpoint_support-1101806-2.patch	6.93 KB	christianchristensen
	multiple_endpoint_support-1.patch	0 bytes	christianchristensen

Comments

Comment #1

christianchristensen commented 22 March 2011 at 19:06

Status	File	Size
new	multiple_endpoint_support-1101806-2.patch	6.93 KB

Patch correction...

Comment #2

christianchristensen commented 22 March 2011 at 21:13

Status	File	Size
new	multiple_endpoint_support-1101806-2.make_.patch	5.51 KB

Patchfile for drush make.

Comment #3

christianchristensen commented 7 April 2011 at 21:27

Status	File	Size
new	multiple_endpoint_support-1101806-2.make_.patch	5.53 KB

Another quick update - include an array_filter on the provided defaults to filter out the non-defaults.

Comment #4

travist commented 15 April 2011 at 19:27

Status	File	Size
new	multiple_endpoint_support-1101806-4.make_.patch	4.7 KB

Found an issue with the patch above. It is possible to fall into an infinite function recursion if the user does not have a send_method already defined in their settings. This patch is better.

Comment #5

christianchristensen commented 18 April 2011 at 19:59

Nice catch travist! That patch is indeed better (but, I think it still provides the flexibility to messaging is now able to ease into multiple endpoints)

Comment #6

pdrake commented 2 August 2011 at 01:26

Status	File	Size
new	multiple_endpoint_support-1101806-6.D6.patch	4.72 KB

Fixed a couple of logic errors. See attached. Thanks for the help on this travist.

Comment #7

pdrake commented 2 August 2011 at 12:43

Status	File	Size
new	multiple_endpoint_support-1101806-7.D6.patch	4.73 KB

Slightly fixed version of the above patch.

Comment #8

travist commented 30 April 2012 at 18:08

Status	File	Size
new	multiple_endpoint_support-1101806-8.D6.patch	7.91 KB

Had to pull in the patch from #1075862: Notifications queuing adds a lot of time to node/comment inserts into this because it was causing a conflict. Here is a re-roll with that patch included.

Comment #9

travist commented 30 April 2012 at 19:13

Status	File	Size
new	multiple_endpoint_support-1101806-9.D6.patch	9.94 KB

Apparently, a previous patch removed a portion of the original. Here is the updated patch.

Comment #10

travist commented 30 April 2012 at 20:50

Status	File	Size
new	multiple_endpoint_support-1101806-10.D6.patch	9.94 KB

Changed a variable to control the queue immediate flag.

Comment #11

travist commented 8 May 2012 at 16:23

Status	File	Size
new	multiple_endpoint_support-1101806-11.D6.patch	6.79 KB

Changed this patch back to the original by removing the reference to #1075862: Notifications queuing adds a lot of time to node/comment inserts since it really doesn't apply here.

Comment #12

christianchristensen commented 19 June 2012 at 18:45

Status	File	Size
new	multiple_endpoint_support-1101806-12.D6.patch	5.45 KB

Updated this patch to be cognizant of notification_lite enqueue options with multiple send methods.

Comment #13

christianchristensen commented 19 June 2012 at 18:57

Status	File	Size
new	multiple_endpoint_support-1101806-13.D6.patch	5.47 KB

Clean up whitespace diff

Multiple messaging endpoints support.