infopark_fiona7 0.71.1.1 → 0.71.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6138dff9bbb3210c8e77bb4b11d3972134f83954
-  data.tar.gz: 10aece82213930a7034b49dd23b6a74bfc5586d6
+  metadata.gz: 473b5c069dec107647330585c7a2d486d58787f8
+  data.tar.gz: 07cc316117ad7a1703d18ee7947e82bceb7d96d9
 SHA512:
-  metadata.gz: 97616528eb79c32a0f34b4ffe12ab72632baf778cb5baa26299d461b36c697b5b796ebc3c5de95b9191ee7a2c0eb36c40737205b5500b59d9ea61bfff7bcad12
-  data.tar.gz: 53f3de235ab3adc20b6693758da37db68dd09f6577d5236726917fea813d6415ee4c0c9ca20a853857ff234b499e1324abbb3d6fd3062b4160a09b07129625e6
+  metadata.gz: 1d3b776871dc5e054a165ec9a8ddae50f587aed738b21723aad6752c2f755a3cbf2133e383b49c02ef6a1f62d27c3f5ce435f855e96019e0d7a294aebe1b6b42
+  data.tar.gz: 0772f0b1b25d78663dea2f8ea374cc36eab105e31c07fc5b1a688eb36b89cc3f27052232f049822d8960c6048995dc16198810228113793650de0cfe69a19504

data/app/assets/javascripts/fiona7_ui.js

@@ -43,42 +43,42 @@
     'commands.release_obj.not_modified_obj': 'Diese Seite wurde nicht geändert. Daher gibt es nichts zu freigeben.',
     'commands.release_obj.dialog.title': 'Änderungen an dieser Seite veröffentlichen?',
     'commands.release_obj.dialog.description': 'Eine Arbeitsversion zu veröffentlichen ist endgültig. Dieser Vorgang kann nicht rückgängig gemacht werden.',
-    'commands.release_obj.dialog.placeholder': 'Kommentar',
+    'commands.release_obj.dialog.placeholder': 'Bitte Kommentar eintragen',
     'commands.release_obj.dialog.confirm': 'Freigeben',
     'commands.release_obj.failed': 'Freigabe fehlgeschlagen',
 
     'commands.forward_obj.title': 'Seite weiterleiten',
     'commands.forward_obj.dialog.title': 'Seite weiterleiten?',
     'commands.forward_obj.dialog.description': 'Die Arbeitsversion wird an einen weiteren Bearbeiter weitergeleitet. Dieser Vorgang kann nicht rückgängig gemacht werden.',
-    'commands.forward_obj.dialog.placeholder': 'Kommentar',
+    'commands.forward_obj.dialog.placeholder': 'Bitte Kommentar eintragen',
     'commands.forward_obj.dialog.confirm': 'Weiterleiten',
     'commands.forward_obj.failed': 'Weiterleitung fehlgeschlagen',
 
     'commands.reject_obj.title': 'Seite ablehnen',
     'commands.reject_obj.dialog.title': 'Seite ablehnen?',
     'commands.reject_obj.dialog.description': 'Die Arbeitsversion wird abgelehnt. Dieser Vorgang kann nicht rückgängig gemacht werden.',
-    'commands.reject_obj.dialog.placeholder': 'Kommentar',
+    'commands.reject_obj.dialog.placeholder': 'Bitte Kommentar eintragen',
     'commands.reject_obj.dialog.confirm': 'Ablehnen',
     'commands.reject_obj.failed': 'Ablehnung fehlgeschlagen',
 
     'commands.sign_obj.title': 'Seite abzeichnen',
     'commands.sign_obj.dialog.title': 'Seite abzeichnen?',
     'commands.sign_obj.dialog.description': 'Die eingereichte Version wird abgezeichnet. Dieser Vorgang kann nicht rückgängig gemacht werden.',
-    'commands.sign_obj.dialog.placeholder': 'Kommentar',
+    'commands.sign_obj.dialog.placeholder': 'Bitte Kommentar eintragen',
     'commands.sign_obj.dialog.confirm': 'Abzeichnen',
     'commands.sign_obj.failed': 'Abzeichnung fehlgeschlagen',
 
     'commands.commit_obj.title': 'Seite einreichen',
     'commands.commit_obj.dialog.title': 'Seite einreichen?',
     'commands.commit_obj.dialog.description': 'Die Arbeitsversion wird eingereicht. Dieser Vorgang kann nicht rückgängig gemacht werden.',
-    'commands.commit_obj.dialog.placeholder': 'Kommentar',
+    'commands.commit_obj.dialog.placeholder': 'Bitte Kommentar eintragen',
     'commands.commit_obj.dialog.confirm': 'Einreichen',
     'commands.commit_obj.failed': 'Einreichung fehlgeschlagen',
 
     'commands.take_obj.title': 'Seite übernehmen',
     'commands.take_obj.dialog.title': 'Seite übernehmen?',
     'commands.take_obj.dialog.description': 'Die Arbeitsversion wird übernommen. Dieser Vorgang kann nicht rückgängig gemacht werden.',
-    'commands.take_obj.dialog.placeholder': 'Kommentar',
+    'commands.take_obj.dialog.placeholder': 'Bitte Kommentar eintragen',
     'commands.take_obj.dialog.confirm': 'Übernehmen',
     'commands.take_obj.failed': 'Übernahme fehlgeschlagen'
   }, 'de');

data/lib/fiona7/builder/obj_builder.rb

@@ -78,6 +78,8 @@ module Fiona7
         @widget_path_map = resolver.path_map
 
         if @widget_pool && !@widget_pool.empty?
+          @new_full_text = rewrite_full_text
+
           @widgets = BatchWidgetWriter.new(@obj, @id_map, @widget_pool, @widget_path_map)
           @widgets.write
 
@@ -254,7 +256,9 @@ module Fiona7
             raise Scrivito::ClientError.new("Unknown attribute type: #{virtual_type}", 418)
           end
         end
-        @obj.set(:X_full_text, rewrite_full_text) if @new_widget_pool
+        if (@new_full_text)
+          @obj.set(:X_full_text, @new_full_text)
+        end
         @obj.set(:X_widget_pool, @new_widget_pool) if @new_widget_pool
         @obj.set(:permalink, @permalink) if @permalink
 
@@ -286,10 +290,14 @@ module Fiona7
       end
 
       def rewrite_full_text
-        full_text = ::JSON.parse(@obj.attr_values["X_full_text"]) rescue {}
+        full_text = ::YAML.load(@obj.attr_values["X_full_text"])
+        full_text = {} unless full_text.kind_of?(Hash)
         full_text["_widget_pool"] ||= {}
         full_text["_widget_pool"].deep_merge!(@widget_pool)
-        full_text.to_json
+        full_text.to_yaml
+      rescue => e
+        Rails.logger.error("Unable to store information for search engine: #{e.message}")
+        nil
       end
 
       def upload_file(file)

data/lib/fiona7/current_user_map.rb

@@ -2,11 +2,12 @@ module Fiona7
   module CurrentUserMapHelper
     def self.groups_for_user(user)
       user_name = user.id rescue ""
+      return [] if user_name.blank?
       Reactor::Sudo.su(Fiona7.root) do
         Reactor::Cache::User.instance.get(user_name).groups
       end
     rescue => e
-      Rails.logger.error "Error occured reading user groups of #{user_name} (#{e.message})"
+      Rails.logger.error "Error occured while reading user groups of #{user_name} (#{e.message})"
       []
     end
   end

data/lib/fiona7/json/obj_decorator.rb

@@ -13,17 +13,23 @@ require 'fiona7/widget_resolver'
 module Fiona7
   module JSON
     class ObjDecorator
-      def initialize(klass, obj)
+      def initialize(klass, obj, widget_resolver=nil)
         @klass = klass
         @obj = obj
 
         @link_converter = LinkConverter::FionaToScrivito.new(obj)
+
+        if widget_resolver
+          @widget_resolver = widget_resolver
+        elsif (widget_links = @obj.attr_values["X_widget_pool"])
+          @widget_resolver = WidgetResolver.new(widget_links, @klass)
+        end
       end
 
       def as_json(*args)
-        attrs = standard_attrs
-        add_custom_attributes(attrs, *args)
+        @attrs = attrs = standard_attrs
         replace_widget_pool(attrs, *args)
+        add_custom_attributes(attrs, *args)
 
         attrs
       end
@@ -150,7 +156,14 @@ module Fiona7
           widget_ids = []
           widgets = linklist.map do |link|
             widget_id = link["title"]
-            widget_ids << widget_id
+            # sometimes a released page references an unreleased widget
+            # it is then impossible to load it into widget_pool
+            # but the reference in the widgetlist field is still there
+            # therefore we need to check if the widget has been loaded
+            # into the pool - only then it can be included
+            if valid_widget_id?(widget_id)
+              widget_ids << widget_id
+            end
           end
 
           widget_ids
@@ -170,10 +183,14 @@ module Fiona7
         end
       end
 
+      def valid_widget_id?(widget_id)
+        @widget_resolver && @widget_resolver.path_map[widget_id]
+      end
+
       def replace_widget_pool(attrs, *args)
-        if (widget_links = @obj.attr_values["X_widget_pool"])
+        if (widget_links = @obj.attr_values["X_widget_pool"]) && @widget_resolver
 
-          resolver = WidgetResolver.new(widget_links, @klass)
+          resolver = @widget_resolver
 
           Assert.success(
             resolver.instance_map.all? {|_, instance| !instance.nil?},
@@ -182,7 +199,7 @@ module Fiona7
 
           result = Hash[
             resolver.instance_map.map {|widget_id, widget|
-              [widget_id, WidgetDecorator.new(@klass, widget).as_json(*args)]
+              [widget_id, WidgetDecorator.new(@klass, widget, @widget_resolver).as_json(*args)]
             }
           ]
 

data/lib/fiona7/version.rb

@@ -1,3 +1,3 @@
 module Fiona7
-  VERSION = "0.71.1.1"
+  VERSION = "0.71.1.2"
 end

data/lib/fiona7/widget_resolver.rb

@@ -33,7 +33,7 @@ module Fiona7
         widget_id_to_object_id_map.map {|widget_id, obj_id|
           object = object_id_to_widget_instance_map[obj_id]
           if !object
-            Rails.logger.error("Unable to find widget with id #{widget_id} (obj_id = #{obj_Id})")
+            Rails.logger.error("Unable to find widget with id #{widget_id} (obj_id = #{obj_id})")
             next
           end
           [widget_id, object]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: infopark_fiona7
 version: !ruby/object:Gem::Version
-  version: 0.71.1.1
+  version: 0.71.1.2
 platform: ruby
 authors:
 - Tomasz Przedmojski
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-16 00:00:00.000000000 Z
+date: 2015-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails