graph_starter 0.15.5 → 0.15.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5cc82c346899feb9018ae55485a4819f1c94c373
-  data.tar.gz: 0ed8f9e24fbece6fdc19a646ef9cefc46f9d7315
+  metadata.gz: 846d2b3c24ae439f3e2431a4cfc1662fdf320f47
+  data.tar.gz: 7d0bb849a37a60e4c77f5318288ee82282fecc5e
 SHA512:
-  metadata.gz: aa4f4da974616a2271132ae5db5fe4c621f5880e9965b7fdc2eccfc97ce845c202912b796289c94b769a939bfba257bb586171125ab3302123eb7abfea696a05
-  data.tar.gz: 1af8ab75b2d8a1bc739ce0e12b0cbbd298d1d35b5e02bb93c3f05bf739e57df97e44339b98bae7d9c93d84522825a164bb4dd0ddeba2a36f15250a750d5620ff
+  metadata.gz: 1f27817808730ca4c850a96bd5551d843b9fadb1be1d2c76de88dccce7e09da1701d6aa6c891e30aa631fd426a84418a3373bade7fe92481630ee73d31984de9
+  data.tar.gz: f58a50e1a8dd9b1749c76123d71c647db8c33d3f1fb6b69de20ed6f80dad8ebe875aa38b6729f1dc79e773b8b638075c0bcfc0a9286799114f87ddc5b3d66b1e

data/app/helpers/graph_starter/application_helper.rb

@@ -11,7 +11,7 @@ module GraphStarter
     def render_body(asset, model_slug)
       views = Dir.glob(Rails.root.join("app/views/#{model_slug}/_body.html.*"))
 
-      partial_path = views.present? ? "#{model_slug}/body" : 'body'
+      partial_path = views.present? ? "#{model_slug}/body" : '/graph_starter/assets/body'
 
       render partial: partial_path, locals: {asset: asset}
     end

data/app/models/graph_starter/asset.rb

@@ -1,4 +1,5 @@
 require 'graph_starter/query_authorizer'
+require 'babosa'
 
 module GraphStarter
   class Asset
@@ -11,6 +12,28 @@ module GraphStarter
 
     property :summary
 
+
+    # This is doing something strange, commenting out for now...
+    # def self.inherited(subclass)
+    #   subclass.property :slug
+    #   subclass.before_validation :place_slug
+    #   subclass.validates :slug, presence: true
+    #   subclass.constraint :slug, type: :unique
+    # end
+
+    def place_slug
+      return if self.slug.present?
+
+      name_value = read_attribute(self.class.name_property)
+      self.slug = self.class.unique_slug_from(name_value)
+        name_value.to_slug.normalize.to_s if name_value
+    end
+
+    def self.unique_slug_from(string)
+      base = string.to_slug.normalize.to_s
+    end
+
+
     if GraphStarter.configuration.user_class
       #has_many :in, :creators, type: :CREATED, model_class: GraphStarter.configuration.user_class
 
@@ -95,7 +118,7 @@ module GraphStarter
     def self.enumerable_property(property_name, values)
       fail "values needs to be an Array, was #{values.inspect}" if !values.is_a?(Array)
 
-      validates :status, inclusion: {in: values}
+      validates property_name.to_sym, inclusion: {in: values}
 
       enumerable_property_values[self.name.to_sym] ||= {}
       enumerable_property_values[self.name.to_sym][property_name.to_sym] ||= values

data/app/views/graph_starter/assets/_admin_buttons.html.slim

@@ -1,17 +1,18 @@
-.ui.buttons
-  a.ui.labeled.icon.button.right.floated data-authorizable="#{asset.to_json}"
-    | Permissions
+- if asset.persisted?
+  .ui.buttons
+    a.ui.labeled.icon.button.right.floated data-authorizable="#{asset.to_json}"
+      | Permissions
 
-  .or
+    .or
 
-  a.ui.labeled.icon.button.right.floated href="#{edit_asset_path(id: asset)}"
-    i.edit.icon
-    | Edit
+    a.ui.labeled.icon.button.right.floated href="#{graph_starter.edit_asset_path(id: asset)}"
+      i.edit.icon
+      | Edit
 
-.ui.divider
+  .ui.divider
 
-a.ui.labeled.icon.red.button.right.floated href="#{destroy_asset_path(id: asset)}"
-  i.delete.icon
-  | Delete
+  a.ui.labeled.icon.red.button.right.floated href="#{graph_starter.destroy_asset_path(id: asset)}"
+    i.delete.icon
+    | Delete
 
 

data/app/views/graph_starter/assets/_associations.html.slim

@@ -10,12 +10,12 @@
           - if as_cards
             .ui.link.cards
               - Array(result).each do |object|
-                = render partial: 'card', locals: {asset: object}
+                = render partial: '/graph_starter/assets/card', locals: {asset: object}
           - else
             .ui.middle.aligned.big.divided.list
               - Array(result).each do |object|
                 .item
                   .content
-                    = render partial: 'icon', locals: {asset: object}
+                    = render partial: '/graph_starter/assets/icon', locals: {asset: object}
                     = link_to object.title, asset_path(object)
 

data/app/views/graph_starter/assets/_body.html.slim

@@ -1,3 +1,3 @@
 - if asset.body.present?
   = asset.body
-= render partial: 'images', locals: {asset: asset}
+= render partial: '/graph_starter/assets/images', locals: {asset: asset}

data/app/views/graph_starter/assets/show.html.slim

@@ -14,24 +14,24 @@ javascript:
   .ui.grid
     - if asset_presenter.left_sidebar_exists?
       div class="#{left_width} wide column"
-        = render partial: 'dynamic_items', locals: {asset: @asset}
+        = render partial: '/graph_starter/assets/dynamic_items', locals: {asset: @asset}
 
     - if asset_presenter.main_column_exists?
       div class="#{center_width} wide column"
         - if asset_presenter.associations_in_body?
-          = render partial: 'associations', locals: {asset: @asset, as_cards: true}
+          = render partial: '/graph_starter/assets/associations', locals: {asset: @asset, as_cards: true}
         - else
           = render_body(@asset, @model_slug)
 
     div class="#{right_width} wide column" id="right-column"
       - if app_user_is_admin?
-        = render partial: 'admin_buttons', locals: {asset: @asset}
+        = render partial: '/graph_starter/assets/admin_buttons', locals: {asset: @asset}
 
       - if !asset_presenter.left_sidebar_exists?
         .ui.items
           - if !asset_presenter.associations_in_body?
-            = render partial: 'associations', locals: {asset: @asset}
-          = render partial: 'recommendations', locals: {asset: @asset}
+            = render partial: '/graph_starter/assets/associations', locals: {asset: @asset}
+          = render partial: '/graph_starter/assets/recommendations', locals: {asset: @asset}
 
-      = render partial: 'property_items', locals: {asset: @asset}
+      = render partial: '/graph_starter/assets/property_items', locals: {asset: @asset}
 

data/lib/graph_starter/version.rb

@@ -1,3 +1,3 @@
 module GraphStarter
-  VERSION = "0.15.5"
+  VERSION = "0.15.6"
 end

data/spec/asset_slug_spec.rb

@@ -0,0 +1,35 @@
+# require './spec/rails_helper'
+# 
+# describe 'Asset slugs' do
+#   class Foo < GraphStarter::Asset
+#     property :title
+# 
+#     name_property :title
+#   end
+# 
+#   before { clear_db }
+# 
+#   describe '.unique_slug_from' do
+#     subject { GraphStarter::Asset.unique_slug_from(string) }
+# 
+#     context 'Without assets with slugs' do
+#       let_context string: 'Test title' do
+#         it { should eq 'test-title' }
+#       end
+# 
+#       let_context string: 'Gölcük, Turkey Number 2!' do
+#         it { should eq 'golcuk-turkey-number-2' }
+#       end
+#     end
+# 
+#     context 'With existing asset slugs' do
+#       before { Foo.create(title: 'Gölcük, Turkey', slug: 'golcuk-turkey') }
+# 
+#       let_context string: 'Gölcük, Turkey!' do
+#         it { should eq 'golcuk-turkey-2' }
+#       end
+#     end
+# 
+#   end
+# end
+

data/spec/spec_helper.rb

@@ -1,3 +1,29 @@
+
+# Introduces `let_context` helper method
+# This allows us to simplify the case where we want to
+# have a context which contains one or more `let` statements
+module LetContextHelpers
+  # Supports giving either a Hash or a String and a Hash as arguments
+  # In both cases the Hash will be used to define `let` statements
+  # When a String is specified that becomes the context description
+  # If String isn't specified, Hash#inspect becomes the context description
+  def let_context(*args, &block)
+    classes = args.map(&:class)
+    context_string, hash =
+      case classes
+      when [String, Hash] then ["#{args[0]} #{args[1]}", args[1]]
+      when [Hash] then args + args
+      end
+
+    context(context_string) do
+      hash.each { |var, value| let(var) { value } }
+
+      instance_eval(&block)
+    end
+  end
+end
+
+
 # This file was generated by the `rails generate rspec:install` command. Conventionally, all
 # specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
 # The generated `.rspec` file contains `--require spec_helper` which will cause
@@ -40,6 +66,8 @@ RSpec.configure do |config|
     mocks.verify_partial_doubles = true
   end
 
+  config.extend LetContextHelpers
+
 # The settings below are suggested to provide a good initial experience
 # with RSpec, but feel free to customize to your heart's content.
 =begin

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: graph_starter
 version: !ruby/object:Gem::Version
-  version: 0.15.5
+  version: 0.15.6
 platform: ruby
 authors:
 - Brian Underwood
@@ -94,6 +94,20 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 0.0.3
+- !ruby/object:Gem::Dependency
+  name: babosa
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.0.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.0.2
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
@@ -223,6 +237,7 @@ files:
 - lib/graph_starter/version.rb
 - lib/tasks/graph_starter_tasks.rake
 - spec/asset_authorization_spec.rb
+- spec/asset_slug_spec.rb
 - spec/dummy/README.rdoc
 - spec/dummy/Rakefile
 - spec/dummy/app/assets/javascripts/application.js
@@ -289,6 +304,7 @@ specification_version: 4
 summary: A Rails engine to get a UI for a Neo4j up and running quickly
 test_files:
 - spec/asset_authorization_spec.rb
+- spec/asset_slug_spec.rb
 - spec/dummy/app/assets/javascripts/application.js
 - spec/dummy/app/assets/stylesheets/application.css
 - spec/dummy/app/controllers/application_controller.rb