Cadoles/daddy
Cadoles
/
daddy
19
0
Fork 0
Code Issues 16 Pull Requests Releases Wiki Activity

Permettre de gérer les options proposées dans un DAD #19

Manually merged
tcornaut merged 6 commits from feature/options into develop 2020-08-31 15:32:40 +02:00
Conversation 0 Commits 6 Files Changed 6 +8 -6
2 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 2d66888ed3 - Show all commits

10
client/src/components/DecisionSupportFilePage/OptionsSection.tsx
View File

@ -51,16 +51,18 @@ export const OptionsSection: FunctionComponent<OptionsSectionProps> = ({ dsf, up
} }
const onAddOptionClick = (evt: MouseEvent) => { const onAddOptionClick = (evt: MouseEvent) => {
const option = newOption("Décision", "", ""); var options = JSON.parse(JSON.stringify(state.section.options))
wpetit commented 2020-08-31 14:45:36 +02:00
Review
Copy Link

JSON.parse(JSON.stringify(state.section.options)) est utilisé ici pour faire une copie ?

Pourquoi ne pas plutôt faire const options = [ ...state.section.options ]; ? Cette méthode est plus performante qu'une sérialisation/dé-sérialisation JSON du tableau.

`JSON.parse(JSON.stringify(state.section.options))` est utilisé ici pour faire une copie ? Pourquoi ne pas plutôt faire `const options = [ ...state.section.options ];` ? Cette méthode est plus performante qu'une sérialisation/dé-sérialisation JSON du tableau.
tcornaut commented 2020-08-31 15:14:19 +02:00
Review
Copy Link

Malheureusement la méthode de copie que tu cites ne lève pas l'interdiction d'écrire dans options, je n'ai pas trouvé d'autre méthode que JSON.parse(JSON.stringify(state.section.options)) pour en faire une copie modifiable.

Malheureusement la méthode de copie que tu cites ne lève pas l'interdiction d'écrire dans options, je n'ai pas trouvé d'autre méthode que JSON.parse(JSON.stringify(state.section.options)) pour en faire une copie modifiable.
setState(state => ({ ...state, changed: true, section: { ...state.section, options: [ ...state.section.options, option ] }})); var option = newOption("Décision", "", "");
wpetit commented 2020-08-31 14:46:15 +02:00
Review
Copy Link

Si il n'y aura pas d'assignation de nouvelle valeur à la variable, préférer const à var pour identifier assez vite les erreurs liées à l'écrasement de valeurs "constantes".

Si il n'y aura pas d'assignation de nouvelle valeur à la variable, préférer `const` à `var` pour identifier assez vite les erreurs liées à l'écrasement de valeurs "constantes".
options.push(option);
setState(state => ({ ...state, changed: true, section: { ...state.section, options }}));
}; };
const onOptionChange = (id: string, attrName: string, evt: ChangeEvent<HTMLInputElement>) => { const onOptionChange = (id: string, attrName: string, evt: ChangeEvent<HTMLInputElement>) => {
const target = evt.currentTarget; const target = evt.currentTarget;
const value = target.hasOwnProperty('checked') ? target.checked : target.value; const value = target.hasOwnProperty('checked') ? target.checked : target.value;
var options = state.section.options; var options = JSON.parse(JSON.stringify(state.section.options))
wpetit commented 2020-08-31 14:46:57 +02:00
Review
Copy Link

Idem qu'un peu plus haut.

Idem qu'un peu plus haut.
options[id][attrName] = value; options[id][attrName] = value;
setState(state => ({ ...state, changed: true, section: { ...state.section, options: options }})); setState(state => ({ ...state, changed: true, section: { ...state.section, options }}));
}; };
return ( return (

4
client/src/gql/queries/dsf.tsx
View File

@ -7,7 +7,7 @@ export const QUERY_DECISION_SUPPORT_FILES = gql`
decisionSupportFiles(filter: $filter) { decisionSupportFiles(filter: $filter) {
id, id,
title, title,
sections sections,
createdAt, createdAt,
closedAt, closedAt,
votedAt, votedAt,
@ -18,7 +18,7 @@ export const QUERY_DECISION_SUPPORT_FILES = gql`
members { members {
id id
} }
} },
} }
} }
`; `;