To recreate: -Be on 20.05.x -Go to returns.tt -See the error Uncaught SyntaxError: expected expression, got keyword 'var'. -Check the page source and you will see that: var columns_settings = [% TablesSettings.GetColumns( 'circ', 'returns', 'checkedintable', 'json' ) | $raw %] is being evaluated to: var columns_settings =
Unfortunately I think 20.05.03 was packaged with this error
This is the result of a bad rebase of 25940. I rebased with TablesSettings instead if ColumnSettings. TableSettings is not in 20.05 or below
Created attachment 109568 [details] [review] Bug 26361: switch TableSettings to ColumnSettings Test Plan: -Be on 20.05.x -Go to returns.tt -See the error Uncaught SyntaxError: expected expression, got keyword 'var'. -Apply patch -Reload returns.tt, no error
Created attachment 109580 [details] [review] Bug 26361: switch TableSettings to ColumnSettings Test Plan: -Be on 20.05.x -Go to returns.tt -See the error Uncaught SyntaxError: expected expression, got keyword 'var'. -Apply patch -Reload returns.tt, no error Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Backport error indeed. Also needed to 19.11.x
Lucas, you need to setup a pre-push hook to catch occurrences of TableSettings in template files.
Created attachment 109587 [details] [review] Bug 26361: switch TableSettings to ColumnSettings Test Plan: -Be on 20.05.x -Go to returns.tt -See the error Uncaught SyntaxError: expected expression, got keyword 'var'. -Apply patch -Reload returns.tt, no error Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
backported to 20.05.x for 20.05.04
(In reply to Fridolin SOMERS from comment #5) > Backport error indeed. > Also needed to 19.11.x hi Aleisha, can you add this to 19.11.x branch
backported to 19.11.x for 19.11.10
*** Bug 26399 has been marked as a duplicate of this bug. ***
IIUC if bug 25940 is backported to oldoldstable/19.05 this bug should be backported at the same time.
(In reply to Victor Grousset/tuxayo from comment #12) > IIUC if bug 25940 is backported to oldoldstable/19.05 this bug should be > backported at the same time. No, backport the original patch (the one from master), and correctly fix the merge conflict.
Since the javascript code breaks completely on the returns view (which i believe will case a lot of confusion about reserved items) whould it not be appropriate to mark this as a blocker and post a message in the release notes of 19.11.09 and 20.05.03 to discourage libraries from installing or upgrading to these versions?
(In reply to Andreas Jonsson from comment #14) > Since the javascript code breaks completely on the returns view (which i > believe will case a lot of confusion about reserved items) whould it not be > appropriate to mark this as a blocker and post a message in the release > notes of 19.11.09 and 20.05.03 to discourage libraries from installing or > upgrading to these versions? The new maintenance release should be out within a few days now (around 22nd) - so this will be fixed soon. We haven't seen any libraries running into this in the chat and on the mailing list - so I hope that not too many have run into this. An email to the mailing list might come a little late now :(
(In reply to Katrin Fischer from comment #15) > (In reply to Andreas Jonsson from comment #14) > > Since the javascript code breaks completely on the returns view (which i > > believe will case a lot of confusion about reserved items) whould it not be > > appropriate to mark this as a blocker and post a message in the release > > notes of 19.11.09 and 20.05.03 to discourage libraries from installing or > > upgrading to these versions? > > The new maintenance release should be out within a few days now (around > 22nd) - so this will be fixed soon. We haven't seen any libraries running > into this in the chat and on the mailing list - so I hope that not too many > have run into this. An email to the mailing list might come a little late > now :( this patch has been added to 19.11.09-2 and 20.05.03-2 releases, so its fixed already :)
Awesome, thx Mason!
(In reply to Katrin Fischer from comment #15) > (In reply to Andreas Jonsson from comment #14) > > Since the javascript code breaks completely on the returns view (which i > > believe will case a lot of confusion about reserved items) whould it not be > > appropriate to mark this as a blocker and post a message in the release > > notes of 19.11.09 and 20.05.03 to discourage libraries from installing or > > upgrading to these versions? > > The new maintenance release should be out within a few days now (around > 22nd) - so this will be fixed soon. We haven't seen any libraries running > into this in the chat and on the mailing list - so I hope that not too many > have run into this. An email to the mailing list might come a little late > now :( Our largest collaboration of libraries, which I upgraded on Thursday night, was affected by this. We were only lucky to notice this as quickly as we did so only about 50 reserved items where affected. But it could easily have been a thousand. If the issue is likely to cause major problems to some libraries it would be a good idea to warn the community even if the maintenance release is just a few days away, in my opinion.
(In reply to Mason James from comment #16) > (In reply to Katrin Fischer from comment #15) > > (In reply to Andreas Jonsson from comment #14) > > > Since the javascript code breaks completely on the returns view (which i > > > believe will case a lot of confusion about reserved items) whould it not be > > > appropriate to mark this as a blocker and post a message in the release > > > notes of 19.11.09 and 20.05.03 to discourage libraries from installing or > > > upgrading to these versions? > > > > The new maintenance release should be out within a few days now (around > > 22nd) - so this will be fixed soon. We haven't seen any libraries running > > into this in the chat and on the mailing list - so I hope that not too many > > have run into this. An email to the mailing list might come a little late > > now :( > > this patch has been added to 19.11.09-2 and 20.05.03-2 releases, so its > fixed already :) Were are these releases announced? I don't even see any tags for them on GitHub.
Are you building your own packages or are you using the community ones? I think this might have been fixed in the packages published, but not on git - could this be the issue?
(In reply to Katrin Fischer from comment #20) > Are you building your own packages or are you using the community ones? I > think this might have been fixed in the packages published, but not on git - > could this be the issue? Yes, indeed we are building our own packages, because we have a few custom patches that we are using. But the releases on the download page are 19.11.09 and 20.05.03 both from 2020-08-31, which is before this issue was reported.
> But the releases on the download page are 19.11.09 and 20.05.03 both from > 2020-08-31, which is before this issue was reported. I am not sure what happened there then with what Mason said. It's good to think about how to improve workflows for such hotfixes.