search

jimregan / mlode

Automatically exported from code.google.com/p/mlode
0 stars 0 forks source link

a lot of white spaces #28

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?

There are a lot of wight spaces, this one example:
curl -L -H "Accept: application/rdf+xml" 
http://www.glottolog.org/resource/languoid/id/tuva1238.rdf | grep 'geo:'

What is the expected output? What do you see instead?

Output:
<geo:long> 179.17 </geo:long>
    <geo:lat> -8.5 </geo:lat>

Defect:
white spaces around the numbers 

Right output:
<geo:long>179.17</geo:long>
<geo:lat>-8.5</geo:lat>

How many triples are affected? (if less than 3-5% of the whole data set,
please set priority to _low_)

Please use labels and text to provide additional information.

Original issue reported on code.google.com by mohamedd...@gmail.com on 31 Jul 2012 at 2:04

GoogleCodeExporter commented 9 years ago 
fixed white spaces

Original comment by sebastia...@googlemail.com on 2 Aug 2012 at 8:16

GoogleCodeExporter commented 9 years ago 
This was virtually everywhere:
   <rdfs:label xml:lang="en"> Tuvaluan </rdfs:label>
    <dcterms:title xml:lang="en"> Tuvaluan </dcterms:title>
    <skos:prefLabel xml:lang="en"> Tuvaluan </skos:prefLabel>
    <skos:provenance> MPI Composite 2011 </skos:provenance>
Please fix this *everywhere*.
you might want to consider using:
grep -R '> ' * 
or 
sed -i 's/> />/' *
on the templates

*Warning better test the sed first. It might have side effects. 

Bug is definitely not fixed!

Original comment by kur...@googlemail.com on 2 Aug 2012 at 8:22

GoogleCodeExporter commented 9 years ago 
increased priority

Original comment by kur...@googlemail.com on 2 Aug 2012 at 8:23

GoogleCodeExporter commented 9 years ago 
whitespace should now be removed everywhere in abcd1234.rdf eg
http://www.glottolog.org/resource/languoid/id/tuva1238.rdf

Original comment by sebastia...@googlemail.com on 2 Aug 2012 at 8:31

GoogleCodeExporter commented 9 years ago 
ok, we will test this on a new dump, after you created one. 
Please don't open a new issue, but post the link of the dump here (or validate 
it yourself).

you can test the dump with 
grep '> ' dump.rdf
and 
grep ' <' dump.rdf

Original comment by kur...@googlemail.com on 2 Aug 2012 at 8:36

GoogleCodeExporter commented 9 years ago 
please verify

Original comment by sebastia...@googlemail.com on 23 Aug 2012 at 1:56

GoogleCodeExporter commented 9 years ago

Original comment by mohamedd...@gmail.com on 23 Aug 2012 at 2:34