Hi, sorry for my English googletranslate :)

I installed the module subdomain and encountered an error specified in the title.
The fact is that to check the variable $cookie_domain, going line by line reading of settings.php and enter $cookie_domain search in the resulting array.

In settings.php comment present

* between your various domains. Make sure to always start the $cookie_domain

that satisfies the search condition, and is transmitted in parts quality of the argument to the function eval(), this results in an error.

It might be easier to get a value of $cookie_doman, declaring it to the global scope:
global $cookie_domain;

Comments

Ser_Mir’s picture

The fast way to fix the error (if you need to use module right now) before the developers release a new version of the module - change line 453 in includes/subdomain.admin.inc

before:

 if (strpos($setting, '$cookie_domain') !== FALSE) {

after:

 if (strpos($setting, '$cookie_domain =') !== FALSE) {
stewart.adam’s picture

Attached patch makes subdomain read $cookie_domain from the global variable, as per Andruxa's recommendation.

inventlogic’s picture

the method used to obtain the $cookie_domain setting from the settings.php file is to parse the whole file. This means it snags the first occurrence of the $cookie_domain within the file which is mentioned at least twice with default settings. If you place your own $cookie_domain= '.yoursite.com' at the end of the file. The $cookie_domain value that is grabbed is the first occurrence which is not your domain names value.

You can edit the settings.php and change all occurrences of $cookie_domain to something like $cokie_domain. leave the $cookie_domain = '.yoursite.com' as the only one.

Or apply the patch above to check the $cookie_domain from the global variables

lovelykaushik86’s picture

Thanks @Ser_Mir for solution

agrozyme’s picture

Issue summary: View changes
StatusFileSize
new867 bytes

remake patch

agrozyme’s picture

StatusFileSize
new2.79 KB

when path is empty, use original_path replace path.

agrozyme’s picture

StatusFileSize
new3.57 KB

Add support the "Content: Has taxonomy term ID (with depth)" which is contextual filter of view.

honza pobořil’s picture

Status: Active » Needs review

Can anybody else confirm if the patch #8 works?

  • Commit f755051 on 7.x-2.x by Bobík:
    Issue #1366682 by agrozyme: Fixed many small errors
    
honza pobořil’s picture

Version: 7.x-2.0-beta2 » 7.x-2.x-dev

I adapted your patch to newest dev version and commited it.

Could anybody confirm the newest dev fixes errors reported here? I did not tested it in term mode.

honza pobořil’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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