File_Adapter: refactoring-refactor to match ReplaceInMemory refactoring
@alelom arbeitet bereits daran.
Seit 02.7.2019.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
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.
- Vorherrschende Sprache
- C#
- Sterne
- 0
- Forks
- 7
- Ø Merge
- 5 T. 8 Std.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus BHoM/File_Toolkit
-
type:feature
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 45/100
BHoM/File_Toolkit#206 ·
-
JSON push always wrapped in array structureEvtl. wieder frei @michaelhoehn hat das vor 813 Tagen übernommen, und es ist kein Pull Request offen. Offentype:feature
BHoM/File_Toolkit#193 · 1 zugewiesene Person ·
-
type:bug type:feature
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 38/100
BHoM/File_Toolkit#191 ·
-
Create Unit Tests to automate regression testingEvtl. wieder frei @alelom hat das vor 908 Tagen übernommen, und es ist kein Pull Request offen. Offentype:test-script
BHoM/File_Toolkit#189 · 1 zugewiesene Person ·
-
Enable writing to JSON without including BHoM-specific attributesEvtl. wieder frei @alelom hat das vor 1609 Tagen übernommen, und es ist kein Pull Request offen. Offentype:feature
BHoM/File_Toolkit#133 · 1 Kommentar · 1 zugewiesene Person ·
Alle Issues in BHoM/File_Toolkit
Ähnliche Issues
-
WPF: each page's `Title` overwrites the window title, and returning to a page does not restore itOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
agentic-workflows area/Docs partner/agentic-workflows
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
jamesmontemagno/tiny-clips#378 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
go 🏃 testing 🧪
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
valkey-io/valkey-glide#7239 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
SubtitleEdit/subtitleedit#15462 ·
Maintainer antworten meist innerhalb von 1 Tag