[jsword-devel] [JIRA] (JS-273) Is 53 not found in ESV

[Chris Burrell]

I'd agree with your latest
On 31 May 2013 16:43, "Martin Denham (JIRA)" <jira at crosswire.org> wrote:

>        Martin Denham<http://www.crosswire.org/bugs/secure/ViewProfile.jspa?name=mjdenham>commented on [image:
> Bug] JS-273 <http://www.crosswire.org/bugs/browse/JS-273>
>  *Is 53 not found in ESV* <http://www.crosswire.org/bugs/browse/JS-273>
>
> So the current implementation of contains(key) is containsAllOf(key) but
> in this particular instance we are expecting containsAnyOf(key). Is that
> right?
>
> If so then we could either consider i) changing contains to be
> containsAnyOf or ii) adding a different function containsAnyOf or iii)
> changing contains to be containsAnyOf when the key is a chapter.
>
> From an And Bible perspective book.contains(key) is used to work out if
> there is anything that can be shown on the screen so contains(key) is
> always used in the sense of containsAnyOf so my initial preference would be
> to change the implementation of book.contains() to be containsAnyOf but
> there may be use cases others would raise that I haven't thought of.
>
> ----------
> I just remembered I emailed JSword-devel regarding a different issue
> related to VerseRange.contains() on 4th May so maybe contains() needs
> consideration in general. I need to raise a Jira for that because I don't
> think one was raised and I don't believe it has been fixed (I worked around
> it in AB). I'll raise a jira but hopefully nobody gets confused between the
> two different contains() problems.
> ----------
>
> Martin
>         This message is automatically generated by JIRA.
> If you think it was sent incorrectly, please contact your JIRA
> administrators
> For more information on JIRA, see: http://www.atlassian.com/software/jira
>
> _______________________________________________
> jsword-devel mailing list
> jsword-devel at crosswire.org
> http://www.crosswire.org/mailman/listinfo/jsword-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.crosswire.org/pipermail/jsword-devel/attachments/20130531/51d0c1e9/attachment-0001.html>