https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |martin.renvoize@ptfs-europe | |.com --- Comment #1 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- *** Bug 22546 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #3 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 94425 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=94425&action=edit Bug 23849: [DO NOT PUSH] list of sysprefs already shared -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #5 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 94427 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=94427&action=edit Bug 23849: [DO NOT PUSH] Not in DB - comments -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #7 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- Created attachment 94429 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=94429&action=edit Bug 23849: [DO NOT PUSH] List of sysprefs in DB we do not share - comments -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #9 from Jonathan Druart <jonathan.druart@bugs.koha-community.org> --- (In reply to Jonathan Druart from comment #0)

We are going to have an updated list for 19.11. For next releases it should be a new post release task (https://wiki.koha-community.org/wiki/Post_major_release_tasks).

It should actually be a pre release task! -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Martin Renvoize <martin.renvoize@ptfs-europe.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact|testopia@bugs.koha-communit |katrin.fischer@bsz-bw.de |y.org | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #10 from Martin Renvoize <martin.renvoize@ptfs-europe.com> --- OK, I've gone through your lists and I agree with all your observations.. I was going to get Katrin to take a look too as she's a stickler for detail here :) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #12 from Paul Poulain <paul.poulain@biblibre.com> --- Comment on attachment 94428 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=94428 Bug 23849: [DO NOT PUSH] List of sysprefs in DB we do not share Looking at the list, I see that you made an SQL check. There are some sysprefs that we will never share, and even some that we can't share. For example casServerUrl, or BakerTaylorPassword. Where we wrote HEA, we selected only the ones we thought useful. I agree we must update this list, but maybe by checking those who have been introduced since HEA was pushed (and if someone think an old one is missing, then we still can add it) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #14 from Paul Poulain <paul.poulain@biblibre.com> --- (In reply to Jonathan Druart from comment #13)

(In reply to Paul Poulain from comment #11)

...

Comment on attachment 94426 [details] [review] [review] Bug 23849: [DO NOT PUSH] List of sysprefs we share but not exist in DB

Good job Jonathan ! looking at this list, they used to exist, so there's a value on hea for most of them. So the task is both remove them on Koha side, as well as on HEA

question: version, are you sure it doesn't exist ?

it's because of "version" vs "Version".

(In reply to Paul Poulain from comment #12)

...

Comment on attachment 94428 [details] [review] [review] Bug 23849: [DO NOT PUSH] List of sysprefs in DB we do not share

Looking at the list, I see that you made an SQL check. There are some sysprefs that we will never share, and even some that we can't share. For example casServerUrl, or BakerTaylorPassword. Where we wrote HEA, we selected only the ones we thought useful. I agree we must update this list, but maybe by checking those who have been introduced since HEA was pushed (and if someone think an old one is missing, then we still can add it)

Yes but it seems easier to redo the whole check and start again with a clean base. See the version of the patches " - comments" to see the ones I have removed from the list.

do you mean https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=94429&action=diff ? I've read the diff, most of the lines have a "> no" => does it mean we wouldn't share them on HEA ? (if yes, I agree) The only one that would be added is "AnonymousPatron" (and yes, I think it should be added) Idea : what about adding some statistics like "intranetuserjs yes/no" => yes if there's something, no if it's empty ? (intranetuserjs, opacuserjs, opac & intranet css, opacheader, opacnav, opacbottom). For sysprefs *XSLT* => we could send "empty/default/local". -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://bugs.koha-community | |.org/bugzilla3/show_bug.cgi | |?id=23897 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Jonathan Druart <jonathan.druart@bugs.koha-community.org> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also|https://bugs.koha-community |https://bugs.koha-community |.org/bugzilla3/show_bug.cgi |.org/bugzilla3/show_bug.cgi |?id=23897 |?id=23898 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Victor Grousset/tuxayo <victor@tuxayo.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |victor@tuxayo.net --- Comment #17 from Victor Grousset/tuxayo <victor@tuxayo.net> --- After reviewing in a local Koha all the sysprefs of the file "in_DB_but_not_shared" here are some notes. == Pretty sure they should be > No == Or just share if it's empty/default or custom. AdlibrisCoversURL EmailAddressForSuggestions ILS-DI:AuthorizedIPs NewItemsDefaultLocation OpacHiddenItems OpacHiddenItemsExceptions ReplyToDefault RestrictedPageLocalIPs RestrictedPageTitle ReturnpathDefault SelfCheckAllowByIPRanges UpdateItemLocationOnCheckin == Maybe false positive, could be yes (share) instead of no == OverDrivePasswordRequired OverDriveUsername NovelistSelectStaffEnabled NovelistSelectView OAI-PMH:AutoUpdateSets OAI-PMH:MaxCount opaclanguages OpacLocationOnDetail OPACResultsLibrary RESTOAuth2ClientCredentials SyndeticsAuthorNotes SyndeticsCoverImages SyndeticsCoverImageSize SyndeticsEditions SyndeticsExcerpt SyndeticsReviews SyndeticsSeries SyndeticsSummary SyndeticsTOC == We could just share if it's empty/default or custom == This list will be copied to bug 23898 AmazonAssocTag IntranetCirculationHomeHTML IntranetFavicon IntranetmainUserblock IntranetNav IntranetReportsHomeHTML IntranetSlipPrinterJS intranetstylesheet OpacCustomSearch OPACHoldsIfAvailableAtPickupExceptions OpacLoginInstructions OpacMainUserBlock OpacMoreSearches OPACResultsSidebar OPACSearchForTitleIn ProcessingFeeNote RestrictedPageContent RoutingListNote SpineLabelFormat XSLTDetailsDisplay == Not sure, does any know more about these sysprefs? == Can field lists be an issue? They might just be useless actually. Or we could just want to know if it's empty/default or custom. Precedent: NotesBlacklist is currently marked as > No AuthoritySeparator BibtexExportAdditionalFields ExportRemoveFields ISBD MarcFieldsToOrder MarcItemFieldsToOrder MergeReportFields OPACISBD PatronQuickAddFields RisExportAdditionalFields StatisticsFields SubfieldsToAllowForRestrictedBatchmod SubfieldsToAllowForRestrictedEditing SubfieldsToUseWhenPrefill SubscriptionDuplicateDroppedInput Similar doubts as previous paragraph: ItemsDeniedRenewal LinkerOptions MARCAuthorityControlField008 UNIMARCAuthorityField100 Are backends lists an issue? Or is it actually very useful? ILLOpacbackends Could the directory name be private and thus not sharable? intranet_includes Could some codes be so rare that they are equivalent of sharing the library name? MARCOrgCode That should be ok right? borrowerRelationship BorrowersTitles == We might want to filter the content of these == To only get Koha's official fields and not custom DB fields. They could be ignored now and be in a follow up bug. Or we could just want to know if it's empty/default or custom. BorrowerMandatoryField BorrowerUnwantedField DefaultPatronSearchFields UniqueItemFields Filter to only have Koha's official statues: decreaseLoanHighHoldsIgnoreStatuses == prefs with > no == They could be ignored now and in a follow up bug, we could share if it's empty/default or custom. == Maybe a typo in the list == ArticleRequestsMandatoryFieldsItemsOnly ↓ ArticleRequestsMandatoryFieldsItemOnly OpacSuppressionByIPRangeµ ↓ OpacSuppressionByIPRange == Not in the UI (Administration › System preferences) what does it mean? == ElasticsearchIndexStatus_authorities ElasticsearchIndexStatus_biblios INTRAdidyoumean OPACdidyoumean OpacMainUserBlock printcirculationslips opacheader -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #18 from Victor Grousset/tuxayo <victor@tuxayo.net> --- @Paul: What was the humorous movement/committee/proposal (I don't remember the nature) of yours that you invented years ago about reducing the number of sysprefs? Because I'm sign up! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #20 from Paul Poulain <paul.poulain@biblibre.com> --- It was FAUSP : "Fighters Against Useless System Preferences" :D -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #22 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hm, why is seearchengine not turning up in the list, didn't we start the discussion there? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #24 from Victor Grousset/tuxayo <victor@tuxayo.net> --- (In reply to Paul Poulain from comment #20)

It was FAUSP : "Fighters Against Useless System Preferences" :D

YES ###### (In reply to Katrin Fischer from comment #21)

...

== Pretty sure they should be > No == Or just share if it's empty/default or custom. AdlibrisCoversURL EmailAddressForSuggestions ILS-DI:AuthorizedIPs NewItemsDefaultLocation OpacHiddenItems OpacHiddenItemsExceptions ReplyToDefault RestrictedPageLocalIPs RestrictedPageTitle ReturnpathDefault SelfCheckAllowByIPRanges UpdateItemLocationOnCheckin

I'd go for not sharing whenever in doubt. I think they would not gain us much, but have the potential to make people insecure about sharing when turning up on Hea maybe?

«have the potential to make people insecure» If there was a way to know what was shared without looking at the code. ^^" But yes, if it's not useful let's not shared these.

...

== Maybe false positive, could be yes (share) instead of no == OverDrivePasswordRequired OverDriveUsername NovelistSelectStaffEnabled NovelistSelectView OAI-PMH:AutoUpdateSets OAI-PMH:MaxCount opaclanguages OpacLocationOnDetail OPACResultsLibrary RESTOAuth2ClientCredentials SyndeticsAuthorNotes SyndeticsCoverImages SyndeticsCoverImageSize SyndeticsEditions SyndeticsExcerpt SyndeticsReviews SyndeticsSeries SyndeticsSummary SyndeticsTOC

Everything that turns a feature on/off (boolean) is probably ok. Prefs set up with strings might not be very helpful and more dangerous.

It seems most of these are not useful, especially if there is a main syspref shared that already tells that OverDrive, Novelist, OAI-PMH, Syndetics is in use. I interpreted "> No" as: "there is/could be private info" But I should have filtered the above list to not put the obviously not useful ones.

opaclanguages would be interesting to see, but would need some extra processing as it's a string of language codes separated by pipe - as a string useless, you'd want to list how often which language is installed.

opaclanguages, yes very interesting! Should the processing be on Koha's side or Hea's side? So besides opaclanguages (which might not be shared directly) any other prefs from this list worth sharing?

...

Are backends lists an issue? Or is it actually very useful? ILLOpacbackends

These are not the installed ones, but what works in OPAC, so maybe not.

Not sharing right?

...

Could some codes be so rare that they are equivalent of sharing the library name? MARCOrgCode

That should be ok right?

The MARCOrgCode is the unique identifier of a library - so it IS equivalent to sharing the library name. Official MARCOrgCodes can be looked up easily.

Thanks for the info. I though it was pointing to e.g. the Library Of Congress in the USA, not the library itself. So yeah, not sharable.

...

== Maybe a typo in the list == ArticleRequestsMandatoryFieldsItemsOnly ↓ ArticleRequestsMandatoryFieldsItemOnly

OpacSuppressionByIPRangeµ ↓ OpacSuppressionByIPRange

Where is the typo?

In the file in_DB_but_not_shared

...

== Not in the UI (Administration › System preferences) what does it mean? == ElasticsearchIndexStatus_authorities ElasticsearchIndexStatus_biblios INTRAdidyoumean OPACdidyoumean

These are set up on their own configuration pages, but systempreferences is used for storage. That's why they are not showing in the regular GUI.

Ok, no worries then.

...

OpacMainUserBlock opacheader

Both of those have been moved into News and should no longer be present, where did you spot them as prefs?

In the file in_DB_but_not_shared. It seems Joubu got them from the DB. So maybe updatedatabase spared them.

...

printcirculationslips

This one has its own bug somewhere I think.

bug 10014 bug 17845 ###### (In reply to Katrin Fischer from comment #22)

Hm, why is seearchengine not turning up in the list, didn't we start the discussion there?

It's not marked as no in the file in_DB_but_not_shared so I assumed it will be shared right? ###### (In reply to Jonathan Druart from comment #23)

Victor, we don't want more sysprefs.

It was a joke after I went through almost all sysprefs (in_DB_but_not_shared) in the UI ^^" (To double-check what data they could contain) Even knowing the number of them I though 1.5 hours would be enough, but wrong!

The goal was to double-check, from the whitelist I provided, if they were all valid and relevant

wait, which is the whitelist? I might have gone totally in the wrong direction 😱

we don't want personal data to be shared.

No misundestanding on this. This what I had in mind.

Don't rethink/redo the whole process please.

So "in_DB_but_not_shared" was the wrong file to check?? 😱 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #26 from Victor Grousset/tuxayo <victor@tuxayo.net> --- (In reply to Jonathan Druart from comment #25)

Victor, yes it's the correct file. I am sorry but those comments are hard to read, you should apply the patches and provide patches on top. IMO that will be more readable.

Ok, I'll try to integrate the content of my comments via patches. I wasn't confident enough to change the patches (fear of "corrupting" your work) And I'm still wondering how to deal with the many "categories" that I needed to do in my long comment. Thought thanks to the responses from Katrin and you, now I don't need that many categories.

Basically the ones marked with "> No" are the ones I processed already, so no need to talk more about them, unless you find something wrong (ie. marked as No but should be shared, however I am pretty sure there is no error on them).

I'll double check my list of potential false positives. Because I included those not having a privacy risk. Because I'm much less competent about the usefulness.

Work to be done (on top of the patches/files): 1. Identify the "Not sure" and give your opinion

ok

[...] 4. (optional) Do the same work for the new prefs added since I worked on this (19.06.00.044 - f4254cca37)

This is great, you got the exact version!! :D That's what I didn't know how to reliably find.

5. Remove the "no" (and "not sure") and provide a patch for Hea.

Thanks for the plan! -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Jonathan Druart <jonathan.druart+koha@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|jonathan.druart+koha@gmail. |koha-bugs@lists.koha-commun |com |ity.org -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Aleisha Amohia <aleisha@catalyst.net.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aleisha@catalyst.net.nz --- Comment #27 from Aleisha Amohia <aleisha@catalyst.net.nz> --- *** Bug 36210 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |arthur.suzuki@biblibre.com Status|In Discussion |Needs Signoff -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Comment #29 is|1 |0 private| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #31 from Jonathan Druart <jonathan.druart@gmail.com> --- (In reply to Arthur Suzuki from comment #29)

Hi, Given Jonathan's files and method I've updated C4/UsageStats.pm. I've also checked for new sysprefs in the installer/data/mysql/mandatory/sysprefs.sql. Some of the newer sysprefs in the SQL file I didn't add to share because they obviously are not meant to be shared (like OverdriveAuthname or ElectreAPIKey). I think we also need to somehow keep track of any syspref we don't want to get shared, in order not to add them later by error. A colleague suggested we make a table in HEA to store the syspref value we don't want to be shared.

We "simply" should enforce the rule to add new sysprefs to this list at the same time they are added to sysprefs.sql -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #33 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- Created attachment 175178 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175178&action=edit Bug 23849: (QA follow-up) remove duplicates on deprecated shared sysprefs -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #35 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- Created attachment 175314 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=175314&action=edit Bug 23849: Update the list of systempreferences shared HEA Added around 60 new or renamed systempreferences which were previously not shared to HEA. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #37 from Victor Grousset/tuxayo <victor@tuxayo.net> --- Given comment 31 and comment 32, here is an intermediate step that was possible to immediately do as it's just a suggestion: https://wiki.koha-community.org/wiki/Database_updates#Additional_hints Is that enough? Is that the right place? -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |ASSIGNED --- Comment #39 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- (In reply to Katrin Fischer from comment #38)

(In reply to Victor Grousset/tuxayo from comment #37)

...

Given comment 31 and comment 32, here is an intermediate step that was possible to immediately do as it's just a suggestion: https://wiki.koha-community.org/wiki/Database_updates#Additional_hints

Is that enough? Is that the right place?

Thanks Victor! I think it's a good start, but having a check in the qa-test-tools would probably be the next step to make this more visible... who reads the small print? :)

Hi Katrin, Got this, I've been thinking about it actually and I definitely think its a good idea as well ;) Otherwise the problem will keep repeating itself over and over... I'll see how we can implement that. Maybe a check like "if an atomicbug add a new syspref, warn about C4/UsageStats". Putting this bug back in "ASSIGNED" status. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #41 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- Hi, It's been a while ^^ I've made a merge request in the qatools project on gitlab : https://gitlab.com/koha-community/qa-test-tools/-/merge_requests/75 -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #43 from Tomás Cohen Arazi (tcohen) <tomascohen@gmail.com> --- The message we are seeing when running the QA script is weird. It should be a warn I'd say: ``` [FAIL] installer/data/mysql/atomicupdate/bug_20638.pl FAIL forbidden_patterns forbidden pattern: Have you considered sharing the new syspref to HEA? (see bug 23849 and C4/UsageStats.pm) (line 13) ``` -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Lucas Gass (lukeg) <lucas@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |lucas@bywatersolutions.com Status|Needs Signoff |Patch doesn't apply -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #94424|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #94426|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #94428|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #175003|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #175315|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #175314|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196117|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Fridolin Somers <fridolin.somers@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |ASSIGNED --- Comment #49 from Fridolin Somers <fridolin.somers@biblibre.com> --- And some prefs have been removed : BakerTaylorEnabled GoogleIndicTransliteration INTRAdidyoumean OPACFineNoRenewals (has been splitted) OPACHoldingsDefaultSortField And renamed system preferences : 'ESPreventAutoTruncate' to 'ElasticsearchPreventAutoTruncate' 'GDPR_Policy' to 'PrivacyPolicyConsent' 'NewsLog' to 'AdditionalContentLog' 'OPACFineNoRenewalsBlockAutoRenew' to 'FineNoRenewalsBlockAutoRenew' 'OPACFineNoRenewalsIncludeCredits' to 'FineNoRenewalsIncludeCredits' 'UseEmailReceipts' to 'AutomaticEmailReceipts' And : Typo in 'HoldRationDefault' => 'HoldRatioDefault' Wrong uppercase 'IllModuleDisclaimerByType' => 'ILLModuleDisclaimerByType' And : ElasticsearchIndexStatus_authorities, ElasticsearchIndexStatus_biblios Those are not "real" preferences, useless for HEA -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #50 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- Created attachment 196654 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=196654&action=edit Bug 23849: Update the list of systempreferences shared HEA Added around 60 new or renamed systempreferences which were previously not shared to HEA. Also got them sorted alphabetically with sort -u -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #196654|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |david@davidnind.com --- Comment #52 from David Nind <david@davidnind.com> --- I've signed off, bit have no idea how to test this. Feel free to change back to needs signoff, with some instructions on what to test or look for if this is required. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 Arthur Suzuki <arthur.suzuki@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Signed Off Sponsorship status|--- |Sponsored Patch complexity|--- |Trivial patch Comma delimited| |BibLibre list of Sponsors| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23849 --- Comment #53 from Arthur Suzuki <arthur.suzuki@biblibre.com> --- Created attachment 209112 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=209112&action=edit Bug 23849: Update the list of systempreferences shared HEA Added around 60 new or renamed systempreferences which were previously not shared to HEA. Also got them sorted alphabetically with sort -u Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.