refactoring

src/fragmentarium/application/FragmentService.ts

@@ -56,7 +56,7 @@ export interface FragmentRepository {
   ): Bluebird<Fragment>
   findInCorpus(number: string): Bluebird<ReadonlyArray<ManuscriptAttestation>>
   fetchGenres(): Bluebird<string[][]>
-  fetchArchaeologies(): Bluebird<string[][]>
+  fetchProvenances(): Bluebird<string[][]>
   fetchPeriods(): Bluebird<string[]>
   updateGenres(number: string, genres: Genres): Bluebird<Fragment>
   updateScript(number: string, script: Script): Bluebird<Fragment>
@@ -177,8 +177,8 @@ export class FragmentService {
   fetchGenres(): Bluebird<string[][]> {
     return this.fragmentRepository.fetchGenres()
   }
-  fetchArchaeologies(): Bluebird<string[][]> {
-    return this.fragmentRepository.fetchArchaeologies()
+  fetchProvenances(): Bluebird<string[][]> {
+    return this.fragmentRepository.fetchProvenances()
   }
 
   fetchPeriods(): Bluebird<string[]> {

src/fragmentarium/infrastructure/FragmentRepository.ts

@@ -196,7 +196,7 @@ class ApiFragmentRepository
   fetchGenres(): Promise<string[][]> {
     return this.apiClient.fetchJson('/genres', false)
   }
-  fetchArchaeologies(): Promise<string[][]> {
+  fetchProvenances(): Promise<string[][]> {
     return this.apiClient.fetchJson('/provenances', false)
   }
 

src/fragmentarium/ui/ArchaeologySearchForm.sass → src/fragmentarium/ui/ProvenanceSearchForm.sass

@@ -1,3 +1,3 @@
-.archaeology-selector
+.provenance-selector
     &__menu
         z-index: 99 !important

src/fragmentarium/ui/ArchaeologySearchForm.tsx → src/fragmentarium/ui/ProvenanceSearchForm.tsx

@@ -2,7 +2,7 @@ import React from 'react'
 import withData from 'http/withData'
 import Select from 'react-select'
 import FragmentService from 'fragmentarium/application/FragmentService'
-import './ArchaeologySearchForm.sass'
+import './ProvenanceSearchForm.sass'
 
 export default withData<
   {
@@ -13,26 +13,26 @@ export default withData<
   ReadonlyArray<ReadonlyArray<string>>
 >(
   ({ data, value, onChange }) => {
-    const options = data.map((archaeology) => ({
-      value: archaeology[0],
-      label: archaeology[0],
+    const options = data.map((site) => ({
+      value: site.join(' '),
+      label: site.join(' '),
     }))
     const defaultOption = value ? { value: value, label: value } : null
 
     return (
       <Select
-        aria-label="select-archaeology"
+        aria-label="select-provenance"
         placeholder="Provenance"
         options={options}
         value={defaultOption}
         onChange={(selection) => {
           onChange(selection?.value || null)
         }}
         isSearchable={true}
-        classNamePrefix={'archaeology-selector'}
+        classNamePrefix={'provenance-selector'}
         isClearable
       />
     )
   },
-  (props) => props.fragmentService.fetchArchaeologies()
+  (props) => props.fragmentService.fetchProvenances()
 )

src/fragmentarium/ui/SearchForm.test.tsx

@@ -89,9 +89,7 @@ beforeEach(async () => {
     Promise.resolve(Object.keys(Periods))
   )
   fragmentService.fetchGenres.mockReturnValue(Promise.resolve(genres))
-  fragmentService.fetchArchaeologies.mockReturnValue(
-    Promise.resolve(provenances)
-  )
+  fragmentService.fetchProvenances.mockReturnValue(Promise.resolve(provenances))
   wordService.searchLemma.mockReturnValue(Promise.resolve([word]))
   wordService.findAll.mockReturnValue(Promise.resolve([]))
   session.isAllowedToReadFragments.mockReturnValue(true)
@@ -228,11 +226,11 @@ describe('Script period selection form', () => {
 
 describe('Provenance selection form', () => {
   beforeEach(() => {
-    userEvent.type(screen.getByLabelText('select-archaeology'), 'Assur')
+    userEvent.type(screen.getByLabelText('select-provenance'), 'Assur')
   })
   it('displays user input', async () => {
     await waitFor(() =>
-      expect(screen.getByLabelText('select-archaeology')).toHaveValue('Assur')
+      expect(screen.getByLabelText('select-provenance')).toHaveValue('Assur')
     )
   })
 

src/fragmentarium/ui/SearchForm.tsx

@@ -33,7 +33,7 @@ import {
 import GenreSearchForm from './GenreSearchForm'
 import BibliographyService from 'bibliography/application/BibliographyService'
 import { ResearchProjects } from 'research-projects/researchProject'
-import ArchaeologySearchForm from './ArchaeologySearchForm'
+import ProvenanceSearchForm from './ProvenanceSearchForm'
 
 interface State {
   number: string | null
@@ -85,7 +85,7 @@ class SearchForm extends Component<Props, State> {
       scriptPeriod: fragmentQuery.scriptPeriod || '',
       scriptPeriodModifier: fragmentQuery.scriptPeriodModifier || '',
       genre: fragmentQuery.genre || '',
-      site: fragmentQuery.archaeology || '',
+      site: fragmentQuery.provenance || '',
       isValid: isValidNumber(fragmentQuery.number),
     }
 
@@ -141,7 +141,7 @@ class SearchForm extends Component<Props, State> {
           : '',
         scriptPeriod: state.scriptPeriod,
         genre: state.genre,
-        site: state.site,
+        site: state.site ? state.site.split(' ')[0] : '',
         project: this.props.project,
       },
       (value) => !value
@@ -244,7 +244,7 @@ class SearchForm extends Component<Props, State> {
               <HelpTrigger overlay={ProvenanceSearchHelp()} />
             </Col>
             <Col>
-              <ArchaeologySearchForm
+              <ProvenanceSearchForm
                 fragmentService={this.props.fragmentService}
                 onChange={this.onChange('site')}
                 value={this.state.site}

src/fragmentarium/ui/front-page/Fragmentarium.test.tsx

@@ -59,7 +59,7 @@ beforeEach(() => {
   wordService.findAll.mockReturnValue(Promise.resolve([]))
   fragmentService.fetchPeriods.mockReturnValueOnce(Promise.resolve([]))
   fragmentService.fetchGenres.mockReturnValueOnce(Promise.resolve([]))
-  fragmentService.fetchArchaeologies.mockRejectedValueOnce(Promise.resolve([]))
+  fragmentService.fetchProvenances.mockRejectedValueOnce(Promise.resolve([]))
 })
 
 describe('Statistics', () => {

src/fragmentarium/ui/search/FragmentariumSearch.test.tsx

@@ -77,7 +77,7 @@ beforeEach(async () => {
   session = new MemorySession(['read:fragments'])
   fragmentService.fetchPeriods.mockReturnValueOnce(Promise.resolve([]))
   fragmentService.fetchGenres.mockReturnValueOnce(Promise.resolve([]))
-  fragmentService.fetchArchaeologies.mockReturnValueOnce(Promise.resolve([]))
+  fragmentService.fetchProvenances.mockReturnValueOnce(Promise.resolve([]))
 })
 
 describe('Search', () => {

src/query/FragmentQuery.ts

@@ -18,7 +18,7 @@ export type FragmentQuery = Partial<{
   scriptPeriod: PeriodString
   scriptPeriodModifier: PeriodModifierString
   genre: string
-  archaeology: string
+  provenance: string
   project: keyof typeof ResearchProjects | null
   latest: boolean
 }>