Add support for GenericForeignKey

Regarding: Does every other object in the system get a get_comment_list method?

It should be possible to specify what models you can relate to from a given GenericFK, so you'd say: object = GenericForeignKey?(Laptop, Server, Desktop). This way the above problem goes away.

This also solves the problem of our object instances being cleanly and quickly removed when a Laptop is removed. Without specifying the models that we relate to, every model on deletion would have to do a DELETE from each of the tables of models with GenericForeignKey?'s. And if we knew what models a GenericForeignKey? can relate to - we would only issue one DELETE - to the right table.

This seems kind of evil from a relational point of view...

What is the disadvantage of having a Comment superclass ( that could be table-less), and just subclassing for each thing you want to have comments on?

class Comment(meta.AbstractModel?):

... some fields...

class LaptopComment?(Comment):

content_object = meta.ForeignKey?(Laptop)

class ServerComment?(Comment):

content_object = meta.ForeignKey?(Server)

To make this seamless, superclasses should aggregate list methods of their subclasses. Also, I'm not sure what the right thing to do is if the target of a ForeignKey? has subclasses... I don't think any solution is great there.

This is tangenitally discussed here: http://groups.google.com/group/django-developers/browse_thread/thread/ea5e0bf903058fac/31734cc4ddf6ff75#31734cc4ddf6ff75

After a bit of discussion on IRC with rjwittams, I have a solution. Basically, I need this functionality by the end of the week, and am going to have to shortcut it until a better solution is discussed.

Essentially, i'd like to have this model:

from django.core import meta

class Comment (meta.Model):
	body = meta.TextField()
	
	self.get_subject = self.get_commentobject().get_comment_subject
	
class BlogEntry (meta.Model):
	body = meta.TextField()
	who = meta.CharField(max_length=32)
	
class DocEntry (meta.Model):
	body = meta.TextField()
	function = meta.CharField(max_length=32)

class FAQEntry (meta.Model):
	question = meta.TextField()
	answer = meta.TextField()
	
class CommentObject (meta.Model):
	comment = meta.OneToOneField(Comment)
	blogentry = meta.OneToOneField(BlogEntry, null=True)
	docentry = meta.OneToOneField(DocEntry, null=True)
	faqentry = meta.OneToOneField(FAQEntry, null=True)
	
	def get_comment_subject(self):
		for field in (blogentry, docentry, faqentry):
			if field:
				return field
		return None

Shortcutted to: {{{from django.core import meta

class Comment (meta.Model):

body = meta.TextField?() subject = meta.GenericForeignKey?([BlogEntry?, DocEntry?, FAQEntry])

class BlogEntry? (meta.Model):

body = meta.TextField?() who = meta.CharField?(max_length=32)

class DocEntry? (meta.Model):

body = meta.TextField?() function = meta.CharField?(max_length=32)

class FAQEntry (meta.Model):

question = meta.TextField?() answer = meta.TextField?()

}}}

comment.get_subject() should return either a BlogEntry?, FAQEntry or DocEntry?. Not entirely sure how this should look in the admin interface though.

Preview function is my friend

from django.core import meta

class Comment (meta.Model):
	body = meta.TextField()
	subject = meta.GenericForeignKey([BlogEntry, DocEntry, FAQEntry])
	
class BlogEntry (meta.Model):
	body = meta.TextField()
	who = meta.CharField(max_length=32)
	
class DocEntry (meta.Model):
	body = meta.TextField()
	function = meta.CharField(max_length=32)

class FAQEntry (meta.Model):
	question = meta.TextField()
	answer = meta.TextField()

Ignore my last two comments, they're wrong. rjwittams' workaround is:

from django.core import meta

class Comment (meta.Model):
	body = meta.TextField()

	def get_parent(self):
	    if not self.getters:
		getters = [self.get_comment_reply, self.get_blogcomment, self.get_faqcomment, self.get_doccomment]
	    for getter in self.getters:
		ext = getter()
		if ext:
		   return ext.parent      

class BlogEntry (meta.Model):
	body = meta.TextField()
	who = meta.CharField(max_length=32)
	
class DocEntry (meta.Model):
	body = meta.TextField()
	function = meta.CharField(max_length=32)

class FAQEntry (meta.Model):
	question = meta.TextField()
	answer = meta.TextField()
	
class BlogComment (meta.Model):
	comment = meta.OneToOneField(Comment)
	parent = meta.ForeignKey(BlogEntry)

class CommentReply (meta.Model):
	comment = meta.OneToOneField(Comment)
	parent = meta.ForeignKey(Comment)

class DocComment(meta.Model):
	comment = meta.OneToOneField(Comment)
	parent =  meta.ForeignKey(DocEntry)
	
class FaqComment(meta.Model):
	comment = meta.OneToOneField(Comment)
	parent = meta.ForeignKey(FAQEntry)

(In [3134]) Added generic foreign key support to Django. Much thanks to Ian Holsman and Luke Plant -- most of this code is theirs. Documentation is to follow; for now see the example/unit test. Fixes #529.

Remove CC to prevent spam - sorry

Milestone Version 1.0 deleted