Show patrons age in checkout search, member search and patrons circulation details (top left menu).
Created attachment 45882 [details] [review] Bug 15400: Show patron age in search results Display patron age in search results and checkout details. Test plan ========= 1. Go to checkout page for one patron (cgi-bin/koha/circ/circulation.pl) 2. In the circulation menu to the left, under the patrons name you should see a new field "Age:" showing the patrons age, if a date of birth is available. 3. Enter a search term that will match more than one patron in the checkout search field 4. A new table column "Age" should be visible and display each patrons age 5. In cgi-bin/koha/members/member.pl enter a search filter that will match more than one patron 6. Each patrons age should be visible within parentheses to the right of the patron name
How is this related to "Bug 15206 - Show patron's age when filling date of birth in memberentry.pl"
They look related, but not contradicting or overlapping. Bug 15206 seems unnecessary code complex at first glance though.
Is this similar to Bug 14763, but in more places?
$ git bz apply 15400 Bug 15400 - display patron age in useful places 45882 - Bug 15400: Show patron age in search results Apply? [(y)es, (n)o, (i)nteractive] y Applying: Bug 15400: Show patron age in search results Using index info to reconstruct a base tree... M koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc M koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt M koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt Falling back to patching base and 3-way merge... Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/members/tables/members_results.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc Failed to merge in the changes. Patch failed at 0001 Bug 15400: Show patron age in search results
Also, I have a hunch this: </a>[% IF age %]([% age%])[% END %] might be better written as: </a>[% IF age %] ([% age %])[% END %] to 1. Give a space between the link and the opening parenthesis 2. Make the layout of TT tags consistent
Created attachment 46935 [details] [review] Bug 15400: Show patron age in search results Display patron age in search results and checkout details. Test plan ========= 1. Go to checkout page for one patron (cgi-bin/koha/circ/circulation.pl) 2. In the circulation menu to the left, under the patrons name you should see a new field "Age:" showing the patrons age, if a date of birth is available. 3. Enter a search term that will match more than one patron in the checkout search field 4. A new table column "Age" should be visible and display each patrons age 5. In cgi-bin/koha/members/member.pl enter a search filter that will match more than one patron 6. Each patrons age should be visible within parentheses to the right of the patron name NOTE: rebased to handle conflict with 13931, but I'm not sure this is fully needed, as 13931 adds date of birth.
I get a software error at Step 3-4 in your test plan: Template process failed: undef error - Date::Calc::Delta_YMD(): not a valid date at /usr/lib/perl5/Template/Plugin/Date.pm line 185. I think the patch for Bug 14763 to show the age on the details page is a better place than above the address like in this patch. I think that if we do want to show the age in brackets on member.pl search, maybe it should say (x years)? Doesn't make much sense to have all these random numbers next to names.
Created attachment 81892 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions
Created attachment 82714 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com>
Is this info really useful? For everybody? To me it seems unnecessary additional info
(In reply to Jonathan Druart from comment #11) > Is this info really useful? For everybody? > > To me it seems unnecessary additional info I think it's useful. The biggest issue I have is the wording takes up a lot of real estate. I think "Date of birth: 05/30/1958 (60 years)" could be changed to "Born: 05/30/1958 (60)", or "DOB: 05/30/1958 (60)" or something similar. Other than that, I think this is a good patch.
Created attachment 85432 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Created attachment 85479 [details] [review] Bug 15400: (follow-up) Updating text display Now shows as 'Born: [date of birth] (age)' in circ sidebar menu and search results Sponsored-by: Catalyst IT
Patch applies correctly except for "Check out" in the header (i still have "NAME, surname (cardnumber) birthdate adress"). Change to Failed [hoping not for too long :)]
Created attachment 85494 [details] [review] Bug 15400: (followup) Adding age to check out search dropdown Confirm age shows in the suggestions dropdown when using the checkout search in header (if user set DOB). Confirm 'Age unknown' shows if no DOB set.
Everything works as described now, thanks Aleisha !
Patch tested with a sandbox, by Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 85495 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 85496 [details] [review] Bug 15400: (follow-up) Updating text display Now shows as 'Born: [date of birth] (age)' in circ sidebar menu and search results Sponsored-by: Catalyst IT Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 85497 [details] [review] Bug 15400: (followup) Adding age to check out search dropdown Confirm age shows in the suggestions dropdown when using the checkout search in header (if user set DOB). Confirm 'Age unknown' shows if no DOB set. Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Comment on attachment 85497 [details] [review] Bug 15400: (followup) Adding age to check out search dropdown Review of attachment 85497 [details] [review]: ----------------------------------------------------------------- ::: circ/ysearch.pl @@ +85,4 @@ > firstname => $b->firstname // '', > cardnumber => $b->cardnumber // '', > dateofbirth => format_sqldatetime($b->dateofbirth, undef, undef, 1) // '', > + age => $b->get_age // 'Age unknown', This is bad, string 'Age unknown' is not translatable
Also, I do not like the 'years' word removed, but that's only my personal opinion
(In reply to Josef Moravec from comment #22) > Comment on attachment 85497 [details] [review] [review] > Bug 15400: (followup) Adding age to check out search dropdown > > Review of attachment 85497 [details] [review] [review]: > ----------------------------------------------------------------- > > ::: circ/ysearch.pl > @@ +85,4 @@ > > firstname => $b->firstname // '', > > cardnumber => $b->cardnumber // '', > > dateofbirth => format_sqldatetime($b->dateofbirth, undef, undef, 1) // '', > > + age => $b->get_age // 'Age unknown', > > This is bad, string 'Age unknown' is not translatable When woudl this display? I think if the user has no date of birth, we should just display 'nothing'. Thx Josef for spotting this! I also liked the 'years'.
Created attachment 86457 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 86458 [details] [review] Bug 15400: (follow-up) Updating text display Now shows as 'Born: [date of birth] (age)' in circ sidebar menu and search results Sponsored-by: Catalyst IT Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 86459 [details] [review] Bug 15400: (follow-up) Adding age to check out search dropdown Confirm age shows in the suggestions dropdown when using the checkout search in header (if user set DOB). Confirm 'Age unknown' shows if no DOB set. Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 86460 [details] [review] Bug 15400: (follow-up) Restore 'years' string, remove untranslatable string
Hi Josef, this works nicely, with one exception: on the guarantors search the number doesn't show, only ( years). As there has been some controversy about showing this information - could you consider wrapping a span with a class around the years part? Thx!
Created attachment 88274 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 88275 [details] [review] Bug 15400: (follow-up) Updating text display Now shows as 'Born: [date of birth] (age)' in circ sidebar menu and search results Sponsored-by: Catalyst IT Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 88276 [details] [review] Bug 15400: (follow-up) Adding age to check out search dropdown Confirm age shows in the suggestions dropdown when using the checkout search in header (if user set DOB). Confirm 'Age unknown' shows if no DOB set. Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Created attachment 88277 [details] [review] Bug 15400: (follow-up) Restore 'years' string, remove untranslatable string
Created attachment 88278 [details] [review] Bug 15400: (follow-up) Add class to years span
(In reply to Katrin Fischer from comment #29) > Hi Josef, > > this works nicely, with one exception: on the guarantors search the number > doesn't show, only ( years). Sorry, I can't reproduce it now... > > As there has been some controversy about showing this information - could > you consider wrapping a span with a class around the years part? Done in folow-up > > Thx! Thanks for testing ;)
> As there has been some controversy about showing this information - could > you consider wrapping a span with a class around the years part? Hi Josef, totally my fault for not being clear :( ... I meant a span around the _whole_ age (number + text). So you can choose not to see it. Guarantor search is ok now (not sure what happened before) - just this little bit left.
Created attachment 88845 [details] [review] Bug 15400: (follow-up) Add class to years span
Created attachment 88846 [details] [review] Bug 15400: (follow-up) Add class to years span
Created attachment 88977 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 88978 [details] [review] Bug 15400: (follow-up) Updating text display Now shows as 'Born: [date of birth] (age)' in circ sidebar menu and search results Sponsored-by: Catalyst IT Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 88979 [details] [review] Bug 15400: (follow-up) Adding age to check out search dropdown Confirm age shows in the suggestions dropdown when using the checkout search in header (if user set DOB). Confirm 'Age unknown' shows if no DOB set. Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 88980 [details] [review] Bug 15400: (follow-up) Restore 'years' string, remove untranslatable string Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 88981 [details] [review] Bug 15400: (follow-up) Add class to years span Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Needs a minor rebase
Created attachment 100047 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 100048 [details] [review] Bug 15400: (follow-up) Updating text display Now shows as 'Born: [date of birth] (age)' in circ sidebar menu and search results Sponsored-by: Catalyst IT Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 100049 [details] [review] Bug 15400: (follow-up) Adding age to check out search dropdown Confirm age shows in the suggestions dropdown when using the checkout search in header (if user set DOB). Confirm 'Age unknown' shows if no DOB set. Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 100050 [details] [review] Bug 15400: (follow-up) Restore 'years' string, remove untranslatable string Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 100051 [details] [review] Bug 15400: (follow-up) Add class to years span Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Rebased on master. This was PQA before, but as it was so long then I finally rebased it (sorry for that!) I am marking it back to signed off state.
*** Bug 24770 has been marked as a duplicate of this bug. ***
Created attachment 100065 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 100066 [details] [review] Bug 15400: (follow-up) Updating text display Now shows as 'Born: [date of birth] (age)' in circ sidebar menu and search results Sponsored-by: Catalyst IT Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 100067 [details] [review] Bug 15400: (follow-up) Adding age to check out search dropdown Confirm age shows in the suggestions dropdown when using the checkout search in header (if user set DOB). Confirm 'Age unknown' shows if no DOB set. Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 100068 [details] [review] Bug 15400: (follow-up) Restore 'years' string, remove untranslatable string Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com>
Created attachment 100069 [details] [review] Bug 15400: (follow-up) Add class to years span Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com>
+ [%- IF ( patron.get_age ) %] ([% patron.get_age | html %] years) + [%- ELSIF ( patron.age ) %] ([% patron.age | html %] years)[% END -%] That smells. You should either test for the type of patron, or pass another variable patron_object vs patron or patron vs borrower (that reads wrong but it is what is usually done in controllers)
Created attachment 100118 [details] [review] Bug 15400: Clarify usage of patron-age.inc
(In reply to Jonathan Druart from comment #57) > + [%- IF ( patron.get_age ) %] ([% patron.get_age | html %] years) > + [%- ELSIF ( patron.age ) %] ([% patron.age | html %] years)[% END > -%] > > That smells. > You should either test for the type of patron, or pass another variable > patron_object vs patron > or > patron vs borrower (that reads wrong but it is what is usually done in > controllers) Sorry, I was reading the patch and not the final change. I added a follow-up. However I still think it's lot of info that could be considered not needed by some. Especially in the circ autocomplete. Letting someone else adding the QA stamp.
Applying these patches, patron search does no longer work for me: SyntaxError: JSON.parse: bad control character in string literal at line 22 column 20 of the JSON data Please double check!
I'm getting a different error that I'm not sure how to fix Source map error: Error: request failed with status 404 Resource URL: http://localhost:8081/intranet-tmpl/lib/datatables/datatables.min_19.1200060.js Source Map URL: pdfmake.min.js.map Anyone?
Created attachment 103373 [details] [review] Bug 15400: (follow-up) Remove line break
(In reply to Katrin Fischer from comment #60) > Applying these patches, patron search does no longer work for me: > SyntaxError: JSON.parse: bad control character in string literal at line 22 > column 20 of the JSON data > > Please double check! I fixed the bad line break, should be OK now.
(In reply to Aleisha Amohia from comment #61) > I'm getting a different error that I'm not sure how to fix > > Source map error: Error: request failed with status 404 > Resource URL: > http://localhost:8081/intranet-tmpl/lib/datatables/datatables.min_19.1200060. > js > Source Map URL: pdfmake.min.js.map > > Anyone? I do not think it is related with this bug anyhow...
Created attachment 106133 [details] [review] Bug 15400: Display date of birth and age more consistantly Test plan: 0) Apply the patch 1) Go to all of these pages Patron detail Other patron pages - look on the left side (circ-menu) Patron search Guarantor search ( go to child patron -> edit -> in guarantor section click "Set to patron" Search through "Check out" (in the header) 2) Confirm that does show date of birth and date consistantly, try it on patrons with and without date of birth set to find possible reggressions Signed-off-by: Aleisha Amohia <aleishaamohia@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 106134 [details] [review] Bug 15400: (follow-up) Updating text display Now shows as 'Born: [date of birth] (age)' in circ sidebar menu and search results Sponsored-by: Catalyst IT Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 106135 [details] [review] Bug 15400: (follow-up) Adding age to check out search dropdown Confirm age shows in the suggestions dropdown when using the checkout search in header (if user set DOB). Confirm 'Age unknown' shows if no DOB set. Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 106136 [details] [review] Bug 15400: (follow-up) Restore 'years' string, remove untranslatable string Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 106137 [details] [review] Bug 15400: (follow-up) Add class to years span Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Michal Denar <black23@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 106138 [details] [review] Bug 15400: Clarify usage of patron-age.inc Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 106139 [details] [review] Bug 15400: (follow-up) Remove line break Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Created attachment 106140 [details] [review] Bug 15400: Add class on patron brief info to allow hiding date of birth This just makes a small addition by adding a class to the new "Born:" entry in the patron brief information area to allow for hiding it easily with css. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
There is a misalignment: https://snipboard.io/EqYJmM.jpg Why did you use the .label class for "Born"?
(In reply to Jonathan Druart from comment #73) > There is a misalignment: https://snipboard.io/EqYJmM.jpg > > Why did you use the .label class for "Born"? I do not really remember that.... so probably we could remove it, if someone want to hide the word "Born" it is possible through the class added in last Katrin's follow-up (thanks for that!)
Are you willing to fix the alignment issue, Josef?
Created attachment 106307 [details] [review] Bug 15400: (follow-up) Remove unnecessary class Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
(In reply to Jonathan Druart from comment #75) > Are you willing to fix the alignment issue, Josef? Yes, it it done now
Pushed to master for 20.11, thanks to everybody involved!
not backporting this enhancement to the 20.05.x series
(In reply to Aleisha Amohia from comment #61) > I'm getting a different error that I'm not sure how to fix > > Source map error: Error: request failed with status 404 > Resource URL: > http://localhost:8081/intranet-tmpl/lib/datatables/datatables.min_19.1200060. > js > Source Map URL: pdfmake.min.js.map While working on a datatable somehwere else, I noticed this warn/error too in the browser console. Looks to me that we are missing something in the datatables files somehow. But cant figure out what action specifically triggers it? Dont know why I dont see it on some other datatable pages?
Need a way to turn this off! This patch seems to have been added without any way to suppress the display of patron age or date of birth, and it is all over Koha. This library does not want people's age displayed every time we check out. That is sensitive personal information. Actually we do not collect it, so Koha now wastes screen space showing "No date of birth stored" in multiple places. Please add a preference to suppress this if required by each library. Thank you. ;-)
(In reply to Agnes Rivers-Moore from comment #81) > Need a way to turn this off! This patch seems to have been added without any > way to suppress the display of patron age or date of birth, and it is all > over Koha. This library does not want people's age displayed every time we > check out. That is sensitive personal information. Actually we do not > collect it, so Koha now wastes screen space showing "No date of birth > stored" in multiple places. Please add a preference to suppress this if > required by each library. Thank you. ;-) Hi Agnes, actually this has a way to turn it off. See here: class='age_years' So all you need to do is add this line to your IntranetUserCSS: .age_years { display:none; } Let me know if that works in all places you need it to and if somewhere is missing, I am sure we can figure it out.
I also expressed a couple of times that I didn't like the feature. Maybe we should turn it off by default?
I think for off by default we should add an actual preference instead of using CSS. But the CSS will hopefully help for now :) We had no comments this or that way from our mostly academic/special libraries. I imagine the feature might be interesting mostly for publics?
(In reply to Katrin Fischer from comment #84) > I think for off by default we should add an actual preference instead of > using CSS. But the CSS will hopefully help for now :) > > We had no comments this or that way from our mostly academic/special > libraries. I imagine the feature might be interesting mostly for publics? As I remember, the request for this development really came from public library.
(In reply to Jonathan Druart from comment #83) > I also expressed a couple of times that I didn't like the feature. Maybe we > should turn it off by default? Thank you Jonathan! I don't know any reason public libraries would want to see this in day-to-day operations. Some use age to set child or adult patron type, and sending a birthday message might be cute (I find it creepy). None of that is reason to see it daily, but libraries in another country might. (I have worked libraries in England, South Africa and Canada.)
There are two or three fields we need to suppress. One is the date of birth, one is the age in years, and the third is the message that displays if there is nothing to display. Born: 03/29/1951 (70 years) or No date of birth stored. We tried the CSS that Katrin suggested, and it does not cover all those cases. It would need to hide DOB and Age. Also as we do not collect DOB, we need to hide whatever generates the message if DOB is empty. That would likely be useful for libraries that collect DOB only for children, not adults. Thank you!
This is privacy issue too. GDPR etc. Note that we (as a special library) removed all birthdates from our system. Hopefully we dont have lots of uninitialized warnings too?
I don't think that displaying age and date of birth in the interface for librarians is in privacy with security and GDPR. It depends on how the library has the area covered in their documents. Our libraries use the age information in the normal course of business in some cases to verify the reader if they don't have a library card, etc. Certainly this feature can be optional, but it should be available.
I don't think that displaying age and date of birth in the interface for librarians is privacy issue or against GDPR rules. It depends on how the library has the area covered in their documents. Our libraries use the age information in the normal course of business in some cases to verify the reader if they don't have a library card, etc. Certainly this feature can be optional, but it should be available.
(In reply to Michal Denar from comment #90) > I don't think that displaying age and date of birth in the interface for > librarians is privacy issue or against GDPR rules. It depends on how the > library has the area covered in their documents. Our libraries use the age > information in the normal course of business in some cases to verify the > reader if they don't have a library card, etc. Certainly this feature can be > optional, but it should be available. Sure. s/is/could be/ More what I meant.
(In reply to Agnes Rivers-Moore from comment #87) > There are two or three fields we need to suppress. One is the date of birth, > one is the age in years, and the third is the message that displays if there > is nothing to display. > > Born: 03/29/1951 (70 years) or > No date of birth stored. > > We tried the CSS that Katrin suggested, and it does not cover all those > cases. It would need to hide DOB and Age. Also as we do not collect DOB, we > need to hide whatever generates the message if DOB is empty. That would > likely be useful for libraries that collect DOB only for children, not > adults. Thank you! Hi Agnes, the date of birth and the message are - as you say - separate issues. But we can also help with CSS there and maybe prefs there. There are some options. In the example above, the years bit should be gone now, correct? And you do collect the date of birth from your patrons, but you don't want to display it in circulation? Check for the HidePersonalPatronDetailOnCirculation system preference. It should do just that :)
(In reply to Katrin Fischer from comment #92) > (In reply to Agnes Rivers-Moore from comment #87) > > There are two or three fields we need to suppress. One is the date of birth, > > one is the age in years, and the third is the message that displays if there > > is nothing to display. > > > > Born: 03/29/1951 (70 years) or > > No date of birth stored. > > > > We tried the CSS that Katrin suggested, and it does not cover all those > > cases. It would need to hide DOB and Age. Also as we do not collect DOB, we > > need to hide whatever generates the message if DOB is empty. That would > > likely be useful for libraries that collect DOB only for children, not > > adults. Thank you! > > Hi Agnes, the date of birth and the message are - as you say - separate > issues. But we can also help with CSS there and maybe prefs there. > > There are some options. In the example above, the years bit should be gone > now, correct? > And you do collect the date of birth from your patrons, but you don't want > to display it in circulation? Check for the > HidePersonalPatronDetailOnCirculation system preference. It should do just > that :) Hi Katrin We have all of it hidden now with a combination of CSS and JS, but there is a flicker when the screen redraws. So, our library has a fix in, but still would like to see this as a preference. We do not collect DOB for all patrons. We don't want to hide ALL personal information - we need phone and email. The preference hides everything. We only want to hide age/DOB. Thanks!