From 6bd4d69e4cc610a1399836479492698dc0ee1625 Mon Sep 17 00:00:00 2001
From: Alexey Kasyanchuk <degitx@gmail.com>
Date: Tue, 26 Jun 2018 19:31:18 +0300
Subject: [PATCH] fix(downloading): always show downloading on list even if
 download not started

---
 src/app/download-page.js |  5 +++--
 src/app/torrent.js       | 31 +++++++++++++++--------------
 src/background/api.js    | 42 +++++++++++++++++++++-------------------
 3 files changed, 41 insertions(+), 37 deletions(-)

diff --git a/src/app/download-page.js b/src/app/download-page.js
index 781d788..17d64e6 100644
--- a/src/app/download-page.js
+++ b/src/app/download-page.js
@@ -45,8 +45,9 @@ export default class TopPage extends Page {
   				}} />
   				<List style={{paddingBottom: '70px', minWidth: '60%'}} className='animated recent-torrents'>
   					{
-  						this.downloads.map((download, index) => {
-  							return <TorrentLine key={index} torrent={download.torrentObject} download={download} />
+  						this.downloads.map((torrentDownload, index) => {
+  							const {torrentObject: torrent, ...download} = torrentDownload
+  							return <TorrentLine key={index} torrent={torrent} download={download} />
   						})
   					}
   				</List>
diff --git a/src/app/torrent.js b/src/app/torrent.js
index 6fdb3d0..e6212f2 100644
--- a/src/app/torrent.js
+++ b/src/app/torrent.js
@@ -171,6 +171,7 @@ export default class Torrent extends Component {
   		this.state.downloadProgress = {
   			progress, downloaded, downloadSpeed
   		}
+  		this.state.downloading = true
   	}
   }
   componentDidMount()
@@ -303,21 +304,21 @@ export default class Torrent extends Component {
                   	/>
                   	<div className='row center pad0-5 fs0-85 text-nowrap' style={{color: torrentRating >= 50 ? '#00E676' : '#FF3D00', width: '190px'}}>{__('Torrent rating')}: {torrentRating}%</div>
                   </div>
-							  }
-							  {
-  									this.state.downloading
-									&&
-									<div className='row w100p inline text-nowrap' style={{maxWidth: 580}}>
-										<div style={{marginRight: 5, color: 'rgb(0, 188, 212)'}}>{__('downloading')}: </div>
-										<LinearProgress 
-											style={{height: '5px', width: '44%', marginTop: 2}}
-											mode="determinate" 
-											value={this.state.downloadProgress && (this.state.downloadProgress.progress ? this.state.downloadProgress.progress : 0) * 100}
-										/>
-										<div className='pad0-75' style={{marginLeft: 20}} style={{color: 'rgb(0, 188, 212)'}}>{this.state.downloadProgress && (this.state.downloadProgress.progress * 100).toFixed(1)}%</div>
-										<div style={{marginLeft: 5, color: 'rgb(0, 188, 212)'}}>{this.state.downloadProgress && formatBytes(this.state.downloadProgress.downloadSpeed || 0, 0)}/s</div>
-									</div>
-  								}
+  							}
+  							{
+  								this.state.downloading
+                                    &&
+                                    <div className='row w100p inline text-nowrap' style={{maxWidth: 580}}>
+                                    	<div style={{marginRight: 5, color: 'rgb(0, 188, 212)'}}>{__('downloading')}: </div>
+                                    	<LinearProgress 
+                                    		style={{height: '5px', width: '44%', marginTop: 2}}
+                                    		mode="determinate" 
+                                    		value={this.state.downloadProgress && (this.state.downloadProgress.progress ? this.state.downloadProgress.progress : 0) * 100}
+                                    	/>
+                                    	<div className='pad0-75' style={{marginLeft: 20}} style={{color: 'rgb(0, 188, 212)'}}>{this.state.downloadProgress && (this.state.downloadProgress.progress * 100).toFixed(1)}%</div>
+                                    	<div style={{marginLeft: 5, color: 'rgb(0, 188, 212)'}}>{this.state.downloadProgress && formatBytes(this.state.downloadProgress.downloadSpeed || 0, 0)}/s</div>
+                                    </div>
+  							}
   						</div>
   					</a>
   				}
diff --git a/src/background/api.js b/src/background/api.js
index 2948c9d..49cd749 100644
--- a/src/background/api.js
+++ b/src/background/api.js
@@ -585,31 +585,33 @@ module.exports = async ({
 			return
 		}
 
-		torrentClient.add(magnet, {path: config.client.downloadPath}, (torrent) =>{
-			torrentClientHashMap[torrent.infoHash] = magnet
-			torrent.torrentObject = torrentObject
+		const torrent = torrentClient.add(magnet, {path: config.client.downloadPath})
+		torrentClientHashMap[torrent.infoHash] = magnet
+		torrent.torrentObject = torrentObject
+
+		torrent.on('torrent', () => {
 			console.log('start downloading', torrent.infoHash)
 			send('downloading', torrent.infoHash)
+		})
 
-			torrent.on('done', () => { 
-				console.log('download done', torrent.infoHash)
-				delete torrentClientHashMap[torrent.infoHash]
-				send('downloadDone', torrent.infoHash)
-			})
+		torrent.on('done', () => { 
+			console.log('download done', torrent.infoHash)
+			delete torrentClientHashMap[torrent.infoHash]
+			send('downloadDone', torrent.infoHash)
+		})
 
-			let now = Date.now()
-			torrent.on('download', (bytes) => {
-				if(Date.now() - now < 100)
-					return
-				now = Date.now()
+		let now = Date.now()
+		torrent.on('download', (bytes) => {
+			if(Date.now() - now < 100)
+				return
+			now = Date.now()
 
-				send('downloadProgress', torrent.infoHash, {
-					received: bytes,
-					downloaded: torrent.downloaded,
-					downloadSpeed: torrent.downloadSpeed,
-					progress: torrent.progress,
-					timeRemaining: torrent.timeRemaining
-				})
+			send('downloadProgress', torrent.infoHash, {
+				received: bytes,
+				downloaded: torrent.downloaded,
+				downloadSpeed: torrent.downloadSpeed,
+				progress: torrent.progress,
+				timeRemaining: torrent.timeRemaining
 			})
 		})
 	});