#545 ✓not-applicable
Jamie Macey

dm-is-versioned docs need fixing

Reported by Jamie Macey | August 28th, 2008 @ 12:20 PM

Docs say to use :on => [:property], but the code does an equality test (line 62 of dm-is-versioned/lib/dm-is-versioned/is/versioned.rb) with property names, which fails. As a result, the versions table has only a primary key of (id), which causes problems creating a second version of something.

Flipping the test to on.include?(property.name) seems to work for me, diff included.

Comments and changes to this ticket

  • Jonathan Stott (namelessjon)

    Jonathan Stott (namelessjon) December 11th, 2008 @ 10:13 AM

    • Assigned user cleared.
    • State changed from “new” to “not-applicable”
    • Title changed from “dm-is-versioned gets wrong key” to “dm-is-versioned docs need fixing”

    I'm going to mark this as not applicable. All the code and the specs seem to be using a singular property, it's just the docs which need fixing.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

Attachments

Tags

Pages