File_Adapter: refactoring-refactor to match ReplaceInMemory refactoring
@alelom y travaille déjà.
Depuis le 2/7/2019.
Évaluation
Cette issue n'a pas encore été évaluée.
Description
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.
- Langage dominant
- C#
- Étoiles
- 0
- Forks
- 7
- Merge moyen
- 5 j 8 h
- PR mergées (30 j)
- 2
Préparer son environnement
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de BHoM/File_Toolkit
-
type:feature
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
BHoM/File_Toolkit#206 ·
-
JSON push always wrapped in array structurePeut-être à nouveau libre @michaelhoehn l’a pris il y a 813 jours, et aucune pull request n’est ouverte. Ouvertetype:feature
BHoM/File_Toolkit#193 · 1 personne assignée ·
-
type:bug type:feature
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
BHoM/File_Toolkit#191 ·
-
Create Unit Tests to automate regression testingPeut-être à nouveau libre @alelom l’a pris il y a 907 jours, et aucune pull request n’est ouverte. Ouvertetype:test-script
BHoM/File_Toolkit#189 · 1 personne assignée ·
-
Enable writing to JSON without including BHoM-specific attributesPeut-être à nouveau libre @alelom l’a pris il y a 1608 jours, et aucune pull request n’est ouverte. Ouvertetype:feature
BHoM/File_Toolkit#133 · 1 commentaire · 1 personne assignée ·
Toutes les issues de BHoM/File_Toolkit
Issues similaires
-
[Simple] NavigationBar primary commands do not render AppBarButton.Content when it is a UIElementOuvertecontrol/navigationbar kind/bug triage/untriaged
Difficulté 2/5 1-3 heures Accessibilité débutants 90/100
unoplatform/uno.toolkit.ui#1652 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
microsoft/copilot-camp#1053 ·
Les mainteneurs répondent en général sous 1 jour
-
bug effort:S P3
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
nightscout/nocturne#1861 ·
Les mainteneurs répondent en général sous 1 jour
-
agentic-workflows area/Docs partner/agentic-workflows
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
microsoft/fluentui-blazor#5364 ·
Les mainteneurs répondent en général sous 1 jour