steve8x8 / geotoad

Geocaching query tool written in Ruby
https://buymeacoffee.com/steve8x8
Other
28 stars 8 forks source link

After 2014-01-14 site update: Trailing whitespace in owner field etc. #297

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The "improvements" rolled out with the latest site update added trailing 
whitespace (including a CR/LF combo) to the cache owner field in the details - 
apparently, the code tweakers have reached the cdpf part of the site :(

While this _may_ be invisible in the GPX, it heavily distorts every list 
output, therefore considered of medium importance.

3.20.0 will be out soon, I guess.

Original issue reported on code.google.com by Steve8x8 on 15 Jan 2014 at 9:05

GoogleCodeExporter commented 9 years ago
If you like to help me testing this: here's my first attempt at a patch for the 
issue.

Original comment by Steve8x8 on 15 Jan 2014 at 9:31

Attachments:

GoogleCodeExporter commented 9 years ago
While this patch went to SVN almost unmodified, it doesn't cover all changes. 
Even more importantly, coordinate parsing has also gone (also due to an added 
line feed in the HTML code) - and I'm now trying to compare old and new cdpf 
files to find out what else might have changed.
Still planning to release tomorrow...

Original comment by Steve8x8 on 15 Jan 2014 at 2:18

GoogleCodeExporter commented 9 years ago
I had a hard job going through all those patterns, and may have missed one or 
two.
At least, my tests didn't show any regressions, and there will be 3.20.1 ... 

Original comment by Steve8x8 on 16 Jan 2014 at 6:22

GoogleCodeExporter commented 9 years ago
Should be fixed now (with 3.20.1 out). Until the next apprentice changes the 
HTML again.

Original comment by Steve8x8 on 25 Feb 2014 at 11:25