scrivito_sdk 1.15.0 → 1.17.0.rc2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 89b1e59c229915c692327f0d8a4e4ab3cf3ca52c7c82b1798f6c206d609f4646
-  data.tar.gz: 1faa4464b7871e33525cfda460262afc0a9aa55080a712dcadaf0425f4a4678d
+  metadata.gz: 146486584acb9660d1f7ceb8d7ecbe072bdb0b3e2ea9fab5437db0d66ded97a7
+  data.tar.gz: d8d757425944ffd948c56e5513c1e01a91092a205aaec407143ae302b31350e6
 SHA512:
-  metadata.gz: d4462657b54b3e78c7262ab7d1ee59162639278c5505dfbed35472d20c2e0d463af546a10ce175376fb91ecffc8cb1643c318de87b700b4c3914dd72ae16e688
-  data.tar.gz: 5fc475b928f74c0acab2abc6a45b9f42ee06ad0f41d8630a9afad0b2b67639ec9bff6c9322766523e5117e7690803da915de781f2dfff1d9eba461f5fee9ea85
+  metadata.gz: ded66fee7754e12b25d6ebbaed8063190990c3e0c218f44d3e8386cd81a7a98ed9885002eaaf21ba880237d06efc9e2ddd8d655164c6cb73dd61b3df4a5ca60e
+  data.tar.gz: 9ee6a548c78305663fac85d0b407a77bbac3eb2879bca4ef0afebd7d6604b9c3e2eed34fd400055f92a56a199004c96f891ac6af001d7f7c0af599e11141e2fc

data/README.md

@@ -12,7 +12,7 @@ of concepts and APIs, tutorials, and release notes.
 
 ## License
 
-Copyright (c) 2009 - 2018 Infopark AG (http://www.infopark.com)
+Copyright (c) 2009 - 2020 Infopark Group GmbH (http://www.infopark.com)
 
 This software can be used and modified under the LGPL-3.0. Please refer to
 http://www.gnu.org/licenses/lgpl-3.0.html for the license text.

data/app/cms/scrivito/attribute_serializer.rb

@@ -29,6 +29,8 @@ class AttributeSerializer < Struct.new(:obj_class, :obj_id)
       _permalink
     ].include?(attribute_name)
       [attribute_name, attribute_value.to_s]
+    elsif attribute_name == '_restriction'
+      [attribute_name, attribute_value]
     else
       raise_unknown_attribute(attribute_name)
     end

data/app/cms/scrivito/basic_obj.rb

@@ -1,4 +1,3 @@
-require 'json'
 require 'ostruct'
 require 'active_model/naming'
 
@@ -25,15 +24,18 @@ module Scrivito
     PublicSystemAttributeDefinition = Class.new(AttributeDefinition)
 
     SYSTEM_ATTRIBUTES = AttributeDefinitionCollection.new(
-      '_id'           => PublicSystemAttributeDefinition.new(:_id, :string),
-      '_last_changed' => PublicSystemAttributeDefinition.new(:_last_changed, :date),
-      '_obj_class'    => PublicSystemAttributeDefinition.new(:_obj_class, :string),
-      '_path'         => PublicSystemAttributeDefinition.new(:_path, :string),
-      '_permalink'    => PublicSystemAttributeDefinition.new(:_permalink, :string),
-
-      '_conflicts'    => AttributeDefinition.new(:_conflicts, nil),
-      '_modification' => AttributeDefinition.new(:_modification, nil),
-      '_widget_pool'  => AttributeDefinition.new(:_widget_pool, nil),
+      '_id'                 => PublicSystemAttributeDefinition.new(:_id, :string),
+      '_created_at'         => PublicSystemAttributeDefinition.new(:_created_at, :date),
+      '_last_changed'       => PublicSystemAttributeDefinition.new(:_last_changed, :date),
+      '_first_published_at' => PublicSystemAttributeDefinition.new(:_first_published_at, :date),
+      '_published_at'       => PublicSystemAttributeDefinition.new(:_published_at, :date),
+      '_obj_class'          => PublicSystemAttributeDefinition.new(:_obj_class, :string),
+      '_path'               => PublicSystemAttributeDefinition.new(:_path, :string),
+      '_permalink'          => PublicSystemAttributeDefinition.new(:_permalink, :string),
+
+      '_conflicts'          => AttributeDefinition.new(:_conflicts, nil),
+      '_modification'       => AttributeDefinition.new(:_modification, nil),
+      '_widget_pool'        => AttributeDefinition.new(:_widget_pool, nil),
     )
 
     UNIQ_ATTRIBUTES = %w[
@@ -44,7 +46,10 @@ module Scrivito
 
     GENERATED_ATTRIBUTES = %w[
       _conflicts
+      _created_at
       _last_changed
+      _first_published_at
+      _published_at
       _modification
       _obj_type
       _text_links
@@ -138,14 +143,14 @@ module Scrivito
       else
         attributes = build_attributes_with_defaults(attributes, context)
         attributes = prepare_attributes_for_instantiation(attributes)
-        api_attributes, widget_properties = prepare_attributes_for_rest_api(attributes)
+        extraction, api_attributes = prepare_attributes_for_rest_api(attributes)
 
         workspace = Workspace.current
         obj_data = workspace.create_obj(obj: api_attributes)
         obj = BasicObj.instantiate(obj_data)
         obj.revision = workspace.revision
 
-        CmsRestApi::WidgetExtractor.notify_persisted_widgets(obj, widget_properties)
+        extraction.notify_persisted(obj)
         obj
       end
     end
@@ -301,7 +306,7 @@ module Scrivito
     # @api public
     def self.find_by_path(path)
       Workspace.current.objs.find_one_by(:path, path,
-          obj_class_name: type_computer.obj_class_name_for_type(self))
+          type_computer.obj_class_name_for_type(self))
     end
 
     # Find an {Scrivito::BasicObj Obj} with the given name.
@@ -329,7 +334,7 @@ module Scrivito
     # @api public
     def self.find_by_permalink(permalink)
       Workspace.current.objs.find_one_by(:permalink, permalink,
-          obj_class_name: type_computer.obj_class_name_for_type(self))
+          type_computer.obj_class_name_for_type(self))
     end
 
     # Returns the {Scrivito::BasicObj Obj} with the given permalink, or raises
@@ -414,9 +419,9 @@ module Scrivito
     #     }
     #   )
     def update(attributes)
-      api_attributes, widget_properties = prepare_attributes_for_rest_api(attributes)
+      extraction, api_attributes = prepare_attributes_for_rest_api(attributes)
       update_data(workspace.update_obj(id, obj: api_attributes))
-      CmsRestApi::WidgetExtractor.notify_persisted_widgets(self, widget_properties)
+      extraction.notify_persisted(self)
       self
     end
 
@@ -633,7 +638,7 @@ module Scrivito
       return [] if binary?
       toclist = children
       toclist = toclist.reject { |toc| toc.binary? } unless args.include?(:all)
-      toclist
+      toclist.sort_by(&:id)
     end
 
     # @api public
@@ -659,13 +664,23 @@ module Scrivito
     SYSTEM_KEYS = Set.new(%w[
       body
       _id
+      _created_at
       _last_changed
+      _first_published_at
+      _published_at
       _path
       _permalink
       _obj_class
       title
     ])
 
+    DATE_KEYS = Set.new(%w[
+      _created_at
+      _last_changed
+      _first_published_at
+      _published_at
+    ])
+
     # Returns the value of a system or custom attribute specified by its name.
     # Passing an invalid key will not raise an error but return +nil+.
     # @api public
@@ -704,11 +719,30 @@ module Scrivito
       }
     end
 
+    # @return [nil, ActiveSupport::TimeWithZone]
+    # @api public
+    def created_at
+      read_attribute('_created_at')
+    end
+
+    # @return [ActiveSupport::TimeWithZone]
     # @api public
     def last_changed
       read_attribute('_last_changed')
     end
 
+    # @return [nil, ActiveSupport::TimeWithZone]
+    # @api public
+    def first_published_at
+      read_attribute('_first_published_at')
+    end
+
+    # @return [nil, ActiveSupport::TimeWithZone]
+    # @api public
+    def published_at
+      read_attribute('_published_at')
+    end
+
     def new?(revision=workspace.base_revision)
       quick_modification(revision) == "new"
     end
@@ -820,7 +854,7 @@ module Scrivito
 
     # @api public
     # This method returns the byte length of the binary of the Obj.
-    # @return [Fixnum] If no binary is set, 0 is returned.
+    # @return [Integer] If no binary is set, 0 is returned.
     def binary_length
       binary.try(:content_length) || 0
     end
@@ -991,15 +1025,16 @@ module Scrivito
     end
 
     def self.prepare_attributes_for_rest_api(attributes, obj = nil)
-      widget_pool_attributes, obj_attributes = CmsRestApi::WidgetExtractor.call(attributes, obj)
+      extraction = CmsRestApi::WidgetExtractor.call(attributes, obj)
+      obj_attributes = extraction.attributes
       obj_id = obj ? obj.id : obj_attributes.fetch(:_id)
       workspace = obj ? obj.revision.workspace : Workspace.current
 
       api_attributes = serialize_attributes(
-        obj_attributes, widget_pool_attributes, workspace, obj_id
+        obj_attributes, extraction.widget_pool_attributes, workspace, obj_id
       )
 
-      [api_attributes, widget_pool_attributes]
+      [extraction, api_attributes]
     end
 
     def self.find_attribute_definitions(obj_class, basic_class = self)
@@ -1025,7 +1060,7 @@ module Scrivito
 
     private_class_method def self.serialize_widget_pool_attributes(serializer, widget_pool_attributes)
       {}.tap do |serialized_attributes|
-        widget_pool_attributes.each_pair do |widget, widget_attributes|
+        widget_pool_attributes.each_value do |(widget, widget_attributes)|
           obj_class = widget_attributes['_obj_class']
           widget_serializer = AttributeSerializer.new(obj_class, serializer.obj_id)
 
@@ -1066,8 +1101,7 @@ module Scrivito
 
     def instantiate_widget(widget_id, widget_data)
       BasicWidget.instantiate(widget_data).tap do |widget|
-        widget.id = widget_id
-        widget.obj = self
+        widget.attach_to_obj(self, widget_id)
       end
     end
 
@@ -1151,7 +1185,7 @@ module Scrivito
 
     def value_of_system_attribute(attribute_name)
       attribute_value = data_from_cms.value_of(attribute_name)
-      if attribute_name == '_last_changed'
+      if DATE_KEYS.include?(attribute_name)
         DateConversion.deserialize_from_backend(attribute_value)
       else
         attribute_value

data/app/cms/scrivito/basic_widget.rb

@@ -95,8 +95,6 @@ class BasicWidget
 
   attr_accessor :container, :container_attribute_name
 
-  attr_writer :obj, :id
-
   attr_reader :attributes_to_be_saved
 
   delegate :widget_from_pool, to: :obj
@@ -221,12 +219,29 @@ class BasicWidget
     @attributes_to_be_saved.nil?
   end
 
+  def attach_to_obj(an_obj, widget_id = compute_widget_id(an_obj))
+    @obj = an_obj
+    @id = widget_id
+
+    attributes_to_be_saved
+  end
+
   def ==(other)
-    other.respond_to?(:obj) && obj == other.obj && other.respond_to?(:id) && id == other.id
+    eql?(other)
   end
 
   def eql?(other)
-    self == other
+    other.respond_to?(:equality_identifier, true) &&
+      equality_identifier.eql?(other.equality_identifier)
+  end
+
+  def hash
+    unless persisted?
+      # An unbound widget will likely be attached soon, which mutates its #hash value. Evil.
+      raise ScrivitoError, 'a new widget is not allowed to be used as a hash key'
+    end
+
+    equality_identifier.hash
   end
 
   #
@@ -262,7 +277,7 @@ class BasicWidget
           { obj: {_widget_pool: {id => previous_widget_content}} })
       reload
     else
-      raise ScrivitoError, "cannot revert changes, since widget is #{modification}."
+      raise ScrivitoError, "cannot revert changes, since widget is #{modification}"
     end
   end
 
@@ -297,14 +312,6 @@ class BasicWidget
     end
   end
 
-  def hash
-    if @obj && @id
-      (id + obj.id).hash
-    else
-      super
-    end
-  end
-
   # Returns the entity ({Scrivito::BasicObj} or {Scrivito::BasicWidget})
   # that references this widget.
   # @api public
@@ -360,7 +367,8 @@ class BasicWidget
     end
   end
 
-  def forget_unsaved_attributes
+  def notify_persisted(obj)
+    @obj = obj
     @attributes_to_be_saved = nil
     reload_data
   end
@@ -398,6 +406,12 @@ class BasicWidget
     referenced + referenced.map { |widget| widget.contained_widgets }.flatten
   end
 
+  protected
+
+  def equality_identifier
+    [id, obj.id]
+  end
+
   private
 
   def workspace
@@ -433,6 +447,14 @@ class BasicWidget
   def value_of_system_attribute(attribute_name)
     data_from_cms.value_of(attribute_name)
   end
+
+  def compute_widget_id(obj)
+    @attributes_to_be_saved.delete('_id') || generate_widget_id(obj)
+  end
+
+  def generate_widget_id(obj)
+    obj.present? ? obj.generate_widget_pool_id : BasicObj.generate_widget_pool_id
+  end
 end
 
 end

data/app/cms/scrivito/binary.rb

@@ -195,7 +195,7 @@ class Binary
   #
   # @param [Hash] definition transformation definition
   #
-  # @option definition [Fixnum,String] :width The width in pixels of the output image. Must be a
+  # @option definition [Integer,String] :width The width in pixels of the output image. Must be a
   #   positive integer.
   #
   #   If only this dimension has been specified, the other dimension is calculated automatically to
@@ -214,7 +214,7 @@ class Binary
   #
   #   If the given width and height are adjusted, the aspect ratio is preserved.
   #
-  # @option definition [Fixnum,String] :height The height in pixels of the output image. Must be a
+  # @option definition [Integer,String] :height The height in pixels of the output image. Must be a
   #   positive integer.
   #
   #   If only this dimension has been specified, the other dimension is calculated automatically to
@@ -248,7 +248,7 @@ class Binary
   #   i.e. the center of the image is preserved. The area to preserve can be configured using
   #   the +crop+ parameter.
   #
-  # @option definition [Fixnum,String] :quality Controls the output quality of lossy file formats.
+  # @option definition [Integer,String] :quality Controls the output quality of lossy file formats.
   #   Applies if the format is +"jpg"+. Valid values are in the range from +0+ to +100+.
   #   The default value is +75+.
   #

data/app/cms/scrivito/cms_backend.rb

@@ -206,7 +206,7 @@ module Scrivito
     private_class_method :blob_data_cache_key
 
     def self.search_params_cache_key(params)
-      MultiJson.encode(normalize_search_param(params))
+      JSON.generate(normalize_search_param(params))
     end
     private_class_method :search_params_cache_key
 

data/app/cms/scrivito/cms_field_tag.rb

@@ -67,7 +67,7 @@ class CmsFieldTag < Struct.new(:view, :tag_name, :obj_or_widget, :editing_option
     if FIELD_TYPES_WITH_ORIGINAL_CONTENT.include?(field_type)
       original_value = view.scrivito_value(current_value)
       original_content = original_content(field_type, original_value, current_value)
-      encoded_content = Base64.strict_encode64(MultiJson.encode(original_content))
+      encoded_content = Base64.strict_encode64(JSON.generate(original_content))
       options['private-field-original-content'] = encoded_content
     end
 
@@ -161,7 +161,7 @@ class CmsFieldTag < Struct.new(:view, :tag_name, :obj_or_widget, :editing_option
   end
 
   def valid_values
-    MultiJson.encode(obj_or_widget.attribute_definitions[field_name].try(:values) || [])
+    JSON.generate(obj_or_widget.attribute_definitions[field_name].try(:values) || [])
   end
 
   def assert_valid_attribute

data/app/cms/scrivito/cms_rest_api.rb

@@ -123,7 +123,7 @@ module Scrivito
       timer)
       request = method_to_net_http_class(method).new(path(resource_path))
       set_headers(request)
-      request.body = MultiJson.encode(payload) if payload.present?
+      request.body = JSON.generate(payload) if payload.present?
 
       response = retry_once_on_network_error(method, timer) do
         CmsRestApi::RateLimit.retry_on_rate_limit(timer) do
@@ -138,12 +138,12 @@ module Scrivito
     private_class_method def self.handle_response(resource_path, response)
       http_code = response.code.to_i
       if response.code.start_with?('2')
-        MultiJson.load(response.body)
+        JSON.parse(response.body)
       elsif response.code == '403'
         raise AccessDenied.new(response.body)
       else
         begin
-          error_body = MultiJson.decode(response.body)
+          error_body = JSON.parse(response.body)
           specific_output = error_body['error']
 
           if response.code.start_with?('4')
@@ -155,7 +155,7 @@ module Scrivito
           else # 3xx and >500 are treated as NetworkErrors
             raise NetworkError.new(response.body, http_code)
           end
-        rescue MultiJson::DecodeError
+        rescue JSON::ParserError, TypeError
           raise NetworkError.new(response.body, http_code)
         end
       end
@@ -170,6 +170,9 @@ module Scrivito
         message = task_data["message"] ||
           "Missing error message in task response #{task_data}"
 
+        if task_data['status'] == 'exception'
+          raise BackendError.new(message, 500)
+        end
         backend_code = task_data['code']
         raise ClientError.new(message, backend_code: backend_code)
       end

data/app/cms/scrivito/cms_rest_api/rate_limit.rb

@@ -12,7 +12,7 @@ module RateLimit
       time_to_sleep = calculate_time_to_sleep(response['Retry-After'].to_f, retry_count)
 
       if request_timer.cover?(Time.now + time_to_sleep.seconds)
-        Warning.warn("Rate limit exceeded. Will retry after #{time_to_sleep} seconds.")
+        Scrivito::Warning.warn("Rate limit exceeded. Will retry after #{time_to_sleep} seconds.")
         sleep time_to_sleep
         internal_retry(request_proc, request_timer, retry_count + 1)
       else

data/app/cms/scrivito/cms_rest_api/widget_extractor.rb

@@ -1,53 +1,68 @@
 module Scrivito
   class CmsRestApi
     class WidgetExtractor
-
       # Extract widgets from a given attribute tree
       # first return param: a flat hash of widget_like => updated widget attributes
       # second return param: the updated attributes
       def self.call(attributes, obj=nil)
+        new(obj).extract_widgets(attributes)
+      end
+
+      def initialize(obj)
+        @obj = obj
+      end
+
+      def extract_widgets(attributes)
+        @extracted_widgets = {}
         updated_attributes = attributes.dup
-        extracted_widgets = {}
+        widget_pool = updated_attributes.delete(:_widget_pool)
+        extract_widgets_from_pool(widget_pool) if widget_pool
+        extract_widgets_from_attributes(updated_attributes)
 
-        (updated_attributes.delete(:_widget_pool) || {}).each do |widget, widget_properties|
-          unless widget.persisted?
-            raise ScrivitoError, 'new top-level widgets are not allowed in the _widget_pool'
-          end
-          extracted_widgets[widget] = widget_properties
-          extracted_widgets.merge!(call(widget_properties, obj).first)
+        Extraction.new(extracted_widgets, updated_attributes)
+      end
+
+      private
+
+      attr_reader :obj
+      attr_reader :extracted_widgets
+
+      def extract_widgets_from_pool(widget_pool)
+        widget_pool.each do |widget, widget_properties|
+          extracted_widgets[widget.id] = [widget, widget_properties]
+          extract_widgets_from_attributes(widget_properties)
         end
+      end
 
-        updated_attributes.each do |attribute_name, value|
+      def extract_widgets_from_attributes(attributes)
+        attributes.each do |attribute_name, value|
           if value.is_a?(Array) && value.all? { |w| w.is_a?(BasicWidget) }
             value.each do |widget|
               unless widget.persisted?
-                attach_widget_to_obj(widget, obj)
-                extracted_widgets[widget] = widget.attributes_to_be_saved
-                extracted_widgets.merge!(call(widget.attributes_to_be_saved, obj).first)
+                widget_attributes = widget.attach_to_obj(obj)
+                extracted_widgets[widget.id] = [widget, widget_attributes]
+                extract_widgets_from_attributes(widget_attributes)
               end
             end
           end
         end
-
-        return extracted_widgets, updated_attributes
       end
 
-      def self.attach_widget_to_obj(widget, obj)
-        widget.id = widget.attributes_to_be_saved.delete('_id') || generate_widget_pool_id(obj)
-        widget.obj = obj
-      end
+      class Extraction
+        attr_reader :widget_pool_attributes, :attributes
 
-      def self.notify_persisted_widgets(obj, widget_hash)
-        widget_hash.each do |widget, attributes|
-          if attributes.present?
-            widget.obj = obj
-            widget.forget_unsaved_attributes
-          end
+        def initialize(widget_pool_attributes, attributes)
+          @widget_pool_attributes = widget_pool_attributes
+          @attributes = attributes
         end
-      end
 
-      def self.generate_widget_pool_id(obj)
-        obj.presence ? obj.generate_widget_pool_id : BasicObj.generate_widget_pool_id
+        def notify_persisted(obj)
+          widget_pool_attributes.each_value do |(widget, attributes)|
+            if attributes.present?
+              widget.notify_persisted(obj)
+            end
+          end
+        end
       end
     end
   end