[sword-devel] preverse divs

[DM Smith]

On 02/29/2012 07:21 AM, Peter von Kaehne wrote:
> Ok,
>
> Just my 2p to the matter:
>
> There are several versions of osis2mod about which produce different results.

This is unfortunate.

Revision 2358, 28 April 2009: The x-preverse div was added. With that 
change all interverse material was retained. As described in another 
email in this thread.
 From 2358 to 2502 (9 January 2010) bugs and compile warning/errors were 
fixed. Each of these bug fixes were in response to a module maker having 
a problem.

Revision 2519, 4 June 2010: The x-preverse div was commented out. The 
interverse material, especially titles, is no longer marked as preverse, 
leaving it to the module maker to put it in.
 From 2519 to 2667 (12 Nov 2011) compile warning/errors were fixed.
All modules made with a version of osis2mod from 2519 to 2667 having 
significant interverse markup will have problems.

Revision 2668, 12 Nov 2011: The x-preverse div was put back in.
 From 2668 to 2671 (12 Nov 2011) bug fixes were put in. Significantly, 
osis2mod was aborting because of the improper handling a chapter not in 
the v11n.

>
> I learned the hard way that other than by adding x-preverse to most titles etc things do NOT work.

There is a bug/shortcoming in osisheadings.cpp.

>
> But recently things have become more broken and it has become a mess to figure out.
>
> It is not all engine as modules produced in different times have different level of "properness".
>
> It would be great if we as module makers had a clear specification to work to - whatever that is - and then could dump the module into osis2mod and get a working module.
The intention is that the module maker creates an OSIS xml file that 
matches the spec and does not do anything special for SWORD.
The current osis2mod supports this. However, SWORD and the frontends 
built on it do not. Osisheadings.cpp needs to be modified and when it is 
released, modules need to be marked as requiring that version of SWORD. 
If we knew what that number would be, we could mark modules with that now.

>
> At the moment we do not.
Where does that leave us:
Create modules with revision 2355 or earlier. Test the module to ensure 
it works. (Non-av11n should work with 1.5.9).
or
Create modules with the latest and wait for osisheadings.cpp to be changed.

In Him,
     DM

>
> Peter
>
> -------- Original-Nachricht --------
>> Datum: Wed, 29 Feb 2012 12:39:19 +0100
>> Von: "Troy A. Griffitts"<scribe at crosswire.org>
>> An: SWORD Developers\' Collaboration Forum<sword-devel at crosswire.org>
>> Betreff: Re: [sword-devel] preverse divs
>> I don't know what's wrong except what I noted in the previous post: that
>> there is something wrong.
>>
>> I want to clear up one thing that seems to be a general misunderstanding:
>>
>> Module makers should never care or know about x-preverse.
>> They should make their module how they think best conforms to the OSIS
>> specification.
>> osis2mod will add the x-preverse div to the section it things should be
>> associated with a verse but display before the verse marker.
>>
>> .... in a perfect world anyway.  That's the goal, at least, and
>> depending on who's reverted whom's revert, it might work that way now.
>>
>>
>>
>> On 02/29/2012 11:09 AM, David Haslam wrote:
>>> Hi Troy,
>>>
>>> I just tried wrapping the first "section title" for Matt.1.1 as follows:
>>>
>>> <div type="section">
>>> <div type="x-milestone" subType="x-preverse" sID="pv1"/>
>>> <title>Lignez Jezuz</title>
>>> <div type="x-milestone" subType="x-preverse" eID="pv1"/>
>>> <verse sID="Matt.1.1" osisID="Matt.1.1"/>
>>> Levr lignez Jezuz-Krist, Mab David, Mab Abraham.
>>> <verse eID="Matt.1.1"/>
>>>
>>> After rebuilding the module, the title "Lignez Jezuz" had disappeared
>> when
>>> viewed with *Xiphos 3.1.5* (in Windows).  In its place there seemed to
>> be an
>>> extra space.
>>>
>>> It was there before, albeit displayed after the verse tag.
>>>
>>> So what's wrong?
>>>
>>> cf. In *BibleDesktop 1.6*, the same title is still displayed, and still
>>> after the verse tag and a line break.
>>>
>>> The phrase "not yet supported" seems to be a considerable
>> understatement.
>>>
>>> Regards,
>>>
>>> David
>>>
>>> --
>>> View this message in context:
>> http://sword-dev.350566.n4.nabble.com/preverse-divs-tp4429131p4431114.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
>>
>> _______________________________________________
>> 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