File_Adapter: refactoring-refactor to match ReplaceInMemory refactoring
@alelom ci sta già lavorando.
Dal 2/7/2019.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Description
As explained in https://github.com/BHoM/BHoM_Adapter/issues/106
Since File_Adapter does not set AdapterConfig.ProcessInMemory, that defaults to true.
This means that the call to Create will always have replaceAll set to true.
Therefore the following calls will have replaceAll always set to true:
https://github.com/BHoM/BHoM_Adapter/blob/49d63e82db3eefa5eecb960e1bd84ab410fc2c51/File_Adapter/CRUD/Create.cs#L40-L46
So these lines never get called
https://github.com/BHoM/BHoM_Adapter/blob/49d63e82db3eefa5eecb960e1bd84ab410fc2c51/File_Adapter/CRUD/Create.cs#L80-L81
As well as this is never called with Append:
https://github.com/BHoM/BHoM_Adapter/blob/49d63e82db3eefa5eecb960e1bd84ab410fc2c51/File_Adapter/CRUD/Create.cs#L57
Proposed change
If anything, the option for the Create method to override should be specifiable dynamically, rather than hard-coded.
This might be solved by making the so called config parameter available to all CRUD methods.
I would remove replaceAll as a parameter of the Create() and use another parameter like
config["CreateOverride"] = true.
- Lingua principale
- C#
- Stelle
- 0
- Fork
- 7
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di BHoM/File_Toolkit
-
type:feature
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
BHoM/File_Toolkit#206 ·
-
JSON push always wrapped in array structureForse di nuovo libera @michaelhoehn l’ha presa 811 giorni fa e non c’è nessuna pull request aperta. Apertatype:feature
BHoM/File_Toolkit#193 · 1 assegnatario ·
-
type:bug type:feature
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
BHoM/File_Toolkit#191 ·
-
Create Unit Tests to automate regression testingForse di nuovo libera @alelom l’ha presa 905 giorni fa e non c’è nessuna pull request aperta. Apertatype:test-script
BHoM/File_Toolkit#189 · 1 assegnatario ·
-
Enable writing to JSON without including BHoM-specific attributesForse di nuovo libera @alelom l’ha presa 1606 giorni fa e non c’è nessuna pull request aperta. Apertatype:feature
BHoM/File_Toolkit#133 · 1 commento · 1 assegnatario ·
Tutte le issue di BHoM/File_Toolkit
Issue simili
-
0 - Backlog Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
BrighterCommand/Brighter#4444 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
area:frontend bug FE P3
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
klasolsson81/jobbliggaren#1915 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
microsoft/vscode-copilotstudio#431 ·
I maintainer di solito rispondono entro 2 giorni