safrano 0.5.5 → 0.6.0

data/lib/odata/model_ext.rb

@@ -104,9 +104,8 @@ module Safrano
     def new_from_hson_h(hash)
       # enty = new
       # enty.set_fields(hash, data_fields, missing: :skip)
-      enty = create(hash)
+      create(hash)
       # enty.set(hash)
-      enty
     end
 
     def attrib_path_valid?(path)
@@ -353,20 +352,18 @@ module Safrano
 
     def build_default_template
       @default_template = { all_values: EMPTYH }
-      if @nav_entity_attribs || @nav_collection_attribs
-        @default_template[:deferr] = (@nav_entity_attribs&.keys || []) + (@nav_collection_attribs&.keys || EMPTY_ARRAY)
-      end
+      @default_template[:deferr] = (@nav_entity_attribs&.keys || []) + (@nav_collection_attribs&.keys || EMPTY_ARRAY) if @nav_entity_attribs || @nav_collection_attribs
     end
 
     def build_casted_cols(service)
       # cols needed catsting before final json output
       @casted_cols = {}
-      db_schema.each { |col, props|
+      db_schema.each do |col, props|
         # first check if we have user-defined type mapping
         usermap = nil
         dbtyp = props[:db_type]
         metadata = @cols_metadata[col]
-        if (service.type_mappings.values.find { |map| usermap = map.match(dbtyp) })
+        if service.type_mappings.values.find { |map| usermap = map.match(dbtyp) }
 
           metadata[:edm_type] = usermap.edm_type
 
@@ -374,17 +371,17 @@ module Safrano
           next # this will override our rules below !
         end
 
-        if (metadata[:edm_precision] && (metadata[:edm_type] =~ /\AEdm.Decimal\(/i))
+        if metadata[:edm_precision] && (metadata[:edm_type] =~ /\AEdm.Decimal\(/i)
           # we save the precision and/or scale in the lambda (binding!)
 
           @casted_cols[col] = if metadata[:edm_scale]
-                                ->(x) {
+                                lambda { |x|
                                   # not sure if these copies are really needed, but feels better that way
                                   # output  decimal with precision and scale
                                   x&.toDecimalPrecisionScaleString(metadata[:edm_precision], metadata[:edm_scale])
                                 }
                               else
-                                ->(x) {
+                                lambda { |x|
                                   # not sure if these copies are really needed, but feels better that way
                                   # output  decimal with precision only
                                   x&.toDecimalPrecisionString(metadata[:edm_precision])
@@ -404,13 +401,13 @@ module Safrano
           @casted_cols[col] = ->(x) { Base64.encode64(x) unless x.nil? } # Base64
           next
         end
-        # TODO check this more in details
+        # TODO: check this more in details
         # NOTE: here we use :type which is the sequel defined ruby-type
-        if props[:type] == :datetime
-          @casted_cols[col] = ->(x) { x&.iso8601 }
-
+        if props[:type] == :datetime || props[:type] == :date
+          #          @casted_cols[col] = ->(x) { x&.iso8601 }
+          @casted_cols[col] = ->(x) { x&.to_edm_json }
         end
-      }
+      end
     end
 
     def finalize_publishing(service)
@@ -440,7 +437,7 @@ module Safrano
       build_entity_allowed_transitions
 
       # for media
-      finalize_media if self.respond_to? :finalize_media
+      finalize_media if respond_to? :finalize_media
     end
 
     KEYPRED_URL_REGEXP = /\A\(\s*'?([\w=,'\s]+)'?\s*\)(.*)/.freeze
@@ -449,7 +446,7 @@ module Safrano
         @pk_names = []
         @pk_cast_from_string = {}
         odata_upk_build = []
-        primary_key.each { |pk|
+        primary_key.each do |pk|
           @pk_names << pk.to_s
           kvpredicate = case db_schema[pk][:type]
                         when :integer
@@ -459,19 +456,19 @@ module Safrano
                           "'?'"
                         end
           odata_upk_build << "#{pk}=#{kvpredicate}"
-        }
+        end
         @odata_upk_parts = odata_upk_build.join(',').split('?')
 
         # regex parts for unordered matching
-        @iuk_rgx_parts = primary_key.map { |pk|
+        @iuk_rgx_parts = primary_key.map do |pk|
           kvpredicate = case db_schema[pk][:type]
                         when :integer
-                          "(\\d+)"
+                          '(\\d+)'
                         else
                           "'(\\w+)'"
                         end
           [pk, "#{pk}=#{kvpredicate}"]
-        }.to_h
+        end.to_h
 
         # single regex assuming the key fields are ordered !
         @iuk_rgx = /\A#{@iuk_rgx_parts.values.join(',\s*')}\z/
@@ -485,7 +482,7 @@ module Safrano
         kvpredicate = case db_schema[primary_key][:type]
                       when :integer
                         @pk_cast_from_string = ->(str) { Integer(str) }
-                        "(\\d+)"
+                        '(\\d+)'
                       else
                         "'(\\w+)'"
                       end
@@ -617,14 +614,14 @@ module Safrano
         scan_rgx_parts = @iuk_rgx_parts.dup
         mdch = {}
 
-        mid.split(/\s*,\s*/).each { |midpart|
+        mid.split(/\s*,\s*/).each do |midpart|
           mval = nil
-          mpk, mrgx = scan_rgx_parts.find { |pk, rgx|
+          mpk, mrgx = scan_rgx_parts.find do |_pk, rgx|
             if (md = rgx.match(midpart))
               mval = md[1]
             end
-          }
-          if mpk and mval
+          end
+          if mpk && mval
             mdch[mpk] = if (pk_cast = @pk_cast_from_string[mpk])
                           pk_cast.call(mval)
                         else
@@ -634,7 +631,7 @@ module Safrano
           else
             return Contract::NOK
           end
-        }
+        end
         # normally arriving here we have mdch filled with key values pairs,
         # but not in the model key ordering. lets just re-order the values
         mdc = @iuk_rgx_parts.keys.map { |pk| mdch[pk] }
@@ -653,7 +650,7 @@ module Safrano
 
     def parse_odata_key(rawid)
       if (md = @iuk_rgx.match(rawid))
-        if (@pk_cast_from_string)
+        if @pk_cast_from_string
           Contract.valid(@pk_cast_from_string.call(md[1]))
         else
           Contract.valid(md[1]) # no cast needed, eg for string
@@ -677,7 +674,7 @@ module Safrano
     # 2. Create relationship if needed
     def odata_create_entity_and_relation(req, assoc = nil, parent = nil)
       # TODO: this is for v2 only...
-      req.with_parsed_data do |data|
+      req.with_parsed_data(self) do |data|
         data.delete('__metadata')
 
         # validate payload column names

data/lib/odata/request/json.rb

@@ -0,0 +1,171 @@
+require 'json'
+require 'time'
+
+# client parsing functionality to ease testing
+
+module Safrano
+  module OData
+    # this is used to parse inbound json payload on POST / PUT & co
+    # it does not do symbolize but proper (hopefully) type casting when needed
+    module JSON
+      # def self.parse(*args)
+      #  ::JSON.parse(*args)
+      # end
+
+      def self.cast_values_in(resd, typ)
+        typ.db_schema.each do |f, props|
+          metadata = typ.cols_metadata[f]
+
+          case props[:type]
+          when :datetime, :date
+            fstr = f.to_s
+            # resd[fstr] = Time.parse(resd[f]) if resd[fstr]
+            resd[fstr] = Sequel.datetime_class.from_edm_json(resd[fstr]) if resd[fstr]
+          end
+        end
+
+        resd
+      end
+
+      def self.parse_one(*args, type)
+        cast_values_in(::JSON.parse(*args), type)
+      end
+    end
+  end
+  # This is used from the Test-suite code !
+  # it does recursive / deep symbolize additionally to inbound casting
+  module XJSON
+    def self.get_class_from_meta(meta)
+      # type is normally namespaced !  --> split.last is the class
+      meta[:type].split('.').last.constantize
+    end
+
+    # symbolise keys and cast/parse values back to the proper ruby type;
+    # proper type meaning the one that Sequel chooses when loading data from the DB
+    # apply recursively to nested navigation attributes sub-structures
+    def self.cast_values_in(resd)
+      resd.symbolize_keys!
+
+      if (defered = resd[:__deferred])
+        defered.symbolize_keys!
+      elsif meta = resd[:__metadata]
+        meta.symbolize_keys!
+
+        # type is normally namespaced !  --> split.last is the class
+        typ = get_class_from_meta(meta)
+
+        typ.db_schema.each do |f, props|
+          metadata = typ.cols_metadata[f]
+          case props[:type]
+          when :datetime
+            #              resd[f] = Time.parse(resd[f]) if resd[f]
+            #              resd[f] = DateTime.strptime(resd[f], '/Date(%Q)').to_time if resd[f]
+            resd[f] = Sequel.datetime_class.from_edm_json(resd[f]) if resd[f]
+          when :decimal
+            resd[f] = BigDecimal(resd[f])
+          when :float
+            resd[f] = Float(resd[f])
+          else
+            # TODO: better typ-system
+            # Currently Sequel loads Numeric(x,y) as Float
+            resd[f] = Float(resd[f]) if metadata[:edm_type] =~ /\A\s*Edm.Decimal/
+          end
+        end
+
+        if typ.nav_entity_attribs
+          typ.nav_entity_attribs.each_key do |nattr|
+            cast_values_in(resd[nattr.to_sym]) if resd[nattr.to_sym]
+          end
+        end
+
+        if typ.nav_collection_attribs
+          typ.nav_collection_attribs.each_key do |ncattr|
+            if (resd_attr = resd[ncattr.to_sym])
+              if (defered = resd_attr['__deferred'])
+                defered.symbolize_keys! if defered
+              else
+                resd_attr.symbolize_keys! #  'results' --> :results
+                resd_attr[:results].each { |enty| cast_values_in(enty) }
+              end
+            end
+          end
+        end
+      end
+      resd
+    end
+
+    # symbolise attrib names (h-keys)
+    # apply recursively to nested navigation attributes sub-structures
+    def self.symbolize_attribs_in(resd)
+      resd.symbolize_keys!
+
+      if (defered = resd[:__deferred])
+        defered.symbolize_keys!
+      elsif meta = resd[:__metadata]
+        meta.symbolize_keys!
+
+        typ = get_class_from_meta(meta)
+
+        if typ.nav_entity_attribs
+          typ.nav_entity_attribs.each_key do |nattr|
+            symbolize_attribs_in(resd[nattr.to_sym])
+          end
+        end
+
+        if typ.nav_collection_attribs
+          typ.nav_collection_attribs.each_key do |ncattr|
+            if (defered = resd[ncattr.to_sym]['__deferred'])
+              defered.symbolize_keys!
+            else
+              resd[ncattr.to_sym].each { |enty| symbolize_attribs_in(enty) }
+            end
+          end
+        end
+
+      end
+      resd
+    end
+
+    def self.parse_one_nocast(*args)
+      resh = ::JSON.parse(*args)
+      symbolize_attribs_in(resh['d'])
+    end
+
+    def self.parse_one(*args)
+      resh = ::JSON.parse(*args)
+      cast_values_in(resh['d'])
+    end
+
+    def self.v1_parse_coll(*args)
+      resh = ::JSON.parse(*args)
+
+      resh['d'].map! { |currd| cast_values_in(currd) }
+
+      resh['d']
+    end
+
+    def self.parse_coll(*args)
+      resh = ::JSON.parse(*args)
+
+      resh['d']['results'].map! { |currd| cast_values_in(currd) }
+
+      resh['d']
+    end
+
+    def self.v1_parse_coll_nocast(*args)
+      resh = ::JSON.parse(*args)
+
+      resh['d'].map! { |currd| symbolize_attribs_in(currd) }
+
+      resh['d']
+    end
+
+    def self.parse_coll_nocast(*args)
+      resh = ::JSON.parse(*args)
+
+      resh['d']['results'].map! { |currd| symbolize_attribs_in(currd) }
+
+      resh['d']
+    end
+  end
+end

data/lib/odata/transition.rb

@@ -12,8 +12,8 @@ module Safrano
     attr_accessor :rgx
     attr_reader :remain_idx
 
-    EMPTYSTR = ''.freeze
-    SLASH = '/'.freeze
+    EMPTYSTR = ''
+    SLASH = '/'
 
     RESULT_BAD_REQ_ERR = [nil, :error, ::Safrano::BadRequestError].freeze
     RESULT_NOT_FOUND_ERR = [nil, :error, ::Safrano::ErrorNotFound].freeze

data/lib/odata/url_parameters.rb

@@ -68,20 +68,20 @@ module Safrano
       return Contract::OK unless @params['$top']
 
       itop = number_or_nil(@params['$top'])
-      (itop.nil? || itop.zero?) ? BadRequestError : Contract::OK
+      itop.nil? || itop.zero? ? BadRequestError : Contract::OK
     end
 
     def check_skip
       return Contract::OK unless @params['$skip']
 
       iskip = number_or_nil(@params['$skip'])
-      (iskip.nil? || iskip.negative?) ? BadRequestError : Contract::OK
+      iskip.nil? || iskip.negative? ? BadRequestError : Contract::OK
     end
 
     def check_inlinecount
       return Contract::OK unless (icp = @params['$inlinecount'])
 
-      ((icp == 'allpages') || (icp == 'none')) ? Contract::OK : BadRequestInlineCountParamError
+      (icp == 'allpages') || (icp == 'none') ? Contract::OK : BadRequestInlineCountParamError
     end
 
     def check_filter

data/lib/odata/walker.rb

@@ -183,7 +183,7 @@ module Safrano
     end
 
     def finalize
-      (@status == :end) ? Contract.valid(@end_context) : @error
+      @status == :end ? Contract.valid(@end_context) : @error
     end
   end
 end

data/lib/safrano/multipart.rb

@@ -411,9 +411,7 @@ module MIME
 
         def unparse(bodyonly = false)
           b = +String.new
-          unless bodyonly
-            b << "#{Safrano::CONTENT_TYPE}: #{@hd[CTT_TYPE_LC]}#{CRLF}"
-          end
+          b << "#{Safrano::CONTENT_TYPE}: #{@hd[CTT_TYPE_LC]}#{CRLF}" unless bodyonly
 
           b << crbdcr = "#{CRLF}--#{@boundary}#{CRLF}"
           b << @content.map(&:unparse).join(crbdcr)

data/lib/safrano/rack_app.rb

@@ -10,7 +10,7 @@ module Safrano
   module MethodHandlers
     def odata_options
       @walker.finalize.tap_error { |err| return err.odata_get(@request) }
-             .if_valid do |context|
+             .if_valid do |_context|
         # cf. stackoverflow.com/questions/22924678/sinatra-delete-response-headers
         headers.delete('Content-Type')
         @response.headers.delete('Content-Type')
@@ -119,7 +119,7 @@ module Safrano
         @response = Safrano::Response.new
 
         before.tap_error { |err| dispatch_error(err) }
-              .tap_valid { |res| dispatch }
+              .tap_valid { |_res| dispatch }
 
       # handle remaining Sequel errors that we couldnt prevent with our
       # own pre-checks
@@ -161,15 +161,3 @@ module Safrano
     end
   end
 end
-
-# deprecated
-# REMOVE 0.6
-module OData
-  class ServerApp < Safrano::ServerApp
-    def self.publish_service(&block)
-      ::Safrano::Deprecation.deprecate('OData::ServerApp',
-                                       'Use Safrano::ServerApp instead')
-      super
-    end
-  end
-end

data/lib/safrano/rack_builder.rb

@@ -17,18 +17,3 @@ module Rack
     end
   end
 end
-
-# deprecated
-# REMOVE 0.6
-module Rack
-  module OData
-    class Builder < ::Rack::Safrano::Builder
-      def initialize(default_app = nil, &block)
-        ::Safrano::Deprecation.deprecate('Rack::OData::Builder',
-                                         'Use Rack::Safrano::Builder instead')
-
-        super
-      end
-    end
-  end
-end

data/lib/safrano/request.rb

@@ -144,11 +144,11 @@ module Safrano
       end
     end
 
-    def with_parsed_data
+    def with_parsed_data(type)
       if content_type == APPJSON
         # Parse json payload
         begin
-          data = JSON.parse(body.read)
+          data = Safrano::OData::JSON.parse_one(body.read, type)
         rescue JSON::ParserError => e
           ON_CGST_ERROR.call(self)
           return [400, EMPTY_HASH, ['JSON Parser Error while parsing payload : ',
@@ -239,7 +239,7 @@ module Safrano
         get_minversion.if_valid do |minv|
           return MAX_LT_MIN_DTSV_ERROR if minv > maxv
 
-          get_version.if_valid do |v|
+          get_version.if_valid do |_v|
             @service = nil
             @service = case maxv
                        when '1'

data/lib/safrano/response.rb

@@ -18,8 +18,8 @@ module Safrano
     end
 
     def body=(value)
-      value = value.body while ::Rack::Response === value
-      @body = String === value ? [value.to_str] : value
+      value = value.body while value.is_a?(::Rack::Response)
+      @body = value.is_a?(String) ? [value.to_str] : value
     end
 
     def each
@@ -51,7 +51,7 @@ module Safrano
     private
 
     def calculate_content_length?
-      headers['Content-Type'] && !headers['Content-Length'] && (Array === body)
+      headers['Content-Type'] && !headers['Content-Length'] && body.is_a?(Array)
     end
 
     def calculated_content_length

data/lib/safrano/service.rb

@@ -161,6 +161,8 @@ module Safrano
       @function_import_keys = []
       @cmap = {}
       @type_mappings = {}
+      # enabled per default starting from 0.6
+      @bugfix_create_response = true
       instance_eval(&block) if block_given?
     end
 
@@ -209,9 +211,8 @@ module Safrano
       (@v2.xserver_url = @xserver_url) if @v2
     end
 
-    # keep the bug active for now, but allow to activate the fix,
-    # later we will change the default to be fixed
-    def bugfix_create_response(bool = false)
+    # keep the bug active for now, but allow to de-activate the fix
+    def bugfix_create_response(bool)
       @bugfix_create_response = bool
     end
 
@@ -246,6 +247,7 @@ module Safrano
       other.function_imports = @function_imports
       other.function_import_keys = @function_import_keys
       other.type_mappings = @type_mappings
+      other.bugfix_create_response(@bugfix_create_response)
       other
     end
 
@@ -386,10 +388,10 @@ module Safrano
         klass.build_uri(@uribase)
 
         # Output create (POST) as single entity (Standard) or as array (non-standard buggy)
-        klass.include ( @bugfix_create_response ? Safrano::EntityCreateStandardOutput : Safrano::EntityCreateArrayOutput)
+        klass.include(@bugfix_create_response ? Safrano::EntityCreateStandardOutput : Safrano::EntityCreateArrayOutput)
 
         # define the most optimal casted_values method for the given model(klass)
-        if (klass.casted_cols.empty?)
+        if klass.casted_cols.empty?
           klass.send(:define_method, :casted_values) do |cols = nil|
             cols ? selected_values_for_odata(cols) : values_for_odata
           end
@@ -413,10 +415,16 @@ module Safrano
       case Sequel::Model.db.adapter_scheme
       when :postgres
         Safrano::Filter::FuncTree.include Safrano::Filter::FuncTreePostgres
+        Safrano::Filter::DateTimeLit.include Safrano::Filter::DateTimeDefault
+        Safrano::Filter::DateTimeOffsetLit.include Safrano::Filter::DateTimeDefault
       when :sqlite
         Safrano::Filter::FuncTree.include Safrano::Filter::FuncTreeSqlite
+        Safrano::Filter::DateTimeLit.include Safrano::Filter::DateTimeSqlite
+        Safrano::Filter::DateTimeOffsetLit.include Safrano::Filter::DateTimeSqlite
       else
         Safrano::Filter::FuncTree.include Safrano::Filter::FuncTreeDefault
+        Safrano::Filter::DateTimeLit.include Safrano::Filter::DateTimeDefault
+        Safrano::Filter::DateTimeOffsetLit.include Safrano::Filter::DateTimeDefault
       end
     end
 

data/lib/safrano/type_mapping.rb

@@ -45,17 +45,17 @@ module Safrano
       end
 
       def match(curtyp)
-        if (@bui2 && (m = @bui2.match(curtyp)))
+        if @bui2 && (m = @bui2.match(curtyp))
           m
-        elsif (@bui1 && (m = @bui1.match(curtyp)))
+        elsif @bui1 && (m = @bui1.match(curtyp))
           m
-        elsif  @rgx.match(curtyp)
+        elsif @rgx.match(curtyp)
           type_mapping
         end
       end
 
       def type_mapping
-        # TODO perf; return always same object when called multiple times
+        # TODO: perf; return always same object when called multiple times
         FixedTypeMapping.new(self)
       end
     end # Builder

data/lib/safrano/version.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-
 module Safrano
-  VERSION = '0.5.5'
+  VERSION = '0.6.0'
 end

data/lib/safrano.rb

@@ -14,9 +14,12 @@ require_relative 'odata/entity'
 require_relative 'odata/attribute'
 require_relative 'odata/navigation_attribute'
 require_relative 'odata/model_ext'
+require_relative 'odata/request/json'
 require_relative 'safrano/service'
 require_relative 'odata/walker'
 require 'sequel'
 require_relative 'safrano/sequel_join_by_paths'
 require_relative 'safrano/rack_app'
 require_relative 'safrano/rack_builder'
+
+Sequel.extension :named_timezones