Some implementations may want to add additional filter on the list of broadcast they want to receive beside the case code that can be explicitely passed in through drush.

As a first easy step, adding an alter hook would allow site specific modules to alter the query and restrict the result set as needed.

CommentFileSizeAuthor
#4 2538538-4-broadcast-query-alter.patch2.25 KBDeFr
#2 2538538-1-broadcast-query-alter.patch1.11 KBDeFr
#1 2538538-1-broadcast-query-alter.patch0 bytesDeFr

Comments

DeFr’s picture

DeFr commented
Status: Active » Needs review
StatusFileSize
new 2538538-1-broadcast-query-alter.patch0 bytes
DeFr’s picture

DeFr commented
StatusFileSize
new 2538538-1-broadcast-query-alter.patch1.11 KB

Looks like there was a problem with the upload of the patch in #2. Let's try again.

pvhee’s picture

pvhee commented
Status: Needs review » Needs work

Thanks for the patch! I think this is ready to go in, but we should also document the patch itself in *.api.php or something and potentially adding this to the README.

DeFr’s picture

DeFr commented
Status: Needs work » Needs review
StatusFileSize
new 2538538-4-broadcast-query-alter.patch2.25 KB

Nice catch. Fixed.

I think there's a bunch of other problems with the module documentation though, the main one being that all the links for the API documentation in the main module README.md are dead links (or maybe links to repository I don't have access to). Filing a new issue for that.

  • pvhee committed f6e6568 on 7.x-1.x authored by DeFr 
    Issue #2538538 by DeFr: Allow other modules to alter the broadcast query
pvhee’s picture

pvhee commented
Status: Needs review » Fixed

Patch applied, thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.