The Materials specified (field 3) is show on the staff catalog in each item row in the record view, but not on the patron side. This would be helpful to see on the patron side. As it is now, patrons can see one general description of items in the bib record from field 300e. However, if there were some minor discrepancies between items (fewer discs, no bonus disc), your choices here are to either hope no one cares, or create a separate bib record. With this item field showing, you could list minor material differences. Christopher
I think it would be good to make this optional and not turn on by default for existing installation as the field might have been used to store informaton that should not be visible for patrons in the OPAC in the past.
(In reply to Katrin Fischer from comment #1) > I think it would be good to make this optional and not turn on by default > for existing installation as the field might have been used to store > informaton that should not be visible for patrons in the OPAC in the past. Agreed. Christopher
Tested sandbox, modded to production 20 June 2015. This allows the use of 952$3 (we use for e.g. v. hb [1384 p., 85 col. plates]) in the OPAC. It is already in staff. /usr/share/koha/opac/htdocs/opac-tmpl/prog/en/modules/opac-detail.tt line 713 insert [% IF itemdata_materials %]<th id="item_materials">Materials specified</th>[% END %] line 739 insert [% IF itemdata_materials %]<td>[% ITEM_RESULT.materials %]</td>[% END %] chown koha:koha, chmod 644 /usr/share/koha/opac/cgi-bin/opac/opac-detail.pl line 513 add 'materials' foreach (qw(ccode enumchron materials copynumber itemnotes uri)) { ^^^^^^^^^ line 561 insert itemdata_materials => $itemfields{materials}, chown koha:koha, chmod 755
Hi, With 18.05, item_materials is not in column configuration for holdingst. could it be added? Regards, Marjorie
(In reply to Marjorie Barry-Vila from comment #4) > With 18.05, item_materials is not in column configuration for holdingst. > > could it be added? +1
Created attachment 100199 [details] [review] Bug 13547: Item field 3 (Materials specified) would be useful to see on OPAC Test plan: 1- View a record with Materials specified (field 3) data in the opac 2- Apply patch 3- Log in to staff client 4- Home->Administration->Column Settings->OPAC->holdingst 5- Set item_materials visibility 6- Refresh OPAC page 7- Confirm that the materials specified column has been added after the Call number column.
Created attachment 100279 [details] [review] Bug 13547: Add item field 3 (Materials specified) to the OPAC holdings table Test plan: 1- View a record with Materials specified (field 3) data in the opac 2- Apply patch 3- Log in to staff client 4- Home->Administration->Column Settings->OPAC->holdingst 5- Set item_materials visibility 6- Refresh OPAC page 7- Confirm that the materials specified column has been added after the Call number column. Signed-off-by: Owen Leonard <oleonard@myacpl.org>
This patch works as expected, but there is a little catch: For libraries, that have used this field to add information that is patron privacy relevant (like the name of someone), this change might be problematic. I have seen this happening, as the field will show up on checkout/checkin in a prominent way. (When returned, hand over to X next or something like that). Even when the materials column is hidden, it's still fully visible in the source code. Would it be possible not to pass it to the template if the column configuration is set to not display it?
(In reply to Katrin Fischer from comment #8) > This patch works as expected, but there is a little catch: For libraries, > that have used this field to add information that is patron privacy relevant > (like the name of someone), this change might be problematic. I have seen > this happening, as the field will show up on checkout/checkin in a prominent > way. (When returned, hand over to X next or something like that). > > Even when the materials column is hidden, it's still fully visible in the > source code. > > Would it be possible not to pass it to the template if the column > configuration is set to not display it? Is it fair to withhold this functionality because some might be using it for something it wasn't intended for? How many other fields are used in a similar fashion? I can understand this caution on a field that was meant to hold sensitive information, but I am not sure about this one. Maybe it could be suppressed with column configuration, but unless we treat other fields the same way, this suppression might get lost in future upgrades because it doesn't coincide with other column practices.
Hi Christopher, I am not withholding anything - I've not set this to Failed QA even. Only checking if we might be able to fix it in an easy way before we push this. The problem is, we only have one field right now that is not visible in the OPAC and shows up on checkin/circulation - so I wouldn't blame anyone using it for different purposes. I won't block this, but I felt it was worth asking.
Koha developments most often (if not always) keep one eye towards current users as well as future users. At times it's hard to balance the needs of a library (or libraries) and workflows that seem so important with the needs and workflows of the many. The things that seem obvious to me are not always obvious to others. I rely on the input of others in the community to give perspective on how other libraries operate. I believe someone once said "show me a library with a mad scheme and I'll show you another with a madder one"! Speaking from someone with experience migrating thousands of libraries to Koha, I can confidently say that libraries have indeed used this field with the expectation that it was NOT visible in the OPAC. There is the distinct possibility that patron information (or staff information/direction) is available in this field. A number of libraries come to my mind who have been using Koha for years in a way that is built upon the original design (not visible in opac). If this is to be implemented and visible in the OPAC, it must be made so it is optional/configurable.
(In reply to Joy Nelson from comment #11) > Koha developments most often (if not always) keep one eye towards current > users as well as future users. At times it's hard to balance the needs of a > library (or libraries) and workflows that seem so important with the needs > and workflows of the many. The things that seem obvious to me are not > always obvious to others. I rely on the input of others in the community to > give perspective on how other libraries operate. I believe someone once > said "show me a library with a mad scheme and I'll show you another with a > madder one"! > > Speaking from someone with experience migrating thousands of libraries to > Koha, I can confidently say that libraries have indeed used this field with > the expectation that it was NOT visible in the OPAC. There is the distinct > possibility that patron information (or staff information/direction) is > available in this field. A number of libraries come to my mind who have > been using Koha for years in a way that is built upon the original design > (not visible in opac). > > If this is to be implemented and visible in the OPAC, it must be made so it > is optional/configurable. I guess then there should be a configuration. Anything to move this forward.
Created attachment 102014 [details] [review] Bug 13547: (follow-up) Item field 'Materials specified' would be useful to see in OPAC We've changed the patch adding the possibility of control for its visibility at opac-detail.tt with is_hidden function added to the Koha/Template/Plugin/ColumnsSettings.pm file
Created attachment 102015 [details] [review] Bug 13547: (follow-up) Item field 'Materials specified' would be useful to see in OPAC We've changed the patch adding the possibility of control for its visibility at opac-detail.tt with is_hidden function added to the Koha/Template/Plugin/ColumnsSettings.pm file
Created attachment 103172 [details] [review] Bug 13547: (follow-up) Item field 'Materials specified' would be useful to see in OPAC Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Thanks Owen
Thx a lot for the follow-up - I am really happy to see that this could be 'fixed'! I really want to PQA on this and will attach signed-off patches. The only problem I see is that we don't have unit tests for the new method (that I am to blame for). I will leave this in Signed off hoping that a quick follow-up can be provided.
Created attachment 103188 [details] [review] Bug 13547: Add item field 3 (Materials specified) to the OPAC holdings table Test plan: 1- View a record with Materials specified (field 3) data in the opac 2- Apply patch 3- Log in to staff client 4- Home->Administration->Column Settings->OPAC->holdingst 5- Set item_materials visibility 6- Refresh OPAC page 7- Confirm that the materials specified column has been added after the Call number column. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 103189 [details] [review] Bug 13547: (follow-up) Item field 'Materials specified' would be useful to see in OPAC Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 104402 [details] [review] Bug 13547: (follow-up) Improve POD for template plugin
Created attachment 104403 [details] [review] Bug 13547: (QA follow-up) Add basic tests
With those two followups adding tests and improving the POD I think we can safely PQA
https://wiki.koha-community.org/wiki/Holdings_data_fields_(9xx) likely needs updating to reflect this bug.
(In reply to Martin Renvoize from comment #23) > https://wiki.koha-community.org/wiki/Holdings_data_fields_(9xx) likely needs > updating to reflect this bug. Updated.
Hi Katrin, Bug number is wrong in the document.
(In reply to Devinim from comment #25) > Hi Katrin, > Bug number is wrong in the document. Thank you - I guess I copied from the wrong tab. Now fixed!
Nice work everyone! Pushed to master for 20.05
enhancement not backported to 19.11