Merge pull request #166 from patrick-rodgers/dev

making update from PR #145 that doesn't include all of the files
SharePoint · Jul 27, 2016 · fbeb341 · fbeb341
2 parents dd94e1c + deaec1a
commit fbeb341
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/sharepoint/rest/roles.ts b/src/sharepoint/rest/roles.ts
@@ -29,7 +29,7 @@ export class RoleAssignments extends QueryableCollection {
      * 
      */
     public add(principalId: number, roleDefId: number): Promise<void> {
-        let a = new RoleAssignments(this, `addroleassignment(principalid=${principalId}, roledefid=${roleDefId}`);
+        let a = new RoleAssignments(this, `addroleassignment(principalid=${principalId}, roledefid=${roleDefId})`);
         return a.post();
     }
 
@@ -41,7 +41,7 @@ export class RoleAssignments extends QueryableCollection {
      * 
      */
     public remove(principalId: number, roleDefId: number): Promise<void> {
-        let a = new RoleAssignments(this, `removeroleassignment(principalid=${principalId}, roledefid=${roleDefId}`);
+        let a = new RoleAssignments(this, `removeroleassignment(principalid=${principalId}, roledefid=${roleDefId})`);
         return a.post();
     }