search

amir-jakoby / crawler-commons

Automatically exported from code.google.com/p/crawler-commons
0 stars 0 forks source link

[Sitemaps] SitemapUrl "setPriority(String str)" should check for proper value #55

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
setPriority (double d) - checks for a proper value which is a value between 0 
to 1.0

But the same overrided method which gets a string as an argument doesn't check 
for a proper value.

And also a default value should be set when a non parsable string is sent to 
this method as indicated in the JavaDocs

Original issue reported on code.google.com by avrah...@gmail.com on 13 Oct 2014 at 6:29

GoogleCodeExporter commented 8 years ago 
Attaching a Patch which contains the fix + new JUnit Test file

Original comment by avrah...@gmail.com on 13 Oct 2014 at 7:25

Attachments:

GoogleCodeExporter commented 8 years ago 
I am +1 for commit

Original comment by lewis.mc...@gmail.com on 13 Oct 2014 at 3:10

GoogleCodeExporter commented 8 years ago 
Fixed in Revision: r152

Original comment by avrah...@gmail.com on 26 Oct 2014 at 11:50