Removed ?per_page=100 as canvas memberships via oauth1 with paging may

result in duplicate records
ahalya123 · Jun 25, 2018 · 1ea82f3 · 1ea82f3
1 parent 3cd04c8
commit 1ea82f3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/WebworkBridge/Bridges/LTIBridge.pm b/lib/WebworkBridge/Bridges/LTIBridge.pm
@@ -596,8 +596,8 @@ sub _getAndParseRosterMemberships
 	my $parser = WebworkBridge::Bridges::LTIParser->new($r, $course_ref, $users_ref);
 
 	#uoft
-	#$self->{memberships_url} = $r->param("custom_context_memberships_url");
-	$self->{memberships_url} = $r->param("custom_context_memberships_url") . "?per_page=100";
+	$self->{memberships_url} = $r->param("custom_context_memberships_url");
+	#$self->{memberships_url} = $r->param("custom_context_memberships_url") . "?per_page=100";
 
 	if (defined($self->{memberships_url}))
 	{

diff --git a/lib/WebworkBridge/Bridges/LTIParser.pm b/lib/WebworkBridge/Bridges/LTIParser.pm
@@ -9,7 +9,7 @@ use WebworkBridge::Importer::Error;
 use WeBWorK::Debug;
 use Data::Dumper;
 use JSON;
-use experimental 'smartmatch';  #uoft added else warnings as this is older version of perl
+#use experimental 'smartmatch';  #uoft added else warnings as this is older version of perl
 
 ##### Exported Functions #####
 sub new