[sword-devel] XML attribute delimiters in OSIS files?

[DM Smith]

Ah, now I understand. This is a bug. And should be fixed. (BTW, not having the entire thread reproduced in each email makes it harder to understand the context of the email. I don't like having to go digging for the context. Having looked, I see that the first email in the thread defines delimiters.)

But I'm not sure where it should be fixed. I haven't looked at the code, but as I recall, we use the SWORD parser to obtain the attribute value. My guess is that it is returning it with the quotes. If the problem is there and we fix it there, it may break a whole host of other things. (This parser is not a true XML parser, but one that is highly optimized for speed and thus we work with it's definition.)

It should be easy to change osis2mod to work. I'll look into doing this soon.

That said, it is and has been the recommendation that double quotes be used to wrap attribute values. It is valid to use single quotes, but it may (does) expose bugs. Fixing this bug does not change this recommendation.

Until osis2mod has been changed and it is available, it is advisable to change the input so that the quoting of sID/eID pairs to be identical.

In Him,
	DM

On Oct 26, 2011, at 6:38 AM, David Haslam wrote:

> Mixing double and single quotes, as per earlier messages in this thread.
> 
> Example (minus the chaff):
> 
> sID="reference"
> .....
> eID='reference'
> 
> But this time for the same verse, just as Chris replied, rather than in
> completely separate OSIS elements.
> 
> As this is just an observation, I see no immediate need to give a detailed
> example of what happens to the module.
> To locate the places where I spotted it yesterday would take some time.
> 
> Perhaps the most interesting thing is that there was no error message from
> osis2mod.
> 
> And I agree with Chris, the OSIS needs fixing first, before using as input
> for osis2mod.
> 
> David
> 
> 
> 
> 
> --
> View this message in context: http://sword-dev.350566.n4.nabble.com/XML-attribute-delimiters-in-OSIS-files-tp3907261p3940110.html
> Sent from the SWORD Dev mailing list archive at Nabble.com.
> 
> _______________________________________________
> sword-devel mailing list: sword-devel at crosswire.org
> http://www.crosswire.org/mailman/listinfo/sword-devel
> Instructions to unsubscribe/change your settings at above page