[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2026-03-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Irene  changed:

   What|Removed |Added

 CC||[email protected]

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2026-01-20 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Liddle  changed:

   What|Removed |Added

 CC||[email protected]

--- Comment #105 from David Liddle  ---
Hello! I was directed here from Bug 38677 by Lucas. What you've done here seems
fantastic, though I have yet to give it a try in a sandbox. I will need to do
that soon and pass the results on to my colleagues. Our group chose to follow
LTS versions from 22.11 onward, but we may need to make an exception on this
occasion. Thank you each for the work you put into this effort!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-12-11 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Michelle Spinney  changed:

   What|Removed |Added

 CC||[email protected]

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-12-10 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #104 from Anneli Österman  ---
Here are some examples that we have tested. They are created with the help of
Copilot. We haven't had need for controlfield's so no examples of those ones.



[% IF record.subfield('020' , 'q') %]
  
  Binding form: 
  [% record.subfield('020' , 'q') %]
  
[% END %]



[% IF record.subfield('049','c') %]
  
  Age limit: 
  [% record.subfield('049','c') %]
  
[% END %]




[% SET values = [] %]

[% FOREACH f IN record.fields %]
  [% IF f.tag == '257' %]
[% FOREACH sf IN f.subfields %]
  [% IF sf.0 == 'a' && sf.1 %]
[% values.push(sf.1) %]
  [% END %]
[% END %]
  [% END %]
[% END %]

[% IF values.size %]
  
Producing country: 
[% values.join(', ') %]
  
[% END %]




[% IF record.subfield('310' , 'a') %]
  
  Current publication frequency: 
  [% record.subfield('310' , 'a') %]
  
[% END %]




[% vals = [] %]
[% FOREACH f IN record.fields %]
  [% IF f.tag == '347' %]
[% FOREACH sf IN f.subfields %]
  [% IF sf.0 == 'b' %]
[% vals.push(sf.1) %]
  [% END %]
[% END %]
  [% END %]
[% END %]

[% IF vals.size %]
  
Encoding format: 
[% vals.join(', ') %]
  
[% END %]




[% SET parts = [] %]
[% IF record.subfield('370','c'); parts.push(record.subfield('370','c')); END
%]
[% IF record.subfield('370','f'); parts.push(record.subfield('370','f')); END
%]
[% IF record.subfield('370','g'); parts.push(record.subfield('370','g')); END
%]

[% IF parts.size %]
  
  Associated place: 
  [% parts.join(', ') %]
  
[% END %]


[% SET allparts = [] %]




[% SET allparts = [] %]

[% FOREACH f IN record.fields %]
  [% IF f.tag != '382' %]
[% NEXT %]
  [% END %]

  [% SET mediums  = [] %]   
  [% SET soloists = [] %]

  [% FOREACH sf IN f.subfields %]
[% IF sf.0 == 'a' %]
  [% SET m = { a => sf.1, n => '' } %]
  [% mediums.push(m) %]
[% ELSIF sf.0 == 'n' %]
  [% SET mcount = mediums.size %]
  [% IF mcount > 0 %]
[% SET last_idx = mcount - 1 %]
[% IF !mediums.$last_idx.n %]
  [% SET mediums.$last_idx.n = sf.1 %]
[% ELSE %]
  [% SET mediums.$last_idx.n = sf.1 %]
[% END %]
  [% END %]
[% ELSIF sf.0 == 'b' %]
  [% soloists.push(sf.1) %]
[% END %]
  [% END %]

  [% SET parts = [] %]

  [% FOREACH m IN mediums %]
[% IF m.a %]
  [% IF m.n %]
[% parts.push(m.a _ ' (' _ m.n _ ')') %]
  [% ELSE %]
[% parts.push(m.a) %]
  [% END %]
[% END %]
  [% END %]

  [% IF soloists.size %]
[% FOREACH b IN soloists %]
  [% parts.push(b) %]
[% END %]
  [% END %]

  [% IF parts.size %]
[% allparts.push(parts.join(', ')) %]
  [% END %]
[% END %]

[% IF allparts.size %]
  
Esityskokoonpano: 
[% allparts.join('; ') %]
  
[% END %]




[% SET parts = [] %]
[% IF record.subfield('386','m'); parts.push(record.subfield('386','m')); END
%]
[% IF record.subfield('386','a'); parts.push(record.subfield('386','a')); END
%]

[% IF parts.size %]
  
  Creator/contributor characteristics: 
  [% parts.join(': ') %]
  
[% END %]





[% SET values = [] %]

[% FOREACH f IN record.fields %]
  [% IF f.tag == '388' %]
[% FOREACH sf IN f.subfields %]
  [% IF sf.0 == 'a' %]
[% values.push(sf.1) %]
  [% END %]
[% END %]
  [% END %]
[% END %]

[% IF values.size %]
  
Creation time: 
[% values.join(', ') %]
  
[% END %]

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-12-10 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #103 from Katrin Fischer  ---
Displaying a single subfield works really well. Could we have some examples on
how to display controlfield contents and full fields or a list of subfields as
well?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-12-04 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Chip Halvorsen  changed:

   What|Removed |Added

 CC||Chip.Halvorsen@WestlakeLibr
   ||ary.org

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-07 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

   Keywords|additional_work_needed  |

--- Comment #102 from Lucas Gass (lukeg)  ---
follow-up pushed to main

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Miranda Nero  changed:

   What|Removed |Added

 CC||[email protected]

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Jonathan Druart  changed:

   What|Removed |Added

 CC||[email protected]
   Keywords||additional_work_needed

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #101 from Jonathan Druart  ---
Created attachment 189146
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=189146&action=edit
Bug 39860: Fix selenium/opac_holds.t

#   Failed test 'Correctly in search results page'
#   at t/db_dependent/selenium/opac_holds.t line 88.
#   'Error: Can't use an undefined value as a HASH
reference at /kohadevbox/koha/C4/XSLT.pm line 247'
# doesn't match '(?^u:Results of search for 'opac holds test place
hold button')'

might be a conflict with bug 40943

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #100 from Lucas Gass (lukeg)  ---
Nice work everyone!

Pushed to main for 25.11

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Version(s)||25.11.00
released in||
 Status|Passed QA   |Pushed to main

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 QA Contact|[email protected] |[email protected]
   |y.org   |

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #99 from Lisette Scheer  ---
(In reply to David Cook from comment #98) 
> Could you elaborate?
> 
> The dropdown is shown either way. This just makes the dropdown show the
> correct option(s). 
> 
> Are you saying it would be better to just not show the dropdown all
> together? I think you'd probably be correct there.

Ah, yeah I misread where in the building of the button that was. I would think
there doesn't need to be a dropdown because there's only one way to edit it.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-11-02 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #98 from David Cook  ---
(In reply to Lisette Scheer from comment #97)
> Wouldn't this:
> 
>  [% IF ( wysiwyg || category == 'record_display' ) %]
> 
> make it show the dropdown when there's a record display even though it
> should just be loading the text editor, not the dropdown?

Could you elaborate?

The dropdown is shown either way. This just makes the dropdown show the correct
option(s). 

Are you saying it would be better to just not show the dropdown all together? I
think you'd probably be correct there.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-31 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #97 from Lisette Scheer  ---
(In reply to David Cook from comment #96)
> Created attachment 188731 [details] [review]
> Bug 39860: (QA follow-up) fix XSS vulnerability and fix disabling of WYSIWYG
> 
> Signed-off-by: David Cook 

Wouldn't this:

 [% IF ( wysiwyg || category == 'record_display' ) %]

make it show the dropdown when there's a record display even though it should
just be loading the text editor, not the dropdown?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #96 from David Cook  ---
Created attachment 188731
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188731&action=edit
Bug 39860: (QA follow-up) fix XSS vulnerability and fix disabling of WYSIWYG

Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #187898|0   |1
is obsolete||

--- Comment #95 from David Cook  ---
Created attachment 188730
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188730&action=edit
Bug 39860: Disable WYSIWYG for record display entries

Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #185905|0   |1
is obsolete||

--- Comment #94 from David Cook  ---
Created attachment 188729
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188729&action=edit
Bug 39860: Update scrubber profile and add Scrubber.t test

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 
Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #185904|0   |1
is obsolete||

--- Comment #93 from David Cook  ---
Created attachment 188728
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188728&action=edit
Bug 39860: Add record_display srubber profile

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 
Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #185903|0   |1
is obsolete||

--- Comment #92 from David Cook  ---
Created attachment 188727
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188727&action=edit
Bug 39860: Add options for opac/staff lists

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 
Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #185902|0   |1
is obsolete||

--- Comment #91 from David Cook  ---
Created attachment 188726
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188726&action=edit
Bug 39860: Fix OPAC display location

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 
Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #185901|0   |1
is obsolete||

--- Comment #90 from David Cook  ---
Created attachment 188725
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188725&action=edit
Bug 39860: Move display control from system preference to additional contents

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 
Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #185900|0   |1
is obsolete||

--- Comment #89 from David Cook  ---
Created attachment 188724
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=188724&action=edit
Bug 39860: Allow for the defining of custom MARC fields on the staff interface
record display pages

To test:
0. APPLY PATCH, restart_all
1. Find or create a record that has some MARC fields that don't show up by
default in the record display. ( I tested using a 538$a )
2. Go to Tools and you should see a new link for 'Record display
customizations'.
3. From Record display customizations click 'New Entry'
4. Choose a display location, the choices should be StaffResultsPage,
StaffDetailPage, OPACResultsPage, and OPACDetailPage
5. Add the following to the entry:

[% IF record.subfield('538' , 'a') %]
  
  The 538a: 
  [% record.subfield('538' , 'a') %]
  
[% END %]

6. Go to the record from step 1, you should see the 538$a labeled as 'The
538a:' with the appropriate data in it.
7. This should display in the appropriate location depending on if you chose
StaffResultsPage, StaffDetailPage, OPACResultsPage, or OPACDetailPage. Check
each of the locations.
8. Make a new, or edit existing, Record display customization and mark the
'Library:' field for a specific branch. Make sure that the Record display
customization only shows up when you are logged in at that branch.
9. Install a new language pack, I tested with es-ES.
10. Make a new, or edit existing, Record display customization and fill out the
language specific tab. Make sure the Record display customization shows
correctly with the language you chose.

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 
Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Status|Signed Off  |Passed QA

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #88 from David Cook  ---
I noticed the "Preview content" was still vulnerable to XSS, so I've fixed that
in a follow-up as well.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #87 from David Cook  ---
Test plan doesn't mention OPACListPage and StaffListPage but they do work so
that's awesome :D

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #86 from David Cook  ---
(In reply to David Cook from comment #85)
> Comment on attachment 187898 [details] [review]
> Bug 39860: Disable WYSIWYG for record display entries
> 
> Review of attachment 187898 [details] [review]:
> -
> 
> Hmm... actually this patch doesn't seem to work at all... and the typo
> didn't seem to do anything which is interesting...

Ah my bad. I'm looking at the wrong spot. The patch changes "New entry", but
the problem is with the toggle next to the "Edit" button in the Action column.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #85 from David Cook  ---
Comment on attachment 187898
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=187898
Bug 39860: Disable WYSIWYG for record display entries

Review of attachment 187898:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=39860&attachment=187898)
-

Hmm... actually this patch doesn't seem to work at all... and the typo didn't
seem to do anything which is interesting...

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-30 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #84 from David Cook  ---
Comment on attachment 187898
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=187898
Bug 39860: Disable WYSIWYG for record display entries

Review of attachment 187898:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=39860&attachment=187898)
-

::: koha-tmpl/intranet-tmpl/prog/en/modules/tools/additional-contents.tt
@@ +351,4 @@
>  > data-bs-toggle="dropdown" aria-expanded="false"> 
>  
>  
> +[% IF ( wysiwy || category == 'record_display' ) %]

It looks like you've got a little typo there where you've turned "wysiwyg" into
"wysiwy". I can just add a follow-up for that...

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-23 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Blocks||40659


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40659
[Bug 40659] Allow "My virtual card" format and content to be customizable
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=40659

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #80 from Lucas Gass (lukeg)  ---
(In reply to David Cook from comment #78)
> Lucas, do you think you could do a patch to force the editor to just be the
> code editor and not the WYSIWYG? Just to help protect people from themselves
> a bit?

Yes! I made a follow-up to disable the WYSIWYG for this.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Mathieu Saby  changed:

   What|Removed |Added

 CC||[email protected]

--- Comment #81 from Mathieu Saby  ---
I did not read all the comments but I am not sure to understand the purpose of
this bug.
In my library we wanted to display some new fields in our records, and to hide
others, and we just made changes to the XSLT files. It's very easy to do,
especially with the help of an LLM. Why add a new way to manage the display of
records?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #82 from Lucas Gass (lukeg)  ---
(In reply to Mathieu Saby from comment #81)
> I did not read all the comments but I am not sure to understand the purpose
> of this bug.
> In my library we wanted to display some new fields in our records, and to
> hide others, and we just made changes to the XSLT files. It's very easy to
> do, especially with the help of an LLM. Why add a new way to manage the
> display of records?

Mathieu, thanks for having a look here! 

Using custom XSLT is not a sustainable solution for us. First of all, it is not
something that can be done by most library staff. Writing XSLT is a technical
bar for many. Template Toolkit is much easier and is something that many
librarians are already familiar with from notices and slips. 

Custom XSLT also must be rebased against changes to the XSLT that happen in
community. This can take a lot of time OR you end up with XSLT files that are
far behind the community versions.

So the purpose of this bug, and the reason to add a new way to manage record
display, is to provide a tool that librarians can use to modify there record
display in a sustainable way.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-18 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #79 from Lucas Gass (lukeg)  ---
Created attachment 187898
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=187898&action=edit
Bug 39860: Disable WYSIWYG for record display entries

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-17 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

  Text to go in the||Adds a way to enhance
  release notes||bibliographic displays
   ||including the staff
   ||interface result and detail
   ||pages, and the OPAC's
   ||result and detail pages.
   ||This is done through
   ||"Tools" > "Additional
   ||content" >  "Record display
   ||customizations".

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-10-17 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #83 from David Cook  ---
(In reply to Lucas Gass (lukeg) from comment #82)
> (In reply to Mathieu Saby from comment #81)
> > I did not read all the comments but I am not sure to understand the purpose
> > of this bug.
> > In my library we wanted to display some new fields in our records, and to
> > hide others, and we just made changes to the XSLT files. It's very easy to
> > do, especially with the help of an LLM. Why add a new way to manage the
> > display of records?
>
> Using custom XSLT is not a sustainable solution for us. 

When supporting many Koha instances across many platforms around the world,
it's also quite onerous for devs to push changes for a tiny little MARC display
customization.

I think this enhancement will improve maintainability and customization of
Koha. 

I'll be doing another round of review today on this, and hopefully moving it
forward!

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-09-07 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Katie Bliss  changed:

   What|Removed |Added

 CC||[email protected]

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-09-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Michael Adamyk  changed:

   What|Removed |Added

 CC||michael.adamyk@bywatersolut
   ||ions.com

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-31 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #78 from David Cook  ---
Lucas, do you think you could do a patch to force the editor to just be the
code editor and not the WYSIWYG? Just to help protect people from themselves a
bit?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-31 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #77 from David Cook  ---
(In reply to Anneli Österman from comment #76)
> I'm still a bit worried about how poorly WYSIWYG handles things but
> otherwise everything worked according to test plan so I signed this off.

That's a good point actually. Maybe we should add a patch that forces the code
editor for this type of HTML customization since it just can't work with
WYSIWYG at this stage. 

For bug 11583 I worked on making the WYSIWYG editor compatible with the <<>>
and [%%] syntax, but... it couldn't work with Template Toolkit embedded in
HTML.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #76 from Anneli Österman  ---
I'm still a bit worried about how poorly WYSIWYG handles things but otherwise
everything worked according to test plan so I signed this off.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

OpenFifth Sandboxes  changed:

   What|Removed |Added

 Attachment #185869|0   |1
is obsolete||

--- Comment #74 from OpenFifth Sandboxes  ---
Created attachment 185904
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185904&action=edit
Bug 39860: Add record_display srubber profile

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

OpenFifth Sandboxes  changed:

   What|Removed |Added

 Attachment #185865|0   |1
is obsolete||

--- Comment #70 from OpenFifth Sandboxes  ---
Created attachment 185900
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185900&action=edit
Bug 39860: Allow for the defining of custom MARC fields on the staff interface
record display pages

To test:
0. APPLY PATCH, restart_all
1. Find or create a record that has some MARC fields that don't show up by
default in the record display. ( I tested using a 538$a )
2. Go to Tools and you should see a new link for 'Record display
customizations'.
3. From Record display customizations click 'New Entry'
4. Choose a display location, the choices should be StaffResultsPage,
StaffDetailPage, OPACResultsPage, and OPACDetailPage
5. Add the following to the entry:

[% IF record.subfield('538' , 'a') %]
  
  The 538a: 
  [% record.subfield('538' , 'a') %]
  
[% END %]

6. Go to the record from step 1, you should see the 538$a labeled as 'The
538a:' with the appropriate data in it.
7. This should display in the appropriate location depending on if you chose
StaffResultsPage, StaffDetailPage, OPACResultsPage, or OPACDetailPage. Check
each of the locations.
8. Make a new, or edit existing, Record display customization and mark the
'Library:' field for a specific branch. Make sure that the Record display
customization only shows up when you are logged in at that branch.
9. Install a new language pack, I tested with es-ES.
10. Make a new, or edit existing, Record display customization and fill out the
language specific tab. Make sure the Record display customization shows
correctly with the language you chose.

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

OpenFifth Sandboxes  changed:

   What|Removed |Added

 Attachment #185870|0   |1
is obsolete||

--- Comment #75 from OpenFifth Sandboxes  ---
Created attachment 185905
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185905&action=edit
Bug 39860: Update scrubber profile and add Scrubber.t test

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

OpenFifth Sandboxes  changed:

   What|Removed |Added

 Attachment #185868|0   |1
is obsolete||

--- Comment #73 from OpenFifth Sandboxes  ---
Created attachment 185903
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185903&action=edit
Bug 39860: Add options for opac/staff lists

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

OpenFifth Sandboxes  changed:

   What|Removed |Added

 Attachment #185867|0   |1
is obsolete||

--- Comment #72 from OpenFifth Sandboxes  ---
Created attachment 185902
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185902&action=edit
Bug 39860: Fix OPAC display location

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-29 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

OpenFifth Sandboxes  changed:

   What|Removed |Added

 Attachment #185866|0   |1
is obsolete||

--- Comment #71 from OpenFifth Sandboxes  ---
Created attachment 185901
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185901&action=edit
Bug 39860: Move display control from system preference to additional contents

Signed-off-by: David Cook 
Signed-off-by: Anneli Österman 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Status|Needs Signoff   |Signed Off

--- Comment #69 from David Cook  ---
Thanks for all your work on this and your patience, Lucas. It looks great. 

If someone else wants to Sign Off on this, I'd be happy to upgrade mine to a
Passed QA.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #185835|0   |1
is obsolete||

--- Comment #68 from David Cook  ---
Created attachment 185870
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185870&action=edit
Bug 39860: Update scrubber profile and add Scrubber.t test

Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #183774|0   |1
is obsolete||

--- Comment #67 from David Cook  ---
Created attachment 185869
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185869&action=edit
Bug 39860: Add record_display srubber profile

Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #183011|0   |1
is obsolete||

--- Comment #66 from David Cook  ---
Created attachment 185868
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185868&action=edit
Bug 39860: Add options for opac/staff lists

Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #183010|0   |1
is obsolete||

--- Comment #65 from David Cook  ---
Created attachment 185867
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185867&action=edit
Bug 39860: Fix OPAC display location

Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #183009|0   |1
is obsolete||

--- Comment #64 from David Cook  ---
Created attachment 185866
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185866&action=edit
Bug 39860: Move display control from system preference to additional contents

Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Attachment #183008|0   |1
is obsolete||

--- Comment #63 from David Cook  ---
Created attachment 185865
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185865&action=edit
Bug 39860: Allow for the defining of custom MARC fields on the staff interface
record display pages

To test:
0. APPLY PATCH, restart_all
1. Find or create a record that has some MARC fields that don't show up by
default in the record display. ( I tested using a 538$a )
2. Go to Tools and you should see a new link for 'Record display
customizations'.
3. From Record display customizations click 'New Entry'
4. Choose a display location, the choices should be StaffResultsPage,
StaffDetailPage, OPACResultsPage, and OPACDetailPage
5. Add the following to the entry:

[% IF record.subfield('538' , 'a') %]
  
  The 538a: 
  [% record.subfield('538' , 'a') %]
  
[% END %]

6. Go to the record from step 1, you should see the 538$a labeled as 'The
538a:' with the appropriate data in it.
7. This should display in the appropriate location depending on if you chose
StaffResultsPage, StaffDetailPage, OPACResultsPage, or OPACDetailPage. Check
each of the locations.
8. Make a new, or edit existing, Record display customization and mark the
'Library:' field for a specific branch. Make sure that the Record display
customization only shows up when you are logged in at that branch.
9. Install a new language pack, I tested with es-ES.
10. Make a new, or edit existing, Record display customization and fill out the
language specific tab. Make sure the Record display customization shows
correctly with the language you chose.

Signed-off-by: David Cook 

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #62 from David Cook  ---
(In reply to David Cook from comment #60)
> I still don't understand how you can specify a key like 'h1|h2|h3|h4|h5|h6'
> or 'ul|ol' but it appears to work. I'll have to dig into that more later...

Oh man... I need to stop using Splinter Review and just look at the patches. I
totally missed that C4/Scrubber.pm was updated in that last patch to not use
'h1|h2|h3|h4|h5|h6' or 'ul|ol'.

I like to say to people if something seems like it's magic or like you're going
crazy, you're probably just looking at the wrong thing. And that's proven to be
true once again...

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #61 from Lucas Gass (lukeg)  ---
> Looking at the rules, I wasn't familiar with "property", "typeof", and
> "resource" but then I recalled we have some use of Schema.org with microdata
> and RDFa. I can't imagine those being needed in many cases, but I suppose I
> can for record display particularly.

Yes, when I was thinking about this scrubber profile I wanted to make sure
staff could add everything to a custom record display that is currently
included in the default record display.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-28 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #60 from David Cook  ---
I've tried out the patches and it seems to work. 

I still don't understand how you can specify a key like 'h1|h2|h3|h4|h5|h6' or
'ul|ol' but it appears to work. I'll have to dig into that more later...

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-27 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #59 from David Cook  ---
Thanks again for all your work on this one.

I'm going to try to get this one soon. 

I was actually setting up a new local scrubber recently, and it's given me some
thoughts on ways of defining profiles in more concise ways. 

Looking at the rules, I wasn't familiar with "property", "typeof", and
"resource" but then I recalled we have some use of Schema.org with microdata
and RDFa. I can't imagine those being needed in many cases, but I suppose I can
for record display particularly.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-08-27 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #58 from Lucas Gass (lukeg)  ---
Created attachment 185835
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=185835&action=edit
Bug 39860: Update scrubber profile and add Scrubber.t test

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-07-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #57 from David Cook  ---
(In reply to Lucas Gass (lukeg) from comment #56)
> David,
> 
> I have written a more comprehensive allow list for 'record display'. It is
> based on the elements I find in the details and results pages. The idea is
> to be able to use the elements I find there to create similar custom
> displays.

I've sent you some feedback on Mattermost.

Overall, I think it's looking OK although there are some rules missing for
elements like "img" and the table related elements.

Are you sure that you can use regex for the key names in the rules? A glance at
HTML::Scrubber source code suggests that won't work. 

I would suggest adding a unit test for the C4/Scrubber.pm to test your profile
to make sure it's working the way you expect it to work. 

Overall, I think it's looking good though.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-07-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #56 from Lucas Gass (lukeg)  ---
David,

I have written a more comprehensive allow list for 'record display'. It is
based on the elements I find in the details and results pages. The idea is to
be able to use the elements I find there to create similar custom displays.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-07-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Attachment #183077|0   |1
is obsolete||

--- Comment #55 from Lucas Gass (lukeg)  ---
Created attachment 183774
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183774&action=edit
Bug 39860: Add record_display srubber profile

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-07-03 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-23 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860
Bug 39860 depends on bug 39184, which changed state.

Bug 39184 Summary: Server-side template injection leading to remote code 
execution
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39184

   What|Removed |Added

 Status|Pushed to oldoldstable  |RESOLVED
 Resolution|--- |FIXED

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-12 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #54 from David Cook  ---
(Apologies if it takes me some time to respond. My mail server is blocking
Bugzilla so I'm not getting email notifications at this time.)

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-12 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #53 from David Cook  ---
(In reply to Lisette Scheer from comment #52)
> (In reply to David Cook from comment #51)
> > So the WYSIWYG thing is an issue for sure. 
> > 
> > We might want to make a change to the HTML customizations feature (or rather
> > DB schema) that allows us to specify that some can only be edited using the
> > text editor and not the WYSIWYG I reckon...
> 
> In my in progress work on css and js snippets I'm doing this to make them
> always load the text:
> 
> [% IF ( wysiwyg ) && category != 'js_snippets' && category != 'css_snippets'
> %]
> [% SET editmode = "wysiwyg" %]

Hard-coding might work for a POC but I think we'll probably want to put that
into the database.

> I worry about being too restrictive when the library is already going to be
> vetting employees. 

I don't think most libraries worldwide are vetting their employees very
carefully or at all. 

> Could we compromise by doing a more restrictive scrub unless you have
> specific permissions and not allow editing of ones that would be scrubbed if
> you didn't have the right permission?

Honestly, I'd say both-and rather than either-or. The person editing the UI
should have higher permissions than someone adding a news item. But it should
still be scrubbed either way. Content management systems do this, and if we're
going to offer content management, we need to do it too.

--

Does Bywater already have this feature locally? If so, then it should be easy
to come up with a list of elements and attributes to allow list. If not, then I
think we use the Wordpress list as a first pass. If there is concern that it
won't be comprehensive enough, then we need to make a way to re-configure it on
the backend.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-11 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lisette Scheer  changed:

   What|Removed |Added

 CC||[email protected]
   ||m

--- Comment #52 from Lisette Scheer  ---
(In reply to David Cook from comment #51)
> So the WYSIWYG thing is an issue for sure. 
> 
> We might want to make a change to the HTML customizations feature (or rather
> DB schema) that allows us to specify that some can only be edited using the
> text editor and not the WYSIWYG I reckon...

In my in progress work on css and js snippets I'm doing this to make them
always load the text:

[% IF ( wysiwyg ) && category != 'js_snippets' && category != 'css_snippets' %]
[% SET editmode = "wysiwyg" %]

I worry about being too restrictive when the library is already going to be
vetting employees. 
Could we compromise by doing a more restrictive scrub unless you have specific
permissions and not allow editing of ones that would be scrubbed if you didn't
have the right permission?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860
Bug 39860 depends on bug 40089, which changed state.

Bug 40089 Summary: Add a less permissive C4::Scrubber profile to replace staff
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40089

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |INVALID

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #42 from Lucas Gass (lukeg)  ---
David, 

I added a specific 'record_display' scrubber profile. Let me know if you think
that is going to restrictive enough.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #41 from Lucas Gass (lukeg)  ---
Created attachment 183077
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183077&action=edit
Bug 39860: Add record_display srubber profile

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #40 from Lucas Gass (lukeg)  ---
Maybe what we need is a profile for all additional contents that was based on
things to deny, instead of things to allow?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #39 from Lucas Gass (lukeg)  ---
(In reply to David Cook from comment #32)
> Sorry but I'm going to fail this one again.
> 
> I think 3/4 of my original issues from Comment 10 have been resolved, which
> is awesome.
> 
> But I notice we're still using the "staff" profile for the HTML scrubber,
> and that's not going to prevent XSS, because it allows everything. 
> 
> In fact... I don't know why that profile was ever created. It looks like it
> goes back to the original creating of C4::Scrubber at f8fecb78634
> 
> Looking at existing use of the C4::Scrubber... we're using the profiles
> "note", "comment", and "default". I think we should actually remove the
> "staff" profile. I'll add a new bug for that...

I'm just curious why we scrub nothing in HTML customization, News, or Pages (
the other additional contents ) but need to scrub here?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #37 from Lucas Gass (lukeg)  ---
(In reply to Anneli Österman from comment #35)
> I tested this in a sandbox and it did not work as expected. I added the 538a
> and the results were like in attached picture detailpage.png.

What exactly did you add? It looks like it was for the OPAC detail page? I am
unable to recreate this.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Depends on||40089


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40089
[Bug 40089] Add a less permissive C4::Scrubber profile to replace staff
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-06 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #38 from Lucas Gass (lukeg)  ---
(In reply to Anneli Österman from comment #35)
> I tested this in a sandbox and it did not work as expected. I added the 538a
> and the results were like in attached picture detailpage.png.

Oh, I think maybe you copied my example into the WYSIWYG?

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #36 from Anneli Österman  ---
Created attachment 183035
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183035&action=edit
Detail page after adding 538a

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #35 from Anneli Österman  ---
I tested this in a sandbox and it did not work as expected. I added the 538a
and the results were like in attached picture detailpage.png.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #34 from David Cook  ---
(In reply to David Cook from comment #32)
> But I notice we're still using the "staff" profile for the HTML scrubber,
> and that's not going to prevent XSS, because it allows everything. 
> 
> In fact... I don't know why that profile was ever created. It looks like it
> goes back to the original creating of C4::Scrubber at f8fecb78634
> 
> Looking at existing use of the C4::Scrubber... we're using the profiles
> "note", "comment", and "default". I think we should actually remove the
> "staff" profile. I'll add a new bug for that...

I've opened bug 40087 to remove the "staff" profile.

I think this bug report should add its own profile to C4::Scrubber. I know it's
hard to think up a list of allowed elements. I do think we need better ways of
allowing for per-instance configuration of C4::Scrubber profiles (while
retaining sensible secure defaults), but for now... especially since this is a
new feature... I think we start with a small list and build up.

I can't find it anymore, but once upon a time I swear I wrote a patch to make
C4::Scrubber per-instance configurable, and I had a comprehensive list of safe
elements. I'm sorry I can't find it now. 

I'd also encourage you to look at bug 38498. I don't have a patch there, but
it's important to keep in mind. That is, we need to block many attributes (for
instance, anything that starts with "on"), but it would be nice to be able to
use some attributes like "id" and "class" at the very least, so we can use CSS.

Happy to work with you on this one...

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

   See Also||https://bugs.koha-community
   ||.org/bugzilla3/show_bug.cgi
   ||?id=40087

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #33 from David Cook  ---
Comment on attachment 183008
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183008
Bug 39860: Allow for the defining of custom MARC fields on the staff interface
record display pages

Review of attachment 183008:
 --> 
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=39860&attachment=183008)
-

::: koha-tmpl/intranet-tmpl/prog/en/xslt/MARC21slim2intranetDetail.xsl
@@ +14,3 @@
>  
>  
>  

Also this looks like an unnecessary whitespace change to this file.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

David Cook  changed:

   What|Removed |Added

 Status|Needs Signoff   |Failed QA

--- Comment #32 from David Cook  ---
Sorry but I'm going to fail this one again.

I think 3/4 of my original issues from Comment 10 have been resolved, which is
awesome.

But I notice we're still using the "staff" profile for the HTML scrubber, and
that's not going to prevent XSS, because it allows everything. 

In fact... I don't know why that profile was ever created. It looks like it
goes back to the original creating of C4::Scrubber at f8fecb78634

Looking at existing use of the C4::Scrubber... we're using the profiles "note",
"comment", and "default". I think we should actually remove the "staff"
profile. I'll add a new bug for that...

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #31 from Lucas Gass (lukeg)  ---
Created attachment 183011
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183011&action=edit
Bug 39860: Add options for opac/staff lists

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #29 from Lucas Gass (lukeg)  ---
Moved this over to additional contents and resetting the status to 'Needs
sign-off'.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Depends on||39184


Referenced Bugs:

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39184
[Bug 39184] Server-side template injection leading to remote code execution
-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #30 from Lucas Gass (lukeg)  ---
Created attachment 183010
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183010&action=edit
Bug 39860: Fix OPAC display location

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Attachment #183007|0   |1
is obsolete||

--- Comment #28 from Lucas Gass (lukeg)  ---
Created attachment 183009
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183009&action=edit
Bug 39860: Move display control from system preference to additional contents

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Status|Failed QA   |Needs Signoff

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Attachment #182233|0   |1
is obsolete||

--- Comment #27 from Lucas Gass (lukeg)  ---
Created attachment 183008
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183008&action=edit
Bug 39860: Allow for the defining of custom MARC fields on the staff interface
record display pages

To test:
0. APPLY PATCH, restart_all
1. Find or create a record that has some MARC fields that don't show up by
default in the record display. ( I tested using a 538$a )
2. Go to Tools and you should see a new link for 'Record display
customizations'.
3. From Record display customizations click 'New Entry'
4. Choose a display location, the choices should be StaffResultsPage,
StaffDetailPage, OPACResultsPage, and OPACDetailPage
5. Add the following to the entry:

[% IF record.subfield('538' , 'a') %]
  
  The 538a: 
  [% record.subfield('538' , 'a') %]
  
[% END %]

6. Go to the record from step 1, you should see the 538$a labeled as 'The
538a:' with the appropriate data in it.
7. This should display in the appropriate location depending on if you chose
StaffResultsPage, StaffDetailPage, OPACResultsPage, or OPACDetailPage. Check
each of the locations.
8. Make a new, or edit existing, Record display customization and mark the
'Library:' field for a specific branch. Make sure that the Record display
customization only shows up when you are logged in at that branch.
9. Install a new language pack, I tested with es-ES.
10. Make a new, or edit existing, Record display customization and fill out the
language specific tab. Make sure the Record display customization shows
correctly with the language you chose.

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

 Attachment #182232|0   |1
is obsolete||

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

--- Comment #26 from Lucas Gass (lukeg)  ---
Created attachment 183007
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=183007&action=edit
Bug 39860: Move display control from system preference to additional contents

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


[Koha-bugs] [Bug 39860] Add a way to allow for additional/custom MARC fields in the record display

2025-06-05 Thread bugzilla-daemon--- via Koha-bugs
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39860

Lucas Gass (lukeg)  changed:

   What|Removed |Added

Summary|Add a system preference to  |Add a way to allow for
   |allow for addtional/custom  |additional/custom MARC
   |MARC fields in the record   |fields in the record
   |display |display

-- 
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/