Merge pull request #39 from Takika/patch-1

thomascube · web-flow · commit 060acc66bb5f · 2022-09-01T22:47:06.000+02:00
Fix composer config variable usage
diff --git a/src/Roundcube/Composer/PluginInstaller.php b/src/Roundcube/Composer/PluginInstaller.php
@@ -23,7 +23,7 @@ protected function confirmInstall($package_name)
     {
         $config = $this->composer->getConfig()->get('roundcube');
 
-        if (!empty($config['enable-plugin'])) {
+        if (isset($config['enable-plugin'])) {
             $answer = $config['enable-plugin'];
         }
         else {
diff --git a/src/Roundcube/Composer/SkinInstaller.php b/src/Roundcube/Composer/SkinInstaller.php
@@ -23,7 +23,7 @@ protected function confirmInstall($package_name)
     {
         $config = $this->composer->getConfig()->get('roundcube');
 
-        if (!empty($config['enable-skin'])) {
+        if (isset($config['enable-skin'])) {
             $answer = $config['enable-skin'];
         }
         else {

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ protected function confirmInstall($package_name)`
`23`	`23`	`{`
`24`	`24`	`$config = $this->composer->getConfig()->get('roundcube');`
`25`	`25`
`26`		`- if (!empty($config['enable-plugin'])) {`
	`26`	`+ if (isset($config['enable-plugin'])) {`
`27`	`27`	`$answer = $config['enable-plugin'];`
`28`	`28`	`}`
`29`	`29`	`else {`