salmans / Razor

5 stars 0 forks source link

Augmentation does not add "fresh" elements to the domain #61

Closed thedotmatrix closed 9 years ago

thedotmatrix commented 9 years ago

When augmenting, we have to add the @element relation for the new elements.

Concrete bug happening from this problem: Facebook example, Query when permission = OnlyMe Augment Read(e^100, e^2) we don't see e^100=e^4 (the new person is equal to the original person)

thedotmatrix commented 9 years ago

@salmans @Element relations are now added to the base and the satIterator. I think the augmentation bug is fixed too. Can you run through the facebook example a couple of times to confirm before we close this?

salmans commented 9 years ago

I think something is broken after the last changes. This is my test-case:

If I am not wrong, when we ran this test a few hours ago, the model was correctly augmented with such tuples.

thedotmatrix commented 9 years ago

I believe we used another query. Some remarks:

Let me know if you agree with these remarks.

salmans commented 9 years ago

My bad! A rule that forces every content to be posted by a user was missing. After I fixed the policy, augmentation is working as it should. I am closing this issue.

Thanks!