I installed the module according to the instructions here.

The theme changes when I visit a subdomain, but the front page value has no effect. It just keeps the main site's front page.

Then on the main site the title changed to "Drupal | [My slogan]"
Not literally "My Slogan" but the slogan for the main site. I use the Page Title module to set the page title on the front page to "[My site] | [My slogan]"

I then tried to uninstall the modules to start fresh. When I did that I got this error:

* user warning: Table 'site_529.purl' doesn't exist query: DELETE FROM purl WHERE provider = 'sites' in /var/aegir/platforms/drupal-6.16/sites/all/modules/sites/sites.install on line 18.
* warning: array_intersect_key() [function.array-intersect-key]: Argument #1 is not an array in /var/aegir/platforms/drupal-6.16/sites/all/modules/purl/purl.module on line 719.
* warning: Invalid argument supplied for foreach() in /var/aegir/platforms/drupal-6.16/sites/all/modules/purl/purl.module on line 743.

It says the modules are now uninstalled, though.

Comments

arvinsingla’s picture

arvinsingla commented

I was getting the same issue for the main site title as well. I tracked it down to the "sites_init" function in sites.module on line 11. It looks like it sets the site_name variable regardless of if you are on a site or not.

Is the module assuming that we will always be on a "site"? My use case is having a primary site at the root domain and sub-sites on url prefixes. By this logic the primary site doesn't have a 'site' configuration only the sub-sites do.

I got around this by altering the sites_init to check if $site does indeed have a site name, otherwise leave it alone.

Original

function sites_init() {
  if(function_exists('strongarm_set_conf')) {
    // Reset strongarm configuration.
    strongarm_set_conf(TRUE);
  }
  
  // Set current site name
  global $conf;
  $site = sites_get_current_site();
  $conf['site_name'] = $site->name;
}

Modified

function sites_init() {
  if(function_exists('strongarm_set_conf')) {
    // Reset strongarm configuration.
    strongarm_set_conf(TRUE);
  }
  
  // Set current site name
  global $conf;
  $site = sites_get_current_site();
  if ($site->name) {
    $conf['site_name'] = $site->name; 
  }
}
brynbellomy’s picture

brynbellomy commented

This is interesting... it never occurred to me that you wouldn't always be on a 'site'. Good thought... I can commit this change.

brynbellomy’s picture

brynbellomy commented

By the way, it looks like you're running Sites in parallel with something that ACTUALLY splits your sites apart (Aegir or Domain or something... whatever it is that's causing your table names to be prefixed with 'site_529')... I can't guarantee Sites will work as expected in this case, since it's supposed to be a super-lightweight replacement for these types of modules.

butler360’s picture

butler360 commented

I am running Aegir, but I think the reason I was having trouble with Domain is that for some features it requires prefixing tables and Aegir is not compatible with that. As far as I know, site_529 isn't prefixed, it's just how it names the databases.

brynbellomy’s picture

brynbellomy commented
Status: Active » Fixed

This has been committed to the 2.x branch but is gonna stay in DEV for a while since it's such a small change ... http://drupal.org/cvs?commit=384184

Status: Fixed » Closed (fixed)

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

butler360’s picture

butler360 commented

I still get * user warning: Table 'site_529.purl' doesn't exist query: DELETE FROM purl WHERE provider = 'sites' in /var/aegir/platforms/drupal-6.16/sites/all/modules/sites/sites.install on line 18. when uninstalling this module.

sunshinee’s picture

sunshinee commented

@arvinsingla: THANK YOU for posting the fix for the site name!