From 9b2f422622de292304b6645e98935ca5921b983e Mon Sep 17 00:00:00 2001 From: Aaron3S Date: Wed, 22 Feb 2023 20:05:31 +0800 Subject: [PATCH] =?UTF-8?q?perf:=20=E4=BC=98=E5=8C=96=E6=8A=A5=E9=94=99?= =?UTF-8?q?=E4=BB=A3=E7=A0=81?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- apps/ops/models/job.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/apps/ops/models/job.py b/apps/ops/models/job.py index bd410a8e7..4cb297191 100644 --- a/apps/ops/models/job.py +++ b/apps/ops/models/job.py @@ -407,12 +407,14 @@ class JobExecution(JMSOrgBaseModel): all_permed_assets = UserPermAssetUtil(self.creator).get_all_assets() has_permed_assets = set(self.current_job.assets.all()) & set(all_permed_assets) + error_assets_count = 0 for asset in self.current_job.assets.all(): if asset not in has_permed_assets: print("\033[31mAsset {}({}) has no access permission\033[0m".format(asset.name, asset.address)) + error_assets_count += 1 - if self.current_job.assets.count() != len(has_permed_assets): - raise Exception("You do not have access rights to some assets") + if error_assets_count > 0: + raise Exception("You do not have access rights to {} assets".format(error_assets_count)) def before_start(self): self.check_assets_perms()