Attachment #173779 for bug #37577

View | Details | Raw Unified | Return to bug 37577
Collapse All | Expand All

Lines 61-66 sub to_api { Link Here

(-)a/Koha/AdditionalField.pm (+1 lines)
61	'aqinvoices' => 'invoice',	61	'aqinvoices' => 'invoice',
62	'erm_licenses' => 'license',	62	'erm_licenses' => 'license',
63	'erm_agreements' => 'agreement',	63	'erm_agreements' => 'agreement',
		64	'erm_packages' => 'package',
64	'aqorders' => 'order',	65	'aqorders' => 'order',
65	};	66	};
66		67

Lines 19-25 use Modern::Perl; Link Here

(-)a/Koha/ERM/EHoldings/Package.pm (-1 / +1 lines)
19		19
20	use Koha::Database;	20	use Koha::Database;
21		21
22	use base qw(Koha::Object);	22	use base qw(Koha::Object::Mixin::AdditionalFields Koha::Object);
23		23
24	use Koha::Acquisition::Booksellers;	24	use Koha::Acquisition::Booksellers;
25	use Koha::ERM::EHoldings::Package::Agreements;	25	use Koha::ERM::EHoldings::Package::Agreements;

Lines 22-28 use Koha::Database; Link Here

(-)a/Koha/ERM/EHoldings/Packages.pm (-1 / +1 lines)
22		22
23	use Koha::ERM::EHoldings::Package;	23	use Koha::ERM::EHoldings::Package;
24		24
25	use base qw(Koha::Objects);	25	use base qw(Koha::Objects::Mixin::AdditionalFields Koha::Objects);
26		26
27	=head1 NAME	27	=head1 NAME
28		28

Lines 88-98 sub add { Link Here

(-)a/Koha/REST/V1/ERM/EHoldings/Packages/Local.pm (+8 lines)
88	my $body = $c->req->json;	88	my $body = $c->req->json;
89		89
90	my $package_agreements = delete $body->{package_agreements} // [];	90	my $package_agreements = delete $body->{package_agreements} // [];
		91	my $extended_attributes = delete $body->{extended_attributes} // [];
91	delete $body->{external_id} unless $body->{external_id};	92	delete $body->{external_id} unless $body->{external_id};
92		93
93	my $package = Koha::ERM::EHoldings::Package->new_from_api($body)->store;	94	my $package = Koha::ERM::EHoldings::Package->new_from_api($body)->store;
94	$package->package_agreements($package_agreements);	95	$package->package_agreements($package_agreements);
95		96
		97	my @extended_attributes = map { {'id' => $_->{field_id}, 'value' => $_->{value}} } @{$extended_attributes};
		98	$package->extended_attributes( \@extended_attributes );
		99
96	$c->res->headers->location($c->req->url->to_string . '/' . $package->package_id);	100	$c->res->headers->location($c->req->url->to_string . '/' . $package->package_id);
97	return $c->render(	101	return $c->render(
98	status => 201,	102	status => 201,
Lines 160-165 sub update { Link Here
160	my $body = $c->req->json;	164	my $body = $c->req->json;
161		165
162	my $package_agreements = delete $body->{package_agreements} // [];	166	my $package_agreements = delete $body->{package_agreements} // [];
		167	my $extended_attributes = delete $body->{extended_attributes} // [];
163	delete $body->{external_id} unless $body->{external_id};	168	delete $body->{external_id} unless $body->{external_id};
164		169
165	$package->set_from_api($body)->store;	170	$package->set_from_api($body)->store;
Lines 168-173 sub update { Link Here
168	# ie. It's coming from EBSCO and we don't have local data linked to it	173	# ie. It's coming from EBSCO and we don't have local data linked to it
169	$package->package_agreements($package_agreements);	174	$package->package_agreements($package_agreements);
170		175
		176	my @extended_attributes = map { {'id' => $_->{field_id}, 'value' => $_->{value}} } @{$extended_attributes};
		177	$package->extended_attributes( \@extended_attributes );
		178
171	$c->res->headers->location($c->req->url->to_string . '/' . $package->package_id);	179	$c->res->headers->location($c->req->url->to_string . '/' . $package->package_id);
172	return $c->render(	180	return $c->render(
173	status => 200,	181	status => 200,

Lines 52-57 sub list { Link Here

(-)a/Koha/REST/V1/ExtendedAttributeTypes.pm (+1 lines)
52	invoice => 'aqinvoices',	52	invoice => 'aqinvoices',
53	license => 'erm_licenses',	53	license => 'erm_licenses',
54	agreement => 'erm_agreements',	54	agreement => 'erm_agreements',
		55	package => 'erm_packages',
55	order => 'aqorders',	56	order => 'aqorders',
56	};	57	};
57		58




# Created by DBIx::Class::Schema::Loader v0.07049 @ 2022-11-11 11:52:09
# DO NOT MODIFY THIS OR ANYTHING ABOVE! md5sum:3YA6xOwdLHDLoZnQJNk8eQ

__PACKAGE__->has_many(
    "additional_field_values",
    "Koha::Schema::Result::AdditionalFieldValue",
    sub {
        my ($args) = @_;

        return {
            "$args->{foreign_alias}.record_id" => { -ident => "$args->{self_alias}.package_id" },

            "$args->{foreign_alias}.field_id" =>
                { -in => \'(SELECT id FROM additional_fields WHERE tablename="erm_packages")' },
        };
    },
    { cascade_copy => 0, cascade_delete => 0 },
);

__PACKAGE__->has_many(
    "extended_attributes",
    "Koha::Schema::Result::AdditionalFieldValue",
    sub {
        my ($args) = @_;

        return {
            "$args->{foreign_alias}.record_id" => { -ident => "$args->{self_alias}.package_id" },

            "$args->{foreign_alias}.field_id" =>
                { -in => \'(SELECT id FROM additional_fields WHERE tablename="erm_packages")' },
        };
    },
    { cascade_copy => 0, cascade_delete => 0 },
);

sub koha_object_class {
    'Koha::ERM::EHoldings::Package';
}




    type:
      - integer
      - "null"
  extended_attributes:
    description: Related additional field values
    type:
      - array
    items:
      $ref: extended_attribute_value.yaml
  _strings:
    type:
      - object
      - "null"

additionalProperties: false
required:

Lines 72-77 Link Here

(-)a/api/v1/swagger/paths/erm_eholdings_packages.yaml (+6 lines)
72	- vendor.name	72	- vendor.name
73	- resources+count	73	- resources+count
74	- resources	74	- resources
		75	- extended_attributes
		76	- +strings
75	collectionFormat: csv	77	collectionFormat: csv
76	responses:	78	responses:
77	200:	79	200:
Lines 193-198 Link Here
193	- package_agreements.agreement	195	- package_agreements.agreement
194	- resources+count	196	- resources+count
195	- vendor	197	- vendor
		198	- extended_attributes
		199	- +strings
196	collectionFormat: csv	200	collectionFormat: csv
197	responses:	201	responses:
198	200:	202	200:
Lines 262-267 Link Here
262	enum:	266	enum:
263	- agreements	267	- agreements
264	- resources	268	- resources
		269	- extended_attributes
		270	- +strings
265	collectionFormat: csv	271	collectionFormat: csv
266	responses:	272	responses:
267	200:	273	200:

Lines 20-25 Link Here

(-)a/api/v1/swagger/paths/extended_attribute_types.yaml (+1 lines)
20	- invoice	20	- invoice
21	- license	21	- license
22	- agreement	22	- agreement
		23	- package
23	- order	24	- order
24	- subscription	25	- subscription
25	- $ref: "../swagger.yaml#/parameters/match"	26	- $ref: "../swagger.yaml#/parameters/match"

Lines 23-29 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/additional-fields.tt (-1 / +2 lines)
23	</head>	23	</head>
24		24
25	[% marcfield_tables = ['subscription', 'aqorders'] %]	25	[% marcfield_tables = ['subscription', 'aqorders'] %]
26	[% searchable_tables = ['subscription', 'aqbasket', 'aqinvoices', 'erm_licenses', 'erm_agreements'] %]	26	[% searchable_tables = ['subscription', 'aqbasket', 'aqinvoices', 'erm_licenses', 'erm_agreements', 'erm_packages'] %]
27	[% show_marcfield = marcfield_tables.grep('^' _ tablename _ '$').size ? 1 : 0 %]	27	[% show_marcfield = marcfield_tables.grep('^' _ tablename _ '$').size ? 1 : 0 %]
28	[% show_searchable = searchable_tables.grep('^' _ tablename _ '$').size ? 1 : 0 %]	28	[% show_searchable = searchable_tables.grep('^' _ tablename _ '$').size ? 1 : 0 %]
29		29
Lines 138-143 Link Here
138	[% IF CAN_user_erm %]	138	[% IF CAN_user_erm %]
139	[% WRAPPER table_option value="erm_licenses" %]ERM Licenses[% END %]	139	[% WRAPPER table_option value="erm_licenses" %]ERM Licenses[% END %]
140	[% WRAPPER table_option value="erm_agreements" %]ERM Agreements[% END %]	140	[% WRAPPER table_option value="erm_agreements" %]ERM Agreements[% END %]
		141	[% WRAPPER table_option value="erm_packages" %]ERM Packages[% END %]
141	[% END %]	142	[% END %]
142	</ul>	143	</ul>
143	[% ELSE %]	144	[% ELSE %]

Lines 63-68 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalPackagesFormAdd.vue (+12 lines)
63	</li>	63	</li>
64	</ol>	64	</ol>
65	</fieldset>	65	</fieldset>
		66	<AdditionalFieldsEntry
		67	resource_type="package"
		68	:additional_field_values="erm_package.extended_attributes"
		69	@additional-fields-changed="additionalFieldsChanged"
		70	/>
66	<EHoldingsPackageAgreements	71	<EHoldingsPackageAgreements
67	:package_agreements="erm_package.package_agreements"	72	:package_agreements="erm_package.package_agreements"
68	/>	73	/>
Lines 87-92 Link Here
87	<script>	92	<script>
88	import { inject } from "vue"	93	import { inject } from "vue"
89	import EHoldingsPackageAgreements from "./EHoldingsLocalPackageAgreements.vue"	94	import EHoldingsPackageAgreements from "./EHoldingsLocalPackageAgreements.vue"
		95	import AdditionalFieldsEntry from "../AdditionalFieldsEntry.vue"
90	import FormSelectVendors from "../FormSelectVendors.vue"	96	import FormSelectVendors from "../FormSelectVendors.vue"
91	import { setMessage, setError, setWarning } from "../../messages"	97	import { setMessage, setError, setWarning } from "../../messages"
92	import { APIClient } from "../../fetch/api-client.js"	98	import { APIClient } from "../../fetch/api-client.js"
Lines 116-121 export default { Link Here
116	created_on: null,	122	created_on: null,
117	resources: null,	123	resources: null,
118	package_agreements: [],	124	package_agreements: [],
		125	extended_attributes: [],
119	},	126	},
120	initialized: false,	127	initialized: false,
121	}	128	}
Lines 172-177 export default { Link Here
172	delete erm_package.vendor	179	delete erm_package.vendor
173	delete erm_package.resources_count	180	delete erm_package.resources_count
174	delete erm_package.is_selected	181	delete erm_package.is_selected
		182	delete erm_package._strings
175		183
176	erm_package.package_agreements = erm_package.package_agreements.map(	184	erm_package.package_agreements = erm_package.package_agreements.map(
177	({ package_id, agreement, ...keepAttrs }) => keepAttrs	185	({ package_id, agreement, ...keepAttrs }) => keepAttrs
Lines 200-209 export default { Link Here
200	)	208	)
201	}	209	}
202	},	210	},
		211	additionalFieldsChanged(additionalFieldValues) {
		212	this.erm_package.extended_attributes = additionalFieldValues
		213	},
203	},	214	},
204	components: {	215	components: {
205	EHoldingsPackageAgreements,	216	EHoldingsPackageAgreements,
206	FormSelectVendors,	217	FormSelectVendors,
		218	AdditionalFieldsEntry,
207	},	219	},
208	name: "EHoldingsEBSCOPackagesFormAdd",	220	name: "EHoldingsEBSCOPackagesFormAdd",
209	}	221	}

Lines 17-22 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalPackagesList.vue (-2 / +49 lines)
17	<KohaTable	17	<KohaTable
18	ref="table"	18	ref="table"
19	v-bind="tableOptions"	19	v-bind="tableOptions"
		20	:searchable_additional_fields="searchable_additional_fields"
		21	:searchable_av_options="searchable_av_options"
20	@show="doShow"	22	@show="doShow"
21	@edit="doEdit"	23	@edit="doEdit"
22	@delete="doDelete"	24	@delete="doDelete"
Lines 74-84 export default { Link Here
74	return {	76	return {
75	package_count: 0,	77	package_count: 0,
76	initialized: false,	78	initialized: false,
		79	searchable_additional_fields: [],
		80	searchable_av_options: [],
77	tableOptions: {	81	tableOptions: {
78	columns: this.getTableColumns(),	82	columns: this.getTableColumns(),
79	url: "/api/v1/erm/eholdings/local/packages",	83	url: "/api/v1/erm/eholdings/local/packages",
80	options: {	84	options: {
81	embed: "resources+count,vendor.name",	85	embed: "resources+count,vendor.name,extended_attributes,+strings",
82	searchCols: [	86	searchCols: [
83	{ search: filters.package_name },	87	{ search: filters.package_name },
84	null,	88	null,
Lines 109-115 export default { Link Here
109	},	113	},
110	beforeRouteEnter(to, from, next) {	114	beforeRouteEnter(to, from, next) {
111	next(vm => {	115	next(vm => {
112	vm.getPackageCount().then(() => (vm.initialized = true))	116	vm.getPackageCount().then(() =>
		117	vm
		118	.getSearchableAdditionalFields()
		119	.then(() =>
		120	vm
		121	.getSearchableAVOptions()
		122	.then(() => (vm.initialized = true))
		123	)
		124	)
113	})	125	})
114	},	126	},
115	methods: {	127	methods: {
Lines 121-126 export default { Link Here
121	},	133	},
122	error => {}	134	error => {}
123	)	135	)
		136	},
		137	async getSearchableAdditionalFields() {
		138	const client = APIClient.additional_fields
		139	await client.additional_fields.getAll("package").then(
		140	searchable_additional_fields => {
		141	this.searchable_additional_fields =
		142	searchable_additional_fields.filter(
		143	field => field.searchable
		144	)
		145	},
		146	error => {}
		147	)
		148	},
		149	async getSearchableAVOptions() {
		150	const client_av = APIClient.authorised_values
		151	let av_cat_array = this.searchable_additional_fields
		152	.filter(field => field.authorised_value_category_name)
		153	.map(field => field.authorised_value_category_name)
		154
		155	await client_av.values
		156	.getCategoriesWithValues([
		157	...new Set(av_cat_array.map(av_cat => '"' + av_cat + '"')),
		158	]) // unique
		159	.then(av_categories => {
		160	av_cat_array.forEach(av_cat => {
		161	let av_match = av_categories.find(
		162	element => element.category_name == av_cat
		163	)
		164	this.searchable_av_options[av_cat] =
		165	av_match.authorised_values.map(av => ({
		166	value: av.value,
		167	label: av.description,
		168	}))
		169	})
		170	})
124	},	171	},
125	doShow: function ({ package_id }, dt, event) {	172	doShow: function ({ package_id }, dt, event) {
126	event.preventDefault()	173	event.preventDefault()

Lines 102-107 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/js/vue/components/ERM/EHoldingsLocalPackagesShow.vue (+10 lines)
102	/>	102	/>
103	</div>	103	</div>
104	</fieldset>	104	</fieldset>
		105	<AdditionalFieldsDisplay
		106	resource_type="package"
		107	:additional_field_values="
		108	erm_package._strings.additional_field_values
		109	"
		110	/>
105	<fieldset class="action">	111	<fieldset class="action">
106	<router-link	112	<router-link
107	:to="{ name: 'EHoldingsLocalPackagesList' }"	113	:to="{ name: 'EHoldingsLocalPackagesList' }"
Lines 120-125 import EHoldingsPackageTitlesList from "./EHoldingsLocalPackageTitlesList.vue" Link Here
120	import { APIClient } from "../../fetch/api-client.js"	126	import { APIClient } from "../../fetch/api-client.js"
121	import Toolbar from "../Toolbar.vue"	127	import Toolbar from "../Toolbar.vue"
122	import ToolbarButton from "../ToolbarButton.vue"	128	import ToolbarButton from "../ToolbarButton.vue"
		129	import AdditionalFieldsDisplay from "../AdditionalFieldsDisplay.vue"
123		130
124	export default {	131	export default {
125	setup() {	132	setup() {
Lines 149-154 export default { Link Here
149	created_on: null,	156	created_on: null,
150	resources: null,	157	resources: null,
151	package_agreements: [],	158	package_agreements: [],
		159	extended_attributes: [],
		160	_strings: [],
152	},	161	},
153	initialized: false,	162	initialized: false,
154	}	163	}
Lines 206-211 export default { Link Here
206	EHoldingsPackageTitlesList,	215	EHoldingsPackageTitlesList,
207	Toolbar,	216	Toolbar,
208	ToolbarButton,	217	ToolbarButton,
		218	AdditionalFieldsDisplay,
209	},	219	},
210	name: "EHoldingsLocalPackagesShow",	220	name: "EHoldingsLocalPackagesShow",
211	}	221	}

Lines 112-118 export class ERMAPIClient extends HttpClient { Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/js/vue/fetch/erm-api-client.js (-1 / +1 lines)
112	endpoint: "eholdings/local/packages/" + id,	112	endpoint: "eholdings/local/packages/" + id,
113	headers: {	113	headers: {
114	"x-koha-embed":	114	"x-koha-embed":
115	"package_agreements,package_agreements.agreement,resources+count,vendor",	115	"package_agreements,package_agreements.agreement,resources+count,vendor,extended_attributes,+strings",
116	},	116	},
117	}),	117	}),
118	getAll: (query, params) =>	118	getAll: (query, params) =>




    ];
}

function get_no_additional_fields_package() {
    return {
        package_id: 1,
        name: "package 1",
        notes: "my first package",
        content_type: "",
        created_on: "2024-09-12T10:26:40+00:00",
        external_id: null,
        package_agreements: [],
        package_type: "",
        provider: null,
        resources_count: 0,
        vendor_id: 1,
        vendor: [cy.get_vendors_to_relate()[0]],
        extended_attributes: [],
        _strings: {
            "additional_field_values": []
        },
    };
}
function get_package() {
    return {
        package_id: 1,
        name: "package 1",
        notes: "my first package",
        content_type: "",
        created_on: "2024-09-12T10:26:40+00:00",
        external_id: null,
        package_agreements: [],
        package_type: "",
        provider: null,
        resources_count: 0,
        vendor_id: 1,
        vendor: [cy.get_vendors_to_relate()[0]],
        extended_attributes: [
            {
                field_id: 1,
                id: "1",
                record_id: "1",
                value: "REF",
            },
            {
                field_id: 1,
                id: "2",
                record_id: "1",
                value: "NFIC",
            },
            {
                field_id: 2,
                id: "3",
                record_id: "1",
                value: "some text",
            },
            {
                field_id: 3,
                id: "4",
                record_id: "1",
                value: "some repeatable text",
            },
            {
                field_id: 4,
                id: "5",
                record_id: "1",
                value: "AF",
            },
        ],
        _strings: {
            additional_field_values: [
                {
                    field_id: 1,
                    field_label: "AV Repeatable",
                    type: "av",
                    value_str: "Reference, Non-fiction",
                },
                {
                    field_id: 2,
                    field_label: "Text non-repeatable",
                    type: "text",
                    value_str: "some text",
                },
                {
                    field_id: 3,
                    field_label: "Text repeatable",
                    type: "text",
                    value_str: "some repeatable text",
                },
                {
                    field_id: 4,
                    field_label: "AV Searchable",
                    type: "av",
                    value_str: "Afghanistan",
                },
            ],
        },
    };
}

function get_packages_additional_fields() {
    return [
        {
            authorised_value_category_name: "CCODE",
            extended_attribute_type_id: 1,
            marcfield: "",
            marcfield_mode: "get",
            name: "AV Repeatable",
            repeatable: true,
            searchable: true,
            resource_type: "package",
        },
        {
            authorised_value_category_name: null,
            extended_attribute_type_id: 2,
            marcfield: "",
            marcfield_mode: "get",
            name: "Text non-repeatable",
            repeatable: false,
            searchable: false,
            resource_type: "package",
        },
        {
            authorised_value_category_name: null,
            extended_attribute_type_id: 3,
            marcfield: "",
            marcfield_mode: "get",
            name: "Text repeatable",
            repeatable: true,
            searchable: false,
            resource_type: "package",
        },
        {
            authorised_value_category_name: "COUNTRY",
            extended_attribute_type_id: 4,
            marcfield: "",
            marcfield_mode: "get",
            name: "AV Searchable",
            repeatable: false,
            searchable: true,
            resource_type: "package",
        },
    ];
}

function get_av_cats() {
    return [
        {

        ).should("have.length", 2);
    });

    //
    // for packages
    //

    it("Additional Fields display - Table (eHoldings packages)", () => {
        let eholdings_package = get_package();
        let eholdings_packages = [eholdings_package];
        let eholdings_package_additional_fields = get_packages_additional_fields();
        let av_cats = get_av_cats();

        cy.intercept("GET", "/api/v1/extended_attribute_types*", {
            body: eholdings_package_additional_fields,
            statusCode: 200,
        });

        cy.intercept("GET", "/api/v1/erm/eholdings/local/packages*", {
            statusCode: 200,
            body: eholdings_packages,
            headers: {
                "X-Base-Total-Count": "1",
                "X-Total-Count": "1",
            },
        });
        cy.intercept("GET", "/api/v1/erm/eholdings/local/packages/*", eholdings_package);
        cy.visit("/cgi-bin/koha/erm/eholdings/local/packages");
        cy.get("#packages_list").contains("Showing 1 to 1 of 1 entries");

        cy.get("#packages_list table tbody tr:first").contains(
            get_description_from_av_value(
                av_cats,
                eholdings_package.extended_attributes[0]
            )
        );
        cy.get("#packages_list table tbody tr:first").contains(
            get_description_from_av_value(
                av_cats,
                eholdings_package.extended_attributes[0]
            ) +
                ", " +
                get_description_from_av_value(
                    av_cats,
                    eholdings_package.extended_attributes[1]
                )
        );
    });

    it("Additional Fields display - Show (eHoldings packages)", () => {
        let empty_eholdings_package = get_no_additional_fields_package();
        let eholdings_package = get_package();
        let eholdings_packages = [eholdings_package];
        let vendors = cy.get_vendors_to_relate();
        let eholdings_package_additional_fields = get_packages_additional_fields();
        let av_cats = get_av_cats();

        // Click the 'Edit' button from the list
        cy.intercept("GET", "/api/v1/erm/eholdings/local/packages*", {
            statusCode: 200,
            body: eholdings_packages,
            headers: {
                "X-Base-Total-Count": "1",
                "X-Total-Count": "1",
            },
        });
        cy.intercept("GET", "/api/v1/erm/eholdings/local/packages/*", empty_eholdings_package).as(
            "get-empty-eholdings-package"
        );

        //Intercept vendors request
        cy.intercept("GET", "/api/v1/acquisitions/vendors*", {
            statusCode: 200,
            body: vendors,
        });
        cy.intercept("GET", "/api/v1/extended_attribute_types*", {
            body: [],
            statusCode: 200,
        }).as("empty-additional-fields");

        //Empty additional fields, should not display
        cy.visit("/cgi-bin/koha/erm/eholdings/local/packages");
        cy.get("#packages_list table tbody tr:first td:first a").click();
        cy.wait("@get-empty-eholdings-package");
        cy.get("#packages_list #additional_fields").should("not.exist");

        cy.intercept("GET", "/api/v1/extended_attribute_types*", {
            body: eholdings_package_additional_fields,
            statusCode: 200,
        }).as("existing-additional-fields");

        cy.intercept(
            {
                pathname: "/api/v1/authorised_value_categories",
                query: {
                    q: '{"me.category_name":["CCODE", "COUNTRY"]}',
                },
            },
            {
                body: av_cats,
                statusCode: 200,
            }
        ).as("avcategories");

        cy.intercept("GET", "/api/v1/erm/eholdings/local/packages/*", eholdings_package).as(
            "get-eholdings-package"
        );

        // There are additional fields, fieldset should exist
        cy.visit("/cgi-bin/koha/erm/eholdings/local/packages");
        cy.get("#packages_list table tbody tr:first td:first a").click();
        cy.wait("@get-eholdings-package");
        cy.get("#packages_show #additional_fields").should("exist");

        // All fields are presented correctly
        cy.get(
            "#packages_show #additional_fields label[for='additional_field_1']"
        ).contains(eholdings_package_additional_fields[0].name);
        cy.get(
            "#packages_show #additional_fields label[for='additional_field_1']"
        )
            .parent()
            .children("span")
            .contains(
                get_description_from_av_value(
                    av_cats,
                    eholdings_package.extended_attributes[0]
                ) +
                    ", " +
                    get_description_from_av_value(
                        av_cats,
                        eholdings_package.extended_attributes[1]
                    )
            );

        cy.get(
            "#packages_show #additional_fields label[for='additional_field_2']"
        ).contains(eholdings_package_additional_fields[1].name);
        cy.get(
            "#packages_show #additional_fields label[for='additional_field_2']"
        )
            .parent()
            .children("span")
            .contains(eholdings_package.extended_attributes[2].value);

        cy.get(
            "#packages_show #additional_fields label[for='additional_field_3']"
        ).contains(eholdings_package_additional_fields[2].name);
        cy.get(
            "#packages_show #additional_fields label[for='additional_field_3']"
        )
            .parent()
            .children("span")
            .contains(eholdings_package.extended_attributes[3].value);

        cy.get(
            "#packages_show #additional_fields label[for='additional_field_4']"
        ).contains(eholdings_package_additional_fields[3].name);
        cy.get(
            "#packages_show #additional_fields label[for='additional_field_4']"
        )
            .parent()
            .children("span")
            .contains(
                get_description_from_av_value(
                    av_cats,
                    eholdings_package.extended_attributes[4]
                )
            );
    });

    it("Additional Fields entry - Add (eHoldings packages)", () => {
        let vendors = cy.get_vendors_to_relate();
        let eholdings_package_additional_fields = get_packages_additional_fields();
        let av_cats = get_av_cats();

        //Intercept vendors request
        cy.intercept("GET", "/api/v1/acquisitions/vendors*", {
            statusCode: 200,
            body: vendors,
        });
        cy.intercept("GET", "/api/v1/extended_attribute_types*", {
            body: [],
            statusCode: 200,
        }).as("empty-additional-fields");

        // No additional fields, fieldset should not exist
        cy.visit("/cgi-bin/koha/erm/eholdings/local/packages/add");
        cy.get("#packages_add form #additional_fields").should("not.exist");

        cy.intercept("GET", "/api/v1/extended_attribute_types*", {
            body: eholdings_package_additional_fields,
            statusCode: 200,
        }).as("existing-additional-fields");

        cy.intercept(
            {
                pathname: "/api/v1/authorised_value_categories",
                query: {
                    q: '{"me.category_name":["CCODE", "COUNTRY"]}',
                },
            },
            {
                body: av_cats,
                statusCode: 200,
            }
        ).as("avcategories");
        // There are additional fields, fieldset should exist
        cy.visit("/cgi-bin/koha/erm/eholdings/local/packages/add");
        cy.get("#packages_add form #additional_fields").should("exist");

        // All additional fields should be listed
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_1']"
        ).contains(eholdings_package_additional_fields[0].name);
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_2']"
        ).contains(eholdings_package_additional_fields[1].name);
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_3']"
        ).contains(eholdings_package_additional_fields[2].name);
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_4']"
        ).contains(eholdings_package_additional_fields[3].name);

        cy.get("#additional_fields #additional_field_1 .vs__selected").should(
            "not.exist"
        ); //new package, no pre-selected value

        // Pick one value
        cy.get("#additional_fields #additional_field_1 .vs__search").click();
        cy.get(
            "#additional_fields #additional_field_1 #vs4__option-0"
        ).contains(av_cats[0].authorised_values[0].description);
        cy.get("#additional_fields #additional_field_1 #vs4__option-0").click();
        cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
            av_cats[0].authorised_values[0].description
        );
        cy.get("#additional_fields #additional_field_1 .vs__selected").should(
            "have.length",
            1
        );

        // Pick a second value for the same repeatable AV field
        cy.get("#additional_fields #additional_field_1 .vs__search").click();
        cy.get(
            "#additional_fields #additional_field_1 #vs4__option-1"
        ).contains(av_cats[0].authorised_values[1].description);
        cy.get("#additional_fields #additional_field_1 #vs4__option-1").click();
        cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
            av_cats[0].authorised_values[1].description
        );
        cy.get("#additional_fields #additional_field_1 .vs__selected").should(
            "have.length",
            2
        );

        // Attempt to pick the same value again - should not be possible
        cy.get("#additional_fields #additional_field_1 .vs__search").click();
        cy.get(
            "#additional_fields #additional_field_1 #vs4__option-1"
        ).contains(av_cats[0].authorised_values[1].description);
        cy.get("#additional_fields #additional_field_1 #vs4__option-1").click();
        cy.get("#additional_fields #additional_field_1 .vs__selected").should(
            "have.length",
            2
        );

        // Remove the second selected value
        cy.get(
            "#additional_fields #additional_field_1 .vs__selected button[title='Deselect " +
                av_cats[0].authorised_values[1].description +
                "'"
        ).click();
        cy.get("#additional_fields #additional_field_1 .vs__selected").should(
            "have.length",
            1
        );
        cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
            av_cats[0].authorised_values[0].description
        );
    });

    it("Additional Fields entry - Edit (eHoldings packages)", () => {
        let eholdings_package = get_package();
        let eholdings_packages = [eholdings_package];
        let vendors = cy.get_vendors_to_relate();
        let eholdings_package_additional_fields = get_packages_additional_fields();
        let av_cats = get_av_cats();

        // Click the 'Edit' button from the list
        cy.intercept("GET", "/api/v1/erm/eholdings/local/packages*", {
            statusCode: 200,
            body: eholdings_packages,
            headers: {
                "X-Base-Total-Count": "1",
                "X-Total-Count": "1",
            },
        });
        cy.intercept("GET", "/api/v1/erm/eholdings/local/packages/*", eholdings_package).as(
            "get-eholdings-package"
        );

        //Intercept vendors request
        cy.intercept("GET", "/api/v1/acquisitions/vendors*", {
            statusCode: 200,
            body: vendors,
        });

        cy.intercept("GET", "/api/v1/extended_attribute_types*", {
            body: eholdings_package_additional_fields,
            statusCode: 200,
        }).as("existing-additional-fields");

        cy.intercept(
            {
                pathname: "/api/v1/authorised_value_categories",
                query: {
                    q: '{"me.category_name":["CCODE", "COUNTRY"]}',
                },
            },
            {
                body: av_cats,
                statusCode: 200,
            }
        ).as("avcategories");

        cy.visit("/cgi-bin/koha/erm/eholdings/local/packages");
        cy.get("#packages_list table tbody tr:first").contains("Edit").click();
        cy.wait("@get-eholdings-package");
        cy.wait(500); // Cypress is too fast! Vue hasn't populated the form yet!

        // All additional fields should be pre-populated
        cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
            get_description_from_av_value(
                av_cats,
                eholdings_package.extended_attributes[0]
            )
        );
        cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
            get_description_from_av_value(
                av_cats,
                eholdings_package.extended_attributes[1]
            )
        );

        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_2']"
        )
            .parent()
            .children("input")
            .should("have.value", eholdings_package.extended_attributes[2].value);

        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_3']"
        )
            .parent()
            .children("input")
            .should("have.value", eholdings_package.extended_attributes[3].value);

        cy.get("#additional_fields #additional_field_4 .vs__selected").contains(
            get_description_from_av_value(
                av_cats,
                eholdings_package.extended_attributes[4]
            )
        );

        // Clear text field works
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_2']"
        )
            .parent()
            .children(".clear_attribute")
            .click();
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_2']"
        )
            .parent()
            .children("input")
            .should("have.value", "");

        // "+New" text field works
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_3']"
        ).should("have.length", 1);
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_3']"
        )
            .parent()
            .children(".clone_attribute")
            .click();
        cy.get(
            "#packages_add form #additional_fields label[for='additional_field_3']"
        ).should("have.length", 2);
    });

});
- 

Return to bug 37577

Lines 201-206 __PACKAGE__->many_to_many("agreements", "erm_eholdings_packages_agreements", "ag Link Here

(-)a/Koha/Schema/Result/ErmEholdingsPackage.pm (+32 lines)
201	# Created by DBIx::Class::Schema::Loader v0.07049 @ 2022-11-11 11:52:09	201	# Created by DBIx::Class::Schema::Loader v0.07049 @ 2022-11-11 11:52:09
202	# DO NOT MODIFY THIS OR ANYTHING ABOVE! md5sum:3YA6xOwdLHDLoZnQJNk8eQ	202	# DO NOT MODIFY THIS OR ANYTHING ABOVE! md5sum:3YA6xOwdLHDLoZnQJNk8eQ
203		203
		204	__PACKAGE__->has_many(
		205	"additional_field_values",
		206	"Koha::Schema::Result::AdditionalFieldValue",
		207	sub {
		208	my ($args) = @_;
		209
		210	return {
		211	"$args->{foreign_alias}.record_id" => { -ident => "$args->{self_alias}.package_id" },
		212
		213	"$args->{foreign_alias}.field_id" =>
		214	{ -in => \'(SELECT id FROM additional_fields WHERE tablename="erm_packages")' },
		215	};
		216	},
		217	{ cascade_copy => 0, cascade_delete => 0 },
		218	);
		219
		220	__PACKAGE__->has_many(
		221	"extended_attributes",
		222	"Koha::Schema::Result::AdditionalFieldValue",
		223	sub {
		224	my ($args) = @_;
		225
		226	return {
		227	"$args->{foreign_alias}.record_id" => { -ident => "$args->{self_alias}.package_id" },
		228
		229	"$args->{foreign_alias}.field_id" =>
		230	{ -in => \'(SELECT id FROM additional_fields WHERE tablename="erm_packages")' },
		231	};
		232	},
		233	{ cascade_copy => 0, cascade_delete => 0 },
		234	);
		235
204	sub koha_object_class {	236	sub koha_object_class {
205	'Koha::ERM::EHoldings::Package';	237	'Koha::ERM::EHoldings::Package';
206	}	238	}

Lines 70-75 properties: Link Here

(-)a/api/v1/swagger/definitions/erm_eholdings_package.yaml (+10 lines)
70	type:	70	type:
71	- integer	71	- integer
72	- "null"	72	- "null"
		73	extended_attributes:
		74	description: Related additional field values
		75	type:
		76	- array
		77	items:
		78	$ref: extended_attribute_value.yaml
		79	_strings:
		80	type:
		81	- object
		82	- "null"
73		83
74	additionalProperties: false	84	additionalProperties: false
75	required:	85	required:

Lines 253-258 function get_agreements_additional_fields() { Link Here

(-)a/t/cypress/integration/AdditionalFields_spec.ts (-1 / +535 lines)
253	];	253	];
254	}	254	}
255		255
		256	function get_no_additional_fields_package() {
		257	return {
		258	package_id: 1,
		259	name: "package 1",
		260	notes: "my first package",
		261	content_type: "",
		262	created_on: "2024-09-12T10:26:40+00:00",
		263	external_id: null,
		264	package_agreements: [],
		265	package_type: "",
		266	provider: null,
		267	resources_count: 0,
		268	vendor_id: 1,
		269	vendor: [cy.get_vendors_to_relate()[0]],
		270	extended_attributes: [],
		271	_strings: {
		272	"additional_field_values": []
		273	},
		274	};
		275	}
		276	function get_package() {
		277	return {
		278	package_id: 1,
		279	name: "package 1",
		280	notes: "my first package",
		281	content_type: "",
		282	created_on: "2024-09-12T10:26:40+00:00",
		283	external_id: null,
		284	package_agreements: [],
		285	package_type: "",
		286	provider: null,
		287	resources_count: 0,
		288	vendor_id: 1,
		289	vendor: [cy.get_vendors_to_relate()[0]],
		290	extended_attributes: [
		291	{
		292	field_id: 1,
		293	id: "1",
		294	record_id: "1",
		295	value: "REF",
		296	},
		297	{
		298	field_id: 1,
		299	id: "2",
		300	record_id: "1",
		301	value: "NFIC",
		302	},
		303	{
		304	field_id: 2,
		305	id: "3",
		306	record_id: "1",
		307	value: "some text",
		308	},
		309	{
		310	field_id: 3,
		311	id: "4",
		312	record_id: "1",
		313	value: "some repeatable text",
		314	},
		315	{
		316	field_id: 4,
		317	id: "5",
		318	record_id: "1",
		319	value: "AF",
320	},
321	],
322	_strings: {
323	additional_field_values: [
324	{
325	field_id: 1,
326	field_label: "AV Repeatable",
327	type: "av",
328	value_str: "Reference, Non-fiction",
329	},
330	{
331	field_id: 2,
332	field_label: "Text non-repeatable",
333	type: "text",
334	value_str: "some text",
335	},
336	{
337	field_id: 3,
338	field_label: "Text repeatable",
339	type: "text",
340	value_str: "some repeatable text",
341	},
342	{
343	field_id: 4,
344	field_label: "AV Searchable",
345	type: "av",
346	value_str: "Afghanistan",
347	},
348	],
349	},
350	};
351	}
352
353	function get_packages_additional_fields() {
354	return [
355	{
356	authorised_value_category_name: "CCODE",
357	extended_attribute_type_id: 1,
358	marcfield: "",
359	marcfield_mode: "get",
360	name: "AV Repeatable",
361	repeatable: true,
362	searchable: true,
363	resource_type: "package",
364	},
365	{
366	authorised_value_category_name: null,
367	extended_attribute_type_id: 2,
368	marcfield: "",
369	marcfield_mode: "get",
370	name: "Text non-repeatable",
371	repeatable: false,
372	searchable: false,
373	resource_type: "package",
374	},
375	{
376	authorised_value_category_name: null,
377	extended_attribute_type_id: 3,
378	marcfield: "",
379	marcfield_mode: "get",
380	name: "Text repeatable",
381	repeatable: true,
382	searchable: false,
383	resource_type: "package",
384	},
385	{
386	authorised_value_category_name: "COUNTRY",
387	extended_attribute_type_id: 4,
388	marcfield: "",
389	marcfield_mode: "get",
390	name: "AV Searchable",
391	repeatable: false,
392	searchable: true,
393	resource_type: "package",
394	},
395	];
396	}
397
256	function get_av_cats() {	398	function get_av_cats() {
257	return [	399	return [
258	{	400	{
Lines 1123-1126 describe("Additional Fields operations", () => { Link Here
1123	).should("have.length", 2);	1265	).should("have.length", 2);
1124	});	1266	});
1125		1267
		1268	//
		1269	// for packages
		1270	//
		1271
		1272	it("Additional Fields display - Table (eHoldings packages)", () => {
		1273	let eholdings_package = get_package();
		1274	let eholdings_packages = [eholdings_package];
		1275	let eholdings_package_additional_fields = get_packages_additional_fields();
		1276	let av_cats = get_av_cats();
		1277
		1278	cy.intercept("GET", "/api/v1/extended_attribute_types*", {
		1279	body: eholdings_package_additional_fields,
		1280	statusCode: 200,
		1281	});
		1282
		1283	cy.intercept("GET", "/api/v1/erm/eholdings/local/packages*", {
		1284	statusCode: 200,
		1285	body: eholdings_packages,
		1286	headers: {
		1287	"X-Base-Total-Count": "1",
		1288	"X-Total-Count": "1",
		1289	},
		1290	});
		1291	cy.intercept("GET", "/api/v1/erm/eholdings/local/packages/*", eholdings_package);
		1292	cy.visit("/cgi-bin/koha/erm/eholdings/local/packages");
		1293	cy.get("#packages_list").contains("Showing 1 to 1 of 1 entries");
		1294
		1295	cy.get("#packages_list table tbody tr:first").contains(
		1296	get_description_from_av_value(
		1297	av_cats,
		1298	eholdings_package.extended_attributes[0]
		1299	)
		1300	);
		1301	cy.get("#packages_list table tbody tr:first").contains(
		1302	get_description_from_av_value(
		1303	av_cats,
		1304	eholdings_package.extended_attributes[0]
		1305	) +
		1306	", " +
		1307	get_description_from_av_value(
		1308	av_cats,
		1309	eholdings_package.extended_attributes[1]
		1310	)
		1311	);
		1312	});
		1313
		1314	it("Additional Fields display - Show (eHoldings packages)", () => {
		1315	let empty_eholdings_package = get_no_additional_fields_package();
		1316	let eholdings_package = get_package();
		1317	let eholdings_packages = [eholdings_package];
		1318	let vendors = cy.get_vendors_to_relate();
		1319	let eholdings_package_additional_fields = get_packages_additional_fields();
		1320	let av_cats = get_av_cats();
		1321
		1322	// Click the 'Edit' button from the list
		1323	cy.intercept("GET", "/api/v1/erm/eholdings/local/packages*", {
		1324	statusCode: 200,
		1325	body: eholdings_packages,
		1326	headers: {
		1327	"X-Base-Total-Count": "1",
		1328	"X-Total-Count": "1",
		1329	},
		1330	});
		1331	cy.intercept("GET", "/api/v1/erm/eholdings/local/packages/*", empty_eholdings_package).as(
1332	"get-empty-eholdings-package"
1333	);
1334
1335	//Intercept vendors request
1336	cy.intercept("GET", "/api/v1/acquisitions/vendors*", {
1337	statusCode: 200,
1338	body: vendors,
1339	});
1340	cy.intercept("GET", "/api/v1/extended_attribute_types*", {
1341	body: [],
1342	statusCode: 200,
1343	}).as("empty-additional-fields");
1344
1345	//Empty additional fields, should not display
1346	cy.visit("/cgi-bin/koha/erm/eholdings/local/packages");
1347	cy.get("#packages_list table tbody tr:first td:first a").click();
1348	cy.wait("@get-empty-eholdings-package");
1349	cy.get("#packages_list #additional_fields").should("not.exist");
1350
1351	cy.intercept("GET", "/api/v1/extended_attribute_types*", {
1352	body: eholdings_package_additional_fields,
1353	statusCode: 200,
1354	}).as("existing-additional-fields");
1355
1356	cy.intercept(
1357	{
1358	pathname: "/api/v1/authorised_value_categories",
1359	query: {
1360	q: '{"me.category_name":["CCODE", "COUNTRY"]}',
1361	},
1362	},
1363	{
1364	body: av_cats,
1365	statusCode: 200,
1366	}
1367	).as("avcategories");
1368
1369	cy.intercept("GET", "/api/v1/erm/eholdings/local/packages/*", eholdings_package).as(
1370	"get-eholdings-package"
1371	);
1372
1373	// There are additional fields, fieldset should exist
1374	cy.visit("/cgi-bin/koha/erm/eholdings/local/packages");
1375	cy.get("#packages_list table tbody tr:first td:first a").click();
1376	cy.wait("@get-eholdings-package");
1377	cy.get("#packages_show #additional_fields").should("exist");
1378
1379	// All fields are presented correctly
1380	cy.get(
1381	"#packages_show #additional_fields label[for='additional_field_1']"
1382	).contains(eholdings_package_additional_fields[0].name);
1383	cy.get(
1384	"#packages_show #additional_fields label[for='additional_field_1']"
1385	)
1386	.parent()
1387	.children("span")
1388	.contains(
1389	get_description_from_av_value(
1390	av_cats,
1391	eholdings_package.extended_attributes[0]
1392	) +
1393	", " +
1394	get_description_from_av_value(
1395	av_cats,
1396	eholdings_package.extended_attributes[1]
1397	)
1398	);
1399
1400	cy.get(
1401	"#packages_show #additional_fields label[for='additional_field_2']"
1402	).contains(eholdings_package_additional_fields[1].name);
1403	cy.get(
1404	"#packages_show #additional_fields label[for='additional_field_2']"
1405	)
1406	.parent()
1407	.children("span")
1408	.contains(eholdings_package.extended_attributes[2].value);
1409
1410	cy.get(
1411	"#packages_show #additional_fields label[for='additional_field_3']"
1412	).contains(eholdings_package_additional_fields[2].name);
1413	cy.get(
1414	"#packages_show #additional_fields label[for='additional_field_3']"
1415	)
1416	.parent()
1417	.children("span")
1418	.contains(eholdings_package.extended_attributes[3].value);
1419
1420	cy.get(
1421	"#packages_show #additional_fields label[for='additional_field_4']"
1422	).contains(eholdings_package_additional_fields[3].name);
1423	cy.get(
1424	"#packages_show #additional_fields label[for='additional_field_4']"
1425	)
1426	.parent()
1427	.children("span")
1428	.contains(
1429	get_description_from_av_value(
1430	av_cats,
1431	eholdings_package.extended_attributes[4]
1432	)
1433	);
1434	});
1435
1436	it("Additional Fields entry - Add (eHoldings packages)", () => {
1437	let vendors = cy.get_vendors_to_relate();
1438	let eholdings_package_additional_fields = get_packages_additional_fields();
1439	let av_cats = get_av_cats();
1440
1441	//Intercept vendors request
1442	cy.intercept("GET", "/api/v1/acquisitions/vendors*", {
1443	statusCode: 200,
1444	body: vendors,
1445	});
1446	cy.intercept("GET", "/api/v1/extended_attribute_types*", {
1447	body: [],
1448	statusCode: 200,
1449	}).as("empty-additional-fields");
1450
1451	// No additional fields, fieldset should not exist
1452	cy.visit("/cgi-bin/koha/erm/eholdings/local/packages/add");
1453	cy.get("#packages_add form #additional_fields").should("not.exist");
1454
1455	cy.intercept("GET", "/api/v1/extended_attribute_types*", {
1456	body: eholdings_package_additional_fields,
1457	statusCode: 200,
1458	}).as("existing-additional-fields");
1459
1460	cy.intercept(
1461	{
1462	pathname: "/api/v1/authorised_value_categories",
1463	query: {
1464	q: '{"me.category_name":["CCODE", "COUNTRY"]}',
1465	},
1466	},
1467	{
1468	body: av_cats,
1469	statusCode: 200,
1470	}
1471	).as("avcategories");
1472	// There are additional fields, fieldset should exist
1473	cy.visit("/cgi-bin/koha/erm/eholdings/local/packages/add");
1474	cy.get("#packages_add form #additional_fields").should("exist");
1475
1476	// All additional fields should be listed
1477	cy.get(
1478	"#packages_add form #additional_fields label[for='additional_field_1']"
1479	).contains(eholdings_package_additional_fields[0].name);
1480	cy.get(
1481	"#packages_add form #additional_fields label[for='additional_field_2']"
1482	).contains(eholdings_package_additional_fields[1].name);
1483	cy.get(
1484	"#packages_add form #additional_fields label[for='additional_field_3']"
1485	).contains(eholdings_package_additional_fields[2].name);
1486	cy.get(
1487	"#packages_add form #additional_fields label[for='additional_field_4']"
1488	).contains(eholdings_package_additional_fields[3].name);
1489
1490	cy.get("#additional_fields #additional_field_1 .vs__selected").should(
1491	"not.exist"
1492	); //new package, no pre-selected value
1493
1494	// Pick one value
1495	cy.get("#additional_fields #additional_field_1 .vs__search").click();
1496	cy.get(
1497	"#additional_fields #additional_field_1 #vs4__option-0"
1498	).contains(av_cats[0].authorised_values[0].description);
1499	cy.get("#additional_fields #additional_field_1 #vs4__option-0").click();
1500	cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
1501	av_cats[0].authorised_values[0].description
1502	);
1503	cy.get("#additional_fields #additional_field_1 .vs__selected").should(
1504	"have.length",
1505	1
1506	);
1507
1508	// Pick a second value for the same repeatable AV field
1509	cy.get("#additional_fields #additional_field_1 .vs__search").click();
1510	cy.get(
1511	"#additional_fields #additional_field_1 #vs4__option-1"
1512	).contains(av_cats[0].authorised_values[1].description);
1513	cy.get("#additional_fields #additional_field_1 #vs4__option-1").click();
1514	cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
1515	av_cats[0].authorised_values[1].description
1516	);
1517	cy.get("#additional_fields #additional_field_1 .vs__selected").should(
1518	"have.length",
1519	2
1520	);
1521
1522	// Attempt to pick the same value again - should not be possible
1523	cy.get("#additional_fields #additional_field_1 .vs__search").click();
1524	cy.get(
1525	"#additional_fields #additional_field_1 #vs4__option-1"
1526	).contains(av_cats[0].authorised_values[1].description);
1527	cy.get("#additional_fields #additional_field_1 #vs4__option-1").click();
1528	cy.get("#additional_fields #additional_field_1 .vs__selected").should(
1529	"have.length",
1530	2
1531	);
1532
1533	// Remove the second selected value
1534	cy.get(
1535	"#additional_fields #additional_field_1 .vs__selected button[title='Deselect " +
1536	av_cats[0].authorised_values[1].description +
1537	"'"
1538	).click();
1539	cy.get("#additional_fields #additional_field_1 .vs__selected").should(
1540	"have.length",
1541	1
1542	);
1543	cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
1544	av_cats[0].authorised_values[0].description
1545	);
1546	});
1547
1548	it("Additional Fields entry - Edit (eHoldings packages)", () => {
1549	let eholdings_package = get_package();
1550	let eholdings_packages = [eholdings_package];
1551	let vendors = cy.get_vendors_to_relate();
1552	let eholdings_package_additional_fields = get_packages_additional_fields();
1553	let av_cats = get_av_cats();
1554
1555	// Click the 'Edit' button from the list
1556	cy.intercept("GET", "/api/v1/erm/eholdings/local/packages*", {
1557	statusCode: 200,
1558	body: eholdings_packages,
1559	headers: {
1560	"X-Base-Total-Count": "1",
1561	"X-Total-Count": "1",
1562	},
1563	});
1564	cy.intercept("GET", "/api/v1/erm/eholdings/local/packages/*", eholdings_package).as(
1565	"get-eholdings-package"
1566	);
1567
1568	//Intercept vendors request
1569	cy.intercept("GET", "/api/v1/acquisitions/vendors*", {
1570	statusCode: 200,
1571	body: vendors,
1572	});
1573
1574	cy.intercept("GET", "/api/v1/extended_attribute_types*", {
1575	body: eholdings_package_additional_fields,
1576	statusCode: 200,
1577	}).as("existing-additional-fields");
1578
1579	cy.intercept(
1580	{
1581	pathname: "/api/v1/authorised_value_categories",
1582	query: {
1583	q: '{"me.category_name":["CCODE", "COUNTRY"]}',
1584	},
1585	},
1586	{
1587	body: av_cats,
1588	statusCode: 200,
1589	}
1590	).as("avcategories");
1591
1592	cy.visit("/cgi-bin/koha/erm/eholdings/local/packages");
1593	cy.get("#packages_list table tbody tr:first").contains("Edit").click();
1594	cy.wait("@get-eholdings-package");
1595	cy.wait(500); // Cypress is too fast! Vue hasn't populated the form yet!
1596
1597	// All additional fields should be pre-populated
1598	cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
1599	get_description_from_av_value(
1600	av_cats,
1601	eholdings_package.extended_attributes[0]
1602	)
1603	);
1604	cy.get("#additional_fields #additional_field_1 .vs__selected").contains(
1605	get_description_from_av_value(
1606	av_cats,
1607	eholdings_package.extended_attributes[1]
1608	)
1609	);
1610
1611	cy.get(
1612	"#packages_add form #additional_fields label[for='additional_field_2']"
1613	)
1614	.parent()
1615	.children("input")
1616	.should("have.value", eholdings_package.extended_attributes[2].value);
1617
1618	cy.get(
1619	"#packages_add form #additional_fields label[for='additional_field_3']"
1620	)
1621	.parent()
1622	.children("input")
1623	.should("have.value", eholdings_package.extended_attributes[3].value);
1624
1625	cy.get("#additional_fields #additional_field_4 .vs__selected").contains(
1626	get_description_from_av_value(
1627	av_cats,
1628	eholdings_package.extended_attributes[4]
1629	)
1630	);
1631
1632	// Clear text field works
1633	cy.get(
1634	"#packages_add form #additional_fields label[for='additional_field_2']"
1635	)
1636	.parent()
1637	.children(".clear_attribute")
1638	.click();
1639	cy.get(
1640	"#packages_add form #additional_fields label[for='additional_field_2']"
1641	)
1642	.parent()
1643	.children("input")
1644	.should("have.value", "");
1645
1646	// "+New" text field works
1647	cy.get(
1648	"#packages_add form #additional_fields label[for='additional_field_3']"
1649	).should("have.length", 1);
1650	cy.get(
1651	"#packages_add form #additional_fields label[for='additional_field_3']"
1652	)
1653	.parent()
1654	.children(".clone_attribute")
1655	.click();
1656	cy.get(
1657	"#packages_add form #additional_fields label[for='additional_field_3']"
1658	).should("have.length", 2);
1659	});
1660
1126	});	1661	});
1127	-