[3.0] has_one association reader returns Hash instead of model object

[wpc]

Found this on 3.0.0.alpha.11 with Cypher session type.

class Request
  include Neo4j::ActiveNode
  property :url
  has_one :out, :next, :model_class => "Request"
end

def test_next
   Neo4j::Transaction.run do
      req1 = Request.create(:url => "/foo")
      req2 = Request.create(:url => "/bar")
      req1.next = req2
      assert_equal req2, req1.next
   end
end

  1) Failure:
RequestTest#test_next
--- expected
+++ actual
@@ -1 +1 @@
-#<UserRequest url: "/foo">
+{"url"=>"/foo"}

Hope this is helpful.

[andreasronge]

Thanks. Yes there are limitations using transactions together with the Neo4j server. I will add this to the documentation. We can probably solve this if models uses id_property (a user defined primary key) since the neo4j server does not return the neo4j ids inside transactions.

[wpc]

Thanks for the suggestion. I will try use id_property

[andreasronge]

id_property will not solve your problem, but maybe you can do a workaround using the id which is available in the hash response. Yes, this is very awkward and might not solve your problem.
Hopefully we will have a solution for this in 3.1

[wpc]

At least I can use the id to find back node and provide a wrapper to the association method, right? Something in my mind:

alias :old_next :next
def next
   record = old_next
   Hash === record ? [find model by record's id property] : record
end

Or are you saying it is not even possible to read the node back once in transaction?

[wpc]

Sorry I accidentally close this. I guess you want to keep this issue open for 3.1.0 milestone.

[andreasronge]

Hmm, looks the workaround I was thinking of was not working.

Did a test and neo4j-core complained. I will investigate.

class FooPerson
  include Neo4j::ActiveNode
  id_property :uuid, auto: :uuid
  property :name
  has_one :out, :next, model_class: self
end

def test
  f1 = FooPerson.create name: 'f1'
  f2 = FooPerson.create name: 'f2'
  f1.next = f2
  n = f1.next
  id = n['uuid']
  puts "F1 #{n.inspect} id #{id}"
  nodes = Neo4j::Label.find_nodes(FooPerson.name, :uuid, id).to_a
  puts "FOUND #{nodes.inspect}"
end

Neo4j::Transaction.run { test }

[subvertallchris]

Realize this one is pretty old but all issues related to working within transactions should be resolved as of 3.0.1. @wpc please let me know if you're still having any trouble.