gazer 0.3.10 → 0.3.11

data/lib/gzr/modules/dashboard.rb

@@ -29,9 +29,13 @@ module Gzr
     def query_dashboard(dashboard_id)
       data = nil
       begin
-        data = @sdk.dashboard(dashboard_id)
-        data&.dashboard_filters&.sort! { |a,b| a.row <=> b.row }
-        data&.dashboard_layouts&.sort_by! { |v| (v.active ? 0 : 1) }
+        data = @sdk.dashboard(dashboard_id).to_attrs
+        data[:dashboard_filters]&.sort! { |a,b| a.row <=> b.row }
+        data[:dashboard_layouts]&.sort_by! { |v| (v[:active] ? 0 : 1) }
+      rescue LookerSDK::Error => e
+        say_error "dashboard #{dashboard_id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error querying dashboard(#{dashboard_id})"
         say_error e
@@ -41,15 +45,17 @@ module Gzr
     end
 
     def delete_dashboard(dash)
-      data = nil
       begin
-        data = @sdk.delete_dashboard(dash)
+        @sdk.delete_dashboard(dash)
+      rescue LookerSDK::Error => e
+        say_error "dashboard #{dash} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error deleting dashboard(#{dash})"
         say_error e
         raise
       end
-      data
     end
 
     def search_dashboards_by_slug(slug, folder_id=nil)
@@ -57,9 +63,9 @@ module Gzr
       begin
         req = { :slug => slug }
         req[:folder_id] = folder_id if folder_id
-        data = @sdk.search_dashboards(req)
+        data = @sdk.search_dashboards(req).collect { |e| e.to_attrs }
         req[:deleted] = true
-        data = @sdk.search_dashboards(req) if data.empty?
+        data = @sdk.search_dashboards(req).collect { |e| e.to_attrs } if data.empty?
       rescue LookerSDK::Error => e
         say_error "Error search_dashboards_by_slug(#{JSON.pretty_generate(req)})"
         say_error e
@@ -73,9 +79,9 @@ module Gzr
       begin
         req = { :title => title }
         req[:folder_id] = folder_id if folder_id
-        data = @sdk.search_dashboards(req)
+        data = @sdk.search_dashboards(req).collect { |e| e.to_attrs }
         req[:deleted] = true
-        data = @sdk.search_dashboards(req) if data.empty?
+        data = @sdk.search_dashboards(req).collect { |e| e.to_attrs } if data.empty?
       rescue LookerSDK::Error => e
         say_error "Error search_dashboards_by_title(#{JSON.pretty_generate(req)})"
         say_error e
@@ -85,11 +91,12 @@ module Gzr
     end
 
     def create_dashboard(dash)
+      data = nil
       begin
-        data = @sdk.create_dashboard(dash)
+        data = @sdk.create_dashboard(dash).to_attrs
         say_error data.inspect if data.respond_to?(:message)
-        data&.dashboard_filters&.sort! { |a,b| a.row <=> b.row }
-        data&.dashboard_layouts&.sort_by! { |v| (v.active ? 0 : 1) }
+        data[:dashboard_filters]&.sort! { |a,b| a.row <=> b.row }
+        data[:dashboard_layouts]&.sort_by! { |v| (v[:active] ? 0 : 1) }
       rescue LookerSDK::Error => e
         say_error "Error creating dashboard(#{JSON.pretty_generate(dash)})"
         say_error e
@@ -99,9 +106,14 @@ module Gzr
     end
 
     def update_dashboard(dash_id,dash)
+      data = nil
       begin
-        data = @sdk.update_dashboard(dash_id,dash)
-        data&.dashboard_filters&.sort! { |a,b| a.row <=> b.row }
+        data = @sdk.update_dashboard(dash_id,dash).to_attrs
+        data[:dashboard_filters]&.sort! { |a,b| a.row <=> b.row }
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard #{dash_id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error updating dashboard(#{dash_id},#{JSON.pretty_generate(dash)})"
         say_error e
@@ -112,139 +124,162 @@ module Gzr
 
     def create_dashboard_element(dash_elem)
       begin
-        data = @sdk.create_dashboard_element(dash_elem)
+        @sdk.create_dashboard_element(dash_elem).to_attrs
       rescue LookerSDK::Error => e
         say_error "Error creating dashboard_element(#{JSON.pretty_generate(dash_elem)})"
         say_error e
         raise
       end
-      data
     end
 
     def update_dashboard_element(id,dash_elem)
       begin
-        data = @sdk.update_dashboard_element(id,dash_elem)
+        @sdk.update_dashboard_element(id,dash_elem).to_attrs
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_element #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error updating dashboard_element(#{id},#{JSON.pretty_generate(dash_elem)})"
         say_error e
         raise
       end
-      data
     end
 
     def delete_dashboard_element(id)
       begin
-        data = @sdk.delete_dashboard_element(id)
+        @sdk.delete_dashboard_element(id)
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_element #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error deleting dashboard_element(#{id})})"
         say_error e
         raise
       end
-      data
     end
 
     def get_dashboard_layout(id)
       begin
-        data = @sdk.dashboard_layout(id)
+        @sdk.dashboard_layout(id).to_attrs
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_layout #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error getting dashboard_layout(#{id})"
         say_error e
         raise
       end
-      data
     end
 
     def create_dashboard_layout(dash_layout)
       begin
-        data = @sdk.create_dashboard_layout(dash_layout)
+        @sdk.create_dashboard_layout(dash_layout).to_attrs
       rescue LookerSDK::Error => e
         say_error "Error creating dashboard_layout(#{JSON.pretty_generate(dash_layout)})"
         say_error e
         raise
       end
-      data
     end
 
     def update_dashboard_layout(id,dash_layout)
       begin
-        data = @sdk.update_dashboard_layout(id,dash_layout)
+        @sdk.update_dashboard_layout(id,dash_layout).to_attrs
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_layout #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error updating dashboard_layout(#{id},#{JSON.pretty_generate(dash_layout)})"
         say_error e
         raise
       end
-      data
     end
 
     def delete_dashboard_layout(id)
       begin
-        data = @sdk.delete_dashboard_layout(id)
+        @sdk.delete_dashboard_layout(id)
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_layout #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error deleting dashboard_layout(#{id})"
         say_error e
         raise
       end
-      data
     end
 
     def get_all_dashboard_layout_components(id)
       begin
-        data = @sdk.dashboard_layout_dashboard_layout_components(id)
-        return nil if data.respond_to?(:message) && data.message == 'Not found'
+        @sdk.dashboard_layout_dashboard_layout_components(id).collect { |e| e.to_attrs }
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_layout #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error getting dashboard_layout_dashboard_layout_components(#{id})"
         say_error e
         raise
       end
-      data
     end
 
     def update_dashboard_layout_component(id,component)
       begin
-        data = @sdk.update_dashboard_layout_component(id,component)
+        @sdk.update_dashboard_layout_component(id,component).to_attrs
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_layout #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error updating dashboard_layout_component(#{id},#{JSON.pretty_generate(component)})"
         say_error e
         raise
       end
-      data
     end
 
     def create_dashboard_filter(dash_filter)
       begin
-        data = @sdk.create_dashboard_filter(dash_filter)
+        @sdk.create_dashboard_filter(dash_filter).to_attrs
       rescue LookerSDK::Error => e
         say_error "Error creating dashboard_filter(#{JSON.pretty_generate(dash_filter)})"
         say_error e
         raise
       end
-      data
     end
 
     def update_dashboard_filter(id,dash_filter)
       begin
-        data = @sdk.update_dashboard_filter(id,dash_filter)
+        @sdk.update_dashboard_filter(id,dash_filter).to_attrs
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_filter #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error updating dashboard_filter(#{id},#{JSON.pretty_generate(dash_filter)})"
         say_error e
         raise
       end
-      data
     end
 
     def delete_dashboard_filter(id)
       begin
-        data = @sdk.delete_dashboard_filter(id)
+        @sdk.delete_dashboard_filter(id)
+      rescue LookerSDK::NotFound => e
+        say_error "dashboard_filter #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error deleting dashboard_filter(#{id})})"
         say_error e
         raise
       end
-      data
     end
 
     def cat_dashboard(dashboard_id)
-      data = query_dashboard(dashboard_id).to_attrs
+      data = query_dashboard(dashboard_id)
       data[:dashboard_elements].each_index do |i|
         element = data[:dashboard_elements][i]
         find_vis_config_reference(element) do |vis_config|
@@ -253,12 +288,11 @@ module Gzr
           end
         end
         alerts = search_alerts(fields: 'id,dashboard_element_id', group_by: 'dashboard', all_owners: true)
-        alerts.map!{|v| v.to_attrs}
         say_warning alerts if @options[:debug]
         data[:dashboard_elements].each do |e|
           alerts_found = alerts.select { |a| a[:dashboard_element_id] == e[:id]}
           say_warning "Found alerts #{alerts_found}" if @options[:debug]
-          alerts_entries = alerts_found.map { |a| get_alert(a[:id]).to_attrs }
+          alerts_entries = alerts_found.map { |a| get_alert(a[:id]) }
           say_warning "Looked up alerts entries #{alerts_entries}" if @options[:debug]
           e[:alerts] = alerts_entries
         end
@@ -357,7 +391,11 @@ module Gzr
 
     def import_lookml_dashboard(id,folder)
       begin
-        return @sdk.import_lookml_dashboard(id,folder)&.to_attrs
+        return @sdk.import_lookml_dashboard(id,folder).to_attrs
+      rescue LookerSDK::NotFound => e
+        say_error "lookml_dashboard #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error import_lookml_dashboard(#{id},#{folder})"
         say_error e
@@ -368,6 +406,10 @@ module Gzr
     def sync_lookml_dashboard(id)
       begin
         return @sdk.sync_lookml_dashboard(id, {})
+      rescue LookerSDK::NotFound => e
+        say_error "lookml_dashboard #{id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error sync_lookml_dashboard(#{id})"
         say_error e

data/lib/gzr/modules/folder.rb

@@ -1,6 +1,6 @@
 # The MIT License (MIT)
 
-# Copyright (c) 2018 Mike DeAngelo Looker Data Sciences, Inc.
+# Copyright (c) 2023 Mike DeAngelo Google, Inc.
 
 # Permission is hereby granted, free of charge, to any person obtaining a copy of
 # this software and associated documentation files (the "Software"), to deal in
@@ -31,38 +31,35 @@ module Gzr
     end
 
     def create_folder(name, parent_id)
-      data = nil
       begin
         req = {:name => name, :parent_id => parent_id}
-        data = @sdk.create_folder(req)
+        @sdk.create_folder(req).to_attrs
       rescue LookerSDK::Error => e
         say_error "Error creating folder(#{JSON.pretty_generate(req)})"
         say_error e
         raise
       end
-      data
     end
 
     def search_folders(name,fields=nil)
-      data = nil
       begin
         req = {:name => name}
         req[:fields] = fields if fields
-        data = @sdk.search_folders(req)
+        @sdk.search_folders(req).collect { |e| e.to_attrs }
+      rescue LookerSDK::NotFound => e
+        []
       rescue LookerSDK::Error => e
         say_error "Error querying search_folders(#{JSON.pretty_generate(req)})"
         say_error e
         raise
       end
-      data
     end
 
     def query_folder(id,fields=nil)
-      data = nil
       begin
         req = {}
         req[:fields] = fields if fields
-        data = @sdk.folder(id, req)
+        @sdk.folder(id, req).to_attrs
       rescue LookerSDK::NotFound
         return nil
       rescue LookerSDK::Error => e
@@ -70,7 +67,6 @@ module Gzr
         say_error e
         raise
       end
-      data
     end
 
     def process_args(args)
@@ -78,7 +74,7 @@ module Gzr
 
       begin
         user = query_me("home_folder_id")
-        folder_ids << user.home_folder_id
+        folder_ids << user[:home_folder_id]
       end unless args && args.length > 0 && !(args[0].nil?)
 
       if args[0] == 'lookml'
@@ -87,26 +83,26 @@ module Gzr
         folder_ids << args[0].to_i
       elsif args[0] == "~" then
         user = query_me("personal_folder_id")
-        folder_ids << user.personal_folder_id
+        folder_ids << user[:personal_folder_id]
       elsif args[0] =~ /^~[0-9]+$/ then
         user = query_user(args[0].sub('~',''), "personal_folder_id")
-        folder_ids << user.personal_folder_id
+        folder_ids << user[:personal_folder_id]
       elsif args[0] =~ /^~.+@.+$/ then
         search_results = search_users( { :email=>args[0].sub('~','') },"personal_folder_id" )
-        folder_ids += search_results.map { |r| r.personal_folder_id }
+        folder_ids += search_results.map { |r| r[:personal_folder_id] }
       elsif args[0] =~ /^~.+$/ then
         first_name, last_name = args[0].sub('~','').split(' ')
         search_results = search_users( { :first_name=>first_name, :last_name=>last_name },"personal_folder_id" )
-        folder_ids += search_results.map { |r| r.personal_folder_id }
+        folder_ids += search_results.map { |r| r[:personal_folder_id] }
       else
         search_results = search_folders(args[0],"id")
-        folder_ids += search_results.map { |r| r.id }
+        folder_ids += search_results.map { |r| r[:id] }
 
         # The built in Shared folder is only availabe by
         # searching for Home. https://github.com/looker/helltool/issues/34994
         if args[0] == 'Shared' then
           search_results = search_folders('Home',"id,is_shared_root")
-          folder_ids += search_results.select { |r| r.is_shared_root }.map { |r| r.id }
+          folder_ids += search_results.select { |r| r[:is_shared_root] }.map { |r| r[:id] }
         end
       end if args && args.length > 0 && !args[0].nil?
 
@@ -114,47 +110,43 @@ module Gzr
     end
 
     def all_folders(fields=nil)
-      data = nil
       begin
         req = {}
         req[:fields] = fields if fields
-        data = @sdk.all_folders(req)
+        @sdk.all_folders(req).collect { |e| e.to_attrs }
       rescue LookerSDK::Error => e
         say_error "Error querying all_folders(#{JSON.pretty_generate(req)})"
         say_error e
         raise
       end
-      data
     end
 
     def query_folder_children(folder_id, fields=nil)
-      data = nil
       req = {}
       req[:fields] = fields if fields
       begin
-        data = @sdk.folder_children(folder_id, req)
+        @sdk.folder_children(folder_id, req).collect { |e| e.to_attrs }
       rescue LookerSDK::NotFound
-        return nil
+        return []
       rescue LookerSDK::Error => e
         say_error "Error querying folder_children(#{folder_id}, #{JSON.pretty_generate(req)})"
         say_error e
         raise
       end
-      data
     end
 
     def delete_folder(folder_id)
-      data = nil
       begin
-        data = @sdk.delete_folder(folder_id)
+        @sdk.delete_folder(folder_id)
       rescue LookerSDK::NotFound
-        return nil
+        say_error "folder #{folder_id} not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error deleting folder #{folder_id}"
         say_error e
         raise
       end
-      data
     end
   end
 end

data/lib/gzr/modules/group.rb

@@ -1,6 +1,6 @@
 # The MIT License (MIT)
 
-# Copyright (c) 2018 Mike DeAngelo Looker Data Sciences, Inc.
+# Copyright (c) 2023 Mike DeAngelo Google, Inc.
 
 # Permission is hereby granted, free of charge, to any person obtaining a copy of
 # this software and associated documentation files (the "Software"), to deal in
@@ -35,7 +35,7 @@ module Gzr
       loop do
         begin
           req[:page] = page
-          scratch_data = @sdk.all_groups(req)
+          scratch_data = @sdk.all_groups(req).collect { |e| e.to_attrs }
         rescue LookerSDK::ClientError => e
           say_error "Unable to get all_groups(#{JSON.pretty_generate(req)})"
           say_error e
@@ -52,9 +52,8 @@ module Gzr
       req = { }
       req[:fields] = fields if fields
 
-      data = Array.new
       begin
-        data = @sdk.all_group_groups(group_id,req)
+        @sdk.all_group_groups(group_id,req).collect { |e| e.to_attrs }
       rescue LookerSDK::NotFound => e
         return []
       rescue LookerSDK::ClientError => e
@@ -62,7 +61,6 @@ module Gzr
         say_error e
         raise
       end
-      data
     end
 
     def query_group_users(group_id,fields=nil,sorts=nil)
@@ -77,7 +75,7 @@ module Gzr
       loop do
         begin
           req[:page] = page
-          scratch_data = @sdk.all_group_users(group_id,req)
+          scratch_data = @sdk.all_group_users(group_id,req).collect { |e| e.to_attrs }
         rescue LookerSDK::ClientError => e
           say_error "Unable to get all_group_users(#{group_id},#{JSON.pretty_generate(req)})"
           say_error e
@@ -93,9 +91,9 @@ module Gzr
     def search_groups(name)
       req = {:name => name }
       begin
-        return @sdk.search_groups(req)
+        return @sdk.search_groups(req).collect { |e| e.to_attrs }
       rescue LookerSDK::NotFound => e
-        return nil
+        return []
       rescue LookerSDK::ClientError => e
         say_error "Unable to search_groups(#{JSON.pretty_generate(req)})"
         say_error e
@@ -107,7 +105,7 @@ module Gzr
       req = Hash.new
       req[:fields] = fields if fields
       begin
-        return @sdk.group(id,req)
+        return @sdk.group(id,req).to_attrs
       rescue LookerSDK::NotFound => e
         return nil
       rescue LookerSDK::ClientError => e
@@ -121,7 +119,7 @@ module Gzr
       req = Hash.new
       req[:value] = value
       begin
-        return @sdk.update_user_attribute_group_value(group_id,attr_id, req)
+        return @sdk.update_user_attribute_group_value(group_id,attr_id, req).to_attrs
       rescue LookerSDK::ClientError => e
         say_error "Unable to update_user_attribute_group_value(#{group_id},#{attr_id},#{JSON.pretty_generate(req)})"
         say_error e

data/lib/gzr/modules/look.rb

@@ -1,6 +1,6 @@
 # The MIT License (MIT)
 
-# Copyright (c) 2018 Mike DeAngelo Looker Data Sciences, Inc.
+# Copyright (c) 2023 Mike DeAngelo Google, Inc.
 
 # Permission is hereby granted, free of charge, to any person obtaining a copy of
 # this software and associated documentation files (the "Software"), to deal in
@@ -24,15 +24,17 @@
 module Gzr
   module Look
     def query_look(look_id)
-      data = nil
       begin
-        data = @sdk.look(look_id)
+        @sdk.look(look_id).to_attrs
+      rescue LookerSDK::NotFoud => e
+          say_error "look(#{look_id}) not found"
+          say_error e
+          raise
       rescue LookerSDK::Error => e
           say_error "Error querying look(#{look_id})"
           say_error e
           raise
       end
-      data
     end
 
     def search_looks_by_slug(slug, folder_id=nil)
@@ -40,9 +42,9 @@ module Gzr
       begin
         req = { :slug => slug }
         req[:folder_id] = folder_id if folder_id
-        data = @sdk.search_looks(req)
+        data = @sdk.search_looks(req).collect { |l| l.to_attrs }
         req[:deleted] = true
-        data = @sdk.search_looks(req) if data.empty?
+        data = @sdk.search_looks(req).collect { |l| l.to_attrs } if data.empty?
       rescue LookerSDK::Error => e
         say_error "Error search_looks_by_slug(#{JSON.pretty_generate(req)})"
         say_error e
@@ -56,9 +58,9 @@ module Gzr
       begin
         req = { :title => title }
         req[:folder_id] = folder_id if folder_id
-        data = @sdk.search_looks(req)
+        data = @sdk.search_looks(req).collect { |l| l.to_attrs }
         req[:deleted] = true
-        data = @sdk.search_looks(req) if data.empty?
+        data = @sdk.search_looks(req).collect { |l| l.to_attrs } if data.empty?
       rescue LookerSDK::Error => e
         say_error "Error search_looks_by_title(#{JSON.pretty_generate(req)})"
         say_error e
@@ -70,36 +72,36 @@ module Gzr
     def create_look(look)
       begin
         look[:public] = false unless look[:public]
-        data = @sdk.create_look(look)
+        @sdk.create_look(look).to_attrs
       rescue LookerSDK::Error => e
         say_error "Error creating look(#{JSON.pretty_generate(look)})"
         say_error e
         raise
       end
-      data
     end
 
     def update_look(id,look)
       begin
-        data = @sdk.update_look(id,look)
+        @sdk.update_look(id,look).to_attrs
+      rescue LookerSDK::NotFound => e
+        say_error "look(#{id}) not found"
+        say_error e
+        raise
       rescue LookerSDK::Error => e
         say_error "Error updating look(#{id},#{JSON.pretty_generate(look)})"
         say_error e
         raise
       end
-      data
     end
 
     def delete_look(look_id)
-      data = nil
       begin
-        data = @sdk.delete_look(look_id)
+        @sdk.delete_look(look_id)
       rescue LookerSDK::Error => e
         say_error "Error deleting look(#{look_id})"
         say_error e
         raise
       end
-      data
     end
 
     def upsert_look(user_id, query_id, folder_id, source, output: $stdout)
@@ -123,8 +125,8 @@ module Gzr
       same_slug = (slug_used&.fetch(:id,nil) == existing_look&.fetch(:id,nil))
 
       if slug_used && !same_slug then
-        say_warning "slug #{slug_used.slug} already used for look #{slug_used.title} in folder #{slug_used.folder_id}", output: output
-        say_warning("That look is in the 'Trash' but not fully deleted yet", output: output) if slug_used.deleted
+        say_warning "slug #{slug_used[:slug]} already used for look #{slug_used[:title]} in folder #{slug_used[:folder_id]}", output: output
+        say_warning("That look is in the 'Trash' but not fully deleted yet", output: output) if slug_used[:deleted]
         say_warning "look will be imported with new slug", output: output
       end
 
@@ -133,14 +135,14 @@ module Gzr
           raise Gzr::CLI::Error, "Look #{source[:title]} already exists in folder #{folder_id}\nDelete it before trying to upate another Look to have that title."
         end
         raise Gzr::CLI::Error, "Look #{existing_look[:title]} with slug #{existing_look[:slug]} already exists in folder #{folder_id}\nUse --force if you want to overwrite it" unless @options[:force]
-        say_ok "Modifying existing Look #{existing_look.id} #{existing_look.title} in folder #{folder_id}", output: output
+        say_ok "Modifying existing Look #{existing_look[:id]} #{existing_look[:title]} in folder #{folder_id}", output: output
         new_look = source.select do |k,v|
           (keys_to_keep('update_look') - [:space_id,:folder_id,:user_id,:query_id,:slug]).include? k
         end
         new_look[:slug] = source[:slug] if source[:slug] && !slug_used
         new_look[:deleted] = false if existing_look[:deleted]
         new_look[:query_id] = query_id
-        return update_look(existing_look.id,new_look)
+        return update_look(existing_look[:id],new_look)
       else
         new_look = source.select do |k,v|
           (keys_to_keep('create_look') - [:space_id,:folder_id,:user_id,:query_id,:slug]).include? k
@@ -191,14 +193,14 @@ module Gzr
     end
 
     def cat_look(look_id)
-      data = query_look(look_id).to_attrs
+      data = query_look(look_id)
       find_vis_config_reference(data) do |vis_config|
         find_color_palette_reference(vis_config) do |o,default_colors|
           rewrite_color_palette!(o,default_colors)
         end
       end
 
-      data[:scheduled_plans] = query_scheduled_plans_for_look(@look_id,"all").map { |e| e.to_attrs }  if @options[:plans]
+      data[:scheduled_plans] = query_scheduled_plans_for_look(@look_id,"all") if @options[:plans]
       data
     end