Here's an experimental module that domain-enables _all_ forms that use system_settings_form().

Might not work in all cases, but may be worth a look.

CommentFileSizeAuthor
#11 domainsettings_README.txt2.91 KBcroryx
#7 Picture 1.png27.54 KBagentrickard
#1 domain_settings.zip4.28 KBagentrickard
domain_settings.zip3.7 KBagentrickard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
FileSize
domain_settings.zip4.28 KB

And a better, sexier version.

marcingy’s picture

marcingy CreditAttribution: marcingy commented

Nice I'll try and give this a roll this weekend as I have some localized patches for variable_set to do this on a case by case basis.

antiorario’s picture

antiorario CreditAttribution: antiorario commented

subscribing

Aniara.io’s picture

Aniara.io CreditAttribution: Aniara.io commented

Subscribing. Going to test it tonight.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented

Note that this only works with forms that don't modify the handling of system_settings_form. Certain core forms (like forums, and date/time) simply won't work.

croryx’s picture

croryx CreditAttribution: croryx commented

I installed the new module and tried it out using the form for the Google Analytics module. Everything worked great. This is a really nice addition both in terms of ease of use and in terms of operating in a way that will be maximally intuitive to new users. Following up on the update to your blog post I'd add a vote for rolling this into Domain Conf.

One minor suggestion (if it's easy) would be to have the Save Settings For option default to the active domain. This would maximize the intuitive nature of the interface - the user goes to the appropriate settings page from the domain of interest, is shown the current settings for that domain, and then (without changing from the default) saves those settings for that domain - and would also reduce the number of instances where the settings for the current domain accidentally overwrite those for the primary domain.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
FileSize
Picture 1.png27.54 KB 
One minor suggestion (if it's easy) would be to have the Save Settings For option default to the active domain.

There is a setting for this on the Domain module settings page. It's just not documented. The theory is that some will prefer the 'normal' behavior, and some won't.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented

If someone would write up some documentation.....

croryx’s picture

croryx CreditAttribution: croryx commented

Thanks, I should have looked around a little more, and thanks again for another great addition to DA.

I can write a first draft of the documentation (apparently after I look around a little more).

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
Category: task » feature
Issue tags: +favorite

Tagging

croryx’s picture

croryx CreditAttribution: croryx commented
FileSize
domainsettings_README.txt2.91 KB

Here is a first attempt at documentation for Domain Settings. It basically just puts together the information from the original blog post and from the queue. I tried to follow the style and structure of the README.txt files for Domain Conf and Domain Nav. Section 2.2 was something I didn't have an example for, but it seemed important to include an explicit description of the cases where the domain-specific settings won't work.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented

Very nice.

So, submodule, or part of Domain Conf? I think submodule, so users can disable it separately.

croryx’s picture

croryx CreditAttribution: croryx commented

I agree - separate submodule.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented

And I think I will go against my earlier pronouncements and simply add this to 6.x.2.2.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
Version: 6.x-2.1 » 6.x-2.x-dev
Status: Needs review » Fixed

Committed to 6--2 and HEAD.

TimAlsop’s picture

TimAlsop CreditAttribution: TimAlsop commented

Is this module included in 6.x-2.2 ? I am using 2.2 and I don't see it. On DA page there is no 6.x-2.x-dev available, which is why I am a bit confused on the status of this settings module/patch.

antiorario’s picture

antiorario CreditAttribution: antiorario commented

It's included. If you don't see it on the modules page you should clear the caches and see if that helps.

TimAlsop’s picture

TimAlsop CreditAttribution: TimAlsop commented

Found it now, thanks.

Status: Fixed » Closed (fixed)

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

pounard’s picture

pounard
Primary language French
CreditAttribution: pounard commented

@#1 your archive contains a lot of MacOS junk.