Service Provider Entity Id should default to 'php-saml' if no alternative is provided. Currently, in settings.php, the line entityId' => variable_get('saml_advanced_settings_sp_entity_id', 'php-saml') does not return 'php-saml' -- probably because saml_advanced_settings_sp_entity_id is the empty string and 'isset' evaluates to TRUE.
This causes downstream errors if the Admin has not provided a Service Provider Entity Id.
Service Provider Entity Id should default to 'php-saml' if no alternative is provided. Currently, in settings.php, the line entityId' => variable_get('saml_advanced_settings_sp_entity_id', 'php-saml') does not return 'php-saml' -- probably because saml_advanced_settings_sp_entity_id is the empty string and 'isset' evaluates to TRUE.
This causes downstream errors if the Admin has not provided a Service Provider Entity Id.