Merge pull request #13541 from Kunde21/php/constructor-cleanup

php: Removed opts checks in BaseStub when Channel is passed as argument
pull/14238/head
Zhouyihai Ding 7 years ago committed by GitHub
commit 9233201cc3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 25
      src/php/lib/Grpc/BaseStub.php

@ -54,6 +54,18 @@ class BaseStub
}
unset($opts['update_metadata']);
}
if (!empty($opts['grpc.ssl_target_name_override'])) {
$this->hostname_override = $opts['grpc.ssl_target_name_override'];
}
if ($channel) {
if (!is_a($channel, 'Grpc\Channel')) {
throw new \Exception('The channel argument is not a'.
'Channel object');
}
$this->channel = $channel;
return;
}
$package_config = json_decode(
file_get_contents(dirname(__FILE__).'/../../composer.json'), true);
if (!empty($opts['grpc.primary_user_agent'])) {
@ -61,9 +73,6 @@ class BaseStub
} else {
$opts['grpc.primary_user_agent'] = '';
}
if (!empty($opts['grpc.ssl_target_name_override'])) {
$this->hostname_override = $opts['grpc.ssl_target_name_override'];
}
$opts['grpc.primary_user_agent'] .=
'grpc-php/'.$package_config['version'];
if (!array_key_exists('credentials', $opts)) {
@ -71,15 +80,7 @@ class BaseStub
'required. Please see one of the '.
'ChannelCredentials::create methods');
}
if ($channel) {
if (!is_a($channel, 'Grpc\Channel')) {
throw new \Exception('The channel argument is not a'.
'Channel object');
}
$this->channel = $channel;
} else {
$this->channel = new Channel($hostname, $opts);
}
$this->channel = new Channel($hostname, $opts);
}
/**

Loading…
Cancel
Save