para 0.8.3.1 → 0.8.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA256:
-  metadata.gz: 57109a780f37db890a9b4fb0a567ff449d0d0e9c73c0ede097f3636f868c7ba8
-  data.tar.gz: 45d5e7ec1dac97d9e49b7871eb7c84c5d6bf93e81c57f861e84492511b3bf6f9
+SHA1:
+  metadata.gz: f7e6c8c33108e3898e0d032ff8c5e1910e4cffa4
+  data.tar.gz: 21447be4a3cdae224fb4bce7b7632e6e2e8f42c4
 SHA512:
-  metadata.gz: 5a230c4d409cb6340cdc5156b79a9dbfe3883f040a6f2f3b20d8bc86d266b7a788dcd8a975e691c6923f2843ca55c5bc2959baa5d572df0611a77a7739879a7f
-  data.tar.gz: 85cf71a9c7499b47f85a4b6eefa2facab6acb0a9894d5b6888f9b28a7a443361a303d56fbfaf57748b0ca105ca4ca3447389f545f02b2eacb79bf987ba68f91a
+  metadata.gz: 99823f9bdf05353edbaea1dbb39fcc9b3dc7784aafa6c748333de2cd6659f41d7f1ffb42d847a1aa37f5ddecca5b8e1ca99590c06289e767fefedcba28196c01
+  data.tar.gz: '0089801d66360f8a7aff0d6527a36d067c10e87d0ec8f5718bb7a4a0cdd8dae43c3425a6601d346062734aef8a11f50ae2753dc747e9d10bcdea86a9a208a07c'

data/app/assets/javascripts/para/admin.coffee

@@ -6,7 +6,8 @@
 #= require jasny-bootstrap
 #= require simple_form_extension
 #= require jquery.scrollto
-#= require html5-sortable
+#= require Sortable
+#= require jquery.sortable
 #= require cocoon
 #= require jquery.remote-modal-form
 #= require jquery.iframe-transport

data/app/assets/javascripts/para/admin/table.coffee

@@ -12,11 +12,9 @@ class Para.ResourceTable
 
     @$tbody.sortable
       handle: '.order-anchor'
-      forcePlaceholderSize: true
-      items: 'tr'
-      placeholder: "<tr><td colspan=\"100%\" class=\"sortable-placeholder\"></td></tr>"
-
-    @$tbody.on('sortupdate', $.proxy(@sortUpdate, this))
+      draggable: 'tr'
+      ghostClass: 'sortable-placeholder'
+      onUpdate: $.proxy(@sortUpdate, this)
 
   sortUpdate: ->
     @$tbody.find('tr').each (i, el) ->

data/app/assets/javascripts/para/admin/tree.coffee

@@ -6,22 +6,75 @@ class Para.ResourceTree
     @orderUrl = @$el.data('url')
     @maxDepth = parseInt @$el.data('max-depth')
 
-    $(".tree")
-      .sortable
-        handle: ".handle"
-        items: ".node"
-        connectWith: ".tree"
-      .on('sortupdate', $.proxy(@sortUpdate, this))
-
-  sortUpdate: (e, data) ->
-    @handlePlaceholder($el) for $el in [data.endparent, data.startparent, data.item.find('.tree')]
+    # Each is needed here as the sortable jQuery plugin doesn't loop over each found node
+    # but initializes the tree on the first found element.
+    $(".tree").each(@initializeSubTree)
+
+  initializeSubTree: (_i, el) =>
+    $(el).sortable(
+      group: "tree"
+      handle: ".handle"
+      draggable: ".node"
+      fallbackOnBody: true
+      swapThreshold: 0.65
+      animation: 150
+      onSort: @handleOrderUpdated
+      onMove: @isMovementValid
+    )
+
+  # Note : This method is called often (many times per second while we're dragging) and
+  # takes quite some processing.
+  isMovementValid: (e) =>
+    $movedNode = $(e.dragged)
+    $target = $(e.related)
+
+    # Calculate the deepness of the moved and target nodes
+    movedNodeDeepness = $movedNode.parents(".node").length - 1
+    # If the target is a node, the moved node root deepness is gonna be the same as the
+    # the target one, else the tree's parent node is counted also
+    targetDeepness = $target.parents(".node").length - 1
+
+    # Find the deepest node in the subtree of the moved node
+    $movedNodeSubtrees = $movedNode.find(".tree")
+    movedNodeTreeDeepness = 0
+
+    # The movedNodeTreeDeepness is the maximum deepness of a child node of the current
+    # moved node, relative to the moved node
+    $movedNodeSubtrees.each (i, el) =>
+      subtreeDeepness = $(el).parents(".node").length - 1
+      subtreeRelativeDeepness = subtreeDeepness - movedNodeDeepness
+      movedNodeTreeDeepness = Math.max(movedNodeTreeDeepness, subtreeRelativeDeepness)
+
+    # Calculate the final subtree deepness once we move the whole moved node subtree to
+    # its target position
+    finalSubtreeDeepnessAfterMove = movedNodeTreeDeepness + targetDeepness
+
+    # We finally validate the move only if the final subtree deepness is lower than the
+    # maximum allowed depth
+    finalSubtreeDeepnessAfterMove <= @maxDepth
+
+  handleOrderUpdated: (e) =>
+
+    # Get all involved tree leaves that may include a subtree
+    treeLeaves = [$(e.target), $(e.from), $(e.item).find('.tree')]
+
+    # Update their placeholder display wether they can be a drop target or not
+    @handlePlaceholder($el) for $el in treeLeaves
+    # Save the tree structure on the server
     @updateOrder()
 
+  # This method checks wether a given tree leaf can be a drop target, depending
+  # on wether it's located at the maximum allowed depth for the tree or not, and adds or
+  # remove a the visual placeholder to indicate its droppable state.
+  #
   handlePlaceholder: ($el) ->
     $placeholder = $el.find("> .placeholder")
-    if $el.parents('.tree').length - 1 >= @maxDepth or $el.find('> .node').length
+    parentsCount = $el.parents('.node').length - 1
+    hasChildren = $el.find('> .node').length
+
+    if parentsCount >= @maxDepth or hasChildren
       $placeholder.hide()
-      $el.children("> .tree").each (index, el) => @handlePlaceholder $(el)
+      $el.children(".tree").each (index, el) => @handlePlaceholder($(el))
     else
       $placeholder.show()
 
@@ -29,9 +82,8 @@ class Para.ResourceTree
     Para.ajax(
       url: @orderUrl
       method: 'patch'
-      data:
-        resources: @buildOrderedData()
-      success: $.proxy(@orderUpdated, this)
+      data: { resources: @buildOrderedData() }
+      success: @orderUpdated
     )
 
   buildOrderedData: ->
@@ -41,11 +93,11 @@ class Para.ResourceTree
       data[index] = {
         id: $this.data("id"),
         position: index,
-        parent_id: $this.parent().parent().data("id")
+        parent_id: $this.parents(".node:first").data("id")
       }
     data
 
-  orderUpdated: ->
+  orderUpdated: =>
     # TODO: Add flash message to display ordering success
 
 $(document).on 'page:change turbolinks:load', ->

data/app/assets/javascripts/para/inputs/multi-select-input.coffee

@@ -122,10 +122,9 @@ class Para.MultiSelectInput extends Vertebra.View
 
     @$selectedItems.sortable
       handle: '.order-anchor'
-      forcePlaceholderSize: true
-      placeholder: "<tr><td colspan='#{ columnsCount }'></td></tr>"
+      animation: 150
 
-    @$selectedItems.on('sortupdate', @selectedItemsSorted)
+    @$selectedItems.on('sort', @selectedItemsSorted)
 
   selectedItemsSorted: =>
     indices = {}

data/app/assets/javascripts/para/inputs/nested_many.coffee

@@ -13,17 +13,17 @@ class Para.NestedManyField
 
     @$fieldsList.sortable
       handle: '.order-anchor'
-      forcePlaceholderSize: true
+      animation: 150
+      onUpdate: $.proxy(@handleOrderingUpdated, this)
 
-    @$fieldsList.on('sortupdate', $.proxy(@sortUpdate, this))
-
-  sortUpdate: ->
-    @$fieldsList.find('.form-fields').each (i, el) ->
+  handleOrderingUpdated: ->
+    @$fieldsList.find('.form-fields:visible').each (i, el) ->
       $(el).find('.resource-position-field').val(i)
 
   initializeCocoon: ->
     @$fieldsList.on 'cocoon:after-insert', $.proxy(@afterInsertField, this)
     @$fieldsList.on 'cocoon:before-remove', $.proxy(@beforeRemoveField, this)
+    @$fieldsList.on 'cocoon:after-remove', $.proxy(@afterRemoveField, this)
 
   afterInsertField: (e, $element) ->
     if ($collapsible = $element.find('[data-open-on-insert="true"]')).length
@@ -32,7 +32,7 @@ class Para.NestedManyField
     if @orderable
       @$fieldsList.sortable('destroy')
       @initializeOrderable()
-      @sortUpdate()
+      @handleOrderingUpdated()
 
     $element.simpleForm()
 
@@ -42,6 +42,10 @@ class Para.NestedManyField
     # create an empty nested resource otherwise
     $nextEl.remove() if $nextEl.is('[data-attributes-mappings]') and not $element.is('[data-persisted]')
 
+  # When a sub field is removed, update every sub field position
+  afterRemoveField: ->
+    @handleOrderingUpdated();
+
   openInsertedField: ($field) ->
     $target = $($field.attr('href'))
     $target.collapse('show')

data/app/controllers/para/admin/search_controller.rb

@@ -1,6 +1,8 @@
 module Para
   module Admin
     class SearchController < ApplicationController
+      include Para::Helpers::ResourceName
+
       def index
         # Parse ids that are provided as string into array
         if params[:q] && params[:q][:id_in].is_a?(String)
@@ -9,8 +11,16 @@ module Para
 
         model = params[:model_name].constantize
         @results = model.ransack(params[:q]).result
+        @results = @results.limit(params[:limit]) if params[:limit]
 
-        render layout: false
+        case params[:mode]
+        when "selectize"
+          render json: @results.map { |res|
+            { text: resource_name(res), value: res.id }
+          }
+        else
+          render layout: false
+        end
       end
     end
   end

data/app/helpers/para/tag_helper.rb

@@ -21,7 +21,7 @@ module Para
       allow_adding_resource = options.fetch(:addable, true)
 
       partial = :list
-      partial = :tree if model.respond_to? :roots
+      partial = :tree if model.respond_to?(:roots) && can?(:tree, model)
 
       render(
         partial: find_partial_for(relation, partial),
@@ -48,6 +48,8 @@ module Para
     end
 
     def add_button_for(component, relation, model)
+      return unless can?(:create, model)
+
       partial_name = if component.subclassable?
         :subclassable_add_button
       else

data/app/views/para/admin/resources/_tree.html.haml

@@ -4,6 +4,10 @@
       = render partial: find_partial_for(relation, :actions), locals: { relation: relation, component: component, model: model, allow_adding_resource: allow_adding_resource }
 
     - if resources.length > 0
+      - if resources.total_count > 100
+        .page-entries-info
+          = page_entries(resources)
+
       %ul.tree.root-tree{ data: { url: component.relation_path(model.model_name.route_key, action: :tree), :"max-depth" => max_depth_for(model) } }
         = render partial: find_partial_for(model, 'tree_item'), collection: resources.roots, as: :root, locals: { model: model}
 

data/app/views/para/inputs/_nested_many.html.haml

@@ -1,10 +1,10 @@
 .nested-many-field{ class: [('orderable' if orderable), ('nested-many-field-inset' if inset)] }
   .fields-list{ id: dom_identifier }
     = form.simple_fields_for attribute_name, resources, nested_attribute_name: attribute_name, orderable: orderable, track_attribute_mappings: render_partial do |nested_form|
-      = render partial: find_partial_for(model, 'nested_many/container', partial_dir: 'inputs'), locals: { form: nested_form, model: nested_form.object.class, subclass: subclass, nested_locals: nested_locals, inset: inset, render_partial: render_partial, remote_partial_params: remote_partial_params }
+      = render partial: find_partial_for(model, 'nested_many/container', partial_dir: 'inputs'), locals: { form: nested_form, model: nested_form.object.class, subclass: subclass, nested_locals: nested_locals, inset: inset, uncollapsed: uncollapsed, render_partial: render_partial, remote_partial_params: remote_partial_params }
   -# Add button
   - if add_button
     - if subclass
-      = render partial: 'para/inputs/nested_many/add_with_subclasses', locals: { form: form, model: model, dom_identifier: dom_identifier, nested_locals: nested_locals, attribute_name: attribute_name, orderable: orderable, subclasses: subclasses, inset: inset, add_button_label: add_button_label, add_button_class: add_button_class, render_partial: render_partial, remote_partial_params: remote_partial_params }
+      = render partial: 'para/inputs/nested_many/add_with_subclasses', locals: { form: form, model: model, dom_identifier: dom_identifier, nested_locals: nested_locals, attribute_name: attribute_name, orderable: orderable, subclasses: subclasses, inset: inset, uncollapsed: uncollapsed, add_button_label: add_button_label, add_button_class: add_button_class, render_partial: render_partial, remote_partial_params: remote_partial_params }
     - else
-      = render partial: 'para/inputs/nested_many/add', locals: { form: form, model: model, dom_identifier: dom_identifier, nested_locals: nested_locals, attribute_name: attribute_name, orderable: orderable, inset: inset, add_button_label: add_button_label, add_button_class: add_button_class, render_partial: render_partial, remote_partial_params: remote_partial_params }
+      = render partial: 'para/inputs/nested_many/add', locals: { form: form, model: model, dom_identifier: dom_identifier, nested_locals: nested_locals, attribute_name: attribute_name, orderable: orderable, inset: inset, uncollapsed: uncollapsed, add_button_label: add_button_label, add_button_class: add_button_class, render_partial: render_partial, remote_partial_params: remote_partial_params }

data/app/views/para/inputs/nested_many/_add.html.haml

@@ -1,2 +1,2 @@
-= link_to_add_association form, attribute_name, partial: find_partial_for(model, 'nested_many/container', partial_dir: 'inputs'), form_name: 'form', class: "btn btn-shadow add-button nested-many-inset-add-button #{add_button_class}", data: { :'association-insertion-node' => "##{ dom_identifier }", :'association-insertion-method' => 'append' }, render_options: { nested_attribute_name: attribute_name, orderable: orderable, locals: { model: model, nested_locals: nested_locals, inset: inset, render_partial: render_partial, remote_partial_params: remote_partial_params } } do
+= link_to_add_association form, attribute_name, partial: find_partial_for(model, 'nested_many/container', partial_dir: 'inputs'), form_name: 'form', class: "btn btn-shadow add-button nested-many-inset-add-button #{add_button_class}", data: { :'association-insertion-node' => "##{ dom_identifier }", :'association-insertion-method' => 'append' }, render_options: { nested_attribute_name: attribute_name, orderable: orderable, locals: { model: model, nested_locals: nested_locals, inset: inset, uncollapsed: uncollapsed, render_partial: render_partial, remote_partial_params: remote_partial_params } } do
   = add_button_label

data/app/views/para/inputs/nested_many/_add_with_subclasses.html.haml

@@ -5,5 +5,5 @@
   %ul.dropdown-menu
     - subclasses.each do |submodel|
       %li
-        = link_to_add_association form, attribute_name, wrap_object: proc { submodel.new }, partial: find_partial_for(submodel, 'nested_many/container', partial_dir: 'inputs'), form_name: 'form', class: 'dropdown-link', data: { :'association-insertion-node' => "##{ dom_identifier }", :'association-insertion-method' => 'append' }, render_options: { nested_attribute_name: attribute_name, orderable: orderable, locals: { model: submodel, nested_locals: nested_locals, inset: inset, render_partial: render_partial, remote_partial_params: remote_partial_params } } do
+        = link_to_add_association form, attribute_name, wrap_object: proc { submodel.new }, partial: find_partial_for(submodel, 'nested_many/container', partial_dir: 'inputs'), form_name: 'form', class: 'dropdown-link', data: { :'association-insertion-node' => "##{ dom_identifier }", :'association-insertion-method' => 'append' }, render_options: { nested_attribute_name: attribute_name, orderable: orderable, locals: { model: submodel, nested_locals: nested_locals, inset: inset, uncollapsed: uncollapsed, render_partial: render_partial, remote_partial_params: remote_partial_params } } do
           = submodel.model_name.human

data/app/views/para/inputs/nested_many/_container.html.haml

@@ -11,7 +11,7 @@
 
     = form.remove_association_button
 
-  .panel-collapse.form-inputs.collapse{ id: form.nested_resource_dom_id, class: ('in' if inset && form.object.persisted?), data: { rendered: render_partial, render_path: @component.path(remote_partial_params), id: form.object.id, :"object-name" => form.object_name, :"model-name" => model.name } }
+  .panel-collapse.form-inputs.collapse{ id: form.nested_resource_dom_id, class: ('in' if uncollapsed && form.object.persisted?), data: { rendered: render_partial, render_path: @component.path(remote_partial_params), id: form.object.id, :"object-name" => form.object_name, :"model-name" => model.name } }
     .panel-body{ data: { :"nested-form-container" => true } }
       - if render_partial
         = render partial: find_partial_for(model, :fields), locals: { form: form }.merge(nested_locals)

data/lib/generators/para/install/templates/initializer.rb

@@ -27,9 +27,17 @@ Para.config do |config|
   # config.resource_name_methods += [:full_name]
 
   # Configure the theme navigation bar color not gray but blue darker
-  # 
+  #
   # config.dark_theme = true
 
+  # Do not uncollapse by default "inset" nested fields, which makes all such nested field
+  # blocks get collapsed on page load, and can be easier to read for big blocks.
+  #
+  # This also speeds up page loading when used in conjunction to remote partial loading
+  # for nested fields
+  #
+  # config.uncollapse_inset_nested_fields = false
+
   # Set if the Para::Breadcrumb::Controller module should be included into the
   # app, allowing easy breadcrumbs management in the app.
   #

data/lib/para/attribute_field/relation.rb

@@ -43,8 +43,12 @@ module Para
       end
 
       # Takes an array of ids and a block. Check for each id if model exists
-      # and create one if not. E.g: [12, "foo"] will try to create a model with
-      # 'foo'
+      # and create one if not.
+      #
+      # Example : [12, "foo"] will try to create a model with
+      #           'foo' as a name, title or any other attribute referenced in
+      #           the `Para.config.resource_name_methods` configuration param.
+      #
       def on_the_fly_creation ids, &block
         Array.wrap(ids).each do |id|
           if !reflection.klass.exists?(id: id)
@@ -56,7 +60,14 @@ module Para
               if resource.respond_to?(setter_name)
                 resource.send(setter_name, id)
 
-                if resource.save
+                # This check avoids multiple creation of the same resource with
+                # a given attribute value and ensure all resources from the form
+                # that reference that new resource name are associated to the
+                # same parent resource.
+                if (existing_resource = reflection.klass.find_by(method_name => id))
+                  block.call(existing_resource, id)
+                  break
+                elsif resource.save
                   block.call(resource, id)
                   break
                 end

data/lib/para/config.rb

@@ -42,6 +42,9 @@ module Para
     mattr_accessor :dark_theme
     @@dark_theme = false
 
+    mattr_accessor :uncollapse_inset_nested_fields
+    @@uncollapse_inset_nested_fields = true
+
     # Allows changing default cache store used by Para to store jobs through
     # the ActiveJob::Status gem
     #

data/lib/para/form_builder/containers.rb

@@ -27,7 +27,9 @@ module Para
       end
 
       def actions_buttons_for(options)
-        names = [:submit, :submit_and_edit, :submit_and_add_another, :cancel]
+        names = [:submit, :submit_and_edit]
+        names << :submit_and_add_another if template.can?(:create, object.class)
+        names << :cancel
 
         names.select! { |name| Array.wrap(options[:only]).include?(name) } if options[:only]
         names.reject! { |name| Array.wrap(options[:except]).include?(name) } if options[:except]

data/lib/para/inputs/nested_many_input.rb

@@ -32,6 +32,7 @@ module Para
               add_button_label: add_button_label,
               add_button_class: add_button_class,
               inset: inset?,
+              uncollapsed: uncollapsed?,
               render_partial: render_partial?,
               remote_partial_params: remote_partial_params
             }
@@ -50,9 +51,13 @@ module Para
       def inset?
         options.fetch(:inset, false)
       end
-      
+
+      def uncollapsed?
+        inset? && Para.config.uncollapse_inset_nested_fields
+      end
+
       def render_partial?
-        options[:render_partial] || object.errors.any? || (object.persisted? && inset?)
+        options[:render_partial] || object.errors.any? || (object.persisted? && uncollapsed?)
       end
 
       def remote_partial_params

data/lib/para/markup/resources_buttons.rb

@@ -2,14 +2,14 @@ module Para
   module Markup
     class ResourcesButtons < Para::Markup::Component
       attr_reader :component
-      
+
       def initialize(component, view)
         @component = component
         super(view)
       end
 
       def clone_button(resource)
-        return unless resource.class.cloneable?
+        return unless resource.class.cloneable? && view.can?(:clone, resource)
 
         path = component.relation_path(
           resource, action: :clone, return_to: view.request.fullpath
@@ -29,6 +29,8 @@ module Para
       end
 
       def edit_button(resource)
+        return unless view.can?(:edit, resource)
+
         path = component.relation_path(
           resource, action: :edit, return_to: view.request.fullpath
         )
@@ -39,6 +41,8 @@ module Para
       end
 
       def delete_button(resource)
+        return unless view.can?(:delete, resource)
+
         path = component.relation_path(resource, return_to: view.request.fullpath)
 
         options = {

data/lib/para/markup/resources_table.rb

@@ -5,7 +5,7 @@ module Para
       self.default_actions = [:edit, :clone, :delete]
 
       attr_reader :component, :model, :orderable, :actions
-      
+
       def initialize(component, view)
         @component = component
         super(view)
@@ -15,7 +15,7 @@ module Para
         @model = options.delete(:model)
 
         if !options.key?(:orderable) || options.delete(:orderable)
-          @orderable = model.orderable?
+          @orderable = model.orderable? && view.can?(:order, model)
         end
 
         @actions = build_actions(options.delete(:actions))
@@ -93,7 +93,7 @@ module Para
 
         content_tag(:th, options) do
           if (sort = options.delete(:sort))
-              view.sort_link(search, *sort, label, hide_indicator: true)
+            view.sort_link(search, *sort, label, hide_indicator: true)
           elsif searchable?(field_name)
             view.sort_link(search, field_name, label, hide_indicator: true)
           else