diff --git a/bin/importPdoMetadata.php b/bin/importPdoMetadata.php index 225472cde8c62d227e6766c3e2668c2eaa53b820..3dbd6cf8caede53c35b0d32507c17fccc7e44162 100755 --- a/bin/importPdoMetadata.php +++ b/bin/importPdoMetadata.php @@ -2,7 +2,7 @@ <?php $baseDir = dirname(dirname(__FILE__)); -require_once $baseDir.DIRECTORY_SEPARATOR . 'lib' . DIRECTORY_SEPARATOR . '_autoload.php'; +require_once $baseDir . DIRECTORY_SEPARATOR . 'lib' . DIRECTORY_SEPARATOR . '_autoload.php'; require_once \SimpleSAML\Utils\Config::getConfigDir() . DIRECTORY_SEPARATOR . 'config.php'; # Iterate through configured metadata sources and ensure diff --git a/bin/initMDSPdo.php b/bin/initMDSPdo.php index 840585d4aefb93469f7840b11e5d54aea08817ec..ca60292003482ebbe7a98a41cee8b3ca46c4b8e8 100755 --- a/bin/initMDSPdo.php +++ b/bin/initMDSPdo.php @@ -8,7 +8,7 @@ $baseDir = dirname(dirname(__FILE__)); require_once $baseDir . DIRECTORY_SEPARATOR . 'lib' . DIRECTORY_SEPARATOR . '_autoload.php'; require_once \SimpleSAML\Utils\Config::getConfigDir() . DIRECTORY_SEPARATOR . 'config.php'; -echo "Initializing Metadata Database...".PHP_EOL; +echo "Initializing Metadata Database..." . PHP_EOL; # Iterate through configured metadata sources and ensure # that a PDO source exists. diff --git a/bin/memcacheSync.php b/bin/memcacheSync.php index f58c83e2f56041f8b9486198be5f9b4c07c37a6b..910841e1c8e698c646b7f620d0184951054a035d 100755 --- a/bin/memcacheSync.php +++ b/bin/memcacheSync.php @@ -21,7 +21,7 @@ if (!class_exists('Memcache') && !class_exists('Memcached')) { $baseDir = dirname(dirname(__FILE__)); // Add library autoloader -require_once($baseDir.'/lib/_autoload.php'); +require_once($baseDir . '/lib/_autoload.php'); // Initialize the configuration $configdir = SimpleSAML\Utils\Config::getConfigDir(); @@ -52,7 +52,7 @@ foreach ($stats as $group) { echo "Total number of keys: " . count($keys) . "\n"; $keys = array_unique($keys); -echo "Total number of unique keys: ".count($keys) . "\n"; +echo "Total number of unique keys: " . count($keys) . "\n"; echo "Starting synchronization.\n" ; @@ -150,7 +150,7 @@ function getServerKeys($server) } } if ($resultSize > 1900000 || count($keys) >= 1000000) { - echo "WARNING: Slab ".$slab." on server " . $host . ":" . $port . + echo "WARNING: Slab " . $slab . " on server " . $host . ":" . $port . " may have contained more keys than we were told about.\n"; $GLOBALS['warnBigSlab'] += 1; } diff --git a/bin/translateAttributes.php b/bin/translateAttributes.php index 07cf01218d03c223bfeb5b07708ffc9e14c792ca..7ed0a2111727f252786c364cae305e29c09c7778 100755 --- a/bin/translateAttributes.php +++ b/bin/translateAttributes.php @@ -38,11 +38,11 @@ unset($languages['no']); // build the list of attributes with their corresponding aliases foreach ($names as $name => $urn) { $lower = str_replace([':', '-'], '_', strtolower($name)); - if (!array_key_exists('attribute_' . $lower, $defs)) { - $defs['attribute_'.$lower] = []; + if (!array_key_exists('attribute_' . $lower, $defs)) { + $defs['attribute_' . $lower] = []; } if (!array_key_exists('attribute_' . $lower, $trans)) { - $trans['attribute_'.$lower] = []; + $trans['attribute_' . $lower] = []; } if (array_key_exists('no', $trans['attribute_' . $lower])) { // fix the locale code @@ -87,8 +87,9 @@ foreach (array_keys($languages) as $language) { continue; } $translation = new Gettext\Translation('', $name); - if (array_key_exists($language, $attribute['translations']) && - !is_null($attribute['translations'][$language]) + if ( + array_key_exists($language, $attribute['translations']) + && !is_null($attribute['translations'][$language]) ) { $t = $strings->find($translation); if ($t) { diff --git a/phpcs.xml b/phpcs.xml index f0fd05728b1a91b5c80614f68dd42095a3823df6..15c8459e584fe7dafc9e5451a6eeb83fa857ce6f 100644 --- a/phpcs.xml +++ b/phpcs.xml @@ -29,6 +29,7 @@ <exclude-pattern>modules/saml/lib/IdP/SAML1.php</exclude-pattern> <exclude-pattern>modules/saml/www/sp/saml1-acs.php</exclude-pattern> <exclude-pattern>tests/lib/SimpleSAML/Auth/TimeLimitedTokenTest.php</exclude-pattern> + <exclude-pattern>www/shib13</exclude-pattern> <!-- Exclude old PHP templates; they are removed in SSP 2.0 --> <exclude-pattern>modules/*/templates/*.tpl.php</exclude-pattern> diff --git a/www/saml2/idp/metadata.php b/www/saml2/idp/metadata.php index fe741a1d364bd783b7025a7139f230250a0bc5d2..bbc874ef4fbdcd5bbe2a1bb1453b1b287e6e0ab9 100644 --- a/www/saml2/idp/metadata.php +++ b/www/saml2/idp/metadata.php @@ -147,7 +147,9 @@ try { ); if (!$idpmeta->hasValue('OrganizationURL')) { - throw new \SimpleSAML\Error\Exception('If OrganizationName is set, OrganizationURL must also be set.'); + throw new \SimpleSAML\Error\Exception( + 'If OrganizationName is set, OrganizationURL must also be set.' + ); } $metaArray['OrganizationURL'] = $idpmeta->getLocalizedString('OrganizationURL'); }