neo4j 6.0.0.alpha.11 → 6.0.0.alpha.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cbd147ccfe5710b9dd99035774e8ccac71f1a2fe
-  data.tar.gz: 0afb13bbc63f47ce1658313f13541586cdbaca8b
+  metadata.gz: 88d57fe8f616166912b89e008686fea6190fee43
+  data.tar.gz: 83cbc8c78c8ae3a8571ed0870699f0537b018a84
 SHA512:
-  metadata.gz: ba9b1a7bed89c5e0d3be9349a527b496d0cd92a025b0602e68a89b38e4bcae5e45de4a667249abeb23c7fa63be7fcf26f152e48b117108eebcb6f38fe061729c
-  data.tar.gz: ccb77857f67f3bc4ac45087b10d71ef2117b6170ed437aae1593a77f4ff35549a709b76d9046430dd429419efc6cb829cd76e80e1e790b5faad49a454e6d88dd
+  metadata.gz: c13bb99309098ea662883902a7491cec10d32c9beb475a96825622e0a3b9e2f032b6d92eed15524613a796ab00da7407516241f5d655c1fc770ee1e0a321f70e
+  data.tar.gz: db933033b16e28f6282dd3e6b46f772587bfa670d125c26e2289f32bba771d93c66c5da3933fe1beb0fab1783495af1604c82ba616179f78179b4f2889a1a55e

data/CHANGELOG.md

@@ -3,6 +3,14 @@ All notable changes to this project will be documented in this file.
 This file should follow the standards specified on [http://keepachangelog.com/]
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## [6.0.0.alpha.12] - 11-5-2015
+
+### Changed
+- `_classname` property has been completely removed, officially dropping support for Neo4j < 2.1.5.
+
+### Added
+- Optional three-argument signature for `ActiveRel#create` and `#create!`, just like `initialize`.
+
 ## [6.0.0.alpha.11] - 11-3-2015
 
 ### Fixed

data/lib/neo4j/active_node.rb

@@ -41,6 +41,11 @@ module Neo4j
     include Neo4j::ActiveNode::Scope
     include Neo4j::ActiveNode::Dependent
 
+    def initialize(args = nil)
+      symbol_args = args.is_a?(Hash) ? args.symbolize_keys : args
+      super(symbol_args)
+    end
+
     def neo4j_obj
       _persisted_obj || fail('Tried to access native neo4j object on a non persisted object')
     end

data/lib/neo4j/active_node/has_n/association.rb

@@ -128,12 +128,6 @@ module Neo4j
           @relationship_class ||= @relationship_class_name && @relationship_class_name.constantize
         end
 
-        def inject_classname(properties)
-          return properties unless relationship_class
-          properties[Neo4j::Config.class_name_property] = relationship_class_name if relationship_class.cached_class?(true)
-          properties
-        end
-
         def unique?
           return relationship_class.unique? if rel_class?
           @origin ? origin_association.unique? : !!@unique

data/lib/neo4j/active_node/node_wrapper.rb

@@ -15,7 +15,7 @@ class Neo4j::Node
 
     def class_to_wrap
       load_classes_from_labels
-      (named_class || ::Neo4j::ActiveNode::Labels.model_for_labels(labels)).tap do |model_class|
+      Neo4j::ActiveNode::Labels.model_for_labels(labels).tap do |model_class|
         Neo4j::Node::Wrapper.populate_constants_for_labels_cache(model_class, labels)
       end
     end
@@ -48,11 +48,5 @@ class Neo4j::Node
     def self.association_model_namespace
       Neo4j::Config.association_model_namespace_string
     end
-
-    def named_class
-      property = Neo4j::Config.class_name_property
-
-      Neo4j::Node::Wrapper.constant_for_label(self.props[property]) if self.props.is_a?(Hash) && self.props.key?(property)
-    end
   end
 end

data/lib/neo4j/active_node/persistence.rb

@@ -130,9 +130,8 @@ module Neo4j::ActiveNode
 
       def find_or_create(find_attributes, set_attributes = {})
         on_create_attributes = set_attributes.merge(on_create_props(find_attributes))
-        on_match_attributes =  set_attributes.merge(on_match_props)
         neo4j_session.query.merge(n: {self.mapped_label_names => find_attributes})
-          .on_create_set(n: on_create_attributes).on_match_set(n: on_match_attributes)
+          .on_create_set(n: on_create_attributes)
           .pluck(:n).first
       end
 

data/lib/neo4j/active_node/query/query_proxy.rb

@@ -178,8 +178,6 @@ module Neo4j
           fail 'Can only create relationships on associations' if !@association
           other_nodes = _nodeify!(*other_nodes)
 
-          properties = @association.inject_classname(properties)
-
           Neo4j::Transaction.run do
             other_nodes.each do |other_node|
               other_node.save unless other_node.neo_id
@@ -221,7 +219,7 @@ module Neo4j
           Array(other_node_or_nodes).each do |other_node|
             node_props = (association.direction == :in) ? {from_node: other_node, to_node: @start_object} : {from_node: @start_object, to_node: other_node}
 
-            association.relationship_class.create(properties.except(:_classname).merge(node_props))
+            association.relationship_class.create(properties.merge(node_props))
           end
         end
 

data/lib/neo4j/active_rel.rb

@@ -19,7 +19,9 @@ module Neo4j
 
     def initialize(from_node = nil, to_node = nil, args = nil)
       load_nodes(node_or_nil(from_node), node_or_nil(to_node))
-      super(hash_or_nil(from_node, args))
+      resolved_args = hash_or_nil(from_node, args)
+      symbol_args = resolved_args.is_a?(Hash) ? resolved_args.symbolize_keys : resolved_args
+      super(symbol_args)
     end
 
     def node_cypher_representation(node)
@@ -53,5 +55,24 @@ module Neo4j
     def hash_or_nil(node_or_hash, hash_or_nil)
       node_or_hash.is_a?(Hash) ? node_or_hash : hash_or_nil
     end
+
+    module ClassMethods
+      [:create, :create!].each do |meth|
+        define_method(meth) do |from_node_or_args = nil, to_node = nil, args = nil|
+          return super(from_node_or_args) if from_node_or_args.is_a?(Hash)
+          args_hash = args || {}
+          args_with_node!(:from_node, from_node_or_args, args_hash)
+          args_with_node!(:to_node, to_node, args_hash)
+          super(args_hash)
+        end
+      end
+
+      private
+
+      def args_with_node!(key, node, args)
+        args[key] = node if node.is_a?(Neo4j::ActiveNode)
+        args
+      end
+    end
   end
 end

data/lib/neo4j/active_rel/rel_wrapper.rb

@@ -3,7 +3,7 @@ class Neo4j::Relationship
     def wrapper
       props.symbolize_keys!
       begin
-        most_concrete_class = sorted_wrapper_classes
+        most_concrete_class = class_from_type
         wrapped_rel = most_concrete_class.constantize.new
       rescue NameError
         return self
@@ -15,10 +15,6 @@ class Neo4j::Relationship
 
     private
 
-    def sorted_wrapper_classes
-      props[Neo4j::Config.class_name_property] || class_from_type
-    end
-
     def class_from_type
       Neo4j::ActiveRel::Types::WRAPPED_CLASSES[rel_type] || Neo4j::ActiveRel::Types::WRAPPED_CLASSES[rel_type] = rel_type.camelize
     end

data/lib/neo4j/active_rel/related_node.rb

@@ -29,7 +29,7 @@ module Neo4j::ActiveRel
 
     # Loads a node from the database or returns the node if already laoded
     def loaded
-      fail NilRelatedNodeError, 'Node not set, cannot load' if @node.nil?
+      fail UnsetRelatedNodeError, 'Node not set, cannot load' if @node.nil?
       @node = @node.respond_to?(:neo_id) ? @node : Neo4j::Node.load(@node)
     end
 

data/lib/neo4j/active_rel/types.rb

@@ -15,9 +15,6 @@ module Neo4j
       #
       # A model is added to WRAPPED_CLASSES when it is initalized AND when the `type` class method is called within a model. This means that
       # it's possible a model will be added twice: once with the rel_type version of the model name, again with the custom type. deal_with_it.gif.
-      #
-      # As an alternative to this, you can call the `set_classname` class method to insert a `_classname` property into your relationship,
-      # which will completely bypass this whole process.
       WRAPPED_CLASSES = {}
 
       included do
@@ -76,7 +73,7 @@ module Neo4j
 
         def assign_type!(given_type, auto)
           @rel_type = (auto ? decorated_rel_type(given_type) : given_type).tap do |type|
-            add_wrapped_class(type) unless uses_classname?
+            add_wrapped_class(type)
           end
         end
       end

data/lib/neo4j/config.rb

@@ -96,10 +96,6 @@ module Neo4j
         configuration.to_yaml
       end
 
-      def class_name_property
-        @_class_name_property = Neo4j::Config[CLASS_NAME_PROPERTY_KEY] || :_classname
-      end
-
       def include_root_in_json
         # we use ternary because a simple || will always evaluate true
         Neo4j::Config[:include_root_in_json].nil? ? true : Neo4j::Config[:include_root_in_json]

data/lib/neo4j/migration.rb

@@ -123,97 +123,5 @@ MESSAGE
         end
       end
     end
-
-    class AddClassnames < Neo4j::Migration
-      def initialize(path = default_path)
-        @classnames_filename = 'add_classnames.yml'
-        @classnames_filepath = File.join(joined_path(path), classnames_filename)
-      end
-
-      def migrate
-        output 'Adding classnames. This make take some time.'
-        execute(true)
-      end
-
-      def test
-        output 'TESTING! No queries will be executed.'
-        execute(false)
-      end
-
-      def setup
-        output "Creating file #{classnames_filepath}. Please use this as the migration guide."
-        FileUtils.mkdir_p('db/neo4j-migrate')
-
-        return if File.file?(classnames_filepath)
-
-        source = File.join(File.dirname(__FILE__), '..', '..', 'config', 'neo4j', classnames_filename)
-        FileUtils.copy_file(source, classnames_filepath)
-      end
-
-      private
-
-      attr_reader :classnames_filename, :classnames_filepath, :model_map
-
-      def execute(migrate = false)
-        file_init
-        map = []
-        map.push :nodes         if model_map[:nodes]
-        map.push :relationships if model_map[:relationships]
-        map.each do |type|
-          model_map[type].each do |action, labels|
-            do_classnames(action, labels, type, migrate)
-          end
-        end
-      end
-
-      def do_classnames(action, labels, type, migrate = false)
-        method = type == :nodes ? :node_cypher : :rel_cypher
-        labels.each do |label|
-          output cypher = self.send(method, label, action)
-          execute_cypher(cypher) if migrate
-        end
-      end
-
-      def file_init
-        @model_map = ActiveSupport::HashWithIndifferentAccess.new(YAML.load_file(classnames_filepath))
-      end
-
-      def node_cypher(label, action)
-        where, phrase_start = action_variables(action, 'n')
-        output "#{phrase_start} _classname '#{label}' on nodes with matching label:"
-        "MATCH (n:`#{label}`) #{where} SET n._classname = '#{label}' RETURN COUNT(n) as modified"
-      end
-
-      def rel_cypher(hash, action)
-        label = hash[0]
-        value = hash[1]
-        from = value[:from]
-        fail "All relationships require a 'type'" unless value[:type]
-
-        from_cypher = from ? "(from:`#{from}`)" : '(from)'
-        to = value[:to]
-        to_cypher = to ? "(to:`#{to}`)" : '(to)'
-        type = "[r:`#{value[:type]}`]"
-        where, phrase_start = action_variables(action, 'r')
-        output "#{phrase_start} _classname '#{label}' where type is '#{value[:type]}' using cypher:"
-        "MATCH #{from_cypher}-#{type}->#{to_cypher} #{where} SET r._classname = '#{label}' return COUNT(r) as modified"
-      end
-
-      def execute_cypher(query_string)
-        output "Modified #{Neo4j::Session.query(query_string).first.modified} records"
-        output ''
-      end
-
-      def action_variables(action, identifier)
-        case action
-        when 'overwrite'
-          ['', 'Overwriting']
-        when 'add'
-          ["WHERE NOT HAS(#{identifier}._classname)", 'Adding']
-        else
-          fail "Invalid action #{action} specified"
-        end
-      end
-    end
   end
 end

data/lib/neo4j/shared/persistence.rb

@@ -2,8 +2,6 @@ module Neo4j::Shared
   module Persistence
     extend ActiveSupport::Concern
 
-    USES_CLASSNAME = []
-
     # @return [Hash] Given a node's state, will call the appropriate `props_for_{action}` method.
     def props_for_persistence
       _persisted_obj ? props_for_update : props_for_create
@@ -18,7 +16,6 @@ module Neo4j::Shared
     # Returns a hash containing:
     # * All properties and values for insertion in the database
     # * A `uuid` (or equivalent) key and value
-    # * A `_classname` property, if one is to be set
     # * Timestamps, if the class is set to include them.
     # Note that the UUID is added to the hash but is not set on the node.
     # The timestamps, by comparison, are set on the node prior to addition in this hash.
@@ -27,7 +24,6 @@ module Neo4j::Shared
       inject_timestamps!
       props_with_defaults = inject_defaults!(props)
       converted_props = props_for_db(props_with_defaults)
-      inject_classname!(converted_props)
       return converted_props unless self.class.respond_to?(:default_property_values)
       inject_primary_key!(converted_props)
     end
@@ -187,61 +183,16 @@ module Neo4j::Shared
       self.updated_at = DateTime.now if respond_to?(:updated_at=) && (updated_at.nil? || (changed? && !updated_at_changed?))
     end
 
-    # Inserts the _classname property into an object's properties during object creation.
-    def inject_classname!(props, check_version = true)
-      props[:_classname] = self.class.name if self.class.cached_class?(check_version)
-    end
-
-    def set_classname(props, check_version = true)
-      warning = 'This method has been replaced with `inject_classname!` and will be removed in a future version'.freeze
-      ActiveSupport::Deprecation.warn warning, caller
-      inject_classname!(props, check_version)
-    end
-
     def inject_timestamps!
       now = DateTime.now
       self.created_at ||= now if respond_to?(:created_at=)
       self.updated_at ||= now if respond_to?(:updated_at=)
     end
 
-
-
     def set_timestamps
       warning = 'This method has been replaced with `inject_timestamps!` and will be removed in a future version'.freeze
       ActiveSupport::Deprecation.warn warning, caller
       inject_timestamps!
     end
-
-    module ClassMethods
-      # Determines whether a model should insert a _classname property. This can be used to override the automatic matching of returned
-      # objects to models.
-      def cached_class?(check_version = true)
-        uses_classname? || (!!Neo4j::Config[:cache_class_names] && (check_version ? neo4j_session.version < '2.1.5' : true))
-      end
-
-      # @return [Boolean] status of whether this model will add a _classname property
-      def uses_classname?
-        Neo4j::Shared::Persistence::USES_CLASSNAME.include?(self.name)
-      end
-
-      # Adds this model to the USES_CLASSNAME array. When new rels/nodes are created, a _classname property will be added. This will override the
-      # automatic matching of label/rel type to model.
-      #
-      # You'd want to do this if you have multiple models for the same label or relationship type. When it comes to labels, there isn't really any
-      # reason to do this because you can have multiple labels; on the other hand, an argument can be made for doing this with relationships since
-      # rel type is a bit more restrictive.
-      #
-      # It could also be speculated that there's a slight performance boost to using _classname since the gem immediately knows what model is responsible
-      # for a returned object. At the same time, it is a bit restrictive and changing it can be a bit of a PITA. Use carefully!
-      def set_classname
-        Neo4j::Shared::Persistence::USES_CLASSNAME << self.name
-      end
-
-      # Removes this model from the USES_CLASSNAME array. When new rels/nodes are create, no _classname property will be injected. Upon returning of
-      # the object from the database, it will be matched to a model using its relationship type or labels.
-      def unset_classname
-        Neo4j::Shared::Persistence::USES_CLASSNAME.delete self.name
-      end
-    end
   end
 end

data/lib/neo4j/shared/property.rb

@@ -84,13 +84,15 @@ module Neo4j::Shared
       end
     end
 
+    DATE_KEY_REGEX = /\A([^\(]+)\((\d+)([if])\)$/
     # Gives support for Rails date_select, datetime_select, time_select helpers.
     def process_attributes(attributes = nil)
       return attributes if attributes.blank?
       multi_parameter_attributes = {}
       new_attributes = {}
       attributes.each_pair do |key, value|
-        if match = key.match(/\A([^\(]+)\((\d+)([if])\)$/)
+        if key.match(DATE_KEY_REGEX)
+          match = key.to_s.match(DATE_KEY_REGEX)
           found_key = match[1]
           index = match[2].to_i
           (multi_parameter_attributes[found_key] ||= {})[index] = value.empty? ? nil : value.send("to_#{$3}")
@@ -105,7 +107,6 @@ module Neo4j::Shared
     def process_multiparameter_attributes(multi_parameter_attributes, new_attributes)
       multi_parameter_attributes.each_with_object(new_attributes) do |(key, values), attributes|
         values = (values.keys.min..values.keys.max).map { |i| values[i] }
-
         if (field = self.class.attributes[key.to_sym]).nil?
           fail MultiparameterAssignmentError, "error on assignment #{values.inspect} to #{key}"
         end

data/lib/neo4j/version.rb

@@ -1,3 +1,3 @@
 module Neo4j
-  VERSION = '6.0.0.alpha.11'
+  VERSION = '6.0.0.alpha.12'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: neo4j
 version: !ruby/object:Gem::Version
-  version: 6.0.0.alpha.11
+  version: 6.0.0.alpha.12
 platform: ruby
 authors:
 - Andreas Ronge, Brian Underwood, Chris Grigg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-11-03 00:00:00.000000000 Z
+date: 2015-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: orm_adapter