Skip to content
This repository has been archived by the owner on Apr 17, 2018. It is now read-only.

issue solved when storage_name is different than class name #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sdesbure
Copy link

Before the patch, dm-rest is displaying the resource name accordingly to the model name and not the storage name.
For example, given this class:
Class BookResource
include DataMapper::Resource
storage_names[:default] = 'books'

      property :id,         Serial
      property :created_at, DateTime
      property :title,      String
      property :author,     String
    end

a POST or a PUT of this class would be:

1
<created_at type='datetime'>2009-05-17T22:38:42-07:00</created_at>
<title>DataMapper</title>
Dan Kubb

instead of the waited XML:

1
<created_at type='datetime'>2009-05-17T22:38:42-07:00</created_at>
<title>DataMapper</title>
Dan Kubb

This patch solve the problem and makes two tests to verify (one for create, one for update)

… to the model name and not the storage name.

For example, given this class:
    Class BookResource
      include DataMapper::Resource
      storage_names[:default] = 'books'

      property :id,         Serial
      property :created_at, DateTime
      property :title,      String
      property :author,     String
    end
a POST or a PUT of this class would be:
    <book-resource>
      <id type='datamapper::types::serial'>1</id>
      <created_at type='datetime'>2009-05-17T22:38:42-07:00</created_at>
      <title>DataMapper</title>
      <author>Dan Kubb</author>
    </book-resource>
instead of the waited XML:
    <book>
      <id type='datamapper::types::serial'>1</id>
      <created_at type='datetime'>2009-05-17T22:38:42-07:00</created_at>
      <title>DataMapper</title>
      <author>Dan Kubb</author>
    </book>

This patch solve the problem and makes two tests to verify (one for create, one for update)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant