Merge pull request #46 from dmitry-viskov/regexp-bugfix

MartinLenord · web-flow · commit 8a42ff760736 · 2020-05-06T11:02:29.000+01:00
Fix regexp in the LTI_Names_Roles_Provisioning_Service
diff --git a/src/lti/LTI_Names_Roles_Provisioning_Service.php b/src/lti/LTI_Names_Roles_Provisioning_Service.php
@@ -31,7 +31,7 @@ public function get_members() {
 
             $next_page = false;
             foreach($page['headers'] as $header) {
-                if (preg_match("/^Link:.*<([^>])>; ?rel=\"next\"/i", $header, $matches)) {
+                if (preg_match("/^Link:.*<([^>]*)>; ?rel=\"next\"/i", $header, $matches)) {
                     $next_page = $matches[1];
                     break;
                 }
@@ -41,4 +41,4 @@ public function get_members() {
 
     }
 }
-?>
+?>

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public function get_members() {`
`31`	`31`
`32`	`32`	`$next_page = false;`
`33`	`33`	`foreach($page['headers'] as $header) {`
`34`		`- if (preg_match("/^Link:.*<([^>])>; ?rel=\"next\"/i", $header, $matches)) {`
	`34`	`+ if (preg_match("/^Link:.<([^>])>; ?rel=\"next\"/i", $header, $matches)) {`
`35`	`35`	`$next_page = $matches[1];`
`36`	`36`	`break;`
`37`	`37`	`}`
`@@ -41,4 +41,4 @@ public function get_members() {`
`41`	`41`
`42`	`42`	`}`
`43`	`43`	`}`
`44`		`-?>`
	`44`	`+?>`