Update SPService.ts

Author: joaojmendes

src/services/SPService.ts

@@ -64,43 +64,37 @@ export default class SPService implements ISPService {
 
    // Get ListAttachments
    public async getListItemAttachments(listId: string, itemId: number, webUrl?: string): Promise<any[]> {
-
-    let spWeb: Web;
-    if (typeof webUrl !== "undefined") {
-      spWeb = new Web(webUrl);
-    } else {
-      spWeb = new Web(this._context.pageContext.web.absoluteUrl);
-    }
-
     try {
-      const files = await spWeb.lists
-        .getById(listId)
-        .items.getById(itemId)
-        .attachmentFiles.get();
-
-      return Promise.resolve(files);
+      const webAbsoluteUrl = !webUrl ? this._context.pageContext.web.absoluteUrl : webUrl;
+      const apiUrl = `${webAbsoluteUrl}/_api/web/lists('${listId}')/items(${itemId})/AttachmentFiles`;
+      const data = await this._context.spHttpClient.get(apiUrl, SPHttpClient.configurations.v1);
+      if (data.ok) {
+        const results = await data.json();
+        if (results && results.value && results.value.length > 0) {
+          return results.value;
+        }
+      }
+      return null;
     } catch (error) {
       console.dir(error);
       return Promise.reject(error);
     }
+    
   }
 
   // delete attachment
   public async deleteAttachment(fileName: string, listId: string, itemId: number, webUrl?: string): Promise<void> {
-    let spWeb: Web;
-    if (typeof webUrl !== "undefined") {
-      spWeb = new Web(webUrl);
-    } else {
-      spWeb = new Web(this._context.pageContext.web.absoluteUrl);
-    }
     try {
-      await spWeb.lists
-        .getById(listId)
-        .items.getById(itemId)
-        .attachmentFiles.getByName(encodeURIComponent(fileName))
-        .delete();
-      return;
+      const spOpts: ISPHttpClientOptions = {
+        headers: { "X-HTTP-Method": 'DELETE', }
+      };
+      const webAbsoluteUrl = !webUrl ? this._context.pageContext.web.absoluteUrl : webUrl;
+      const apiUrl = `${webAbsoluteUrl}/_api/web/lists('${listId}')/items(${itemId})/AttachmentFiles/getByFileName('${encodeURIComponent(fileName)}')`;
+      const data = await this._context.spHttpClient.post(apiUrl, SPHttpClient.configurations.v1, spOpts);
+
+
     } catch (error) {
+      console.dir(error);
       return Promise.reject(error);
     }
   }
@@ -124,12 +118,15 @@ export default class SPService implements ISPService {
         await this.deleteAttachment(fileName, listId, itemId, webUrl);
       }
       // Add Attachment
-      const files = await spWeb.lists
-        .getById(listId)
-        .items.getById(itemId)
-        .attachmentFiles.add(fileName, file);
+      const spOpts: ISPHttpClientOptions = {
+        body: file
+      };
+      const webAbsoluteUrl = !webUrl ? this._context.pageContext.web.absoluteUrl : webUrl;
+      const apiUrl = `${webAbsoluteUrl}/_api/web/lists('${listId}')/items(${itemId})/AttachmentFiles/add(FileName='${encodeURIComponent(fileName)}')`;
+      const data = await this._context.spHttpClient.post(apiUrl, SPHttpClient.configurations.v1, spOpts);
 
       return;
+
     } catch (error) {
       return Promise.reject(error);
     }
@@ -138,21 +135,17 @@ export default class SPService implements ISPService {
   // get Attachment
   //
   public async getAttachment(listId: string, itemId: number, fileName: string, webUrl?: string): Promise<any> {
-    let spWeb: Web;
-    if (typeof webUrl !== "undefined") {
-      spWeb = new Web(webUrl);
-    } else {
-      spWeb = new Web(this._context.pageContext.web.absoluteUrl);
-    }
-    try {
-      const file = await spWeb.lists
-        .getById(listId)
-        .items.getById(itemId)
-        .attachmentFiles.getByName(encodeURIComponent(fileName)).get();
-      return Promise.resolve(file);
-    } catch (error) {
-      return Promise.reject(error);
+    const webAbsoluteUrl = !webUrl ? this._context.pageContext.web.absoluteUrl : webUrl;
+    const apiUrl = `${webAbsoluteUrl}/_api/web/lists('${listId}')/items(${itemId})/AttachmentFiles/GetByFileBame('${fileName}'))`;
+    const data = await this._context.spHttpClient.get(apiUrl, SPHttpClient.configurations.v1);
+    if (data.ok) {
+      const file = await data.json();
+      if (file) {
+        return file;
+      }
     }
+
+    return null;
   }
   // Check if Attachment Exists
   public async checkAttachmentExists(listId: string, itemId: number, fileName: string, webUrl?: string): Promise<any> {
@@ -177,22 +170,15 @@ export default class SPService implements ISPService {
 
   // Get ListName
   public async getListName(listId: string, webUrl?: string): Promise<string> {
-    let spWeb: Web;
-    if (typeof webUrl !== "undefined") {
-      spWeb = new Web(webUrl);
-    } else {
-      spWeb = new Web(this._context.pageContext.web.absoluteUrl);
-    }
-    try {
-      const list = await spWeb.lists
-        .getById(listId)
-        .select('RootFolder/Name')
-        .expand('RootFolder')
-        .get();
-
-      return Promise.resolve(list.RootFolder.Name);
-    } catch (error) {
-      return Promise.reject(error);
+    const webAbsoluteUrl = !webUrl ? this._context.pageContext.web.absoluteUrl : webUrl;
+    const apiUrl = `${webAbsoluteUrl}/_api/web/lists('${listId}')?$select=RootFolder/Name&$expand=RootFolder)`;
+    const data = await this._context.spHttpClient.get(apiUrl, SPHttpClient.configurations.v1);
+    if (data.ok) {
+      const results = await data.json();
+      if (results) {
+        return results.RootFolder.Name;
+      }
     }
+    return;
   }
 }