Bug 23525

Summary: ISBD view uses view policy of ACQ framework
Product: Koha Reporter: Fridolin Somers <fridolin.somers>
Component: Staff interfaceAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: gmcharlt, jonathan.druart, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
19.11.00
Bug Depends on: 19289    
Bug Blocks:    
Attachments: Bug 23525: (bug 19289 follow-up) Use biblio's framework not ACQ
Bug 23525: (bug 19289 follow-up) Use biblio's framework not ACQ
Bug 23525: (bug 19289 follow-up) Use biblio's framework not ACQ

Description Fridolin Somers 2019-08-30 14:01:18 UTC
In staff interface, ISBD view always uses view policy of ACQ framework.
This comes from a change from commit :
  Bug 19289: Use the ACQ framework to display bibliographic details


@@ -97,7 +97,7 @@ my $record_processor = Koha::RecordProcessor->new({
     filters => 'ViewPolicy',
     options => {
         interface => 'intranet',
-        frameworkcode => $framework
+        frameworkcode => 'ACQ'
     },

Is this a mistake ?
Comment 1 Katrin Fischer 2019-09-01 21:41:25 UTC
Yes, I think this should have no effect on the ISBD views in ACQ and/or catalog pages, it seems counter-intuitive.
Comment 2 Jonathan Druart 2019-10-14 07:55:12 UTC
Created attachment 94042 [details] [review]
Bug 23525: (bug 19289 follow-up) Use biblio's framework not ACQ

This patch restores the behavior prior to
  commit 84ce1fb5921cdddf2368f0c330dede0f602742d0
  Bug 19289: Use the ACQ framework to display bibliographic details

Test plan is hard to provide for such a simple and obvious change. I
would recommend you to read the previous commit and confirm that the
change did not make sense.
Comment 3 Katrin Fischer 2019-10-14 20:19:21 UTC
Created attachment 94102 [details] [review]
Bug 23525: (bug 19289 follow-up) Use biblio's framework not ACQ

This patch restores the behavior prior to
  commit 84ce1fb5921cdddf2368f0c330dede0f602742d0
  Bug 19289: Use the ACQ framework to display bibliographic details

Test plan is hard to provide for such a simple and obvious change. I
would recommend you to read the previous commit and confirm that the
change did not make sense.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Nick Clemens (kidclamp) 2019-10-25 12:36:57 UTC
Created attachment 94751 [details] [review]
Bug 23525: (bug 19289 follow-up) Use biblio's framework not ACQ

This patch restores the behavior prior to
  commit 84ce1fb5921cdddf2368f0c330dede0f602742d0
  Bug 19289: Use the ACQ framework to display bibliographic details

Test plan is hard to provide for such a simple and obvious change. I
would recommend you to read the previous commit and confirm that the
change did not make sense.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 5 Martin Renvoize 2019-10-28 12:39:10 UTC
Nice work!

Pushed to master for 19.11.00