Permalink
Browse files

Updated POD

	- Added answers.yahoo.com to engines
	- Removed images.google.* from TODO list as this was already being parsed
  • Loading branch information...
oalders committed Jul 23, 2009
1 parent 33becdb commit 5490ab75864fd5a38de035d87fdc93b62c93ec54
Showing with 17 additions and 33 deletions.
  1. +2 −0 Changes
  2. +7 −33 lib/URI/ParseSearchString/More.pm
  3. +8 −0 t/urls.cfg
View
@@ -6,6 +6,8 @@ Revision history for Perl module URI::ParseSearchString::More
- Added bing.com to engines
- Added fastbrowsersearch.com to engines
- Now requires Data::Dump rather than Data::Dumper
+ - Added answers.yahoo.com to engines
+ - Removed images.google.* from TODO list as this was already being parsed
0.11 2009-07-09
- Wrapped $mech->get in an eval in order to get by the following cryptic
@@ -15,12 +15,14 @@ use URI::Heuristic qw(uf_uristr);
use WWW::Mechanize::Cached;
my %search_regex = (
+ answers => qr{(.*) - Yahoo! Answers},
aol => qr/AOL Search results for "(.*)"/,
as => qr{(?:WeatherStudio|Starware) (.*) Search Results},
dogpile => qr{(.*) - Dogpile Web Search},
);
my %url_regex = (
+ answers => qr{answers.yahoo.com},
aol => qr{aol.com/(?:aol|aolcom)/search\?encquery=},
as => qr{as.\w+.com/dp/search\?x=},
dogpile => qr{http://www.dogpile},
@@ -540,45 +542,17 @@ provide you with a valid search term. Then again, it may not. Caveat emptor.
=head1 TO DO
-Here is a list of some of the engines currently not covered by
-L<URI::ParseSearchString> that may be added to this module:
-
- images.google.*
- www.adelphia.net/google/
- http://answers.yahoo.com/question/index;_ylt=Al7fJtDUTm2S69bM0VvjPDIjzKIX?qid=20061214165004AADtB1I
+I've pretty much added all of the search engines I care about. If you'd like
+something added, please get in touch.
=head1 NOTES
-Despite its low version number, this module actually works. It is,
-however, still very young and the interface is subject to some change.
+Despite its low version number, this module is now stable.
=head1 KNOWN ISSUES
-On some systems, this module dies with the following message when caching is
-enabled:
-
-Can't store CODE items at blib/lib/Storable.pm (autosplit into blib/lib/auto/Storable/_freeze.al) line 339
-
-For this reason, caching is disabled by default as of version 0.08 If caching
-does not fail on your system, I encourage you to enable it. It seems to me
-that this error is not caused by any problem with this module, but I haven't
-really spent too much time looking into it as I can't replicate it on my
-development machine. Leaving it enabled by default would cause a lot of
-failing tests and switching it off only for tests would mean a lot of passing
-tests but failing real world use.
-
-See the documentation it t/005_parse_more.t for information on how to run the
-parsing tests with caching enabled.
-
-NOTE: As of 0.11 the $mech->get call is wrapped in an eval. This gets past the
-die problems, even though I still have no idea what the root cause actually is.
-So, if you have WWW::Mechanize::Cached installed, you should be able to use the
-caching option at this point.
-
-The actual problem may have to do with the following unresolved ticket for
-WWW::Mechanize::Cached:
-
-<L>http://rt.cpan.org/Public/Bug/Display.html?id=42693
+As of 0.12 WWW::Mechanize::Cached 1.33 is required. This solves the errors
+which were being thrown by Storable.
=head1 BUGS
View
@@ -4094,4 +4094,12 @@
<urls>
url = http://www.bing.com/search?q=myspace%20headline&first=71&FORM=PORE
terms = myspace headline
+</urls>
+<urls>
+ url = http://answers.yahoo.com/question/index;_ylt=Al7fJtDUTm2S69bM0VvjPDIjzKIX?qid=20061214165004AADtB1I
+ terms = How can I track the IP address of visitors to my personal site?
+</urls>
+<urls>
+ url = http://images.google.com/images?hl=en&q=vile+richard&btnG=Search+Images&gbv=2&aq=f&oq=
+ terms = vile richard
</urls>

0 comments on commit 5490ab7

Please sign in to comment.