Issue #2134581 by Stevel: Allow role sync rules with commas in values

Author: stevel

simplesamlphp_auth.info

@@ -2,3 +2,4 @@ name = SimpleSAMLphp authentication
 description = Allows users to authenticate to a remote SAML identity provider (IdP) via a locally configured SimpleSAMLphp service point (SP).
 core = 7.x
 configure = admin/config/people/simplesamlphp_auth
+php = 5.3

simplesamlphp_auth.module

@@ -592,7 +592,7 @@ function _simplesamlphp_auth_rolepopulation($rolemap) {
 
         watchdog('simplesamlphp_auth', 'Evaluate role evaulation: %roleruleeval', array('%roleruleeval' => $roleruleevaluation), WATCHDOG_DEBUG);
 
-        $roleruleevaluationdc = explode(',', $roleruleevaluation);
+        $roleruleevaluationdc = str_getcsv($roleruleevaluation);
         if (!_simplesamlphp_auth_evaulaterolerule($roleruleevaluationdc, $attributes)) {
           $addnew = FALSE;
         }