
paranoid properties are ignored when using has many through relationships
Reported by Alex Arnell | May 22nd, 2009 @ 01:18 PM | in 1.0.0
Given
class Photo
include DataMapper::Resource
has n, :taggings
has n, :tags, :through => :taggings
end
class Tag
include DataMapper::Resource
has n, :taggings
has n, :photos, :through => :taggings
end
class Tagging
include DataMapper::Resource
property :deleted_at, ParanoidDateTime
belongs_to :tag
belongs_to :photo
end
After deleting a Tagging
photo.tags
will include the Tag for which the Tagging was deleted.
Comments and changes to this ticket
-
Dan Kubb (dkubb) June 16th, 2009 @ 03:42 AM
- Assigned user set to Dan Kubb (dkubb)
- State changed from unconfirmed to accepted
- Milestone set to 0.10.0
-
Dan Kubb (dkubb) June 23rd, 2009 @ 01:49 AM
- State changed from accepted to resolved
I just wrote a script to verify this behavior, and it works as expected.
Attached is a script that demonstrates the current behavior, and includes comments/output.
@Alex: Please check the script and let me know what you expected from the behavior, or if I should make any changes to the script to reproduce your problem. I am marking this as resolve for now though.
-
Dan Kubb (dkubb) June 23rd, 2009 @ 01:52 AM
- State changed from resolved to accepted
Actually, I take that back. I found an error in the script, and can confirm this is a problem. Reopening the ticket. Attached is the updated script.
-
Dan Kubb (dkubb) October 4th, 2009 @ 09:33 PM
- Milestone changed from 0.10.0 to 0.10.2
[project:id#20609 not-tagged:"0.10.0" milestone:id#51895 bulk edit command]
-
Dan Kubb (dkubb) February 1st, 2010 @ 04:33 PM
- Milestone changed from 0.10.2 to 1.0.0
-
-
-
Dan Kubb (dkubb) February 2nd, 2010 @ 02:47 AM
- State changed from accepted to confirmed
-
Dan Kubb (dkubb) February 4th, 2010 @ 03:49 AM
- State changed from confirmed to resolved
- Assigned user set to Dan Kubb (dkubb)
- Milestone set to 1.0.0
This appears to be fixed on edge dm-core. Paranoid columns are being used to filter out the intermediaries that are deleted now. Marking this as resolved.
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.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »