Skip to content
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

[B06]missingになっている参照の検出 #76

Closed
Kozu-vr opened this issue Jun 9, 2019 · 3 comments
Closed

[B06]missingになっている参照の検出 #76

Kozu-vr opened this issue Jun 9, 2019 · 3 comments
Assignees
Labels
enhancement New feature or request Validator VitDeck機能の一つ。対象が設定されたルールに従っているか検証します。
Projects

Comments

@Kozu-vr
Copy link
Contributor

Kozu-vr commented Jun 9, 2019

#58 個別ルールの実装

項目
セクション Booth
ID B06
クラス名 MissingReferenceRule
ルール missingになっている参照の検出
検証対象 ブースオブジェクト
非アクティブオブジェクトの扱い 検証する
拡張設定 なし
デフォルト値 なし
備考 シーン上のMissing Prefab、GameObjectにアタッチされたコンポーネント(MonoBehaviour)のmissing、SerializedPropertyのobjectReferenceのmissingを検出する

[サンプル]

[Validation]
public IRule missingReferenceRule = new MissingReferenceRule("[B06]missing参照ルール");
@Kozu-vr Kozu-vr added enhancement New feature or request Validator VitDeck機能の一つ。対象が設定されたルールに従っているか検証します。 labels Jun 9, 2019
@Kozu-vr Kozu-vr added this to To do in 1.0.0 Jun 9, 2019
@Kozu-vr Kozu-vr moved this from To do to In progress in 1.0.0 Jun 9, 2019
@sokuhatiku
Copy link
Contributor

@Kozu-vr プルリクエストを出そうとした時に気が付いたのですが、こちら、対象にシーン上のオブジェクトだけでなくベースフォルダ内の全アセットも含んだ方が良いのではないでしょうか?(今はブースオブジェクトのみ検出する形になっています。)

@Kozu-vr
Copy link
Contributor Author

Kozu-vr commented Jun 22, 2019

@Kozu-vr プルリクエストを出そうとした時に気が付いたのですが、こちら、対象にシーン上のオブジェクトだけでなくベースフォルダ内の全アセットも含んだ方が良いのではないでしょうか?(今はブースオブジェクトのみ検出する形になっています。)

自分はどちらかというと不要という立場でMissingの部分にシーン上で新しい参照を設定するような運用もありえるかなと思っていました。
VitDeckではシーン上のブースオブジェクトが最終的な入稿物として見るという方針にしたいですがいかがでしょうか?
作るとしたらAssetMissingReferenceRuleがアセットセクションのルールとして追加される気がします。

@sokuhatiku
Copy link
Contributor

@Kozu-vr なるほど、了解しました。
その場合でもオブジェクト参照を辿れる範囲でmissingフィールドのチェックをしておいた方が良さそうなので、そこだけ追加してからプルリクを出そうと思います。

@sokuhatiku sokuhatiku mentioned this issue Jun 23, 2019
Kozu-vr added a commit that referenced this issue Jun 23, 2019
@Kozu-vr Kozu-vr moved this from In progress to Done in 1.0.0 Jun 23, 2019
@Kozu-vr Kozu-vr closed this as completed Jun 23, 2019
sokuhatiku added a commit that referenced this issue Sep 2, 2021
拡張子のブラックリストルールで、ベースフォルダ外のアセットを検査対象から除外
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Validator VitDeck機能の一つ。対象が設定されたルールに従っているか検証します。
Projects
No open projects
1.0.0
  
Done
Development

No branches or pull requests

2 participants