Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 1.2.1
-
Fix Version/s: 1.2.2
-
Component/s: Extensions
-
Labels:None
Description
There are some typos in your documentation for Taggable extension.
In section related to getPopularTags method, if I use your code, like Doctrine::getTable('BlogPost')->getPopularTags(); I get an exception "Unknown method Doctrine_Table::getPopularTags" (with replacing BlogPost by my model name). In effect, it's not the table for my model that I must use but the table for TaggeableTag.
Besides, in your tests, you use Doctrine::getTable('TaggableTag') ![]()
I have write a patch to fix these typos. It's attached to this ticket.
I believe the TaggableTag table is used to hold all tags, so if you have multiple tables that use tags, TaggableTag will contain the tags for all the tables using the Taggable extension. By doing: Doctrine::getTable('BlogPost')->getPopularTags(); we only want to get the popular tags for that table alone.
However, I could be totally wrong???