From 20468, comment7 by Owen Leonard: By the way, I get an error when I try to print a slip: "Undefined subroutine &C4::Letters::GetPreparedLetter called at /home/vagrant/kohaclone/circ/article-request-slip.pl line 49."
Created attachment 75719 [details] [review] Bug 20851: Add module to circ/article-request-slip.pl See bug 20468, comment7. Although the script seems to compile well, we better add the module here. Trivial change. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Moving trivial fix to SO rightaway
I don't see a sign off or a test plan. What is this supposed to fix? Printing article request slips works without problems for me. Can you please explain and reset to needs signoff?
(In reply to Katrin Fischer from comment #3) > I don't see a sign off or a test plan. What is this supposed to fix? > > Printing article request slips works without problems for me. > Can you please explain and reset to needs signoff? LOL
If you think my request is so silly, maybe you should epxlain it anyway.
(In reply to Katrin Fischer from comment #5) > If you think my request is so silly, maybe you should epxlain it anyway. Sure :) As mentioned in comment1 Owen got an error. And you and I did not (see also commit message). The error comes from a missing module; might be a devbox config issue for Owen. But the script itself calls C4::Letters but does not load it. So it depends on implicit loading by one of the other modules or deeper down the chain. We should not assume it will be loaded. If one day Letters is removed elsewhere, the script does not compile. So what does this oneliner do: add the missing module for that reason. Which is indeed stated ultra compact in the commit msg. No test plan needed. Back to SO.
Created attachment 75764 [details] [review] Bug 20851: Add module to circ/article-request-slip.pl See bug 20468, comment7. Although the script seems to compile well, we better add the module here. Trivial change. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Owen Leonard <oleonard@myacpl.org>
This indeed fixes the error I was getting in my devbox.
Created attachment 75792 [details] [review] Bug 20851: Add module to circ/article-request-slip.pl See bug 20468, comment7. Although the script seems to compile well, we better add the module here. Trivial change. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Thanks, Owen, for the sign-off and retesting. Article slips still work for me.
(In reply to Katrin Fischer from comment #10) > Thanks, Owen, for the sign-off and retesting. > Article slips still work for me. Thx Katrin for QA
Pushed to master for 18.11, awesome work all!
This patch has been pushed to 18.05.x and will be in 18.05.01
Pushed to 17.11.x for 17.11.07