I submitted a ticket and patch #8279. Basically, there are three
differences.
1. Fixed the deprecated :dependent => true
2. Added migration generator (it has really specific requirements on
table layout, so might as well generate it).
3. Expanded the README to explain using the migration and to include a
usage example.
Feedback much appreciated.
http://svn.viney.net.nz/things/rails/plugins/acts_as_taggable_on_steroids/
Thanks for the patch, but I'm not sure it necessarily needs to be applied.
--
Cheers
Koz
http://blog.evanweaver.com/pages/has_many_polymorphs
Evan
On May 7, 1:52 am, "Michael Koziarski" <mich...@koziarski.com> wrote:
> Acts as taggable was only intended as a demonstration of how to use
> the polymorphic associations. We're not really maintaining it as a
> drop-in tagging solution. There are already more complete
> implementations out there like:
>
> http://svn.viney.net.nz/things/rails/plugins/acts_as_taggable_on_ster...
> Another solution is my has_many_polymorphs plugin--it has a generator
> for self-referential Tagging generator in it. I haven't publicized it
> yet because I need to add an example controller. But the models,
> migrations, helper methods all work fine already.
>
> http://blog.evanweaver.com/pages/has_many_polymorphs
and let's face it: any plugin with a "self-referential tagging
generator" is naturally going to pwn all alternatives.
-- tim
I had just thought it was a matter of a bare bones vs. many features I
will not need for my current project.
It would be helpful if there were an indication that it was intended
as a tech demo, perhaps as a notice in the README.
On May 7, 12:52 am, "Michael Koziarski" <mich...@koziarski.com> wrote:
> Acts as taggable was only intended as a demonstration of how to use
> the polymorphic associations. We're not really maintaining it as a
> drop-in tagging solution. There are already more complete
> implementations out there like:
>
> http://svn.viney.net.nz/things/rails/plugins/acts_as_taggable_on_ster...
>
> Thanks for the patch, but I'm not sure it necessarily needs to be applied.
>
> On 5/7/07, monki <gee...@gmail.com> wrote:
>
>
>
>
>
>
>
> > I was working with acts_as_taggable earlier and thought I could make
> > it a bit friendlier.
>
> > I submitted a ticket and patch #8279. Basically, there are three
> > differences.
>
> > 1. Fixed the deprecated :dependent => true
> > 2. Added migration generator (it has really specific requirements on
> > table layout, so might as well generate it).
> > 3. Expanded the README to explain using the migration and to include a
> > usage example.
>
> > Feedback much appreciated.
>
> --
> Cheers
>
> Koz- Hide quoted text -
>
> - Show quoted text -
> It would be helpful if there were an indication that it was intended
> as a tech demo, perhaps as a notice in the README.
+1
Way too many developer hours has been wasted on that thing:
http://dev.rubyonrails.org/search?q=acts_as_taggable&ticket=on
Updated readme: http://dev.rubyonrails.org/ticket/8287
Doesn't help that it's in the Rails plugin dir which is included in ./
script/plugin's default set of search paths.
-- tim
I've updated the readme, and moved it to be a little more prominent.
--
Cheers
Koz
I decided to roll my changes into my own plugin that better fits my
needs, if any one is interested:
http://agilewebdevelopment.com/plugins/acts_as_taggable_redux
On May 11, 6:12 pm, "Michael Koziarski" <mich...@koziarski.com> wrote:
> good call.
>
> I've updated the readme, and moved it to be a little more prominent.
>