File_Adapter: refactoring-refactor to match ReplaceInMemory refactoring
@alelom がすでに取り組んでいます。
2019年7月2日 から。
評価
この issue はまだ評価されていません。
説明
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.
- 主要言語
- C#
- スター
- 0
- フォーク
- 7
- 平均マージ
- 5日 8時間
- マージ済み PR（30日）
- 2
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
BHoM/File_Toolkit のほかの issue
-
Add support for CSV/TXT text-base data files対応中かも @linhnam-nguyen が 389 日前に担当しました。 オープンtype:feature
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
BHoM/File_Toolkit#206 ·
-
JSON push always wrapped in array structure対応中かも @michaelhoehn が 824 日前に担当しました。 オープンtype:feature
BHoM/File_Toolkit#193 · 担当者 1 名 ·
-
type:bug type:feature
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
BHoM/File_Toolkit#191 ·
-
Create Unit Tests to automate regression testing再び着手できるかも @alelom が 918 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンtype:test-script
BHoM/File_Toolkit#189 · 担当者 1 名 ·
-
Enable writing to JSON without including BHoM-specific attributes再び着手できるかも @alelom が 1619 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンtype:feature
BHoM/File_Toolkit#133 · コメント 1 件 · 担当者 1 名 ·
BHoM/File_Toolkit の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
Esri/calcite-dotnet-toolkit#30 · リアクション 1 件 ·
-
VideoViewer: rotated (portrait phone) videos shown sideways when system decimal separator is a commaオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
security
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
MorganHacks/Arctic#182 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Volodymyr-Petrunin/Bankomaten#45 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
thekid/inotify-win#45 ·