Skip to content

Commit e14cfd0

Browse files
Fix pr with suggestions
1 parent 1b0eeea commit e14cfd0

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

app/controllers/miq_request_controller.rb

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,6 @@ def prov_copy
219219

220220
if req.kind_of?(ServiceTemplateProvisionRequest)
221221
@dialog_replace_data = req.options[:dialog].map { |key, val| {:name => key.split('dialog_').last, :value => parse_val(key, val)} }.to_json
222-
# req.options[:dialog]["Array::dialog_param_test1"][1] =="\u001F"
223222
@new_dialog = true
224223
template = find_record_with_rbac(ServiceTemplate, req.source_id)
225224
resource_action = template.resource_actions.find { |r| r.action.downcase == 'provision' && r.dialog_id }
@@ -351,12 +350,9 @@ def parse_val(key, val)
351350
# TODO: Tag Control Field currently does not handle default values
352351
if val.include?("Classification::") # Handle tag control default values
353352
if val.include?("\u001F") # Array of tags
354-
val = val.split("\u001F")
355-
tags = []
356-
val.each do |tag|
357-
tags.push(Classification.find_by(:id => tag.split('::').second).description)
353+
val = val.split("\u001F").map do |tag|
354+
Classification.find_by(:id => tag.split('::').second).description
358355
end
359-
val = tags
360356
else # Single tag
361357
val = Classification.find_by(:id => val.split('::').second).description
362358
end

app/javascript/oldjs/controllers/dialog_user/dialog_user_controller.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ ManageIQ.angular.app.controller('dialogUserController', ['API', 'dialogFieldRefr
2121
_.forEach(vm.dialog.dialog_tabs, function(tab) {
2222
_.forEach(tab.dialog_groups, function(group) {
2323
_.forEach(group.dialog_fields, function(field) {
24-
console.log(dialogReplaceData);
2524
const replaceField = dialogReplaceData ? JSON.parse(dialogReplaceData).find(function (replace) { return replace.name === field.name }) : false;
2625

2726
// Handles multi-select dropdowns with integer values

0 commit comments

Comments
 (0)