neo4j 3.0.0.rc.5 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 26a95704c598f6b1e39d5a05d8cebc03ef2e4705
-  data.tar.gz: b8246ee70c32ae137eaff9bc169ef14646d52868
+  metadata.gz: 8c7dc4797f55611d07474d42b4618485691dc613
+  data.tar.gz: 8b8dd2f9eee185c7a157eeba14afcbe8cfbf043c
 SHA512:
-  metadata.gz: afc6ef946ce72a7bf88afa316218f2e78155fdc269f91d795d6c12e1d9d5edb90d409374cdf746872f5dd1e585c8535c00fe0180522b5ab14b3bc37511ab870c
-  data.tar.gz: 8733527ffa47beccdf15477380a953c8f8789b854df097f656f6a00736ee5a02627632461c8b79f7d66f6e0ff7c5148604516d034db4c8b4b7ba5418ed71b049
+  metadata.gz: 97484101ffda9dc168e5f1ce39bde7fc28f019231dc9cfa82ae74dbc49fc75942279882023da33329453644d3734ea072415d9996fdeb6089f404e952f2aa262
+  data.tar.gz: 2770dacf06cbb2816acf892c0e162e3049ff18af305013b358ab5b850cf20462a556b3197760ed128aad870d57f362302296b00c32906325b23b963ece0419a7

data/CHANGELOG

@@ -1,3 +1,6 @@
+== 3.0.0
+No change from rc 4
+
 == 3.0.0.rc.4
 * UUIDs are now automatically specified on models as neo IDs won't be reliable
 in future versions of neo4j

data/lib/neo4j/active_node.rb

@@ -67,7 +67,7 @@ module Neo4j
       end
 
       Neo4j::Session.on_session_available do |_|
-        id_property :uuid, auto: :uuid
+        id_property :uuid, auto: :uuid unless self.has_id_property?
 
         name = Neo4j::Config[:id_property]
         type = Neo4j::Config[:id_property_type]

data/lib/neo4j/active_node/id_property.rb

@@ -42,12 +42,8 @@ module Neo4j::ActiveNode
 
       def validate_conf(conf)
         return if conf.empty?
-
         raise "Expected a Hash, got #{conf.class} (#{conf.to_s}) for id_property" unless conf.is_a?(Hash)
-
-        unless conf.include?(:auto) || conf.include?(:on)
-          raise "Illegal value #{conf.inspect} for id_property, expected :on or :auto"
-        end
+        raise "Illegal value #{conf.inspect} for id_property, expected :on or :auto" unless conf.include?(:auto) || conf.include?(:on)
       end
 
       def define_property_method(clazz, name)

data/lib/neo4j/active_node/labels.rb

@@ -86,7 +86,7 @@ module Neo4j
           self.query_as(:n).where(n: eval(args.join)).limit(1).pluck(:n).first
         end
 
-        # Like find_by, except that if no record is found, raises a RecordNotFound error. 
+        # Like find_by, except that if no record is found, raises a RecordNotFound error.
         def find_by!(*args)
           a = eval(args.join)
           find_by(args) or raise RecordNotFound, "#{self.query_as(:n).where(n: a).limit(1).to_cypher} returned no results"

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

@@ -37,7 +37,7 @@ module Neo4j
         def include?(other, target=nil)
           raise(InvalidParameterError, ':include? only accepts nodes') unless other.respond_to?(:neo_id)
           query_with_target(target) do |target|
-            self.query.where(target => {@model.primary_key => other.id}).return("count(#{target}) AS count").first.count > 0
+            self.where("#{target}.#{other.class.primary_key} = {other_node_id}").params(other_node_id: other.id).query.return("count(#{target}) as count").first.count > 0
           end
         end
 

data/lib/neo4j/active_node/query_methods.rb

@@ -44,10 +44,10 @@ module Neo4j
       private
 
       def exists_query_start(node_condition)
-        case
-        when node_condition.class == Fixnum
+        case node_condition
+        when Fixnum
           self.query_as(:n).where("ID(n) = #{node_condition}")
-        when node_condition.class == Hash
+        when Hash
           self.where(node_condition.keys.first => node_condition.values.first)
         else
           self.query_as(:n)

data/lib/neo4j/active_node/scope.rb

@@ -61,7 +61,7 @@ module Neo4j::ActiveNode
 
       def _call_scope_context(eval_context, query_params, proc)
         if proc.arity == 1
-          eval_context.instance_exec(query_params,&proc)
+          eval_context.instance_exec(query_params, &proc)
         else
           eval_context.instance_exec(&proc)
         end

data/lib/neo4j/active_node/validations.rb

@@ -31,19 +31,12 @@ module Neo4j
           # TODO: Added as find(:name => nil) throws error
           value = "" if value == nil
 
-          if options[:case_sensitive]
-            conditions[attribute] = value
-          else
-            conditions[attribute] = /^#{Regexp.escape(value.to_s)}$/i
-          end
+          conditions[attribute] = options[:case_sensitive] ? value : /^#{Regexp.escape(value.to_s)}$/i
 
           # prevent that same object is returned
           # TODO: add negative condtion to not return current record
           found = record.class.where(conditions).to_a.select{|f| f.neo_id != record.neo_id}
-
-          if found.count > 0
-            record.errors.add(attribute, :taken, options.except(:case_sensitive, :scope).merge(:value => value))
-          end
+          record.errors.add(attribute, :taken, options.except(:case_sensitive, :scope).merge(:value => value)) if found.count > 0
         end
 
         def message(instance)

data/lib/neo4j/active_rel/persistence.rb

@@ -13,9 +13,7 @@ module Neo4j::ActiveRel
     end
 
     def save!(*args)
-      unless save(*args)
-        raise RelInvalidError.new(self)
-      end
+      raise RelInvalidError.new(self) unless save(*args)
     end
 
     def create_model(*)
@@ -44,21 +42,17 @@ module Neo4j::ActiveRel
 
       # Same as #create, but raises an error if there is a problem during save.
       def create!(*args)
-        unless create(*args)
-          raise RelInvalidError.new(self)
-        end
+        raise RelInvalidError.new(self) unless create(*args)
       end
     end
 
-    private 
+    private
 
     def confirm_node_classes
       [from_node, to_node].each do |node|
         type = from_node == node ? :_from_class : :_to_class
         next if allows_any_class?(type)
-        unless class_as_constant(type) == node.class
-          raise ModelClassInvalidError, "Node class was #{node.class}, expected #{self.class.send(type)}"
-        end
+        raise ModelClassInvalidError, "Node class was #{node.class}, expected #{self.class.send(type)}" unless class_as_constant(type) == node.class
       end
     end
 
@@ -71,10 +65,10 @@ module Neo4j::ActiveRel
 
     def class_as_constant(type)
       given_class = self.class.send(type)
-      case
-      when given_class.is_a?(String)
+      case given_class
+      when String
         given_class.constantize
-      when given_class.is_a?(Symbol)
+      when Symbol
         given_class.to_s.constantize
       else
         given_class

data/lib/neo4j/active_rel/query.rb

@@ -54,7 +54,7 @@ module Neo4j::ActiveRel
         when :inbound
           identifier = '(n2'
           identifier + (_to_class == :any ? ')' : cypher_label(:inbound))
-        end 
+        end
       end
 
       def cypher_label(dir = :outbound)
@@ -63,10 +63,10 @@ module Neo4j::ActiveRel
       end
 
       def as_constant(given_class)
-        case
-        when given_class.is_a?(String)
+        case given_class
+        when String
           given_class.constantize
-        when given_class.is_a?(Symbol)
+        when Symbol
           given_class.to_s.constantize
         else
           given_class
@@ -74,11 +74,10 @@ module Neo4j::ActiveRel
       end
 
       def where_string(args)
-        if args.is_a?(Hash)
-          args.map do |k, v|
-            v.is_a?(Integer) ? "r1.#{k} = #{v}" : "r1.#{k} = '#{v}'"
-          end.join(', ')
-        else 
+        case args
+        when Hash
+          args.map { |k, v| v.is_a?(Integer) ? "r1.#{k} = #{v}" : "r1.#{k} = '#{v}'" }.join(', ')
+        else
           args
         end
       end

data/lib/neo4j/active_rel/rel_wrapper.rb

@@ -5,7 +5,7 @@ class Neo4j::Relationship
       props.symbolize_keys!
       return self unless props.is_a?(Hash) && props.has_key?(Neo4j::Config.class_name_property)
       begin
-      found_class = props[Neo4j::Config.class_name_property].constantize
+        found_class = props[Neo4j::Config.class_name_property].constantize
       rescue NameError
         return self
       end

data/lib/neo4j/migration.rb

@@ -1,5 +1,18 @@
 module Neo4j
   class Migration
+
+    def migrate
+      raise 'not implemented'
+    end
+
+    def output(string = '')
+      puts string unless !!ENV['silenced']
+    end
+
+    def print_output(string)
+      print string unless !!ENV['silenced']
+    end
+
     class AddIdProperty < Neo4j::Migration
       attr_reader :models_filename
 
@@ -9,12 +22,12 @@ module Neo4j
 
       def migrate
         models = ActiveSupport::HashWithIndifferentAccess.new(YAML.load_file(models_filename))[:models]
-        puts "This task will add an ID Property every node in the given file."
-        puts "It may take a significant amount of time, please be patient."
+        output "This task will add an ID Property every node in the given file."
+        output "It may take a significant amount of time, please be patient."
         models.each do |model|
-          puts
-          puts
-          puts "Adding IDs to #{model}"
+          output
+          output
+          output "Adding IDs to #{model}"
           add_ids_to model.constantize
         end
       end
@@ -44,10 +57,10 @@ module Neo4j
           nodes_left = Neo4j::Session.query.match(n: label).where("NOT has(n.#{property})").return("COUNT(n) AS ids").first.ids
 
           time_per_node = last_time_taken / max_per_batch if last_time_taken
-          print "Running first batch...\r"
+          print_output "Running first batch...\r"
           if time_per_node
             eta_seconds = (nodes_left * time_per_node).round
-            print "#{nodes_left} nodes left.  Last batch: #{(time_per_node * 1000.0).round(1)}ms / node (ETA: #{eta_seconds / 60} minutes)\r"
+            print_output "#{nodes_left} nodes left.  Last batch: #{(time_per_node * 1000.0).round(1)}ms / node (ETA: #{eta_seconds / 60} minutes)\r"
           end
 
           return if nodes_left == 0
@@ -58,7 +71,7 @@ module Neo4j
             last_time_taken = id_batch_set(label, property, new_ids, to_set)
           rescue Neo4j::Server::CypherResponse::ResponseError, Faraday::TimeoutError
             new_max_per_batch = (max_per_batch * 0.8).round
-            puts "Error querying #{max_per_batch} nodes.  Trying #{new_max_per_batch}"
+            output "Error querying #{max_per_batch} nodes.  Trying #{new_max_per_batch}"
             max_per_batch = new_max_per_batch
           end
         end
@@ -92,7 +105,6 @@ module Neo4j
     end
 
     class AddClassnames < Neo4j::Migration
-      attr_reader :classnames_filename, :classnames_filepath
 
       def initialize
         @classnames_filename = 'add_classnames.yml'
@@ -100,33 +112,34 @@ module Neo4j
       end
 
       def migrate
-        puts "Adding classnames. This make take some time."
+        output "Adding classnames. This make take some time."
         execute(true)
       end
 
       def test
-        puts "TESTING! No queries will be executed."
+        output "TESTING! No queries will be executed."
         execute(false)
       end
 
       def setup
-        puts "Creating file #{classnames_filepath}. Please use this as the migration guide."
+        output "Creating file #{classnames_filepath}. Please use this as the migration guide."
         FileUtils.mkdir_p("db/neo4j-migrate")
-        unless File.file?(@classnames_filepath)
+        unless File.file?(classnames_filepath)
           source = File.join(File.dirname(__FILE__), "..", "..", "config", "neo4j", classnames_filename)
           FileUtils.copy_file(source, classnames_filepath)
         end
       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.push :nodes         if model_map[:nodes]
+        map.push :relationships if model_map[:relationships]
         map.each do |type|
-          @model_map[type].each do |action, labels|
+          model_map[type].each do |action, labels|
             do_classnames(action, labels, type, migrate)
           end
         end
@@ -135,7 +148,7 @@ module Neo4j
       def do_classnames(action, labels, type, migrate = false)
         method = type == :nodes ? :node_cypher : :rel_cypher
         labels.each do |label|
-          puts cypher = self.send(method, label, action)
+          output cypher = self.send(method, label, action)
           execute_cypher(cypher) if migrate
         end
       end
@@ -146,7 +159,7 @@ module Neo4j
 
       def node_cypher(label, action)
         where, phrase_start = action_variables(action, 'n')
-        puts "#{phrase_start} _classname '#{label}' on nodes with matching label:"
+        output "#{phrase_start} _classname '#{label}' on nodes with matching label:"
         "MATCH (n:`#{label}`) #{where} SET n._classname = '#{label}' RETURN COUNT(n) as modified"
       end
 
@@ -161,13 +174,13 @@ module Neo4j
         to_cypher = to ? "(to:`#{to}`)" : "(to)"
         type = "[r:`#{value[:type]}`]"
         where, phrase_start = action_variables(action, 'r')
-        puts "#{phrase_start} _classname '#{label}' where type is '#{value[:type]}' using cypher:"
+        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)
-        puts "Modified #{Neo4j::Session.query(query_string).first.modified} records"
-        puts ""
+        output "Modified #{Neo4j::Session.query(query_string).first.modified} records"
+        output ""
       end
 
       def action_variables(action, identifier)

data/lib/neo4j/shared/property.rb

@@ -108,11 +108,7 @@ module Neo4j::Shared
       return nil if values_with_empty_parameters.all? { |v| v.nil? }
       values = values_with_empty_parameters.collect { |v| v.nil? ? 1 : v }
       klass = field[:type]
-      if klass
-        klass.new(*values)
-      else
-        values
-      end
+      klass ? klass.new(*values) : values
     end
 
     module ClassMethods
@@ -163,6 +159,7 @@ module Neo4j::Shared
       end
 
       def default_property(name, &block)
+        reset_default_properties(name) if default_properties.respond_to?(:size)
         default_properties[name] = block
       end
 
@@ -171,6 +168,13 @@ module Neo4j::Shared
         @default_property ||= {}
       end
 
+      def reset_default_properties(name_to_keep)
+        default_properties.each_key do |property|
+          undef_method(property) unless property == name_to_keep
+        end
+        @default_property = {}
+      end
+
       def default_property_values(instance)
         default_properties.each_with_object({}) do |(key, block),result|
           result[key] = block.call(instance)

data/lib/neo4j/shared/validations.rb

@@ -28,13 +28,13 @@ module Neo4j
         super(context)
         errors.empty?
       end
-      
+
       private
-      
+
       def perform_validations(options={})
         perform_validation = case options
-                               when Hash
-                                 options[:validate] != false
+                             when Hash
+                               options[:validate] != false
                              end
 
         if perform_validation

data/lib/neo4j/version.rb

@@ -1,3 +1,3 @@
 module Neo4j
-  VERSION = "3.0.0.rc.5"
+  VERSION = "3.0.0"
 end

data/neo4j.gemspec

@@ -31,7 +31,7 @@ A Neo4j OGM for use in Ruby on Rails and Rack frameworks, intended as a complete
   s.add_dependency("activesupport", "~> 4")
   s.add_dependency("railties", "~> 4")
   s.add_dependency('active_attr', "~> 0.8")
-  s.add_dependency("neo4j-core", "= 3.0.0.rc.5")
+  s.add_dependency("neo4j-core", "= 3.0.0")
 
   if RUBY_PLATFORM =~ /java/
     s.add_dependency("neo4j-community", '~> 2.0')

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: neo4j
 version: !ruby/object:Gem::Version
-  version: 3.0.0.rc.5
+  version: 3.0.0
 platform: ruby
 authors:
 - Andreas Ronge
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-23 00:00:00.000000000 Z
+date: 2014-09-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: orm_adapter
@@ -86,14 +86,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.0.0.rc.5
+        version: 3.0.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 3.0.0.rc.5
+        version: 3.0.0
 description: |
   A Neo4j OGM for use in Ruby on Rails and Rack frameworks, intended as a complete replacement for ActiveRecord.
 email: andreas.ronge@gmail.com
@@ -182,9 +182,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 1.9.1
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: neo4j
 rubygems_version: 2.2.2