Similar to the issue #1462950: More Flexible Implementation this adds the ability to have a blacklist/whitelist option for the domains. Unlike the one mentioned in 1462950 this does not change the class that is being used nor does it replace the checkboxes with a textarea.

I believe that the option to select the domains based on the domain_id instead of listing each domain would be more manageable especially from a dev => live environment or changing of a the domain and accounting for domain aliases without having to adjust the context(s).

This patch adds a new domaincontext option to allow the user to select exclude/include and the execute will set the context appropriately.

CommentFileSizeAuthor
#3 domaincontext-exclude_include_option-1480604-3.patch1.73 KBtorpy
domaincontext-exclude_include_option--0.patch1.69 KBgrndlvl

Comments

apemantus’s picture

apemantus commented
Status: Needs review » Reviewed & tested by the community

Patch works for me.

gabriel.achille’s picture

gabriel.achille commented

Patch works for me too. (but i didn't compare it with the other approach)

torpy’s picture

torpy commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new domaincontext-exclude_include_option-1480604-3.patch1.73 KB

This causes undefined index issues for me. Updated patch attached.