New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DataModel.schemaがversion1.1.8でDataModel._schemaになっている #236

Closed
fukudayasuo opened this Issue Aug 22, 2013 · 2 comments

Comments

Projects
None yet
2 participants
@fukudayasuo

fukudayasuo commented Aug 22, 2013

DataModelがschemaプロパティを持つのは仕様なので、_schemaschemaに戻す。

FW内部でmodels[name].schemaとしている箇所があり(ベーススキーマのチェック箇所(l.576))、それでもテストが通っているので、その箇所を通るようなテストケースがあるかどうか、不足がないかを確認する。

@fukudayasuo

This comment has been minimized.

Show comment
Hide comment
@fukudayasuo

fukudayasuo Aug 28, 2013

ベーススキーマのチェックで.schemaを使っていた箇所は、修正前ではundefinedが返ってきており、正しく動作しておらず、baseにデータモデルを指定した場合はschema定義を省略できる、という仕様を満たしていませんでした。
別にissueを立てました。 #239

修正してテストケースの追加を行いました。

fukudayasuo commented Aug 28, 2013

ベーススキーマのチェックで.schemaを使っていた箇所は、修正前ではundefinedが返ってきており、正しく動作しておらず、baseにデータモデルを指定した場合はschema定義を省略できる、という仕様を満たしていませんでした。
別にissueを立てました。 #239

修正してテストケースの追加を行いました。

fukudayasuo pushed a commit to hifive-labs/hifivemain that referenced this issue Aug 28, 2013

fukudayasuo
#236
ObsItem,DataModelにschemaプロパティを持たせ、DataItem(≠ObsItem)にはschemaは持たせないようにしました。
@fukudayasuo

This comment has been minimized.

Show comment
Hide comment
@fukudayasuo

fukudayasuo Aug 28, 2013

schemaプロパティ(1.1.8では_schemaプロパティ)は、DataModel,DataItem,ObservableItem全てが持っていましたが、DataModel、ObservableItemだけに持たせて、ObservableItemでないDataItemはschemaプロパティを持たないように変更しました。

DataModelに属するDataItemのスキーマオブジェクトは、そのDataItemが属するDataModelから取得してください。

var schema = dataItem.getModel().schema;

fukudayasuo commented Aug 28, 2013

schemaプロパティ(1.1.8では_schemaプロパティ)は、DataModel,DataItem,ObservableItem全てが持っていましたが、DataModel、ObservableItemだけに持たせて、ObservableItemでないDataItemはschemaプロパティを持たないように変更しました。

DataModelに属するDataItemのスキーマオブジェクトは、そのDataItemが属するDataModelから取得してください。

var schema = dataItem.getModel().schema;

fukudayasuo pushed a commit to hifive-labs/hifivemain that referenced this issue Feb 20, 2014

fukudayasuo
#236 ソースコード修正。
createSchemaInfoCacheで引数にmodelを取る必要はないので削除

@simdy simdy closed this Feb 20, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment