fix: bootstrap rm return value when all option is set

Pedro Santos · Pedro Santos · commit 98390c2b235a · 2019-11-22T17:29:09.000Z
diff --git a/src/core/components/bootstrap.js b/src/core/components/bootstrap.js
@@ -45,16 +45,17 @@ module.exports = function bootstrap (self) {
         throw invalidMultiaddrError(multiaddr)
       }
 
+      let res = []
       const config = await self._repo.config.get()
       if (args.all) {
+        res = config.Bootstrap
         config.Bootstrap = []
       } else {
         config.Bootstrap = config.Bootstrap.filter((mh) => mh !== multiaddr)
       }
 
       await self._repo.config.set(config)
 
-      const res = []
       if (!args.all && multiaddr) {
         res.push(multiaddr)
       }

Original file line number	Diff line number	Diff line change
`@@ -45,16 +45,17 @@ module.exports = function bootstrap (self) {`
`45`	`45`	`throw invalidMultiaddrError(multiaddr)`
`46`	`46`	`}`
`47`	`47`
	`48`	`+ let res = []`
`48`	`49`	`const config = await self._repo.config.get()`
`49`	`50`	`if (args.all) {`
	`51`	`+ res = config.Bootstrap`
`50`	`52`	`config.Bootstrap = []`
`51`	`53`	`} else {`
`52`	`54`	`config.Bootstrap = config.Bootstrap.filter((mh) => mh !== multiaddr)`
`53`	`55`	`}`
`54`	`56`
`55`	`57`	`await self._repo.config.set(config)`
`56`	`58`
`57`		`- const res = []`
`58`	`59`	`if (!args.all && multiaddr) {`
`59`	`60`	`res.push(multiaddr)`
`60`	`61`	`}`