* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox