django-taggit icon indicating copy to clipboard operation
django-taggit copied to clipboard

TaggableManager doesn't base itself off of RelatedManager

Open rtpg opened this issue 2 years ago • 0 comments

TaggableManager is, essentially, a RelatedManager in spirit. It's fetching data off of a related table, has its prefetch cache.

Over the years we've had a couple bugs (#802 , #686 ) due to us not relying on RelatedManager and just reproducing features ad-hoc.

I haven't deeply looked into the feasibility of using RelatedManager directly. But if it's doable, it would be very good in my opinion. We might still need to do some magic to maintain some backwards compat.

rtpg avatar May 11 '22 09:05 rtpg