Thanks a lot for the good work. A newbie here, and have a question on the comment model.
Isn't it cleaner to look at the threaded comments in another way, e.g., if comment B is a reply to comment A, then the content_object of B is simply A instead of the original content object a comment is attached to? That way you can get rid of the 'parent', because 'content_object' is the 'parent'. You may calculate the 'level' by recursively looking at the content_object's content_type. If it's not a comment then you've reached the top of the thread. In this way, a non-threaded comment is just a threaded comment with maximum level 1, and the post_comment and reply_comment can be merged. Also a 'modify_date' may be useful for the abstract comment model. One of the benefits you might want to provide to the authenticated users is to modify their comments. Thanks, Zhiwu Xie Thejaswi Puthraya wrote: > Hello folks, > My exams got done in the first week of May. I was feeling quite bored > and so started working on the project. > > I completed a few items from the Todo list but need to complete the > documentation and write more tests. > > The code for my project is at > http://code.google.com/p/django-newcomments/ > and at > http://gitorious.org/projects/django-newcomments > > I also have put up the four demos in the source at: > > Demo-1: Using django-newcomments as a standalone app. > http://sandbox.thejaswi.info/example1/authors/ > > Demo-2: Using django-newcomments placing it in django.contrib.comments > (after removing the current django.contrib.comments) > http://sandbox.thejaswi.info/example2/books/ > > Demo-3: A simple threaded comments demo (inspired from Eric > Florenzano's project). > http://sandbox.thejaswi.info/example3/comics/ > > Demo-4: A custom comments demo > http://sandbox.thejaswi.info/example4/messages/ > > Current Problems faced: > * Abstract models don't allow custom managers. Tickets 7154 and 7252 > handle that (I am running trunk patched with ticket 7154). > * Abstract models don't allow the inheritance of the inner admin > class. After a discussion on IRC, I was assured that this was a small > issue that newforms-admin would cover after it supports model- > inheritance. Till then an abstract model designer cannot pass on a > neat admin UI to the user :( > > Please do let me know of any suggestions/criticisms and/or feedback. > > -- > Cheers > Thejaswi Puthraya > http://thejaswi.info/ > > > --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Django developers" group. To post to this group, send email to django-developers@googlegroups.com To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/django-developers?hl=en -~----------~----~----~----~------~----~------~--~---
