Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

check commentable behavior extension #1

Open
cebe opened this Issue Dec 13, 2011 · 3 comments

Comments

Projects
None yet
2 participants
Owner

cebe commented Dec 13, 2011

check if/how https://github.com/yiiext/commentable-behavior could be merged into this extension

@cebe cebe was assigned Dec 13, 2011

Owner

samdark commented Dec 13, 2011

I think we should remove commentable-behavior since it wasn't finished and still in the draft state.

Owner

cebe commented Dec 13, 2011

I'd like to review the code and see if there is some nice thing that could live further with comment-module and delete it after that, okay?

Owner

samdark commented Dec 13, 2011

Sure. I should mention that using nested set for comments is rarely useful and often just an additional burden so the right decision is to use simplest adjacent list.

It was discussed here: http://yiiframework.ru/forum/viewtopic.php?f=9&t=3385 in Russian but I think it's still worth reading.

btw., please check if there are only my commits in this extension. If not then I probably wrong about code state.

@samdark samdark closed this Dec 13, 2011

@cebe cebe reopened this Dec 13, 2011

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment