Forum: Ruby on Rails Creating/Updating Models from text/xml Requests

Announcement (2017-05-07): is now read-only since I unfortunately do not have the time to support and maintain the forum any more. Please see and for other Rails- und Ruby-related community platforms.
MaggotChild (Guest)
on 2009-03-14 03:34
(Received via mailing list)
I'm curious how others would deal with this situation.

In my case, I have:

class Package
  has_many :deliveries

class Delivery
  belongs_to :package
  belongs_to :region

class Region
  has_many :deliveries

And accept XML requests in the following manner:

  <name>Good Shipment<name>
  <!-- snip other fields -->

Now, my controller would look something like this ugly beast:

class PackagesController
  def create
    if package = params[:package]
      package = Package.find_or_initialize_by_name(package)
      if deliveries = package.delete(:deliveries)
         deliveries.each do |d|
             region = d.delete(:region)
             region = Region.find_or_create_by_name(region[:name])
             break if region.nil?
             package.deliveries[-1].region = region
    #save and more...

As one can see, this is horrible for several reasons. One of the big
ones being request validation and error handling.

Overriding package.from_xml  is an option, but I find myself thinking
that I should be validating against request.body with say a DTD, as
opposed to picking apart the params hash. (Though I'll have to pick it
apart anyways to create the models...)

Then I find my self thinking that creating a course grained validation
method like a DTD is stupid since I have my fine grain validation
logic within the given models.

Yes, I've heard of ROXML.

Any thoughts?
Phlip (Guest)
on 2009-03-14 04:06
(Received via mailing list)
MaggotChild wrote:

> And accept XML requests in the following manner:
> <package>
>   <name>Good Shipment<name>
>   <!-- snip other fields -->
>   <deliveries>
>     <delivery>
>       <status>Failure</status>
>       <region>

If you try this from_xml...

...I think it would clear the make-work out of the way, allowing you to
focus on
the validations. Tell me if you get stuck - I wrote it for your exact
but it can't validate automatically.
This topic is locked and can not be replied to.