all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pbs-devel] [PATCH proxmox-backup 1/2] ui: tape: fix eslint warnings (trailing comma)
@ 2021-02-19 14:40 Dominik Csapak
  2021-02-19 14:40 ` [pbs-devel] [PATCH proxmox-backup 2/2] ui: tape/ChangerStatus: fixup for conditional Dominik Csapak
  2021-02-19 15:53 ` [pbs-devel] applied: [PATCH proxmox-backup 1/2] ui: tape: fix eslint warnings (trailing comma) Dietmar Maurer
  0 siblings, 2 replies; 3+ messages in thread
From: Dominik Csapak @ 2021-02-19 14:40 UTC (permalink / raw)
  To: pbs-devel

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/tape/BackupJobs.js    | 6 +++---
 www/tape/ChangerStatus.js | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/www/tape/BackupJobs.js b/www/tape/BackupJobs.js
index f8a64694..7c23864f 100644
--- a/www/tape/BackupJobs.js
+++ b/www/tape/BackupJobs.js
@@ -13,7 +13,7 @@ Ext.define('pbs-tape-backup-job-status', {
 		let task = Proxmox.Utils.parse_task_upid(data['last-run-upid']);
 		return endtime - task.starttime;
 	    },
-	}
+	},
     ],
     idProperty: 'id',
     proxy: {
@@ -42,7 +42,7 @@ Ext.define('PBS.config.TapeBackupJobView', {
 
 	init: function(view) {
 	    Proxmox.Utils.monStoreErrors(view, view.getStore().rstore);
-	}
+	},
     },
 
     listeners: {
@@ -143,5 +143,5 @@ Ext.define('PBS.config.TapeBackupJobView', {
 	let me = this;
 
 	me.callParent();
-    }
+    },
 });
diff --git a/www/tape/ChangerStatus.js b/www/tape/ChangerStatus.js
index 2e35c55c..5a1c6073 100644
--- a/www/tape/ChangerStatus.js
+++ b/www/tape/ChangerStatus.js
@@ -422,7 +422,7 @@ Ext.define('PBS.TapeManagement.ChangerStatus', {
 		    method: 'GET',
 		    params: {
 			"update-status": false,
-		    }
+		    },
 		});
 
 		let [status, drives, tapes_list] = await Promise.all([status_fut, drives_fut, tapes_fut]);
-- 
2.20.1





^ permalink raw reply	[flat|nested] 3+ messages in thread

* [pbs-devel] [PATCH proxmox-backup 2/2] ui: tape/ChangerStatus: fixup for conditional
  2021-02-19 14:40 [pbs-devel] [PATCH proxmox-backup 1/2] ui: tape: fix eslint warnings (trailing comma) Dominik Csapak
@ 2021-02-19 14:40 ` Dominik Csapak
  2021-02-19 15:53 ` [pbs-devel] applied: [PATCH proxmox-backup 1/2] ui: tape: fix eslint warnings (trailing comma) Dietmar Maurer
  1 sibling, 0 replies; 3+ messages in thread
From: Dominik Csapak @ 2021-02-19 14:40 UTC (permalink / raw)
  To: pbs-devel

the statement !a === "somestring" cannot be true since
!a is either true or false and thus not a string

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 www/tape/ChangerStatus.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/tape/ChangerStatus.js b/www/tape/ChangerStatus.js
index 5a1c6073..05ca8892 100644
--- a/www/tape/ChangerStatus.js
+++ b/www/tape/ChangerStatus.js
@@ -518,7 +518,7 @@ Ext.define('PBS.TapeManagement.ChangerStatus', {
 	control: {
 	    'grid[reference=drives]': {
 		cellclick: function(table, td, ci, rec, tr, ri, e) {
-		    if (!e.position.column.dataIndex === 'state') {
+		    if (e.position.column.dataIndex !== 'state') {
 			return;
 		    }
 
-- 
2.20.1





^ permalink raw reply	[flat|nested] 3+ messages in thread

* [pbs-devel] applied: [PATCH proxmox-backup 1/2] ui: tape: fix eslint warnings (trailing comma)
  2021-02-19 14:40 [pbs-devel] [PATCH proxmox-backup 1/2] ui: tape: fix eslint warnings (trailing comma) Dominik Csapak
  2021-02-19 14:40 ` [pbs-devel] [PATCH proxmox-backup 2/2] ui: tape/ChangerStatus: fixup for conditional Dominik Csapak
@ 2021-02-19 15:53 ` Dietmar Maurer
  1 sibling, 0 replies; 3+ messages in thread
From: Dietmar Maurer @ 2021-02-19 15:53 UTC (permalink / raw)
  To: Proxmox Backup Server development discussion, Dominik Csapak

applied both patches




^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-02-19 15:55 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-19 14:40 [pbs-devel] [PATCH proxmox-backup 1/2] ui: tape: fix eslint warnings (trailing comma) Dominik Csapak
2021-02-19 14:40 ` [pbs-devel] [PATCH proxmox-backup 2/2] ui: tape/ChangerStatus: fixup for conditional Dominik Csapak
2021-02-19 15:53 ` [pbs-devel] applied: [PATCH proxmox-backup 1/2] ui: tape: fix eslint warnings (trailing comma) Dietmar Maurer

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal