If AJAX_PK_ATTR_NAME is set, we want to remove the original reference to "pk" since this is acting as a rename of the same variable

joestump / django-ajax

A simple framework for creating AJAX endpoints in Django.

http://github.com/joestump/django-ajax

BSD 3-Clause "New" or "Revised" License

260 stars 54 forks source link

If AJAX_PK_ATTR_NAME is set, we want to remove the original reference to "pk" since this is acting as a rename of the same variable #27

Closed mattrobenolt closed 12 years ago

mattrobenolt commented 12 years ago

The behavior of this acts as if it's renaming the original "pk" variable to something else. Without this, "pk" still persists as well as the new reference.

Not sure if the original behavior is intended or not, just an observation made. Feel free to reject. :)

joestump commented 12 years ago

I should be deleting the pk field from the data. Going to reject this as it's not quite in line with the original requirement. I'll fix it in a separate commit.