Remove unsupported params from API calls on the UI (#6686)

Author: BryanMLima

ui/src/components/header/SamlDomainSwitcher.vue

@@ -73,7 +73,7 @@ export default {
       const samlAccounts = []
       const getNextPage = () => {
         this.loading = true
-        api('listAndSwitchSamlAccount', { listAll: true, details: 'min', page: page, pageSize: 500 }).then(json => {
+        api('listAndSwitchSamlAccount', { details: 'min', page: page, pageSize: 500 }).then(json => {
           if (json && json.listandswitchsamlaccountresponse && json.listandswitchsamlaccountresponse.samluseraccount) {
             samlAccounts.push(...json.listandswitchsamlaccountresponse.samluseraccount)
           }

ui/src/components/view/SearchView.vue

@@ -449,7 +449,7 @@ export default {
     },
     fetchZones () {
       return new Promise((resolve, reject) => {
-        api('listZones', { listAll: true, showicon: true }).then(json => {
+        api('listZones', { showicon: true }).then(json => {
           const zones = json.listzonesresponse.zone
           resolve({
             type: 'zoneid',
@@ -475,7 +475,7 @@ export default {
     },
     fetchPods () {
       return new Promise((resolve, reject) => {
-        api('listPods', { listAll: true }).then(json => {
+        api('listPods').then(json => {
           const pods = json.listpodsresponse.pod
           resolve({
             type: 'podid',
@@ -488,7 +488,7 @@ export default {
     },
     fetchClusters () {
       return new Promise((resolve, reject) => {
-        api('listClusters', { listAll: true }).then(json => {
+        api('listClusters').then(json => {
           const clusters = json.listclustersresponse.cluster
           resolve({
             type: 'clusterid',

ui/src/components/view/SettingsTab.vue

@@ -142,10 +142,7 @@ export default {
   methods: {
     fetchData (callback) {
       this.tabLoading = true
-      const params = {
-        [this.scopeKey]: this.resource.id,
-        listAll: true
-      }
+      const params = { [this.scopeKey]: this.resource.id }
       if (this.filter) {
         params.keyword = this.filter
       }

ui/src/config/section/offering.js

@@ -247,7 +247,6 @@ export default {
       icon: 'wifi-outlined',
       docHelp: 'adminguide/networking.html#network-offerings',
       permission: ['listNetworkOfferings', 'listInfrastructure'],
-      params: { isrecursive: 'true' },
       columns: ['name', 'state', 'guestiptype', 'traffictype', 'networkrate', 'domain', 'zone', 'order'],
       details: ['name', 'id', 'displaytext', 'guestiptype', 'traffictype', 'internetprotocol', 'networkrate', 'ispersistent', 'egressdefaultpolicy', 'availability', 'conservemode', 'specifyvlan', 'specifyipranges', 'supportspublicaccess', 'supportsstrechedl2subnet', 'service', 'tags', 'domain', 'zone'],
       resourceType: 'NetworkOffering',
@@ -339,7 +338,6 @@ export default {
       icon: 'deployment-unit-outlined',
       docHelp: 'plugins/nuage-plugin.html?#vpc-offerings',
       permission: ['listVPCOfferings', 'listInfrastructure'],
-      params: { isrecursive: 'true' },
       resourceType: 'VpcOffering',
       columns: ['name', 'state', 'displaytext', 'domain', 'zone', 'order'],
       details: ['name', 'id', 'displaytext', 'internetprotocol', 'distributedvpcrouter', 'tags', 'service', 'domain', 'zone', 'created'],

ui/src/store/modules/user.js

@@ -214,7 +214,7 @@ const user = {
           })
         } else {
           const hide = message.loading(i18n.global.t('message.discovering.feature'), 0)
-          api('listZones', { listall: true }).then(json => {
+          api('listZones').then(json => {
             const zones = json.listzonesresponse.zone || []
             commit('SET_ZONES', zones)
           }).catch(error => {

ui/src/views/AutogenView.vue

@@ -876,6 +876,15 @@ export default {
       if (this.$showIcon()) {
         params.showIcon = true
       }
+
+      if (['listAnnotations', 'listRoles', 'listZonesMetrics', 'listPods',
+        'listClustersMetrics', 'listHostsMetrics', 'listStoragePoolsMetrics',
+        'listImageStores', 'listSystemVms', 'listManagementServers',
+        'listConfigurations', 'listHypervisorCapabilities',
+        'listAlerts', 'listNetworkOfferings', 'listVPCOfferings'].includes(this.apiName)) {
+        delete params.listall
+      }
+
       api(this.apiName, params).then(json => {
         var responseName
         var objectName

ui/src/views/compute/DeployVM.vue

@@ -1920,7 +1920,7 @@ export default {
       return new Promise((resolve) => {
         this.loading.zones = true
         const param = this.params.zones
-        const args = { listall: true, showicon: true }
+        const args = { showicon: true }
         if (zoneId) args.id = zoneId
         api(param.list, args).then(json => {
           const zoneResponse = json.listzonesresponse.zone || []
@@ -1952,7 +1952,7 @@ export default {
       param.loading = true
       param.opts = []
       const options = param.options || {}
-      if (!('listall' in options)) {
+      if (!('listall' in options) && !['zones', 'pods', 'clusters', 'hosts', 'dynamicScalingVmConfig', 'hypervisors'].includes(name)) {
         options.listall = true
       }
       api(param.list, options).then((response) => {

ui/src/views/compute/EditVM.vue

@@ -254,7 +254,7 @@ export default {
     fetchOsTypes () {
       this.osTypes.loading = true
       this.osTypes.opts = []
-      api('listOsTypes', { listAll: true }).then(json => {
+      api('listOsTypes').then(json => {
         this.osTypes.opts = json.listostypesresponse.ostype || []
       }).catch(error => {
         this.$notifyError(error)

ui/src/views/iam/SSLCertificateTab.vue

@@ -152,7 +152,6 @@ export default {
   methods: {
     fetchData () {
       const params = {}
-      params.listAll = true
       params.page = this.page
       params.pageSize = this.pageSize
       params.accountid = this.resource.id

ui/src/views/image/AddKubernetesSupportedVersion.vue

@@ -189,7 +189,6 @@ export default {
     },
     fetchZoneData () {
       const params = {}
-      params.listAll = true
       params.showicon = true
       this.zoneLoading = true
       api('listZones', params).then(json => {