Posted by GreenJelly on February 14, 2008 at 9:10am
Jump to:
| Project: | Flag content |
| Version: | 5.x-2.5 |
| Component: | Code |
| Category: | feature request |
| Priority: | minor |
| Assigned: | Unassigned |
| Status: | active |
Issue Summary
just noticed an inconsistency with other modules. When the content is not known you have $content_type and $content_id
small change that will make reading your code easier.
Comments
#1
No.
eid is entity ID.
An entity can be a node (content) as well as comment and user, which are not content.
#2
The reason I mentioned this change is because Taxonomy is under the same circumstance, and they call it "content_id" and "content_title".. that was the consistancy I was mentioning. As stated it is a small change, and doesn't matter much, however I have been actively working with your module as a model of my own, and thats when I saw the difference. In taxonomy, the relationship between content_id and content_title is what defines content. I see all elements displayed to the end user as content, even user accounts, comments, or what ever the next add-on is. Even your own module can be considered content.
BTW; You did good work... I am quite impressed, at every turn of the way I have seen how you eloquently deal with the code. On a few accounts I thought I saw a problem, only to realize how you did it was quite cool!
#3
Ohh I wanted to have a private conversation with you about my module. I dont want it to be public, and I would prefer a discussion in a private chat, so that we may discuss and share ideas.