We should be using defaultValue instead of value to set defaults
Created attachment 191371 [details] [review] Bug 41605: Fix incorrect default key Test plan: 1) Create a new vendor, the status should be inactive by default 2) Apply patch, yarn js:build and hard refresh 3) The vendor should now default to active
When creating a new vendor (before the patch), active is already the default for the add vendor form: 1. Acquisitions > New vendor - Vendor name: enter a name - Ordering information section: . Vendor is: Active: X Inactive: 2. After creating the vendor, for the information shown for a vendor, the "Vendor is" field is empty/blank (shown under the ordering information section). 3. In the database, the active field is set to 1 (koha-mysql kohadev, select * from aqbooksellers;) Looking at the code, before the patch, koha-tmpl/intranet-tmpl/prog/js/vue/components/Vendors/VendorResource.vue already uses defaultValue, so may have been changed by another bug: 461 defaultValue: true, However, for koha-tmpl/intranet-tmpl/prog/js/vue/components/Admin/RecordSources/RecordSourcesResource.vue (record sources), it is still 'value' before the patch: 62 value: false,