[BUG] config.php file is already present in my project

[Sokulev]

Describe the bug
I'm using Magento.
config.php file is already present in my project in root dir.
I don't want / can't alter it
class ET_Client is trying to load it first to get settings instead of using $params when class instantiated.

Causing error:
Uncaught exception 'Exception' with message 'baseUrl is null: Must be provided in config file when instantiating ET_Client' in /src/LittlePassports/lpmage/vendor/salesforce-mc/fuel-sdk-php/src/ET_Client.php on line 131

To Reproduce
Have empty config.php file in your app root folder.

Expected behavior
ET_Client should use params supplied during instantiation or / and config file should be named differently than config.php.

Environment

• SDK Version [e.g. 1.4.0]
• PHP version 5.6

The bug has the severity

• Critical: The defect affects critical functionality or critical data. It does not have a workaround.
• Major: The defect affects major functionality or major data. It has a workaround but is not obvious and is difficult.
• Minor: The defect affects minor functionality or non-critical data. It has an easy workaround.
• Trivial: The defect does not affect functionality or data. It does not even need a workaround. It does not impact productivity or efficiency. It is merely an inconvenience.

[alessandrofilira]

I've same problem with a Symfony application, any news? Thanks.

[garek007]

You can move or create your own class for the config.php code. it doesn't have to live in the root.

[aalwash]

Seriously, how difficult is it to add an extra optional parameter in the ET_Client constructor to avoid reading config.php?

function __construct($getWSDL = false, $debug = false, $params = null) 
	{

to

function __construct($getWSDL = false, $debug = false, $params = null, $readConfig = true) {

    //....
    if($readConfig && file_exists(realpath('config.php'))) {

    }
    
    // ...
}

That simple :)

[aalwash]

There you go a PR ;)

[sp72]

It's ugly but it works...

$bak = getcwd();
chdir('/where/there/is/no/config.php/file...');
$client = new ET_Client(false, false, $params);
chdir($bak);

I hope it will be fixed soon !

Other solution :
Instead of searching the config.php first, read the $params. If empty, search for the file, no constructor modification.