How would I update list item attachments using pnp js?

%3CLINGO-SUB%20id%3D%22lingo-sub-1852332%22%20slang%3D%22en-US%22%3EHow%20would%20I%20update%20list%20item%20attachments%20using%20pnp%20js%3F%3C%2FLINGO-SUB%3E%3CLINGO-BODY%20id%3D%22lingo-body-1852332%22%20slang%3D%22en-US%22%3E%3CP%3EI'm%20trying%20to%20find%20how%20to%20create%20a%20file%20upload%20feature%20for%20an%20spfx%20webpart%20that%20uploads%20to%20the%20attachments%20on%20a%20SharePoint%20list%20item.%20I%20can%20successfully%20upload%2C%20but%20the%20user%20should%20be%20able%20to%20click%20on%20the%20SP%20list%20item%20(it%20then%20brings%20up%20the%20modal%20showing%20the%20attached%20files)%20and%20then%20allow%20them%20to%20delete%20an%20attachment%2C%20then%20finally%20click%20an%20update%20button%20which%20adds%20the%20new%20file%20and%2For%20removes%20the%20unwanted%20file.%3C%2FP%3E%3CP%3EAt%20the%20moment%20I'm%20using%20this%20to%20add%20files%3A%3C%2FP%3E%3CP%3E%26nbsp%3B%3C%2FP%3E%3CPRE%20class%3D%22lia-code-sample%20language-applescript%22%3E%3CCODE%3Eprivate%20_addFile%20%3D%20(event)%20%3D%26gt%3B%20%7B%0A%20%20%20let%20resultFile%20%3D%20event.target.files%3B%0A%20%20%20console.log(resultFile%2C%20'resultFile')%3B%0A%20%20%20let%20fileInfos%20%3D%20%5B%5D%3B%0A%20%20%20for%20(var%20i%20%3D%200%3B%20i%20%26lt%3B%20resultFile.length%3B%20i%2B%2B)%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20var%20resfile%20%3D%20resultFile%5Bi%5D%3B%0A%20%20%20%20%20%20console.log(resfile%2C%20'resfile')%3B%0A%20%20%20%20%20%20%20var%20reader%20%3D%20new%20FileReader()%3B%0A%20%20%20%20%20%20%20%20reader.onload%20%3D%20((file)%20%3D%26gt%3B%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20return%20(e)%20%3D%26gt%3B%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%2F%2FPush%20the%20converted%20file%20into%20array%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20fileInfos.push(%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%22name%22%3A%20file.name%2C%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%22content%22%3A%20e.target.result%0A%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20addedFiles%3A%20%5B...this.state.addedFiles%2C%20...%5Bfile.name%5D%5D%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D%2C()%20%3D%26gt%3B%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20if(this.state.addedFiles%5B0%5D)%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20FileAttachment1%3A%20fileInfos%5B0%5D.name%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20if(this.state.addedFiles%5B1%5D)%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20FileAttachment2%3A%20fileInfos%5B1%5D.name%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20if(this.state.addedFiles%5B2%5D)%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20FileAttachment3%3A%20fileInfos%5B2%5D.name%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D%0A%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7D%3B%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20%20%20%20%7D)(resfile)%3B%0A%20%20%20%20%20%20%20%20%20%20%0A%20%20%20%20%20%20%20%20reader.readAsArrayBuffer(resfile)%3B%0A%20%20%20%20%20%20%20%20%2F%2F%20%40ts-ignore%0A%20%20%20%20%20%20const%20fileListToArr%20%3D%20Array.from(resultFile)%3B%0A%20%20%0A%20%20%20%20%20%20if(this.state.fileArray)%7B%0A%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20%20%20fileArray%3A%20fileListToArr%0A%0A%20%20%20%20%20%20%20%20%20%20%7D%2C%20()%20%3D%26gt%3B%20%7B%0A%20%20%20%20%20%20%20%20%20%20%20%20console.log(this.state.fileArray%2C%20'this.state.fileArray')%3B%0A%20%20%20%20%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%20%20%7D%0A%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20%20%20fileInfos%2C%0A%0A%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%7D%0A%20%20%20%20%20%20%20%0A%20%20%7D%3C%2FCODE%3E%3C%2FPRE%3E%3CP%3EI'm%20using%20this%20to%20retrieve%20files%20(when%20the%20user%20clicks%20the%20SP%20item)%3A%3C%2FP%3E%3CP%3E%26nbsp%3B%3C%2FP%3E%3CPRE%20class%3D%22lia-code-sample%20language-applescript%22%3E%3CCODE%3Elet%20attachmentName%20%3D%20sp.web.lists.getByTitle(%22MyList%22).items.getById(sid)%3B%0A%20%20%20%20%20attachmentName.attachmentFiles.get().then((files)%3D%26gt%3B%7B%20%20%0A%20%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20%20%20%20fileInfos%3A%20files%0A%20%20%20%20%20%20%20%7D)%3B%20%3C%2FCODE%3E%3C%2FPRE%3E%3CP%3E%26nbsp%3B%3C%2FP%3E%3CP%3E%3CSPAN%3EAnd%20I%20have%20a%20delete%20icon%20next%20to%20each%20file%20in%20the%20list%20which%20picks%20up%20the%20id%20of%20the%20deleted%20item%2C%20here's%20the%20JSX%20for%20one%20of%20the%20uploaded%20items%3A%3C%2FSPAN%3E%3C%2FP%3E%3CP%3E%26nbsp%3B%3C%2FP%3E%3CPRE%20class%3D%22lia-code-sample%20language-applescript%22%3E%3CCODE%3E%3CP%3E%7Bthis.state.FileAttachment1%20%3F%20this.state.FileAttachment1%20%3A%20null%7D%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%7Bthis.state.FileAttachment1%20%3F%20%3C%2FP%3E%3CDIV%20classname%3D%22%7Bstyles.editIcon%7D%22%3E%3CICON%20iconname%3D%22Delete%22%20id%3D%22%7B'att1'%7D%22%20onclick%3D%22%7Bthis._removeFile%7D%2F%22%3E%3C%2FICON%3E%3C%2FDIV%3E%20%3A%20null%20%7D%0A%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%20%3CP%3E%3C%2FP%3E%3C%2FCODE%3E%3C%2FPRE%3E%3CP%3E%26nbsp%3B%3C%2FP%3E%3CP%3EI've%20tried%20this%20to%20remove%20the%20files%3A%3C%2FP%3E%3CP%3E%26nbsp%3B%3C%2FP%3E%3CPRE%20class%3D%22lia-code-sample%20language-applescript%22%3E%3CCODE%3Eprivate%20_removeFile%20%3D%20(ev)%20%3D%26gt%3B%20%7B%0A%20%20%20%20const%20cid%20%3D%20ev.target.id%3B%0A%20%20console.log(cid%2C%20'cid')%3B%20%2F%2Fthis%20gives%20the%20id%20for%20the%20selected%20file.%0A%20%20console.log(this.state.fileInfos)%3B%0A%0A%20%20let%20filteredArray%20%3D%20this.state.fileInfos.filter(item%20%3D%26gt%3B%20)%3B%0A%20%20%20%20%20%20%0A%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20fileInfos%3A%20filteredArray%2C%0A%20%20%20%20%7D%2C()%20%3D%26gt%3B%20%0A%20%20%20%20%7Bconsole.log(this.state.fileInfos%2C%20'state.fileInfos')%3B%7D)%3B%0A%20%20%20%20if(cid%20%3D%3D%20'att1')%7B%0A%20%20%20%20%20%20%0A%20%20%20%20%20this.setState(%7B%0A%20%20%20%0A%20%20%20%20%20%7D)%3B%0A%20%20%20%20%7D%0A%20%20%20%20if(cid%20%3D%3D%20'att2')%7B%0A%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%0A%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%20%7D%0A%20%20%20%20if(cid%20%3D%3D%20'att3')%7B%0A%20%20%20%20%20%20this.setState(%7B%0A%20%20%20%20%20%20%0A%20%20%20%20%20%20%7D)%3B%0A%20%20%20%20%20%7D%0A%20%20%7D%3C%2FCODE%3E%3C%2FPRE%3E%3CP%3EAs%20you%20can%20see%20on%20that%20last%20function%20I've%20been%3CSPAN%3E%26nbsp%3B%3C%2FSPAN%3E%3CA%20href%3D%22https%3A%2F%2Fstackoverflow.com%2Fquestions%2F58351957%2Fwhy-not-to-use-splice-with-spread-operator-to-remove-item-from-an-array-in-react%22%20target%3D%22_blank%22%20rel%3D%22nofollow%20noopener%20noreferrer%22%3Etrying%20to%20use%20a%20filter%20to%20remove%20the%20item%20from%20the%20array%3C%2FA%3E%3CSPAN%3E%26nbsp%3B%3C%2FSPAN%3EfileInfos%2C%20but%20whenever%20I%20click%20on%20delete%20to%20remove%20a%20file%20it's%20not%20removing%20it%20from%20the%20fileInfos%20array%20in%20state.%3C%2FP%3E%3CP%3EHas%20anyone%20got%20a%20better%20solution%20or%20better%20yet%2C%20a%20better%20file%20management%20system%20I%20could%20use%3F%20PS.%20I%20know%20it's%20frowned%20upon%20to%20use%20file%20attachments%20on%20list%20items.%3C%2FP%3E%3C%2FLINGO-BODY%3E%3CLINGO-LABS%20id%3D%22lingo-labs-1852332%22%20slang%3D%22en-US%22%3E%3CLINGO-LABEL%3EDeveloper%3C%2FLINGO-LABEL%3E%3C%2FLINGO-LABS%3E
Contributor

I'm trying to find how to create a file upload feature for an spfx webpart that uploads to the attachments on a SharePoint list item. I can successfully upload, but the user should be able to click on the SP list item (it then brings up the modal showing the attached files) and then allow them to delete an attachment, then finally click an update button which adds the new file and/or removes the unwanted file.

At the moment I'm using this to add files:

 

private _addFile = (event) => {
   let resultFile = event.target.files;
   console.log(resultFile, 'resultFile');
   let fileInfos = [];
   for (var i = 0; i < resultFile.length; i++) {
               
     var resfile = resultFile[i];
      console.log(resfile, 'resfile');
       var reader = new FileReader();
        reader.onload = ((file) => {
           return (e) => {
             //Push the converted file into array
              fileInfos.push({
                "name": file.name,
                "content": e.target.result
           
                });
                    
               this.setState({
               addedFiles: [...this.state.addedFiles, ...[file.name]]
               },() => {
                  
                if(this.state.addedFiles[0]){
                 this.setState({
                  FileAttachment1: fileInfos[0].name
                  });
                 }
                 if(this.state.addedFiles[1]){
                  this.setState({
                    FileAttachment2: fileInfos[1].name
                   });
                  }
                  if(this.state.addedFiles[2]){
                   this.setState({
                     FileAttachment3: fileInfos[2].name
                     });
                    }
      
                  });
                  
               };
                  
           })(resfile);
          
        reader.readAsArrayBuffer(resfile);
        // @ts-ignore
      const fileListToArr = Array.from(resultFile);
  
      if(this.state.fileArray){
      this.setState({
        fileArray: fileListToArr

          }, () => {
            console.log(this.state.fileArray, 'this.state.fileArray');
          });
      }
      this.setState({
        fileInfos,

      });
    }
       
  }

I'm using this to retrieve files (when the user clicks the SP item):

 

let attachmentName = sp.web.lists.getByTitle("MyList").items.getById(sid);
     attachmentName.attachmentFiles.get().then((files)=>{  
       this.setState({
         fileInfos: files
       }); 

 

And I have a delete icon next to each file in the list which picks up the id of the deleted item, here's the JSX for one of the uploaded items:

 

<p>{this.state.FileAttachment1 ? this.state.FileAttachment1 : null}
                {this.state.FileAttachment1 ? <div className={styles.editIcon}><Icon iconName="Delete" id={'att1'} onClick={this._removeFile}/></div> : null }
                </p>

 

I've tried this to remove the files:

 

private _removeFile = (ev) => {
    const cid = ev.target.id;
  console.log(cid, 'cid'); //this gives the id for the selected file.
  console.log(this.state.fileInfos);

  let filteredArray = this.state.fileInfos.filter(item => );
      
    this.setState({
      fileInfos: filteredArray,
    },() => 
    {console.log(this.state.fileInfos, 'state.fileInfos');});
    if(cid == 'att1'){
      
     this.setState({
   
     });
    }
    if(cid == 'att2'){
      this.setState({
     
      });
     }
    if(cid == 'att3'){
      this.setState({
      
      });
     }
  }

As you can see on that last function I've been trying to use a filter to remove the item from the array fileInfos, but whenever I click on delete to remove a file it's not removing it from the fileInfos array in state.

Has anyone got a better solution or better yet, a better file management system I could use? PS. I know it's frowned upon to use file attachments on list items.

0 Replies