Skip to content
Snippets Groups Projects
Commit 206fbb48 authored by Esko Ikkala's avatar Esko Ikkala
Browse files

Simplify suggestions and results

parent 60f45d77
No related branches found
No related tags found
No related merge requests found
...@@ -67,7 +67,7 @@ export const fetchResults = () => ({ ...@@ -67,7 +67,7 @@ export const fetchResults = () => ({
type: FETCH_RESULTS, type: FETCH_RESULTS,
}); });
export const updateResults = (results) => ({ export const updateResults = ({ results }) => ({
type: UPDATE_RESULTS, type: UPDATE_RESULTS,
results results
}); });
......
import {
UPDATE_RESULTS,
CLEAR_RESULTS,
} from '../actions';
const results = (state = [], action) => {
switch (action.type) {
case UPDATE_RESULTS:
return action.results.results;
case CLEAR_RESULTS:
return [];
default:
return state;
}
};
export default results;
...@@ -8,17 +8,35 @@ import { ...@@ -8,17 +8,35 @@ import {
UPDATE_RESULTS, UPDATE_RESULTS,
CLEAR_RESULTS CLEAR_RESULTS
} from '../actions'; } from '../actions';
//import suggestions from './suggestions';
import results from './results';
export const INITIAL_STATE = { export const INITIAL_STATE = {
query: '', query: '',
datasets: [ datasets: {
{ id: 'kotus', selected: true}, 'kotus': {
{ id: 'pnr', selected: true}, 'title': 'Institute for the Languages of Finland (Kotus) Digital Names archive',
{ id: 'warsa_karelian_places', selected: true}, 'shortTitle': 'DNA',
{ id: 'warsa_municipalities', selected: true}, 'timePeriod': '1900s',
], 'selected': true
},
'pnr': {
'title': 'Finnish Geographic Names Registry',
'shortTitle': 'FGN',
'timePeriod': 'contemporary',
'selected': true
},
'warsa_karelian_places': {
'title': 'Karelian map names',
'shortTitle': 'KMN',
'timePeriod': '1922-1944',
'selected': true
},
'warsa_municipalities': {
'title': 'Finnish WW2 municipalities',
'shortTitle': 'FWM',
'timePeriod': '1939-1944',
'selected': true
},
},
suggestions: [], suggestions: [],
suggestionsQuery: '', suggestionsQuery: '',
fetchingSuggestions: false, fetchingSuggestions: false,
...@@ -34,12 +52,7 @@ const search = (state = INITIAL_STATE, action) => { ...@@ -34,12 +52,7 @@ const search = (state = INITIAL_STATE, action) => {
case UPDATE_DATASETS: case UPDATE_DATASETS:
return { return {
...state, ...state,
datasets: [ datasets: state.datasets
{ id: 'kotus', selected: true},
{ id: 'pnr', selected: true},
{ id: 'warsa_karelian_places', selected: false},
{ id: 'warsa_municipalities', selected: true},
],
}; };
case FETCH_SUGGESTIONS: case FETCH_SUGGESTIONS:
return { ...state, fetchingSuggestions: true }; return { ...state, fetchingSuggestions: true };
...@@ -69,7 +82,7 @@ const search = (state = INITIAL_STATE, action) => { ...@@ -69,7 +82,7 @@ const search = (state = INITIAL_STATE, action) => {
case UPDATE_RESULTS: case UPDATE_RESULTS:
return { return {
...state, ...state,
results: results(state.results, action), results: action.results,
resultsQuery: state.query, resultsQuery: state.query,
fetchingResults: false fetchingResults: false
}; };
......
import {
UPDATE_SUGGESTIONS,
CLEAR_SUGGESTIONS,
} from '../actions';
const getShortTitle = (datasetId) => datasetId.split('_').map((part) => part.substr(0, 1)).join('');
export const updateSuggestions = ({ suggestions }) =>
suggestions.map((suggestion) => ({
...suggestion,
datasets: suggestion.datasets.map((dataset) => ({
...dataset,
shortTitle: getShortTitle(dataset.datasetId)
}))
}));
const suggestions = (state = [], action) => {
switch (action.type) {
case UPDATE_SUGGESTIONS:
return updateSuggestions(action);
case CLEAR_SUGGESTIONS:
return [];
default:
return state;
}
};
export default suggestions;
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment