@electerm/electerm-react 1.51.3 → 1.51.18

package/client/components/sftp/confirm-modal-store.jsx

@@ -11,6 +11,7 @@ import { FolderOutlined, FileOutlined } from '@ant-design/icons'
 import {
   fileActions
 } from '../../common/constants'
+import deepCopy from 'json-deep-copy'
 import postMessage from '../../common/post-msg'
 
 const e = window.translate
@@ -25,9 +26,7 @@ function formatTimeAuto (strOrDigit) {
 export default function ConfirmModalStore (props) {
   function act (action) {
     const { transferToConfirm } = props
-    window.store.setState(
-      'transferToConfirm', {}
-    )
+    window.store.transferToConfirm = {}
     const {
       fromFile: {
         id: fileId
@@ -39,7 +38,7 @@ export default function ConfirmModalStore (props) {
       transferGroupId,
       fileId,
       id,
-      transfer: transferToConfirm,
+      transfer: deepCopy(transferToConfirm),
       action
     })
   }
@@ -129,13 +128,6 @@ export default function ConfirmModalStore (props) {
         >
           {e('skip')}
         </Button>
-        <Button
-          type='dashed'
-          className='mg1l'
-          onClick={() => act(fileActions.skipAll)}
-        >
-          {e('skipAll')}
-        </Button>
         <Button
           danger
           className='mg1l'

package/client/components/sftp/file-mode-modal.jsx

@@ -6,7 +6,7 @@ import React from 'react'
 import { Modal, Button } from 'antd'
 import resolve from '../../common/resolve'
 import time from '../../common/time'
-import { findIndex, update } from 'lodash-es'
+import { update } from 'lodash-es'
 import { mode2permission, permission2mode } from '../../common/mode2permission'
 import { commonActions } from '../../common/constants'
 import renderPermission from './permission-render'
@@ -65,7 +65,7 @@ export default class FileMode extends React.PureComponent {
   onChangePermission = (name, permName) => {
     const { file } = this.state
     const perms = mode2permission(file.mode)
-    const i = findIndex(perms, p => p.name === name)
+    const i = perms.findIndex(p => p.name === name)
     update(
       perms,
       `[${i}].permission.${permName}`,

package/client/components/sftp/sftp-entry.jsx

@@ -2,7 +2,7 @@ import { Component } from 'react'
 import generate from '../../common/uid'
 import runIdle from '../../common/run-idle'
 import { Spin, Modal, notification } from 'antd'
-import { find, isString, findIndex, isEqual, last, isNumber, some, isArray, pick, uniq, debounce } from 'lodash-es'
+import { find, isString, isEqual, last, isNumber, some, isArray, pick, uniq, debounce } from 'lodash-es'
 import FileSection from './file-item'
 import resolve from '../../common/resolve'
 import wait from '../../common/wait'
@@ -14,7 +14,6 @@ import {
   typeMap, maxSftpHistory, paneMap,
   eventTypes,
   fileTypeMap,
-  terminalSshConfigType,
   terminalSerialType,
   unexpectedPacketErrorDesc,
   sftpRetryInterval,
@@ -207,7 +206,7 @@ export default class Sftp extends Component {
       action === commonActions.sftpList &&
       sessionId === this.props.sessionId
     ) {
-      this[type + 'List']()
+      this[type + 'ListDebounce']()
     }
   }
 
@@ -264,7 +263,7 @@ export default class Sftp extends Component {
 
   getIndex = (file) => {
     const { type } = file
-    return findIndex(this.getFileList(type), f => f.id === file.id)
+    return this.getFileList(type).findIndex(f => f.id === file.id)
   }
 
   onResizeDragEnd = () => {
@@ -482,9 +481,7 @@ export default class Sftp extends Component {
 
   shouldRenderRemote = () => {
     const { props } = this
-    return props.tab?.host &&
-      props.tab?.type !== terminalSshConfigType &&
-      props.tab?.type !== terminalSerialType
+    return props.tab?.host && props.tab?.type !== terminalSerialType
   }
 
   initLocalAll = () => {

package/client/components/sftp/transfer-conflict-store.jsx

@@ -12,9 +12,11 @@ import {
   getFileExt,
   checkFolderSize
 } from './file-read'
-import { findIndex, find } from 'lodash-es'
 import generate from '../../common/uid'
 import resolve from '../../common/resolve'
+import deepCopy from 'json-deep-copy'
+
+const { assign } = Object
 
 export default class TransferConflictStore extends PureComponent {
   state = {
@@ -27,7 +29,7 @@ export default class TransferConflictStore extends PureComponent {
 
   componentDidUpdate (prevProps) {
     if (
-      prevProps._fileTransfers !== this.props._fileTransfers
+      prevProps.fileTransferChanged !== this.props.fileTransferChanged
     ) {
       this.watchFile()
     }
@@ -56,17 +58,16 @@ export default class TransferConflictStore extends PureComponent {
     const newName = ext
       ? `${base}(rename-${renameId}).${ext}`
       : `${base}(rename-${renameId})`
-    const res = {
-      ...tr,
+    assign(tr, {
       renameId,
       newName,
       oldName: base,
       toPath: resolve(path, newName)
-    }
+    })
     if (action) {
-      res.action = action
+      tr.action = action
     }
-    return res
+    return tr
   }
 
   updateTransferAction = (data) => {
@@ -79,65 +80,55 @@ export default class TransferConflictStore extends PureComponent {
       fromFile
     } = transfer
     this.clear()
-    let { fileTransfers } = this.props
+
     const { store } = window
-    const index = findIndex(fileTransfers, d => d.id === id)
+    const { fileTransfers } = store
+    const index = fileTransfers.findIndex(d => d.id === id)
     if (index < 0) {
-      return store.setFileTransfers(fileTransfers)
+      return
     }
-    fileTransfers[index].fromFile = fromFile
-    fileTransfers[index].action = action
+    const tr = fileTransfers[index]
+    tr.fromFile = deepCopy(fromFile)
+    tr.action = action
+    tr.r = Math.random()
     if (action === 'skip') {
-      fileTransfers.splice(index, 1)
+      return fileTransfers.splice(index, 1)
     } else if (action === 'cancel') {
-      fileTransfers = fileTransfers.slice(0, index)
+      return store.skipAllTransfersSinceIndex(index)
     }
     if (action.includes('All')) {
-      fileTransfers = fileTransfers.map((t, i) => {
-        if (i < index) {
-          return t
-        }
-        return {
-          ...t,
-          action: action.replace('All', '')
-        }
+      return store.updateTransfersFromIndex(index, {
+        action: action.replace('All', '')
       })
     }
     if (action.includes('rename')) {
-      fileTransfers[index] = this.rename(fileTransfers[index])
-    } else if (action === 'skipAll') {
-      fileTransfers.splice(index, 1)
+      return this.rename(tr)
     }
-    store.setFileTransfers(fileTransfers)
   }
 
   tagTransferError = (id, errorMsg) => {
-    const { fileTransfers } = this.props
-    const tr = find(fileTransfers, d => d.id === id)
-    if (!tr) {
+    this.clear()
+    const { store } = window
+    const { fileTransfers } = store
+    const index = fileTransfers.findIndex(d => d.id === id)
+    if (index < 0) {
       return
     }
-    const { store } = window
-    store.addTransferHistory({
-      ...tr,
+
+    const [tr] = fileTransfers.splice(index, 1)
+    assign(tr, {
       host: tr.host,
       error: errorMsg,
       finishTime: Date.now()
     })
-    const index = findIndex(fileTransfers, d => d.id === id)
-    if (index >= 0) {
-      fileTransfers.splice(index, 1)
-    }
-    store.setFileTransfers(fileTransfers)
+    store.addTransferHistory(tr)
   }
 
   setConflict (tr) {
     if (this.props.transferToConfirm.id) {
       return
     }
-    window.store.setState(
-      'transferToConfirm', tr
-    )
+    window.store.transferToConfirm = tr
   }
 
   onDecision = (event) => {
@@ -157,6 +148,18 @@ export default class TransferConflictStore extends PureComponent {
     window.addEventListener('message', this.onDecision)
   }
 
+  updateData = () => {
+    const {
+      store
+    } = window
+    const {
+      fileTransfers
+    } = store
+    if (fileTransfers.length > 0) {
+      fileTransfers[0].r = Math.random()
+    }
+  }
+
   setCanTransfer = (fromFile, tr) => {
     this.clear()
     const {
@@ -164,20 +167,20 @@ export default class TransferConflictStore extends PureComponent {
     } = window
     const {
       fileTransfers
-    } = this.props
-    const index = findIndex(fileTransfers, t => {
+    } = store
+    const index = fileTransfers.findIndex(t => {
       return t.id === tr.id
     })
-    if (index >= 0) {
-      const up = {
-        action: 'transfer',
-        fromFile
-      }
-      Object.assign(fileTransfers[index], up)
-    } else if (fileTransfers[0]) {
-      fileTransfers[0].r = Math.random()
+    if (index < 0) {
+      setTimeout(this.updateData, 0)
+      return
     }
-    store.setFileTransfers(fileTransfers)
+    const up = {
+      action: 'transfer',
+      fromFile
+    }
+    assign(fileTransfers[index], up)
+    // may have issue
   }
 
   clear = () => {
@@ -188,25 +191,24 @@ export default class TransferConflictStore extends PureComponent {
     const { store } = window
     const {
       fileTransfers
-    } = this.props
-    if (!fileTransfers.length && this.currentId) {
+    } = store
+    if (!fileTransfers.length) {
       return this.clear()
     }
     const tr = fileTransfers
-      .filter(t => {
+      .find(t => {
         return (
           !t.action ||
           !t.fromFile ||
           t.fromFile.isDirectory
         )
-      })[0]
+      })
     if (!tr) {
       this.onConfirm = false
       return this.clear()
     }
     if (this.currentId) {
-      // fileTransfers[0].r = Math.random()
-      return store.setFileTransfers(fileTransfers)
+      return
     }
     this.currentId = tr.id
     const {
@@ -225,7 +227,6 @@ export default class TransferConflictStore extends PureComponent {
       ? tr.fromFile
       : await this.checkExist(typeFrom, fromPath, sessionId)
     if (!fromFile) {
-      this.currentId = ''
       return this.tagTransferError(id, 'file not exist')
     }
     let toFile = false
@@ -249,33 +250,33 @@ export default class TransferConflictStore extends PureComponent {
       tr.skipExpand = true
     }
     if (fromPath === toPath && typeFrom === typeTo) {
+      assign(tr, {
+        operation: 'cp',
+        fromFile
+      })
       return this.updateTransferAction({
         id,
         action: 'rename',
-        transfer: {
-          ...tr,
-          operation: 'cp',
-          fromFile
-        }
+        transfer: tr
       })
     } else if (toFile && !action && !skipConfirm) {
       this.waitForSignal(id)
       if (!this.onConfirm) {
         this.onConfirm = true
-        return this.setConflict({
-          ...tr,
+        assign(tr, {
           fromFile,
           toFile
         })
+        return this.setConflict(tr)
       }
     } else if (toFile && !tr.fromFile && action) {
+      assign(tr, {
+        fromFile
+      })
       return this.updateTransferAction({
         id,
         action,
-        transfer: {
-          ...tr,
-          fromFile
-        }
+        transfer: tr
       })
     }
     this.setCanTransfer(fromFile, tr)

package/client/components/sftp/transport-action-store.jsx

@@ -1,11 +1,13 @@
 import { Component } from 'react'
 import copy from 'json-deep-copy'
-import { findIndex, isFunction } from 'lodash-es'
+import { isFunction } from 'lodash-es'
 import generate from '../../common/uid'
 import { typeMap, transferTypeMap, commonActions } from '../../common/constants'
 import fs from '../../common/fs'
 import format, { computeLeftTime, computePassedTime } from './transfer-speed-format'
-import { getFolderFromFilePath } from './file-read'
+import {
+  getFolderFromFilePath
+} from './file-read'
 import resolve from '../../common/resolve'
 import delay from '../../common/wait'
 import postMsg from '../../common/post-msg'
@@ -35,15 +37,9 @@ export default class TransportAction extends Component {
       this.initTransfer()
     }
     if (
-      this.props.cancel === true &&
-      prevProps.cancel !== true
+      this.props.pausing !== prevProps.pausing
     ) {
-      this.cancel()
-    }
-    if (
-      this.props.pause !== prevProps.pause
-    ) {
-      if (this.props.pause) {
+      if (this.props.pausing) {
         this.pause()
       } else {
         this.resume()
@@ -54,32 +50,24 @@ export default class TransportAction extends Component {
   componentWillUnmount () {
     this.transport && this.transport.destroy()
     this.transport = null
-    clearTimeout(this.timer)
-    this.timer = null
+    this.inst = null
   }
 
   update = (up) => {
-    const { fileTransfers, transfer } = this.props
+    const { transfer } = this.props
     const {
       store
     } = window
-    const index = findIndex(fileTransfers, t => t.id === transfer.id)
-    if (index < 0) {
-      return store.setFileTransfers(fileTransfers)
-    }
-    store.editTransfer(
-      fileTransfers[index].id,
+    store.updateTransfer(
+      transfer.id,
       up
     )
-    Object.assign(fileTransfers[index], up)
-    store.setFileTransfers(fileTransfers)
   }
 
   insert = (insts) => {
-    const { fileTransfers, transfer } = this.props
-    const index = findIndex(fileTransfers, t => t.id === transfer.id)
+    const { fileTransfers } = window.store
+    const { index } = this.props
     fileTransfers.splice(index, 1, ...insts)
-    window.store.setFileTransfers(fileTransfers)
   }
 
   remoteList = () => {
@@ -110,23 +98,38 @@ export default class TransportAction extends Component {
       typeTo,
       next
     } = transfer
-    const cb = this[typeTo + 'List']
     const finishTime = Date.now()
     if (!config.disableTransferHistory) {
+      const r = copy(transfer)
+      delete transfer.next
+      Object.assign(r, update, {
+        finishTime,
+        startTime: this.startTime,
+        size: transfer.fromFile.size,
+        next: null,
+        speed: format(transfer.fromFile.size, this?.startTime)
+      })
       window.store.addTransferHistory(
-        {
-          ...transfer,
-          ...update,
-          finishTime,
-          startTime: this.startTime,
-          size: transfer.fromFile.size,
-          next: null,
-          speed: format(transfer.fromFile.size, this?.startTime)
-        }
+        r
       )
     }
+    const cbs = [
+      this[typeTo + 'List']
+    ]
     if (next) {
-      this.insert([copy(next)])
+      cbs.push(() => {
+        setTimeout(
+          () => {
+            window.store.fileTransfers.splice(
+              this.props.index, 0, copy(next)
+            )
+          },
+          100
+        )
+      })
+    }
+    const cb = () => {
+      cbs.forEach(cb => cb())
     }
     this.cancel(cb)
   }
@@ -160,21 +163,11 @@ export default class TransportAction extends Component {
       return
     }
     this.onCancel = true
-    const {
-      transfer
-    } = this.props
-    let {
-      fileTransfers
-    } = this.props
-    const { id } = transfer
     this.transport && this.transport.destroy()
     this.transport = null
-    fileTransfers = fileTransfers.filter(t => {
-      return t.id !== id
-    })
-    this.timer = setTimeout(() => {
-      window.store.setFileTransfers(fileTransfers)
-    }, 100)
+    window.store.fileTransfers.splice(
+      this.props.index, 1
+    )
     if (isFunction(callback)) {
       callback()
     }
@@ -247,9 +240,9 @@ export default class TransportAction extends Component {
       originalId: transfer.id,
       id: generate()
     }
-    delete newTrans1.fromFile
     delete newTrans1.inited
     delete newTrans1.zip
+    delete newTrans1.fromFile
     const newTrans2 = copy(newTrans1)
     newTrans2.unzip = true
     newTrans2.id = generate()

package/client/components/sftp/transports-action-store.jsx

@@ -14,7 +14,7 @@ export default class TransportsActionStore extends Component {
 
   componentDidUpdate (prevProps) {
     if (
-      prevProps._fileTransfers !== this.props._fileTransfers
+      prevProps.fileTransferChanged !== this.props.fileTransferChanged
     ) {
       this.control()
     }
@@ -22,11 +22,11 @@ export default class TransportsActionStore extends Component {
 
   control = async () => {
     const { store } = window
-    let {
+    const {
       fileTransfers
-    } = this.props
+    } = store
 
-    fileTransfers = fileTransfers.map(t => {
+    fileTransfers.forEach(t => {
       const {
         typeTo,
         typeFrom,
@@ -37,7 +37,6 @@ export default class TransportsActionStore extends Component {
       if (typeTo === typeFrom && ready && !inited) {
         t.inited = true
       }
-      return t
     })
     // if (pauseAllTransfer) {
     //   return store.setFileTransfers(fileTransfers)
@@ -46,15 +45,16 @@ export default class TransportsActionStore extends Component {
       const {
         typeTo,
         typeFrom,
-        inited
+        inited,
+        pausing
       } = t
-      return typeTo !== typeFrom && inited
+      return typeTo !== typeFrom && inited && pausing !== true
     }).length
     if (count >= maxTransport) {
-      return store.setFileTransfers(fileTransfers)
+      return
     }
     const len = fileTransfers.length
-    const ids = []
+    // const ids = []
     for (let i = 0; i < len; i++) {
       const tr = fileTransfers[i]
       const {
@@ -62,13 +62,11 @@ export default class TransportsActionStore extends Component {
         typeFrom,
         inited,
         fromFile,
-        error,
-        id,
         action
       } = tr
-      if (!error) {
-        ids.push(id)
-      }
+      // if (!error) {
+      //   ids.push(id)
+      // }
       const isTransfer = typeTo !== typeFrom
       const ready = (
         action && fromFile
@@ -90,8 +88,10 @@ export default class TransportsActionStore extends Component {
         count++
         tr.inited = true
       }
+      if (count >= maxTransport) {
+        break
+      }
     }
-    store.setFileTransfers(fileTransfers)
   }
 
   render () {

package/client/components/sftp/transports-ui-store.jsx

@@ -11,13 +11,17 @@ export default function TransportsUI (props) {
   }
   return fileTransfers.map((t, i) => {
     const { id } = t
+    const trProps = {
+      transfer: t,
+      index: i,
+      inited: t.inited,
+      cancel: t.cancel,
+      pausing: t.pausing,
+      config: props.config
+    }
     return (
       <Transport
-        {...props}
-        transfer={t}
-        inited={t.inited}
-        cancel={t.cancel}
-        pause={t.pausing}
+        {...trProps}
         key={id + ':tr:' + i}
       />
     )