yep, still needed...

Author: Billy Clark

themes/material/default/selectidp-links.php

@@ -43,7 +43,7 @@ function setSelectedIdp(id) {
             <?php
             // in order to bypass some built-in simplesaml behavior, an extra idp
             // might've been added.  It's not meant to be displayed.
-            unset($this->data['idplist']['dummy']);  //TODO: is this still needed?
+            unset($this->data['idplist']['dummy']);
 
             $enabledIdps = [];
             $disabledIdps = [];