gooddata 0.6.0.pre11 → 0.6.0

data/lib/gooddata/models/empty_result.rb

@@ -1,8 +1,9 @@
-require File.join(File.dirname(__FILE__), "data_result.rb")
+# encoding: UTF-8
+
+require_relative 'data_result.rb'
 
 module GoodData
   class EmptyResult < DataResult
-
     def initialize(data, options = {})
       super(data)
       @options = options
@@ -10,7 +11,7 @@ module GoodData
     end
 
     def to_s
-      "No Data"
+      'No Data'
     end
 
     def assemble_table

data/lib/gooddata/models/fact.rb

@@ -1,19 +1,19 @@
-require File.join(File.dirname(__FILE__), "metadata")
+# encoding: UTF-8
 
-module GoodData
-  class Fact < GoodData::MdObject 
+require_relative 'metadata'
 
+module GoodData
+  class Fact < GoodData::MdObject
     root_key :fact
 
     class << self
       def [](id)
         if id == :all
           GoodData.get(GoodData.project.md['query'] + '/facts/')['query']['entries']
-        else 
+        else
           super
         end
       end
     end
-
   end
 end

data/lib/gooddata/models/links.rb

@@ -1,3 +1,5 @@
+# encoding: UTF-8
+
 module GoodData
   class Links
     attr_reader :data
@@ -8,7 +10,7 @@ module GoodData
         category = item['category']
         if @data[category] then
           if @data[category]['category'] == category then
-            @data[category] = { @data[category]['identifier'] => @data[category] }
+            @data[category] = {@data[category]['identifier'] => @data[category]}
           end
           @data[category][item['identifier']] = item
         else

data/lib/gooddata/models/metadata.rb

@@ -1,4 +1,6 @@
-require File.join(File.dirname(__FILE__), 'model')
+# encoding: UTF-8
+
+require_relative 'model'
 
 module GoodData
   class MdObject
@@ -7,25 +9,25 @@ module GoodData
 
     class << self
       def root_key(a_key)
-        define_method :root_key, Proc.new { a_key.to_s}
+        define_method :root_key, Proc.new { a_key.to_s }
       end
-      
+
       def [](id)
         raise "Cannot search for nil #{self.class}" unless id
         uri = if id.is_a? Integer or id =~ /^\d+$/
-          "#{GoodData.project.md[MD_OBJ_CTG]}/#{id}"
-        elsif id !~ /\//
-          identifier_to_uri id
-        elsif id =~ /^\//
-          id
-        else
-          raise "Unexpected object id format: expected numeric ID, identifier with no slashes or an URI starting with a slash"
-        end
+                "#{GoodData.project.md[MD_OBJ_CTG]}/#{id}"
+              elsif id !~ /\//
+                identifier_to_uri id
+              elsif id =~ /^\//
+                id
+              else
+                raise 'Unexpected object id format: expected numeric ID, identifier with no slashes or an URI starting with a slash'
+              end
         self.new(GoodData.get uri) unless uri.nil?
       end
 
       def find_by_tag(tag)
-        self[:all].find_all {|r| r["tags"].split(",").include?(tag)}
+        self[:all].find_all { |r| r['tags'].split(',').include?(tag) }
       end
 
       def get_by_id(id)
@@ -36,27 +38,26 @@ module GoodData
       def find_first_by_title(title)
         all = self[:all]
         item = if title.is_a?(Regexp)
-          all.find {|r| r["title"] =~ title}
-        else
-          all.find {|r| r["title"] == title}
-        end
-        self[item["link"]] unless item.nil?
+                 all.find { |r| r['title'] =~ title }
+               else
+                 all.find { |r| r['title'] == title }
+               end
+        self[item['link']] unless item.nil?
       end
 
       def identifier_to_uri(*ids)
-        raise NoProjectError.new "Connect to a project before searching for an object" unless GoodData.project
-        uri      = GoodData.project.md[IDENTIFIERS_CFG]
-        response = GoodData.post uri, { 'identifierToUri' => ids }
+        raise NoProjectError.new 'Connect to a project before searching for an object' unless GoodData.project
+        uri = GoodData.project.md[IDENTIFIERS_CFG]
+        response = GoodData.post uri, {'identifierToUri' => ids}
         if response['identifiers'].empty?
           nil
         else
-          ids = response['identifiers'].map {|x| x['uri']}
+          ids = response['identifiers'].map { |x| x['uri'] }
           ids.count == 1 ? ids.first : ids
         end
       end
 
       alias :id_to_uri :identifier_to_uri
-
     end
 
     def initialize(json)
@@ -66,7 +67,7 @@ module GoodData
     def delete
       if saved?
         GoodData.delete(uri)
-        meta.delete("uri")
+        meta.delete('uri')
         # ["uri"] = nil
       end
     end
@@ -107,19 +108,19 @@ module GoodData
     end
 
     def title=(a_title)
-      data["meta"]["title"] = a_title
+      data['meta']['title'] = a_title
     end
 
     def summary=(a_summary)
-      data["meta"]["summary"] = a_summary
+      data['meta']['summary'] = a_summary
     end
 
     def tags
-      data["meta"]["tags"]
+      data['meta']['tags']
     end
 
     def tags=(list_of_tags)
-      data["meta"]["tags"] = list_of_tags
+      data['meta']['tags'] = list_of_tags
     end
 
     def meta
@@ -136,12 +137,12 @@ module GoodData
 
     def get_usedby
       result = GoodData.get "#{GoodData.project.md['usedby2']}/#{obj_id}"
-      result["entries"]
+      result['entries']
     end
 
     def get_using
       result = GoodData.get "#{GoodData.project.md['using2']}/#{obj_id}"
-      result["entries"]
+      result['entries']
     end
 
     def to_json
@@ -153,7 +154,8 @@ module GoodData
     end
 
     def data
-      raw_data[root_key]
+      key = methods.include?(:root_key) ? root_key : raw_data.keys.first
+      raw_data[key]
     end
 
     def saved?
@@ -161,13 +163,13 @@ module GoodData
     end
 
     def save
-      fail("Validation failed") unless validate
+      fail('Validation failed') unless validate
 
       if saved?
         GoodData.put(uri, to_json)
       else
         result = GoodData.post(GoodData.project.md['obj'], to_json)
-        saved_object = self.class[result["uri"]]
+        saved_object = self.class[result['uri']]
         @json = saved_object.raw_data
       end
       self

data/lib/gooddata/models/metric.rb

@@ -1,17 +1,18 @@
-require File.join(File.dirname(__FILE__), "metadata")
-require File.join(File.dirname(__FILE__), '../goodzilla/goodzilla')
+# encoding: UTF-8
+
+require_relative 'metadata'
+require_relative '../goodzilla/goodzilla'
 
 module GoodData
   # Metric representation
   class Metric < GoodData::MdObject
-
     root_key :metric
 
     class << self
       def [](id)
         if id == :all
           GoodData.get(GoodData.project.md['query'] + '/metrics/')['query']['entries']
-        else 
+        else
           super
         end
       end
@@ -32,43 +33,43 @@ module GoodData
         else
           title = options[:title]
           summary = options[:summary]
-          expression = options[:expression] || fail("Metric has to have its expression defined")
+          expression = options[:expression] || fail('Metric has to have its expression defined')
           extended_notation = options[:extended_notation] || false
         end
 
         expression = if extended_notation
-          dict = {
-            :facts => GoodData::Fact[:all].reduce({}) {|memo, item| memo[item["title"]] = item["link"]; memo},
-            :attributes => GoodData::Attribute[:all].reduce({}) {|memo, item| memo[item["title"]] = item["link"]; memo},
-            :metrics => GoodData::Metric[:all].reduce({}) {|memo, item| memo[item["title"]] = item["link"]; memo},
-          }
-          interpolated_metric = GoodData::SmallGoodZilla.interpolate_metric(expression, dict)
-          interpolated_metric
-        else
-          expression
-        end
+                       dict = {
+                         :facts => GoodData::Fact[:all].reduce({}) { |memo, item| memo[item['title']] = item['link']; memo },
+                         :attributes => GoodData::Attribute[:all].reduce({}) { |memo, item| memo[item['title']] = item['link']; memo },
+                         :metrics => GoodData::Metric[:all].reduce({}) { |memo, item| memo[item['title']] = item['link']; memo },
+                       }
+                       interpolated_metric = GoodData::SmallGoodZilla.interpolate_metric(expression, dict)
+                       interpolated_metric
+                     else
+                       expression
+                     end
 
         Metric.new({
-           "metric" => {
-              "content" => {
-                 "format" => "#,##0",
-                 "expression" => expression
-              },
-              "meta" => {
-                 "tags" => "",
-                 "summary" => summary,
-                 "title" => title,
-              }
-           }
-        })
+                     'metric' => {
+                       'content' => {
+                         'format' => '#,##0',
+                         'expression' => expression
+                       },
+                       'meta' => {
+                         'tags' => '',
+                         'summary' => summary,
+                         'title' => title,
+                       }
+                     }
+                   })
       end
 
       def execute(expression, options={})
         m = if expression.is_a?(String)
-          GoodData::Metric.create({:title => "Temporary metric to be deleted", :expression => expression}.merge(options))
-        else
-          GoodData::Metric.create({:title => "Temporary metric to be deleted"}.merge(expression))
-        end
+              GoodData::Metric.create({:title => 'Temporary metric to be deleted', :expression => expression}.merge(options))
+            else
+              GoodData::Metric.create({:title => 'Temporary metric to be deleted'}.merge(expression))
+            end
         m.execute
       end
 
@@ -79,7 +80,6 @@ module GoodData
           execute(expression.merge({:extended_notation => true}))
         end
       end
-
     end
 
     def execute
@@ -88,13 +88,12 @@ module GoodData
     end
 
     def validate
-      fail "Meric needs to have title" if title.nil?
+      fail 'Meric needs to have title' if title.nil?
       true
     end
 
     def is_metric?
       true
     end
-      
   end
 end

data/lib/gooddata/models/model.rb

@@ -1,13 +1,16 @@
+# encoding: UTF-8
+
+require_relative '../helpers'
+
 require 'open-uri'
 require 'active_support/all'
+
 ##
 # Module containing classes that counter-part GoodData server-side meta-data
 # elements, including the server-side data model.
 #
 module GoodData
-
   module Model
-
     # GoodData REST API categories
     LDM_CTG = 'ldm'
     LDM_MANAGE_CTG = 'ldm-manage'
@@ -24,7 +27,7 @@ module GoodData
     LABEL_PREFIX = 'label'
     FACT_PREFIX = 'fact'
     DATE_FACT_PREFIX = 'dt'
-    DATE_ATTRIBUTE = "date"
+    DATE_ATTRIBUTE = 'date'
     DATE_ATTRIBUTE_DEFAULT_DISPLAY_FORM = 'mdyy'
     TIME_FACT_PREFIX = 'tm.dt'
     TIME_ATTRIBUTE_PREFIX = 'attr.time'
@@ -83,13 +86,13 @@ module GoodData
         pull = {'pullIntegration' => File.basename(dir)}
         link = project.md.links('etl')['pull']
         task = GoodData.post link, pull
-        while (GoodData.get(task["pullTask"]["uri"])["taskStatus"] === "RUNNING" || GoodData.get(task["pullTask"]["uri"])["taskStatus"] === "PREPARED") do
+        while GoodData.get(task['pullTask']['uri'])['taskStatus'] === 'RUNNING' || GoodData.get(task['pullTask']['uri'])['taskStatus'] === 'PREPARED'
           sleep 30
         end
-        if (GoodData.get(task["pullTask"]["uri"])["taskStatus"] == "ERROR")
+        if GoodData.get(task['pullTask']['uri'])['taskStatus'] == 'ERROR'
           s = StringIO.new
           GoodData.download_from_user_webdav(File.basename(dir) + '/upload_status.json', s)
-          js = JSON.parse(s.string)
+          js = MultiJson.load(s.string)
           fail "Load Failed with error #{JSON.pretty_generate(js)}"
         end
       end
@@ -100,20 +103,18 @@ module GoodData
         d = Marshal.load(Marshal.dump(a_schema_blueprint))
         d[:columns] = d[:columns] + b_schema_blueprint[:columns]
         d[:columns].uniq!
-        columns_that_failed_to_merge = d[:columns].group_by {|x| x[:name]}.map {|k, v| [k, v.count]}.find_all {|x| x[1] > 1}
+        columns_that_failed_to_merge = d[:columns].group_by { |x| x[:name] }.map { |k, v| [k, v.count] }.find_all { |x| x[1] > 1 }
         fail "Columns #{columns_that_failed_to_merge} failed to merge. When merging columns with the same name they have to be identical." unless columns_that_failed_to_merge.empty?
         d
       end
-
     end
 
     class ProjectBlueprint
-
       attr_accessor :data
 
       def self.from_json(spec)
         if spec.is_a?(String)
-          ProjectBlueprint.new(JSON.parse(File.read(spec), :symbolize_names => true))
+          ProjectBlueprint.new(MultiJson.load(File.read(spec), :symbolize_keys => true))
         else
           ProjectBlueprint.new(spec)
         end
@@ -140,7 +141,7 @@ module GoodData
       end
 
       def remove_dataset(dataset_name)
-        x = data[:datasets].find {|d| d[:name] == dataset_name}
+        x = data[:datasets].find { |d| d[:name] == dataset_name }
         index = data[:datasets].index(x)
         data[:datasets].delete_at(index)
       end
@@ -205,28 +206,26 @@ module GoodData
 
       def to_wire_model
         {
-          "diffRequest" => {
-              "targetModel" => {
-                  "projectModel" => {
-                      "datasets" => datasets.map {|d| d.to_wire_model},
-                      "dateDimensions" => date_dimensions.map {|d|
-                        {
-                            "dateDimension" => {
-                                "name" => d[:name],
-                                "title" => d[:title] || d[:name].humanize
-                            }
-                        }}
-                      }}}}
+          'diffRequest' => {
+            'targetModel' => {
+              'projectModel' => {
+                'datasets' => datasets.map { |d| d.to_wire_model },
+                'dateDimensions' => date_dimensions.map { |d|
+                  {
+                    'dateDimension' => {
+                      'name' => d[:name],
+                      'title' => d[:title] || d[:name].humanize
+                    }
+                  } }
+              }}}}
       end
 
       def to_hash
         @data
       end
-
     end
 
     class SchemaBlueprint
-
       attr_accessor :data
 
       def change(&block)
@@ -243,8 +242,8 @@ module GoodData
 
       def upload(source, options={})
         project = options[:project] || GoodData.project
-        fail "You have to specify a project into which you want to load." if project.nil?
-        mode = options[:load] || "FULL"
+        fail 'You have to specify a project into which you want to load.' if project.nil?
+        mode = options[:load] || 'FULL'
         project.upload(source, to_schema, mode)
       end
 
@@ -271,7 +270,7 @@ module GoodData
       end
 
       def has_anchor?
-        columns.any? { |c| c[:type].to_s == "anchor" }
+        columns.any? { |c| c[:type].to_s == 'anchor' }
       end
 
       def anchor
@@ -335,17 +334,14 @@ module GoodData
       def ==(other)
         to_hash == other.to_hash
       end
-
     end
 
     class ProjectBuilder
-
       attr_reader :title, :datasets, :reports, :metrics, :uploads, :users, :assert_report, :date_dimensions
 
       class << self
-
-        def create_from_data(blueprint)
-          pb = ProjectBuilder.new
+        def create_from_data(blueprint, title = 'Title')
+          pb = ProjectBuilder.new(title)
           pb.data = blueprint.to_hash
           pb
         end
@@ -355,7 +351,6 @@ module GoodData
           block.call(pb)
           pb
         end
-
       end
 
       def initialize(title)
@@ -370,8 +365,14 @@ module GoodData
         @date_dimensions = []
       end
 
-      def add_date_dimension(name, options={})
-        @date_dimensions << {:urn => options[:urn], :name => name, :title => options[:title]}
+      def add_date_dimension(name, options = {})
+        dimension = {
+          urn: options[:urn],
+          name: name,
+          title: options[:title]
+        }
+
+        @date_dimensions << dimension
       end
 
       def add_dataset(name, &block)
@@ -403,12 +404,12 @@ module GoodData
       end
 
       def load_metrics(file)
-        new_metrics = JSON.parse(open(file).read, :symbolize_names => true)
+        new_metrics = MultiJson.load(open(file).read, :symbolize_keys => true)
         @metrics = @metrics + new_metrics
       end
 
       def load_datasets(file)
-        new_metrics = JSON.parse(open(file).read, :symbolize_names => true)
+        new_metrics = MultiJson.load(open(file).read, :symbolize_keys => true)
         @datasets = @datasets + new_metrics
       end
 
@@ -417,12 +418,12 @@ module GoodData
       end
 
       def upload(data, options={})
-        mode = options[:mode] || "FULL"
+        mode = options[:mode] || 'FULL'
         dataset = options[:dataset]
         @uploads << {
-            :source => data,
-            :mode => mode,
-            :dataset => dataset
+          :source => data,
+          :mode => mode,
+          :dataset => dataset
         }
       end
 
@@ -442,26 +443,24 @@ module GoodData
 
       def to_hash
         {
-            :title => @title,
-            :datasets => @datasets,
-            :uploads => @uploads,
-            :dashboards => @dashboards,
-            :metrics => @metrics,
-            :reports => @reports,
-            :users => @users,
-            :assert_tests => @assert_tests,
-            :date_dimensions => @date_dimensions
+          :title => @title,
+          :datasets => @datasets,
+          :uploads => @uploads,
+          :dashboards => @dashboards,
+          :metrics => @metrics,
+          :reports => @reports,
+          :users => @users,
+          :assert_tests => @assert_tests,
+          :date_dimensions => @date_dimensions
         }
       end
 
       def get_dataset(name)
         datasets.find { |d| d.name == name }
       end
-
     end
 
     class DashboardBuilder
-
       def initialize(title)
         @title = title
         @tabs = []
@@ -476,14 +475,13 @@ module GoodData
 
       def to_hash
         {
-            :name => @name,
-            :tabs => @tabs.map { |tab| tab.to_hash }
+          :name => @name,
+          :tabs => @tabs.map { |tab| tab.to_hash }
         }
       end
     end
 
     class TabBuilder
-
       def initialize(title)
         @title = title
         @stuff = []
@@ -495,31 +493,27 @@ module GoodData
 
       def to_hash
         {
-            :title => @title,
-            :items => @stuff
+          :title => @title,
+          :items => @stuff
         }
       end
-
     end
 
     class SchemaBuilder
-
       attr_accessor :data
 
       class << self
-
         def create_from_data(blueprint)
           sc = SchemaBuilder.new
           sc.data = blueprint.to_hash
           sc
         end
-
       end
 
       def initialize(name=nil)
         @data = {
-            :name => name,
-            :columns => []
+          :name => name,
+          :columns => []
         }
       end
 
@@ -575,20 +569,17 @@ module GoodData
       def to_schema
         Schema.new(to_hash)
       end
-
     end
 
     class ProjectCreator
-
       class << self
         def migrate(options={})
-
-          spec = options[:spec] || fail("You need to provide spec for migration")
+          spec = options[:spec] || fail('You need to provide spec for migration')
           spec = spec.to_hash
 
           token = options[:token]
           project = options[:project] || GoodData::Project.create(:title => spec[:title], :auth_token => token)
-          fail("You need to specify token for project creation") if token.nil? && project.nil?
+          fail('You need to specify token for project creation') if token.nil? && project.nil?
 
           begin
             GoodData.with_project(project) do |p|
@@ -615,8 +606,9 @@ module GoodData
           bp = ProjectBlueprint.new(spec)
           # schema = Schema.load(schema) unless schema.respond_to?(:to_maql_create)
           # project = GoodData.project unless project
-          result = GoodData.post("/gdc/projects/#{GoodData.project.pid}/model/diff", bp.to_wire_model)
-          link = result["asyncTask"]["link"]["poll"]
+          uri = "/gdc/projects/#{GoodData.project.pid}/model/diff"
+          result = GoodData.post(uri, bp.to_wire_model)
+          link = result['asyncTask']['link']['poll']
           response = GoodData.get(link, :process => false)
           # pp response
           while response.code != 200
@@ -630,9 +622,9 @@ module GoodData
           response = GoodData.get(link)
           ldm_links = GoodData.get project.md[LDM_CTG]
           ldm_uri = Links.new(ldm_links)[LDM_MANAGE_CTG]
-          chunks = response["projectModelDiff"]["updateScripts"].find_all {|script| script["updateScript"]["preserveData"] == true && script["updateScript"]["cascadeDrops"] == false}.map {|x| x["updateScript"]["maqlDdlChunks"]}.flatten
+          chunks = response['projectModelDiff']['updateScripts'].find_all { |script| script['updateScript']['preserveData'] == true && script['updateScript']['cascadeDrops'] == false }.map { |x| x['updateScript']['maqlDdlChunks'] }.flatten
           chunks.each do |chunk|
-            GoodData.post ldm_uri, { 'manage' => { 'maql' => chunk } }
+            GoodData.post ldm_uri, {'manage' => {'maql' => chunk}}
           end
 
           bp.datasets.each do |ds|
@@ -669,7 +661,7 @@ module GoodData
         def load(project, spec)
           if spec.has_key?(:uploads)
             spec[:uploads].each do |load|
-              schema = GoodData::Model::Schema.new(spec[:datasets].detect {|d| d[:name] == load[:dataset]})
+              schema = GoodData::Model::Schema.new(spec[:datasets].detect { |d| d[:name] == load[:dataset] })
               project.upload(load[:source], schema, load[:mode])
             end
           end
@@ -716,20 +708,23 @@ module GoodData
         Schema.new JSON.load(open(file))
       end
 
-      def initialize(config, name = nil)
+      def initialize(config, name = 'Default Name', title = 'Default Title')
         super()
         @fields = []
         @attributes = []
         @facts = []
         @folders = {
-            :facts => {},
-            :attributes => {}
+          :facts => {},
+          :attributes => {}
         }
         @references = []
         @labels = []
 
         config[:name] = name unless config[:name]
-        config[:title] = config[:title] || config[:name].humanize
+        config[:title] = config[:name] unless config[:title]
+        config[:title] = title unless config[:title]
+        config[:title] = config[:title].humanize
+
         fail 'Schema name not specified' unless config[:name]
         self.name = config[:name]
         self.title = config[:title]
@@ -739,17 +734,17 @@ module GoodData
       def config=(config)
         config[:columns].each do |c|
           case c[:type].to_s
-          when "attribute"
+          when 'attribute'
             add_attribute c
-          when "fact"
+          when 'fact'
             add_fact c
-          when "date"
+          when 'date'
             add_date c
-          when "anchor"
+          when 'anchor'
             set_anchor c
-          when "label"
+          when 'label'
             add_label c
-          when "reference"
+          when 'reference'
             add_reference c
           else
             fail "Unexpected type #{c[:type]} in #{c.inspect}"
@@ -773,7 +768,7 @@ module GoodData
       # Generates MAQL DDL script to drop this data set and included pieces
       #
       def to_maql_drop
-        maql = ""
+        maql = ''
         [attributes, facts].each do |obj|
           maql += obj.to_maql_drop
         end
@@ -784,9 +779,10 @@ module GoodData
       # Generates MAQL DDL script to create this data set and included pieces
       #
       def to_maql_create
+        # TODO: Use template (.erb)
         maql = "# Create the '#{self.title}' data set\n"
         maql += "CREATE DATASET {#{self.identifier}} VISUAL (TITLE \"#{self.title}\");\n\n"
-        [ attributes, facts, { 1 => @anchor } ].each do |objects|
+        [attributes, facts, {1 => @anchor}].each do |objects|
           objects.values.each do |obj|
             maql += "# Create '#{obj.title}' and add it to the '#{self.title}' data set.\n"
             maql += obj.to_maql_create
@@ -809,7 +805,7 @@ module GoodData
         folders_maql + "\n" + maql + "SYNCHRONIZE {#{identifier}};\n"
       end
 
-      def upload(path, project = nil, mode = "FULL")
+      def upload(path, project = nil, mode = 'FULL')
         if path =~ URI::regexp
           Tempfile.open('remote_file') do |temp|
             temp << open(path).read
@@ -827,31 +823,32 @@ module GoodData
 
       # Generates the SLI manifest describing the data loading
       # 
-      def to_manifest(mode="FULL")
+      def to_manifest(mode = 'FULL')
         {
-            'dataSetSLIManifest' => {
-                'parts' => fields.reduce([]) { |memo, f| val = f.to_manifest_part(mode); memo << val unless val.nil?; memo },
-                'dataSet' => self.identifier,
-                'file' => 'data.csv', # should be configurable
-                'csvParams' => {
-                    'quoteChar' => '"',
-                    'escapeChar' => '"',
-                    'separatorChar' => ',',
-                    'endOfLine' => "\n"
-                }
+          'dataSetSLIManifest' => {
+            'parts' => fields.reduce([]) { |memo, f| val = f.to_manifest_part(mode); memo << val unless val.nil?; memo },
+            'dataSet' => self.identifier,
+            'file' => 'data.csv', # should be configurable
+            'csvParams' => {
+              'quoteChar' => '"',
+              'escapeChar' => '"',
+              'separatorChar' => ',',
+              'endOfLine' => "\n"
             }
+          }
         }
       end
 
       def to_wire_model
         {
-          "dataset" => {
-                "identifier" => identifier,
-                "title" => title,
-                "anchor" => @anchor.to_wire_model,
-                "facts" => facts.map {|f| f.to_wire_model},
-                "attributes" => attributes.map {|a| a.to_wire_model},
-                "references" => references.map {|r| r.is_a?(DateReference) ? r.schema_ref : type_prefix + "." + r.schema_ref }}}
+          'dataset' => {
+            'identifier' => identifier,
+            'title' => title,
+            'anchor' => @anchor.to_wire_model,
+            'facts' => facts.map { |f| f.to_wire_model },
+            'attributes' => attributes.map { |a| a.to_wire_model },
+            'references' => references.map { |r| r.is_a?(DateReference) ? r.schema_ref : type_prefix + '.' + r.schema_ref }}
+        }
       end
 
       private
@@ -902,14 +899,13 @@ module GoodData
         date.parts.values.each { |p| @fields << p }
         date.facts.each { |f| facts << f }
         date.attributes.each { |a| attributes << a }
-        date.references.each {|r| references << r}
+        date.references.each { |r| references << r }
       end
 
       def set_anchor(column)
         @anchor = Anchor.new column, self
         @fields << @anchor
       end
-
     end
 
     ##
@@ -922,7 +918,9 @@ module GoodData
       def initialize(hash, schema)
         super()
         raise ArgumentError.new("Schema must be provided, got #{schema.class}") unless schema.is_a? Schema
-        @name = hash[:name] || raise("Data set fields must have their names defined")
+        raise('Data set fields must have their names defined') if hash[:name].nil?
+
+        @name = hash[:name]
         @title = hash[:title] || hash[:name].humanize
         @folder = hash[:folder]
         @schema = schema
@@ -985,7 +983,7 @@ module GoodData
       end
 
       def table
-        @table ||= "d_" + @schema.name + "_" + name
+        @table ||= 'd_' + @schema.name + '_' + name
       end
 
       def key;
@@ -1001,31 +999,30 @@ module GoodData
 
       def to_manifest_part(mode)
         {
-            'referenceKey' => 1,
-            'populates' => [@primary_label.identifier],
-            'mode' => mode,
-            'columnName' => name
+          'referenceKey' => 1,
+          'populates' => [@primary_label.identifier],
+          'mode' => mode,
+          'columnName' => name
         }
       end
 
       def to_wire_model
         {
-            "attribute" => {
-                "identifier" => identifier,
-                "title" => title,
-                "labels" => labels.map do |l|
-                  {
-                    "label" => {
-                        "identifier" => l.identifier,
-                        "title" => l.title,
-                        "type" => "GDC.text"
-                    }
-                  }
-                end
-            }
+          'attribute' => {
+            'identifier' => identifier,
+            'title' => title,
+            'labels' => labels.map do |l|
+              {
+                'label' => {
+                  'identifier' => l.identifier,
+                  'title' => l.title,
+                  'type' => 'GDC.text'
+                }
+              }
+            end
+          }
         }
       end
-
     end
 
     ##
@@ -1034,30 +1031,31 @@ module GoodData
     # field
     #
     class Label < Column
-
       attr_accessor :attribute
 
-      def type_prefix ; 'label' ; end
+      def type_prefix;
+        'label';
+      end
 
       # def initialize(hash, schema)
       def initialize(hash, attribute, schema)
         super hash, schema
-        attribute = attribute.nil? ? schema.fields.find {|field| field.name === hash[:reference]} : attribute
+        attribute = attribute.nil? ? schema.fields.find { |field| field.name === hash[:reference] } : attribute
         @attribute = attribute
         attribute.labels << self
       end
 
       def to_maql_create
-        "# LABEL FROM LABEL"
+        '# LABEL FROM LABEL'
         "ALTER ATTRIBUTE {#{@attribute.identifier}} ADD LABELS {#{identifier}}" \
               + " VISUAL (TITLE #{title.inspect}) AS {#{column}};\n"
       end
 
       def to_manifest_part(mode)
         {
-            'populates' => [identifier],
-            'mode' => mode,
-            'columnName' => name
+          'populates' => [identifier],
+          'mode' => mode,
+          'columnName' => name
         }
       end
 
@@ -1068,7 +1066,7 @@ module GoodData
       alias :inspect_orig :inspect
 
       def inspect
-        inspect_orig.sub(/>$/, " @attribute=" + @attribute.to_s.sub(/>$/, " @name=#{@attribute.name}") + '>')
+        inspect_orig.sub(/>$/, " @attribute=#{@attribute.to_s.sub(/>$/, " @name=#{@attribute.name}")}>")
       end
     end
 
@@ -1081,14 +1079,14 @@ module GoodData
         if column then
           super
         else
-          super({:type => "anchor", :name => 'id'}, schema)
+          super({:type => 'anchor', :name => 'id'}, schema)
           @labels = []
           @primary_label = nil
         end
       end
 
       def table
-        @table ||= "f_" + @schema.name
+        @table ||= 'f_' + @schema.name
       end
 
       def to_maql_create
@@ -1100,7 +1098,6 @@ module GoodData
         end
         maql
       end
-
     end
 
     ##
@@ -1134,17 +1131,17 @@ module GoodData
 
       def to_manifest_part(mode)
         {
-            'populates' => [identifier],
-            'mode' => mode,
-            'columnName' => name
+          'populates' => [identifier],
+          'mode' => mode,
+          'columnName' => name
         }
       end
 
       def to_wire_model
         {
-          "fact" => {
-            "identifier" => identifier,
-            "title" => title
+          'fact' => {
+            'identifier' => identifier,
+            'title' => title
           }
         }
       end
@@ -1154,7 +1151,6 @@ module GoodData
     # Reference to another data set
     #
     class Reference < Column
-
       attr_accessor :reference, :schema_ref
 
       def initialize(column, schema)
@@ -1193,10 +1189,10 @@ module GoodData
 
       def to_manifest_part(mode)
         {
-            'populates' => [label_column],
-            'mode' => mode,
-            'columnName' => name,
-            'referenceKey' => 1
+          'populates' => [label_column],
+          'mode' => mode,
+          'columnName' => name,
+          'referenceKey' => 1
         }
       end
     end
@@ -1205,14 +1201,13 @@ module GoodData
     # Date as a reference to a date dimension
     #
     class DateReference < Reference
-
       attr_accessor :format, :output_format, :urn
 
       def initialize(column, schema)
         super column, schema
-        @output_format = column["format"] || 'dd/MM/yyyy'
+        @output_format = column['format'] || 'dd/MM/yyyy'
         @format = @output_format.gsub('yyyy', '%Y').gsub('MM', '%m').gsub('dd', '%d')
-        @urn = column[:urn] || "URN:GOODDATA:DATE"
+        @urn = column[:urn] || 'URN:GOODDATA:DATE'
       end
 
       def identifier
@@ -1221,11 +1216,11 @@ module GoodData
 
       def to_manifest_part(mode)
         {
-            'populates' => ["#{identifier}.#{DATE_ATTRIBUTE_DEFAULT_DISPLAY_FORM}"],
-            'mode' => mode,
-            'constraints' => {"date" => output_format},
-            'columnName' => name,
-            'referenceKey' => 1
+          'populates' => ["#{identifier}.#{DATE_ATTRIBUTE_DEFAULT_DISPLAY_FORM}"],
+          'mode' => mode,
+          'constraints' => {'date' => output_format},
+          'columnName' => name,
+          'referenceKey' => 1
         }
       end
 
@@ -1249,10 +1244,10 @@ module GoodData
 
       def to_manifest_part(mode)
         {
-            'populates' => ['label.stuff.mmddyy'],
-            "format" => "unknown",
-            "mode" => mode,
-            "referenceKey" => 1
+          'populates' => ['label.stuff.mmddyy'],
+          'format' => 'unknown',
+          'mode' => mode,
+          'referenceKey' => 1
         }
       end
     end
@@ -1274,7 +1269,6 @@ module GoodData
     # Time as a reference to a time-of-a-day dimension
     #
     class TimeReference < Reference
-
     end
 
     ##
@@ -1343,7 +1337,6 @@ module GoodData
       def to_manifest_part(mode)
         nil
       end
-
     end
 
     ##
@@ -1351,8 +1344,10 @@ module GoodData
     #
     class Folder < MdObject
       def initialize(title)
+        # TODO: should a super be here?
+        # how to deal with name vs title?
         @title = title
-        @name = title
+        @name = GoodData::Helpers.sanitize_string(title)
       end
 
       def to_maql_create
@@ -1366,11 +1361,11 @@ module GoodData
     #
     class AttributeFolder < Folder
       def type;
-        "ATTRIBUTE";
+        'ATTRIBUTE'
       end
 
       def type_prefix;
-        "dim";
+        'dim'
       end
     end
 
@@ -1379,21 +1374,20 @@ module GoodData
     #
     class FactFolder < Folder
       def type;
-        "FACT";
+        'FACT'
       end
 
       def type_prefix;
-        "ffld";
+        'ffld'
       end
     end
 
     class DateDimension < MdObject
-
       def initialize(spec={})
         super()
         @name = spec[:name]
         @title = spec[:title] || @name
-        @urn = spec[:urn] || "URN:GOODDATA:DATE"
+        @urn = spec[:urn] || 'URN:GOODDATA:DATE'
       end
 
       def to_maql_create
@@ -1401,12 +1395,10 @@ module GoodData
         # title = "title"
         # name = "name"
 
-        maql = ""
+        maql = ''
         maql += "INCLUDE TEMPLATE \"#{@urn}\" MODIFY (IDENTIFIER \"#{@name}\", TITLE \"#{@title}\");"
         maql
       end
-
     end
-
   end
 end