Life is grand - Latest Comments in A dirty Ruby

Re: A dirty Ruby

Paul Watson — Thu, 11 May 2006 06:54:08 -0000

How fantastic. I took the above code and cut it down to just this:
if soapResponse.getNewTagsSinceResult.respond_to? "tag"   for tag_current in Array(soapResponse.getNewTagsSinceResult.tag)       tag = Tag.find_by_sync_id(tag_current.id) || Tag.new     tag.name = tag_current.name     tag.sync_id = tag_current.id     tag.save   end end

Thanks Farrel.

Re: A dirty Ruby

Farrel Lifson — Thu, 11 May 2006 03:41:51 -0000

I did some further experimenting and it seems it's recommended to use Array(object) instead of object.to_a.

Re: A dirty Ruby

Paul Watson — Thu, 11 May 2006 03:17:36 -0000

Thank you for those two comments Farrel. The last one is especially interesting. I'll give the to_a trick a try.

Re: A dirty Ruby

Farrel Lifson — Wed, 10 May 2006 16:07:49 -0000

Also if you really want to squeeze things down you can do this:

@tag = Tag.find_by_syn_id(tag.id) || Tag.new
@tag.name = tag.name
@tag.sync_id ||= tag.id
@tag.save

Re: A dirty Ruby

Farrel Lifson — Wed, 10 May 2006 15:18:22 -0000

I haven't used WSDLDriver but doe the tag object respond to to_a? If so you can do the following:

if soapResponse.getNewTagsSinceResult.respond_to? "tag"
for tag in soapResponse.getNewTagsSinceResult.tag.to_a
@tag = Tag.find_by_sync_id(tag.id)
if (@tag)
@tag.name = tag.name
@tag.save
else
@tag = Tag.new
@tag.name = tag.name
@tag.sync_id = tag.id
@tag.save
end
end
end

Just a quick warning that currently all objects respond to to_a but this will soon be deprecated so for future versions it's better to check that the tag object class explicity implements to_a.