Skip to content

Commit eb3dc98

Browse files
Merge pull request ManageIQ#1280 from AllenBW/bug/master/#0000-console_supported
Update console behavior as per api changes
2 parents c60105b + b8d76c7 commit eb3dc98

File tree

4 files changed

+7
-8
lines changed

4 files changed

+7
-8
lines changed

client/app/services/service-details/service-details.component.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -378,13 +378,13 @@ function ComponentController ($stateParams, $state, $window, CollectionsApi, Eve
378378
}
379379

380380
function openConsole (item) {
381-
if (item['supports_console?'] && item['supports_vnc_console?'] && item.power_state === 'on') {
381+
if (item.supported_consoles.vnc.visible && item.supported_consoles.vnc.enabled) {
382382
Consoles.open(item.id)
383383
}
384384
}
385385

386386
function openCockpit (item) {
387-
if (item['supports_console?'] && item['supports_cockpit?'] && item.power_state === 'on') {
387+
if (item.supported_consoles.cockpit.visible && item.supported_consoles.cockpit.enabled) {
388388
$window.open('http://' + item.ipaddresses[0] + ':9090')
389389
}
390390
}

client/app/services/service-details/service-details.html

+2-2
Original file line numberDiff line numberDiff line change
@@ -288,12 +288,12 @@ <h2 translate>Resources</h2>
288288
<ul class="dropdown-menu dropdown-menu-right" uib-dropdown-menu role="menu"
289289
aria-labelledby="btn-append-to-to-body">
290290
<li role="menuitem"
291-
ng-class="{'disabled': !item['supports_console?'] || !item['supports_vnc_console?'] || item.power_state !== 'on' || !$ctrl.customScope.permissions.console}"
291+
ng-class="{'disabled': !item.supported_consoles.vnc.visible || !item.supported_consoles.vnc.enabled || !$ctrl.customScope.permissions.console}"
292292
ng-click="$ctrl.customScope.openConsole(item)">
293293
<a href="#" translate>VM Console</a>
294294
</li>
295295
<li role="menuitem"
296-
ng-class="{'disabled': !item['supports_console?'] || !item['supports_cockpit?'] || item.power_state !== 'on' || !$ctrl.customScope.permissions.cockpit}"
296+
ng-class="{'disabled': !item.supported_consoles.cockpit.visible || !item.supported_consoles.cockpit.enabled || !$ctrl.customScope.permissions.cockpit}"
297297
ng-click="$ctrl.customScope.openCockpit(item)">
298298
<a href="#" translate>Web Console</a>
299299
</li>

client/app/services/services-state.service.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,9 @@ export function ServicesStateFactory (ListConfiguration, CollectionsApi, RBAC) {
3131
'chargeback_report', 'service_template', 'parent_service', 'power_state', 'power_status', 'options', 'vms.ipaddresses',
3232
'vms.snapshots', 'vms.v_total_snapshots', 'vms.v_snapshot_newest_name', 'vms.v_snapshot_newest_timestamp',
3333
'vms.max_mem_usage_absolute_average_avg_over_time_period', 'vms.hardware',
34-
'vms.hardware.aggregate_cpu_speed', 'vms.cpu_usagemhz_rate_average_avg_over_time_period', 'generic_objects.picture', 'generic_objects.generic_object_definition'
34+
'vms.hardware.aggregate_cpu_speed', 'vms.cpu_usagemhz_rate_average_avg_over_time_period', 'generic_objects.picture',
35+
'generic_objects.generic_object_definition', 'vms.supported_consoles'
3536
],
36-
decorators: ['vms.supports_console?', 'vms.supports_cockpit?', 'supports_vnc_console?'],
3737
expand: ['vms', 'orchestration_stacks', 'generic_objects'],
3838
auto_refresh: refresh
3939
}

client/app/services/services-state.service.spec.js

+1-2
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,9 @@ describe('Service: ServicesStateFactory', () => {
2424
'vms.snapshots', 'vms.v_total_snapshots', 'vms.v_snapshot_newest_name', 'vms.v_snapshot_newest_timestamp',
2525
'vms.max_mem_usage_absolute_average_avg_over_time_period', 'vms.hardware', 'vms.hardware.aggregate_cpu_speed',
2626
'vms.cpu_usagemhz_rate_average_avg_over_time_period', 'generic_objects.picture',
27-
'generic_objects.generic_object_definition'
27+
'generic_objects.generic_object_definition', 'vms.supported_consoles'
2828
],
2929
expand: ['vms', 'orchestration_stacks', 'generic_objects'],
30-
decorators: ['vms.supports_console?', 'vms.supports_cockpit?', 'supports_vnc_console?'],
3130
auto_refresh: false
3231
}
3332
done()

0 commit comments

Comments
 (0)