fix: Fix the issues of raise exception#5130
fix: Fix the issues of raise exception#5130wangliang181230 wants to merge 14 commits into1Panel-dev:v2from
raise exception#5130Conversation
|
Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
| for (let index = 0; index < branch_condition_list.length; index++) { | ||
| const element = branch_condition_list[index] | ||
|
|
wangliang181230
changed the title
raise exceptionraise exception
| raise AppApiException(500, _('Application does not exist')) | ||
| if not application.file_upload_enable: | ||
| return AppApiException(500, _('File upload is not enabled')) | ||
| raise AppApiException(500, _('File upload is not enabled')) |
There was a problem hiding this comment.
关键字错误，这个方法并不是一个返回异常对象的方法。
wangliang181230
changed the title
raise exceptionraise exception
| try: | ||
| return source_value is not None and len(source_value) > 0 | ||
| except Exception: | ||
| return True |
There was a problem hiding this comment.
len(source_value) 可能会抛出异常，捕获处理。
因为抛出异常时 source_value is not None 已经判定为 True，表示对象不为空，所以len方法出现异常后，直接 return True
1 participant
What this PR does / why we need it?
fix: Fix the issue of
raise exceptionSummary of your change
修复
抛出异常相关的问题判断器中，
不为空判断出现异常的截图：Please indicate you've done the following: