Pearl1594 commented on a change in pull request #4611:
URL: https://github.com/apache/cloudstack/pull/4611#discussion_r563468761
##########
File path: ui/src/views/AutogenView.vue
##########
@@ -957,7 +957,8 @@ export default {
if (param.name !== key) {
continue
}
- if (input === undefined || input === null || input === '') {
+ if (input === undefined || input === null || (input === '' &&
Review comment:
@utchoang doing so would lead to the second condition not being
evaluated when the input param is empty('') and the api is one among - update
physicalnetwork/storagepool/host as the first condition evaluates to true -
because (!input) = string not being undefined, null or empty.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
