The calling code of the template had been changed to accommodate
new twig templates, but slightly changed the content of the
template variables so preferredidp never matched the list of known
idp's. This fixes it in the least invasive way. Could be
refactored because we now loop the idplist twice, but since this
is legacy code that's likely not worth it.

Thanks Stefan Winter from RESTENA for reporting.

The radius-module was externalized earlier

Because it's used in the _REQUEST array, the name is changed by PHP
to replacethe dot with an underscore, leading it not to match, that
in turn breaking the exception handling.

Thanks pverbaan for reporting this issue.

Closes: #1077

* Externalize consent module

* Update Psalm

* Ignore DocblockTypeContradiction and RedundantConditionGivenDocblockType

* Fix Psalm-errors

* Raise minimum PHP-version to 5.6

* Update lock-file accordingly

* Fix phpunit

* Suppress Psalm-issue

* Create .appveyor.yml

* Fix tests for Windows platform

* Fix builtin webserver for Windows

* Properly kill process on Windows

Handle untranslated tabs more gracefully in discopower-module

We no longer test on HHVM

This commit adds the ability to override the defaults in the generated SP metadata for SLO Location and ACS endpoints. This is necessary for my use case as I have additional ACS endpoints to publish in my metadata beyond the generated ones as well as a custom SLO handler that I need to direct my users to. If unset in the config, it uses the defaults as before.

Stream wrappers were assumed to be relative paths. When using a stream wrapper (e.g. s3://) it was appended to the given base path.

Silence scrutinizer

Make the module-discopower twig template usable

Remove the inline javascript from discopower-module

We'd like to stick with echo throughout the project

Fix logic error in UIInfo generation

* Fix discopower tabs

* Remove duplication of search prototype