in Koha 3.2, the suggestions is not completly linked to the related order. The link is only done through the biblionumber. We will add a field to have a fully qualified link. We will be able to add links to jump from suggestion to orders & from orders to suggestion
If you want to participate to the upstreaming: * assign the bug to yourself * retrieve the patch(es) on git.biblibre.com * the branch name can be found using git branch -a|grep NNNN Usually, it should be biblibre/customer/ft/MTNNNN For example: remotes/origin/stetienne/ft/MT6522 This patch is in branch number 5507
Created attachment 7216 [details] [review] Bug 5346 Linking suggestions and orders Display suggestion info in acquisition module: basket.pl neworderempty.pl orderreceive.pl parcel.pl To Test: Create a suggestion and accept it. Create a new order from this suggestion Receive this order For each step, check if suggestion info are visible.
I ran the tests and noticed that the added suggestion stayed in my database. It's also using a status I have not seen before "STALLED". Can you explain? I think it should delete the suggestion after the tests have been run.
Created attachment 7285 [details] [review] Bug 5346: Linking suggestions and orders. Display suggestion info in acquisition module: basket.pl neworderempty.pl orderreceive.pl parcel.pl To Test: Create a suggestion and accept it. Create a new order from this suggestion Receive this order For each step, check if suggestion info are visible. Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Test provides more unit tests, all complete successfully. perl t/db_dependent/Suggestions.t Note: test case should be cleaned up after running tests. Display changes are consistent and information about the suggestion is shown on every important screen now. I created an order from a suggestion and one from an existing record. No problems found.
Created attachment 7286 [details] [review] Bug 5346: Small display changes Changing display from: Suggested by Admin, Koha (from suggestion #2) To: Suggested by: Admin, Koha (suggestion #2) Adding some missing parentheses and deleting some additional spaces.
Jonathan, could you perhaps sign-off on my second patch and fix the unit tests? I don't think status 'STALLED' was implemented and the test case remains in my database after running the tests.
Katrin, Jonathan asked me to tell you that he's actually not available. He'll come back in two weeks. He also wants to say that the 'stalled' status was already present before his patch, he didn't what it is, so he left it.
Hi Julian, thx for the answer. I don't know what it is either. :( I think it's not existing in current Koha, so we should make the test use an existing status like 'ASKED'. Perhaps someone else could sign-off on my patch? :)
Created attachment 7560 [details] [review] [SIGNED-OFF] Bug 5346: Small display changes
QA comment: * perlcritic still OK after applying the patches * translatable-template and valid-templates still OK after applying the patches * nothing to say about the code itself * one question though: when i'm on a basket, the link to suggestions is "Suggested by: TEST (suggestion #153)", and when clicking on suggestion #153 link, I don't reach the suggestion itself, just a list of suggestions. Am I doing something wrong or is there a problem somewhere ?
Paul: Yes there is a problem :) But I can't redirect to the suggestion, the only existing page is the edit suggestion page. I redirect to "suggestion/suggestion.pl?suggestionid=". This page should redirect (in the future) to an other page, displaying a suggestion.
Main patch implements two new subroutines, with documentation and tests. Introduces those subroutines to scripts, and displays the output if applicable (but not if not). Marking as Passed QA.
Patch pushed. I hesitated, and fear that we will get some feedback from users about the link to jump to the suggestion that won't reach the suggestion itself. A follow-up to solve this annoyance would be highly welcomed !