tags in 0.7.3

I am applying tags to an article and it doesn't seem to work
as
advertised.  It tells me to comma sepperate my tags but that
results
in the tags getting added as one lump tag. so:

rails,ruby,plugin

results in one tag being added to the database named
"rails,ruby,plugin" instead of adding all 3 tags
sepparately.

it does the same thing if I sepperate them with spaces:

rails ruby plugin

results in one tag being added to the database named
"rails ruby
plugin"

the only way it worked is if I sepperated them with commas
and spaces:

rails, ruby, plugin

then it added 3 sepperate tags.

I guess it is due to the regex in /app/models/tag.rb:
tag_parse_regex = /((?:
|)['"]{0,1})['"]?s*(.*?)s*(?:[,'"]|$)(?:
1(?: |$))/

Is everyone else experiencing this same thing?


--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---

Try this:

{{ article | linked_tag_list }}


--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---

I am failrly new to mephisto so I think you are refering to
template
with that post.  This afects me more in terms that I can't
perform
normal tag functions on those post properly.

For instance I can't find all articles "tagged
with" some tag from the
admin and my /tags/ page doesn't show post properly b/c
of the
way it is tagged in the DB as mentioned earlier.

On Feb 12, 2:43 pm, "jaredburns"
<jared5bu...gmail.com> wrote:
> Try this:
>
> {{ article | linked_tag_list }}


--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---

Whoops. I thought you we're referring to the template, but
you're
actually referring to posting a new article in the admin. In
regards
to that, I haven't experienced the issues you're
discribing.


--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---

rails, ruby, plugin

Put a space after each comma.

Mike Gunderloy
http://afreshcup.com

On Feb 12, 12:35 pm, "wyrosdick"
<ben.wyrosd...gmail.com> wrote:
> I am applying tags to an article and it doesn't seem to
work as
> advertised.  It tells me to comma sepperate my tags but
that results
> in the tags getting added as one lump tag. so:
>
> rails,ruby,plugin
>
> results in one tag being added to the database named
> "rails,ruby,plugin" instead of adding all 3
tags sepparately.
>
> it does the same thing if I sepperate them with
spaces:
>
> rails ruby plugin
>
> results in one tag being added to the database named
"rails ruby
> plugin"
>
> the only way it worked is if I sepperated them with
commas and spaces:
>
> rails, ruby, plugin
>
> then it added 3 sepperate tags.
>
> I guess it is due to the regex in /app/models/tag.rb:
> tag_parse_regex = /((?:
|)['"]{0,1})['"]?s*(.*?)s*(?:[,'"]|$)(?:
> 1(?: |$))/
>
> Is everyone else experiencing this same thing?


--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---

On 2/12/07, MikeG1larkfarm.com <Larkwaregmail.com> wrote:
>
> rails, ruby, plugin
>
> Put a space after each comma.

I think rails,ruby,plugin would be reasonable.  Tested patch
anyone?


-- 
Rick Olson
http://weblog.techno-
weenie.net
http://mephistoblog.com

--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---

The admin page says "Enter one or more tags separated
by a comma."
Yet the tests allow space separated tags:
    assert_equal %w(tagging it's weirdness),
Tag.parse('"tagging" "it
's" "weirdness"')
    assert_equal %w(tagging it's weirdness),
Tag.parse("'tagging'
'it's' 'weirdness'")
    # with spaces...
    assert_equal %w(tagging it's weirdness),
Tag.parse("'tagging'
'it's weirdness'")
    assert_equal %w(tagging it's weirdness),
Tag.parse('"tagging" "it
's weirdness"')

Is there a rationale behind this? In the interests of
simplicity,
would it be worth removing support for this?
The regex we have is pretty ugly already - "/((?:
|)['"]{0,1})['"]?
s*(.*?)s*(?:[,'"]|$)(?:1(?: |$))/" - unless
anyone has enough fu to
expand it to match "a,b,c" as well (judging from
the lack of replies,
no), I propose the following patch:

http://pastie.caboo.se/4
1320

... since it is preferable to have the app do what is says
it can do
(separate on commas) than some groovy complex logic that
doesn't
actually work.

Xavier

On Feb 13, 10:20 am, "Rick Olson"
<technowee...gmail.com> wrote:
> On 2/12/07, Mik...larkfarm.com <Larkw...gmail.com> wrote:
>
>
>
> > rails, ruby, plugin
>
> > Put a space after each comma.
>
> I think rails,ruby,plugin would be reasonable.  Tested
patch anyone?
>
> --
> Rick Olsonhttp://weblog.techno-weenie.nethttp://mephistoblog.com



--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---

I agree ... I like the simpler tags

On Feb 18, 10:09 pm, "Xavier Shay"
<Xavier.S...gmail.com> wrote:
> The admin page says "Enter one or more tags
separated by a comma."
> Yet the tests allow space separated tags:
>     assert_equal %w(tagging it's weirdness),
Tag.parse('"tagging" "it
> 's" "weirdness"')
>     assert_equal %w(tagging it's weirdness),
Tag.parse("'tagging'
> 'it's' 'weirdness'")
>     # with spaces...
>     assert_equal %w(tagging it's weirdness),
Tag.parse("'tagging'
> 'it's weirdness'")
>     assert_equal %w(tagging it's weirdness),
Tag.parse('"tagging" "it
> 's weirdness"')
>
> Is there a rationale behind this? In the interests of
simplicity,
> would it be worth removing support for this?
> The regex we have is pretty ugly already - "/((?:
|)['"]{0,1})['"]?
> s*(.*?)s*(?:[,'"]|$)(?:1(?: |$))/" -
unless anyone has enough fu to
> expand it to match "a,b,c" as well (judging
from the lack of replies,
> no), I propose the following patch:
>
> http://pastie.caboo.se/4
1320
>
> ... since it is preferable to have the app do what is
says it can do
> (separate on commas) than some groovy complex logic
that doesn't
> actually work.
>
> Xavier
>
> On Feb 13, 10:20 am, "Rick Olson"
<technowee...gmail.com> wrote:
>
> > On 2/12/07, Mik...larkfarm.com
<Larkw...gmail.com> wrote:
>
> > > rails, ruby, plugin
>
> > > Put a space after each comma.
>
> > I think rails,ruby,plugin would be reasonable. 
Tested patch anyone?
>
> > --
> > Rick Olsonhttp://weblog.techno-weenie.nethttp://mephistoblog.com



--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---

That patch worked nicely for me ... thanks Xavier

is there any way that can get added to the trunk?

On Feb 19, 10:48 am, "wyrosdick"
<ben.wyrosd...gmail.com> wrote:
> I agree ... I like the simpler tags
>
> On Feb 18, 10:09 pm, "Xavier Shay"
<Xavier.S...gmail.com> wrote:
>
> > The admin page says "Enter one or more tags
separated by a comma."
> > Yet the tests allow space separated tags:
> >     assert_equal %w(tagging it's weirdness),
Tag.parse('"tagging" "it
> > 's" "weirdness"')
> >     assert_equal %w(tagging it's weirdness),
Tag.parse("'tagging'
> > 'it's' 'weirdness'")
> >     # with spaces...
> >     assert_equal %w(tagging it's weirdness),
Tag.parse("'tagging'
> > 'it's weirdness'")
> >     assert_equal %w(tagging it's weirdness),
Tag.parse('"tagging" "it
> > 's weirdness"')
>
> > Is there a rationale behind this? In the interests
of simplicity,
> > would it be worth removing support for this?
> > The regex we have is pretty ugly already -
"/((?: |)['"]{0,1})['"]?
> > s*(.*?)s*(?:[,'"]|$)(?:1(?: |$))/" -
unless anyone has enough fu to
> > expand it to match "a,b,c" as well
(judging from the lack of replies,
> > no), I propose the following patch:
>
> >http://pastie.caboo.se/4
1320
>
> > ... since it is preferable to have the app do what
is says it can do
> > (separate on commas) than some groovy complex
logic that doesn't
> > actually work.
>
> > Xavier
>
> > On Feb 13, 10:20 am, "Rick Olson"
<technowee...gmail.com> wrote:
>
> > > On 2/12/07, Mik...larkfarm.com
<Larkw...gmail.com> wrote:
>
> > > > rails, ruby, plugin
>
> > > > Put a space after each comma.
>
> > > I think rails,ruby,plugin would be
reasonable.  Tested patch anyone?
>
> > > --
> > > Rick Olsonhttp://weblog.techno-weenie.nethttp://mephistoblog.com



--~--~---------~--~----~------------~-------~--~----~
Mephisto: http://mephistoblog.com |
Mephisto Help: http://mephisto.stikipad
.com

To post to this group, send email to MephistoBloggooglegroups.com
To unsubscribe from this group, send email to
MephistoBlog-unsubscribegooglegroups.com
For more options, visit this group at http
://groups-beta.google.com/group/MephistoBlog
-~----------~----~----~----~------~----~------~--~---