Re: [Xsltforms-support] Issue with deleting items from repeat list
Brought to you by:
alain-couthures
From: Alain C. <ala...@ag...> - 2017-10-27 16:45:19
|
Interesting! It appears that the XForms control was deleted before yet another event was sent to it. Adding a test to avoid the exception sounds enough in this situation but it is currently not possible for me to check this... It will be committed in the next revision. Thanks! --Alain Le 27/10/2017 à 17:59, Tim Thompson a écrit : > Alain, > > I believe I was able to reproduce this error in Chrome 61. See the > attached screenshot, which references line 3614 in xsltforms.js. > > Best, > Tim > > -- > Tim A. Thompson > Discovery Metadata Librarian > Yale University Library > > > On Fri, Oct 27, 2017 at 10:32 AM, Alain Couthures > <ala...@ag... > <mailto:ala...@ag...>> wrote: > > Hi Mats, > > I have not been able to reproduce this issue: I have tested > with Chrome 61, Chrome 62, Chrome Beta 62 and Chrome Canary 64 > but no exception appeared... > > Could you please send me the line number you saw so I could > check its validity? > > Thanks! > > --Alain > > > Le 23/10/2017 à 14:36, Mats Eklund a écrit : >> Hi Alan, >> >> By chance stumbled upon a potential bug in the xsltforms >> library recently. The attached simple test case shows how an >> exception is thrown when user clicks Delete trigger on the >> last item in a repeat list. Only in Chrome it seems. >> >> It is possible to avoid the exception by putting an extra >> null check in the if() statement on the line where the >> exception is thrown, however, it looks like the issue may >> reveal a deeper issue, whose effects could sometimes be more >> subtle. >> >> / Mats >> >> >> >> >> ------------------------------------------------------------------------------ >> Check out the vibrant tech community on one of the world's most >> engaging tech sites, Slashdot.org!http://sdm.link/slashdot >> >> >> _______________________________________________ >> Xsltforms-support mailing list >> Xsl...@li... >> <mailto:Xsl...@li...> >> https://lists.sourceforge.net/lists/listinfo/xsltforms-support >> <https://lists.sourceforge.net/lists/listinfo/xsltforms-support> > > > > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > _______________________________________________ > Xsltforms-support mailing list > Xsl...@li... > <mailto:Xsl...@li...> > https://lists.sourceforge.net/lists/listinfo/xsltforms-support > <https://lists.sourceforge.net/lists/listinfo/xsltforms-support> > > > > > > ------------------------------------------------------------------------------ > Check out the vibrant tech community on one of the world's most > engaging tech sites, Slashdot.org! http://sdm.link/slashdot > > > _______________________________________________ > Xsltforms-support mailing list > Xsl...@li... > https://lists.sourceforge.net/lists/listinfo/xsltforms-support |