<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<p>Dear yvand,</p>
<p>I believe I've fixed this in osis2mod. osis2mod had code to deal
with linking for ranges, but it checked for the necessity of a
link with ListKey::getCount(), but this only tells how many
elements are in the ListKey, not if any of those elements are
ranges. Oddly, it did have code after this check which would work
find to see if more than 1 verse was referenced-- so close.
Anyway, I think it's now fixed. I added unit tests to check both
osisID verse lists and also annotateRef ranges and both tests
pass.</p>
<p>Thanks for your patience on this one. Please let me know what
you find,</p>
<p>Troy</p>
<p><br>
</p>
<div class="moz-cite-prefix">On 7/25/20 9:30 PM, yvand wrote:<br>
</div>
<blockquote type="cite"
cite="mid:bd0e0298-256a-f912-efb6-61e4f4c7dd90@gmail.com">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<p>I tried using annotateRef="Gen.1.1 Gen.1.2 Gen.1.3 Gen.1.4"
instead of annotateRef="Gen.1.1-Gen.1.4", the behavior is
different. diatheke gives a commentary for each verse, but I am
not sure it is link entries...<br>
</p>
<p>So now, I don't know what I can do to "fix" this bug
(feature?).</p>
<p>I am lost. I still do not know where is exactly the bug, or if
this is just a frontend feature…<br>
</p>
<p>--yvand<br>
</p>
<div class="moz-cite-prefix">Le 24/07/2020 à 10:44, Troy A.
Griffitts a écrit :<br>
</div>
<blockquote type="cite"
cite="mid:9fa858c1-ed5c-1d0d-de74-e2b5529a490d@crosswire.org">
<meta http-equiv="Content-Type" content="text/html;
charset=UTF-8">
<p>Yes, linking is not working with annotateRef. Thanks for the
text. If you wouldn't mind rebuilding the module and changing
one entry to an osisID verse list and then testing again, it
would really help finalize what exactly the problem is. If
osisID verse lists do link correctly, then we simply need to
apply the same logic to annotateRef.<br>
</p>
<div class="moz-cite-prefix">On 7/24/20 10:42 AM, yvand wrote:<br>
</div>
<blockquote type="cite"
cite="mid:d634642e-dbc7-5e27-0491-8653c3c03e28@gmail.com">
<meta http-equiv="Content-Type" content="text/html;
charset=UTF-8">
<p>I tried it again, here is what I got: <br>
<br>
</p>
<blockquote type="cite" style="color: #000000;">$ diatheke -b
FreCJE -k Gen.1.2 <br>
Genesis 1:2: <br>
(FreCJE) <br>
<br>
$ diatheke -b FreCJE -k Gen.1.3 <br>
Genesis 1:3: <br>
(FreCJE) <br>
</blockquote>
<p><br>
</p>
<p>So nothing :-/</p>
<p>--yvand</p>
<p><br>
</p>
<div class="moz-cite-prefix">Le 24/07/2020 à 10:38, Troy A.
Griffitts a écrit :<br>
</div>
<blockquote type="cite"
cite="mid:5a7c3ed0-e3b6-9f46-c603-11ebf2e667aa@crosswire.org">Hi
yvand, <br>
<br>
Well, that's not an authoritative test. You need to ask
ONLY for verse Gen.1.2. <br>
<br>
SWORD has logic to "skip" contiguous linked verses, so
diatheke might be taking advantage of that to prevent it
from printing out the same commentary 19 times in a row.
The output you see is the desired output when a user
requests the entire chapter. One instance of each
commentary entry. <br>
<br>
To test linking, you need to ask for a verse within the
range, but not the first verse. <br>
<br>
Hope this explains a bit, <br>
<br>
Troy <br>
<br>
<br>
On 7/24/20 10:27 AM, yvand wrote: <br>
<blockquote type="cite">Hey, <br>
<br>
Thanks Troy for your help. I use annotateRef and not
osisID, as mentioned in <a class="moz-txt-link-freetext"
href="https://wiki.crosswire.org/OSIS_Commentaries"
moz-do-not-send="true">https://wiki.crosswire.org/OSIS_Commentaries</a>
. I tried diatheke, here is what I got : <br>
<br>
<blockquote type="cite">diatheke -b FreCJE -k Gen.1 | head
<br>
Genesis 1:1: <div annotateRef="Gen.1.1-Gen.1.19"
annotateType="commentary" sID="gen1" type="section"/>
<div sID="gen2" type="x-p"/><hi
type="bold">La création de
l'univers</hi><div eID="gen2" type="x-p"/>
<div sID="gen3" type="x-p"/>Avant que rien
n'existe de tout………….<div eID="gen4" type="x-p"/>
<div annotateRef="Gen.1.1-Gen.1.19"
annotateType="commentary" eID="gen1" type="section"/>
<br>
Genesis 1:2: <br>
Genesis 1:3: <br>
Genesis 1:4: <br>
Genesis 1:5: <br>
Genesis 1:6: <br>
Genesis 1:7: <br>
Genesis 1:8: <br>
Genesis 1:9: <br>
Genesis 1:10: <br>
</blockquote>
I don't know if this is normal: Gen.1.2 is empty, does it
mean no link entry? <br>
<br>
Best regards, <br>
<br>
--yvand <br>
<br>
<br>
Le 23/07/2020 à 23:51, Troy A. Griffitts a écrit : <br>
<blockquote type="cite">Hey yvand, <br>
<br>
My apologies for not getting back with you. osis2mod
isn't my domain these days, so I am not sure what the
current behavior is for annotateRef. I am pretty sure
linking works, and the normal way for linking is the
provide osisID="Gen.1.1 Gen.1.2 Gen1.3" <br>
<br>
You an test linking by using any of the lookup tools,
like diatheke, or I use sword/examples/cmdline/lookup. <br>
<br>
e.g., ./lookup FreCJE Gen.1.2 <br>
<br>
I'd have to dig into the source for osis2mod and how it
handles annotateRef ranges, but I would think it should
work the same as osisID verse lists. <br>
<br>
Troy <br>
<br>
<br>
<br>
On 7/23/20 6:44 PM, yvand wrote: <br>
<blockquote type="cite">Thanks Greg for your quick reply
with explanation! <br>
<br>
I thought there was a bug and "link entries" were not
taken into consideration, when using osis2mod. Is
there a simple way to test if a commentary module
contains link entries? I tried with mod2imp to export
FreCJE but it only shows verses with attached
commentary ($$$Genesis 1:2 is missing for instance).
Maybe I misunderstood "link entries" functionality...
<br>
<br>
I believe it worked as expected in the past (at least
with Xiphos), but maybe I am wrong. <br>
<br>
Unfortunately I am not familiar with C/C++ and with
the sword engine, so I am not able to offer you a
patch. <br>
<br>
I understand the issues you pointed and it doesn't
seem easy. Currently, there are still questions, for
instance: how will operate the engine if there are
multiple commentaries starting with Gen.1.1 in the
source, like : <br>
<br>
<blockquote type="cite"><div ...
annotateRef="Gen.1.1">...</div> <br>
<div ...
annotateRef="Gen.1.1-Gen.1.16">...</div> <br>
<div ...
annotateRef="Gen.1.1-Gen.1.31">...</div> <br>
</blockquote>
I guess I will now give up for this feature. <br>
<br>
--yvand <br>
<br>
_______________________________________________ <br>
sword-devel mailing list: <a
class="moz-txt-link-abbreviated"
href="mailto:sword-devel@crosswire.org"
moz-do-not-send="true">sword-devel@crosswire.org</a>
<br>
<a class="moz-txt-link-freetext"
href="http://www.crosswire.org/mailman/listinfo/sword-devel"
moz-do-not-send="true">http://www.crosswire.org/mailman/listinfo/sword-devel</a>
<br>
Instructions to unsubscribe/change your settings at
above page <br>
</blockquote>
_______________________________________________ <br>
sword-devel mailing list: <a
class="moz-txt-link-abbreviated"
href="mailto:sword-devel@crosswire.org"
moz-do-not-send="true">sword-devel@crosswire.org</a> <br>
<a class="moz-txt-link-freetext"
href="http://www.crosswire.org/mailman/listinfo/sword-devel"
moz-do-not-send="true">http://www.crosswire.org/mailman/listinfo/sword-devel</a>
<br>
Instructions to unsubscribe/change your settings at
above page <br>
</blockquote>
_______________________________________________ <br>
sword-devel mailing list: <a
class="moz-txt-link-abbreviated"
href="mailto:sword-devel@crosswire.org"
moz-do-not-send="true">sword-devel@crosswire.org</a> <br>
<a class="moz-txt-link-freetext"
href="http://www.crosswire.org/mailman/listinfo/sword-devel"
moz-do-not-send="true">http://www.crosswire.org/mailman/listinfo/sword-devel</a>
<br>
Instructions to unsubscribe/change your settings at above
page <br>
</blockquote>
_______________________________________________ <br>
sword-devel mailing list: <a
class="moz-txt-link-abbreviated"
href="mailto:sword-devel@crosswire.org"
moz-do-not-send="true">sword-devel@crosswire.org</a> <br>
<a class="moz-txt-link-freetext"
href="http://www.crosswire.org/mailman/listinfo/sword-devel"
moz-do-not-send="true">http://www.crosswire.org/mailman/listinfo/sword-devel</a>
<br>
Instructions to unsubscribe/change your settings at above
page<br>
</blockquote>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
sword-devel mailing list: <a class="moz-txt-link-abbreviated" href="mailto:sword-devel@crosswire.org" moz-do-not-send="true">sword-devel@crosswire.org</a>
<a class="moz-txt-link-freetext" href="http://www.crosswire.org/mailman/listinfo/sword-devel" moz-do-not-send="true">http://www.crosswire.org/mailman/listinfo/sword-devel</a>
Instructions to unsubscribe/change your settings at above page</pre>
</blockquote>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
sword-devel mailing list: <a class="moz-txt-link-abbreviated" href="mailto:sword-devel@crosswire.org" moz-do-not-send="true">sword-devel@crosswire.org</a>
<a class="moz-txt-link-freetext" href="http://www.crosswire.org/mailman/listinfo/sword-devel" moz-do-not-send="true">http://www.crosswire.org/mailman/listinfo/sword-devel</a>
Instructions to unsubscribe/change your settings at above page</pre>
</blockquote>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<pre class="moz-quote-pre" wrap="">_______________________________________________
sword-devel mailing list: <a class="moz-txt-link-abbreviated" href="mailto:sword-devel@crosswire.org">sword-devel@crosswire.org</a>
<a class="moz-txt-link-freetext" href="http://www.crosswire.org/mailman/listinfo/sword-devel">http://www.crosswire.org/mailman/listinfo/sword-devel</a>
Instructions to unsubscribe/change your settings at above page</pre>
</blockquote>
</body>
</html>