in reply to RFC: A module to evaluate keywords expressions on tag lists.

Neither is particularly compelling.

Having a root namespace "Keywords::" means you can do lots of things with keywords. Is there that much you can do to keywords?

Sounds like it would be better integrated into "Search::" or some other existing top-level namespace.

I would actually prefer if the query parser provided a standard interface that supported multiple query languages. (The one you provided, Google's, etc)

I also have a feeling the matching functionality shouldn't be too heavily integrated.

  • Comment on Re: RFC: A module to evaluate keywords expressions on tag lists.

Replies are listed 'Best First'.
Re^2: RFC: A module to evaluate keywords expressions on tag lists.
by vitoco (Hermit) on Sep 03, 2009 at 20:20 UTC
    Is there that much you can do to keywords?

    I think not, but this module provides a query parser, a regexp builder and some tests methods. Not only the query language is relevant. Also it is the way that a list of keywords is being stored if you want to use some specific options, then a method to clean up the list string is provided. So it's not only a Search thing. That's made me doubt.

    I don't think that "Search::Query" is appropiate, for the same reason as above. May be "Regexp::Keywords"?

    BTW, "Search::Query::Google" made me think on a module to retrieve some results from Google (as "Google::Search")... "Search::Keywords::Google" is not much better, but I got the idea and agree on that.

    I also have a feeling the matching functionality shouldn't be too heavily integrated.

    Could you explain this?

      Could you explain this?

      If query building is separate from matching, then you'll be able to match against inputs other than a flat string.

      Specifically, only supporting flat strings prevents it from being extended to support "is:unread" or "title:foo"

      I'm not saying you should implement those features. I'm saying it would be great if the designed allowed someone to do so.