Bug 25728 - Add the ability to create a new authorised value within the cataloguing module
Summary: Add the ability to create a new authorised value within the cataloguing module
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 25727
Blocks: 27128 29137
  Show dependency treegraph
 
Reported: 2020-06-12 14:59 UTC by Jonathan Druart
Modified: 2022-06-06 20:25 UTC (History)
4 users (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: Small patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
When a librarian is cataloguing a bibliographic record and needs a new authorised value, they will now be able to create it directly from the edit form. It also works on the item and authority editing forms. They will need the necessary permission: manage_auth_values.
Version(s) released in:
20.11.00


Attachments
Bug 25728: Create AV when cataloguing a bibliographic record (10.08 KB, patch)
2020-06-12 15:20 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when adding a new item (16.25 KB, patch)
2020-06-12 15:20 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing an authority record (3.71 KB, patch)
2020-06-12 15:20 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Don't explode in the advanced editor (5.89 KB, patch)
2020-06-12 15:20 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Fix the clone field feature (9.03 KB, patch)
2020-06-12 15:20 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Don't explode if no CAN_* var defined (1.00 KB, patch)
2020-06-12 15:21 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Use a svc script instead of the REST API endpoint (4.95 KB, patch)
2020-06-13 10:38 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing a bibliographic record (10.13 KB, patch)
2020-07-09 10:04 UTC, Hugo Agud
Details | Diff | Splinter Review
Bug 25728: Create AV when adding a new item (16.33 KB, patch)
2020-07-09 10:04 UTC, Hugo Agud
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing an authority record (3.77 KB, patch)
2020-07-09 10:04 UTC, Hugo Agud
Details | Diff | Splinter Review
Bug 25728: Don't explode in the advanced editor (5.94 KB, patch)
2020-07-09 10:04 UTC, Hugo Agud
Details | Diff | Splinter Review
Bug 25728: Fix the clone field feature (9.08 KB, patch)
2020-07-09 10:04 UTC, Hugo Agud
Details | Diff | Splinter Review
Bug 25728: Don't explode if no CAN_* var defined (1.04 KB, patch)
2020-07-09 10:04 UTC, Hugo Agud
Details | Diff | Splinter Review
Bug 25728: Use a svc script instead of the REST API endpoint (5.01 KB, patch)
2020-07-09 10:04 UTC, Hugo Agud
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing a bibliographic record (10.12 KB, patch)
2020-08-16 11:42 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when adding a new item (16.31 KB, patch)
2020-08-16 11:42 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing an authority record (3.75 KB, patch)
2020-08-16 11:42 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Don't explode in the advanced editor (5.93 KB, patch)
2020-08-16 11:43 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Fix the clone field feature (9.07 KB, patch)
2020-08-16 11:43 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Don't explode if no CAN_* var defined (1.04 KB, patch)
2020-08-16 11:43 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Use a svc script instead of the REST API endpoint (5.00 KB, patch)
2020-08-16 11:43 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing a bibliographic record (10.57 KB, patch)
2020-08-22 09:37 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when adding a new item (17.32 KB, patch)
2020-08-22 09:37 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing an authority record (3.78 KB, patch)
2020-08-22 09:37 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Don't explode in the advanced editor (6.00 KB, patch)
2020-08-22 09:37 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Fix the clone field feature (9.37 KB, patch)
2020-08-22 09:37 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Don't explode if no CAN_* var defined (1.04 KB, patch)
2020-08-22 09:38 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Use a svc script instead of the REST API endpoint (5.00 KB, patch)
2020-08-22 09:38 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: fix rebase issues (6.70 KB, patch)
2020-08-22 09:38 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Don't prefill av's code (1.22 KB, patch)
2020-08-22 12:19 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing a bibliographic record (10.64 KB, patch)
2020-08-22 13:05 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25728: Create AV when adding a new item (17.42 KB, patch)
2020-08-22 13:05 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25728: Create AV when cataloguing an authority record (3.85 KB, patch)
2020-08-22 13:05 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25728: Don't explode in the advanced editor (6.07 KB, patch)
2020-08-22 13:05 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25728: Fix the clone field feature (9.43 KB, patch)
2020-08-22 13:05 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25728: Don't explode if no CAN_* var defined (1.10 KB, patch)
2020-08-22 13:05 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25728: Use a svc script instead of the REST API endpoint (5.06 KB, patch)
2020-08-22 13:06 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25728: fix rebase issues (6.77 KB, patch)
2020-08-22 13:06 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 25728: Don't prefill av's code (1.27 KB, patch)
2020-08-22 13:06 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2020-06-12 14:59:08 UTC
If you are editing a bibliographic record, then you realised that you need a new authorised values, you have to go to the admin module, and reload the page.
It would be handy to create it directly from the cataloguing module.
Comment 2 Jonathan Druart 2020-06-12 15:20:44 UTC
Created attachment 105837 [details] [review]
Bug 25728: Create AV when cataloguing a bibliographic record

This whole patch set adds the ability to create a new authorised value
directly from the cataloguing module.
It will prevent the cataloger to leave and reload the page if they need
a new one.

Test plan:
If you are logged in with a patron with the manage_authorised_values
subpermission you will be able to:
* Edit a bibliographic record
* Search for a subfield linked to an authorised values (not the "fake"
AV like itemtypes, branches, cn)
* Click the dropdown list and start searching for a value that does not
exist
* Click enter
* Fill the fields in the popup
* Save
=> Your dropdown list has the new AV :)

Note: This does not work for the advanced editor. It does not have
select2 applied to the selects. I have hardly tried but failed.

Sponsored-by: Orex Digital
Comment 3 Jonathan Druart 2020-06-12 15:20:48 UTC
Created attachment 105838 [details] [review]
Bug 25728: Create AV when adding a new item

We do a bit of refactoring to make the code reusable.

Test plan:
Same as the first patch but when adding/editing an item

QA note: There is a warning from the QA tools
 FAIL   koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
   FAIL   js_in_body
                A <script> tag found inside head, must be moved to the body (see bug 17858)
I don't think how we could avoid it.

Sponsored-by: Orex Digital
Comment 4 Jonathan Druart 2020-06-12 15:20:51 UTC
Created attachment 105839 [details] [review]
Bug 25728: Create AV when cataloguing an authority record

Test plan:
Same as the first patch but when cataloguing an authority record

Sponsored-by: Orex Digital
Comment 5 Jonathan Druart 2020-06-12 15:20:54 UTC
Created attachment 105840 [details] [review]
Bug 25728: Don't explode in the advanced editor

Sponsored-by: Orex Digital
Comment 6 Jonathan Druart 2020-06-12 15:20:57 UTC
Created attachment 105841 [details] [review]
Bug 25728: Fix the clone field feature

Sponsored-by: Orex Digital
Comment 7 Jonathan Druart 2020-06-12 15:21:00 UTC
Created attachment 105842 [details] [review]
Bug 25728: Don't explode if no CAN_* var defined

Sponsored-by: Orex Digital
Comment 8 Jonathan Druart 2020-06-13 10:38:45 UTC
Created attachment 105852 [details] [review]
Bug 25728: Use a svc script instead of the REST API endpoint

The try to implement correctly the REST API endpoint for authorised
values failed.
This patch uses an easy to implement svc script for the POST route.
Comment 9 Katrin Fischer 2020-06-13 18:19:51 UTC
(In reply to Jonathan Druart from comment #1)
> How it looks like:
> https://snipboard.io/FYuAay.jpg
> https://snipboard.io/CL7vTo.jpg
> https://snipboard.io/B5aJ7m.jpg
> https://snipboard.io/HL9lfO.jpg

Pretty!
Comment 10 Hugo Agud 2020-07-09 10:04:33 UTC
Created attachment 106696 [details] [review]
Bug 25728: Create AV when cataloguing a bibliographic record

This whole patch set adds the ability to create a new authorised value
directly from the cataloguing module.
It will prevent the cataloger to leave and reload the page if they need
a new one.

Test plan:
If you are logged in with a patron with the manage_authorised_values
subpermission you will be able to:
* Edit a bibliographic record
* Search for a subfield linked to an authorised values (not the "fake"
AV like itemtypes, branches, cn)
* Click the dropdown list and start searching for a value that does not
exist
* Click enter
* Fill the fields in the popup
* Save
=> Your dropdown list has the new AV :)

Note: This does not work for the advanced editor. It does not have
select2 applied to the selects. I have hardly tried but failed.

Sponsored-by: Orex Digital

Signed-off-by:Hugo Agud <hagud@orex.es>
Comment 11 Hugo Agud 2020-07-09 10:04:36 UTC
Created attachment 106697 [details] [review]
Bug 25728: Create AV when adding a new item

We do a bit of refactoring to make the code reusable.

Test plan:
Same as the first patch but when adding/editing an item

QA note: There is a warning from the QA tools
 FAIL   koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
   FAIL   js_in_body
                A <script> tag found inside head, must be moved to the body (see bug 17858)
I don't think how we could avoid it.

Sponsored-by: Orex Digital

Signed-off-by: Your Full Name <your_email>
Comment 12 Hugo Agud 2020-07-09 10:04:39 UTC
Created attachment 106698 [details] [review]
Bug 25728: Create AV when cataloguing an authority record

Test plan:
Same as the first patch but when cataloguing an authority record

Sponsored-by: Orex Digital

Signed-off-by: Your Full Name <your_email>
Comment 13 Hugo Agud 2020-07-09 10:04:43 UTC
Created attachment 106699 [details] [review]
Bug 25728: Don't explode in the advanced editor

Sponsored-by: Orex Digital

Signed-off-by: Your Full Name <your_email>
Comment 14 Hugo Agud 2020-07-09 10:04:46 UTC
Created attachment 106700 [details] [review]
Bug 25728: Fix the clone field feature

Sponsored-by: Orex Digital

Signed-off-by: Your Full Name <your_email>
Comment 15 Hugo Agud 2020-07-09 10:04:50 UTC
Created attachment 106701 [details] [review]
Bug 25728: Don't explode if no CAN_* var defined

Sponsored-by: Orex Digital

Signed-off-by: Your Full Name <your_email>
Comment 16 Hugo Agud 2020-07-09 10:04:53 UTC
Created attachment 106702 [details] [review]
Bug 25728: Use a svc script instead of the REST API endpoint

The try to implement correctly the REST API endpoint for authorised
values failed.
This patch uses an easy to implement svc script for the POST route.

Signed-off-by: Your Full Name <your_email>
Comment 17 Katrin Fischer 2020-08-16 09:22:20 UTC
This depends on bug 25727 that hasn't been signed off yet. There has also been something gone wrong with the sign-off line on these patches here.
Comment 18 Jonathan Druart 2020-08-16 11:42:47 UTC
Created attachment 108356 [details] [review]
Bug 25728: Create AV when cataloguing a bibliographic record

This whole patch set adds the ability to create a new authorised value
directly from the cataloguing module.
It will prevent the cataloger to leave and reload the page if they need
a new one.

Test plan:
If you are logged in with a patron with the manage_authorised_values
subpermission you will be able to:
* Edit a bibliographic record
* Search for a subfield linked to an authorised values (not the "fake"
AV like itemtypes, branches, cn)
* Click the dropdown list and start searching for a value that does not
exist
* Click enter
* Fill the fields in the popup
* Save
=> Your dropdown list has the new AV :)

Note: This does not work for the advanced editor. It does not have
select2 applied to the selects. I have hardly tried but failed.

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 19 Jonathan Druart 2020-08-16 11:42:52 UTC
Created attachment 108357 [details] [review]
Bug 25728: Create AV when adding a new item

We do a bit of refactoring to make the code reusable.

Test plan:
Same as the first patch but when adding/editing an item

QA note: There is a warning from the QA tools
 FAIL   koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
   FAIL   js_in_body
                A <script> tag found inside head, must be moved to the body (see bug 17858)
I don't think how we could avoid it.

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 20 Jonathan Druart 2020-08-16 11:42:58 UTC
Created attachment 108358 [details] [review]
Bug 25728: Create AV when cataloguing an authority record

Test plan:
Same as the first patch but when cataloguing an authority record

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 21 Jonathan Druart 2020-08-16 11:43:03 UTC
Created attachment 108359 [details] [review]
Bug 25728: Don't explode in the advanced editor

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 22 Jonathan Druart 2020-08-16 11:43:09 UTC
Created attachment 108360 [details] [review]
Bug 25728: Fix the clone field feature

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 23 Jonathan Druart 2020-08-16 11:43:15 UTC
Created attachment 108361 [details] [review]
Bug 25728: Don't explode if no CAN_* var defined

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 24 Jonathan Druart 2020-08-16 11:43:22 UTC
Created attachment 108362 [details] [review]
Bug 25728: Use a svc script instead of the REST API endpoint

The try to implement correctly the REST API endpoint for authorised
values failed.
This patch uses an easy to implement svc script for the POST route.

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 25 Katrin Fischer 2020-08-21 22:11:24 UTC
Can you please rebase?
Comment 26 Jonathan Druart 2020-08-22 09:37:40 UTC
Created attachment 108883 [details] [review]
Bug 25728: Create AV when cataloguing a bibliographic record

This whole patch set adds the ability to create a new authorised value
directly from the cataloguing module.
It will prevent the cataloger to leave and reload the page if they need
a new one.

Test plan:
If you are logged in with a patron with the manage_authorised_values
subpermission you will be able to:
* Edit a bibliographic record
* Search for a subfield linked to an authorised values (not the "fake"
AV like itemtypes, branches, cn)
* Click the dropdown list and start searching for a value that does not
exist
* Click enter
* Fill the fields in the popup
* Save
=> Your dropdown list has the new AV :)

Note: This does not work for the advanced editor. It does not have
select2 applied to the selects. I have hardly tried but failed.

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 27 Jonathan Druart 2020-08-22 09:37:45 UTC
Created attachment 108884 [details] [review]
Bug 25728: Create AV when adding a new item

We do a bit of refactoring to make the code reusable.

Test plan:
Same as the first patch but when adding/editing an item

QA note: There is a warning from the QA tools
 FAIL   koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
   FAIL   js_in_body
                A <script> tag found inside head, must be moved to the body (see bug 17858)
I don't think how we could avoid it.

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 28 Jonathan Druart 2020-08-22 09:37:49 UTC
Created attachment 108885 [details] [review]
Bug 25728: Create AV when cataloguing an authority record

Test plan:
Same as the first patch but when cataloguing an authority record

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 29 Jonathan Druart 2020-08-22 09:37:52 UTC
Created attachment 108886 [details] [review]
Bug 25728: Don't explode in the advanced editor

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 30 Jonathan Druart 2020-08-22 09:37:56 UTC
Created attachment 108887 [details] [review]
Bug 25728: Fix the clone field feature

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 31 Jonathan Druart 2020-08-22 09:38:00 UTC
Created attachment 108888 [details] [review]
Bug 25728: Don't explode if no CAN_* var defined

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 32 Jonathan Druart 2020-08-22 09:38:04 UTC
Created attachment 108889 [details] [review]
Bug 25728: Use a svc script instead of the REST API endpoint

The try to implement correctly the REST API endpoint for authorised
values failed.
This patch uses an easy to implement svc script for the POST route.

Signed-off-by: Hugo Agud <hagud@orex.es>
Comment 33 Jonathan Druart 2020-08-22 09:38:08 UTC
Created attachment 108890 [details] [review]
Bug 25728: fix rebase issues
Comment 34 Jonathan Druart 2020-08-22 09:39:04 UTC
(In reply to Katrin Fischer from comment #25)
> Can you please rebase?

Rebased, that was painful.
The QA failure is ok
"A <script> tag found inside head, must be moved to the body"
I had to put it in head.
Comment 35 Katrin Fischer 2020-08-22 10:09:40 UTC
Hi Joubu,

finally get to test here :)

1) QA test tools:

 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
   OK	  filters
   OK	  forbidden patterns
   OK	  git manipulation
   FAIL	  js_in_body
		A <script> tag found inside head, must be moved to the body (see bug 17858)

2) GUI

I would suggest a small change. Most of the codes should best be added capitalized and without special characters. I'd suggest either pre-filling the code or the description with the entered text, leaning towards the description. One argument for description would be that most of the item status fields only work with numerical codes. Also the code field and description should be required fields in the modal.

What do you think?
Comment 36 Katrin Fischer 2020-08-22 11:34:47 UTC
Talked to Joubu:
- We indeed to not have these fields are required on the admin screen, so we just match the behaviour in the modal.
- We also do not keep people from adding values that won't work... so validation should be handled separately as well.
- The JS error from the QA script was checked and is ok (see comment#34)
Comment 37 Katrin Fischer 2020-08-22 11:39:34 UTC
I've now tested in authorities and cataloguing too. I still feel like we should not pre-fill the authorised value with the text entered, but only the description. Otherwise this works really nicely.
Codes should be picked wisely, I think that would better support this.
Comment 38 Jonathan Druart 2020-08-22 12:19:20 UTC
Created attachment 108922 [details] [review]
Bug 25728: Don't prefill av's code
Comment 39 Katrin Fischer 2020-08-22 13:05:37 UTC
Created attachment 108923 [details] [review]
Bug 25728: Create AV when cataloguing a bibliographic record

This whole patch set adds the ability to create a new authorised value
directly from the cataloguing module.
It will prevent the cataloger to leave and reload the page if they need
a new one.

Test plan:
If you are logged in with a patron with the manage_authorised_values
subpermission you will be able to:
* Edit a bibliographic record
* Search for a subfield linked to an authorised values (not the "fake"
AV like itemtypes, branches, cn)
* Click the dropdown list and start searching for a value that does not
exist
* Click enter
* Fill the fields in the popup
* Save
=> Your dropdown list has the new AV :)

Note: This does not work for the advanced editor. It does not have
select2 applied to the selects. I have hardly tried but failed.

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 40 Katrin Fischer 2020-08-22 13:05:42 UTC
Created attachment 108924 [details] [review]
Bug 25728: Create AV when adding a new item

We do a bit of refactoring to make the code reusable.

Test plan:
Same as the first patch but when adding/editing an item

QA note: There is a warning from the QA tools
 FAIL   koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/additem.tt
   FAIL   js_in_body
                A <script> tag found inside head, must be moved to the body (see bug 17858)
I don't think how we could avoid it.

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 41 Katrin Fischer 2020-08-22 13:05:45 UTC
Created attachment 108925 [details] [review]
Bug 25728: Create AV when cataloguing an authority record

Test plan:
Same as the first patch but when cataloguing an authority record

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 42 Katrin Fischer 2020-08-22 13:05:49 UTC
Created attachment 108926 [details] [review]
Bug 25728: Don't explode in the advanced editor

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 43 Katrin Fischer 2020-08-22 13:05:54 UTC
Created attachment 108927 [details] [review]
Bug 25728: Fix the clone field feature

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 44 Katrin Fischer 2020-08-22 13:05:58 UTC
Created attachment 108928 [details] [review]
Bug 25728: Don't explode if no CAN_* var defined

Sponsored-by: Orex Digital

Signed-off-by: Hugo Agud <hagud@orex.es>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 45 Katrin Fischer 2020-08-22 13:06:02 UTC
Created attachment 108929 [details] [review]
Bug 25728: Use a svc script instead of the REST API endpoint

The try to implement correctly the REST API endpoint for authorised
values failed.
This patch uses an easy to implement svc script for the POST route.

Signed-off-by: Hugo Agud <hagud@orex.es>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 46 Katrin Fischer 2020-08-22 13:06:06 UTC
Created attachment 108930 [details] [review]
Bug 25728: fix rebase issues

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 47 Katrin Fischer 2020-08-22 13:06:10 UTC
Created attachment 108931 [details] [review]
Bug 25728: Don't prefill av's code

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 48 Jonathan Druart 2020-08-24 10:33:09 UTC
Pushed to master for 20.11, thanks to everybody involved!
Comment 49 Lucas Gass 2020-09-04 19:45:13 UTC
missing dependencies for 20.05.x, no backport
Comment 50 Klas Blomberg 2021-09-15 08:13:31 UTC
I don't like this patch at all!
After just a few weeks we have dozens of new locations created by mistake, like HORS, FICT, NON-FI &c, &c

Staff type just the first letters of a location and then hit enter, and voila, there is a new location!

Please make a syspref for this, and make it disabled by default - or set it as a permission that could be turned off, even for superlibrarians
Comment 51 Katrin Fischer 2021-09-15 09:21:57 UTC
(In reply to Klas Blomberg from comment #50)
> I don't like this patch at all!
> After just a few weeks we have dozens of new locations created by mistake,
> like HORS, FICT, NON-FI &c, &c
> 
> Staff type just the first letters of a location and then hit enter, and
> voila, there is a new location!
> 
> Please make a syspref for this, and make it disabled by default - or set it
> as a permission that could be turned off, even for superlibrarians

Hi Klas, this is not how it should work. The intended behaviour is that it opens up a modal where you have to enter further information - code and descripton for the new authorised value. If it works like you describe, this would be a bug.

It is already tied to a permission - you can't add them if you don't have managing authorised values permission. Does that help?

Also: Filing a new bug and linking to the existing bug once released is better, we will have to do this for this to progress and with the link everyone added here will also get emailed.