Skip to content

[Artifact 0015] ガソリンの作成 - #2074

Open
ogachu0331 wants to merge 17 commits into
masterfrom
dev/ogachu_artifact0015
Open

ogachu0331 wants to merge 17 commits into
masterfrom
dev/ogachu_artifact0015

Conversation

@ogachu0331

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread Asset/data/asset/functions/artifact/0015.gasoline/give/2.give.mcfunction Outdated
Comment thread Asset/data/asset/functions/effect/0372.gasoline/register.mcfunction Outdated
Comment thread Asset/data/asset/functions/effect/0372.gasoline/modifier/add.mcfunction Outdated
Comment thread Asset/data/asset/functions/artifact/0015.gasoline/give/2.give.mcfunction Outdated
Comment thread Asset/data/asset/functions/effect/0372.gasoline/register.mcfunction Outdated
Comment thread Asset/data/asset/functions/effect/0372.gasoline/register.mcfunction Outdated
Comment thread Asset/data/asset/functions/artifact/0015.gasoline/melee/particle.mcfunction Outdated
Comment on lines +7 to +8
# 神器の基本的な条件の確認を行うfunction、成功している場合CanUsedタグが付く
#function asset:artifact/common/check_condition/auto

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

やりたいことのニュアンスは理解しつつ、そもそも実行しない、という状態は避けたいので
↓のやり方で同じことできないかな

Suggested change
# 神器の基本的な条件の確認を行うfunction、成功している場合CanUsedタグが付く
#function asset:artifact/common/check_condition/auto
# 神器の基本的な条件の確認を行うfunction、成功している場合CanUsedタグが付く
data modify storage asset:artifact DisabledCheckFlag set value {TypeCooldown:true,Believe:true}
function asset:artifact/common/check_condition/auto

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これだとmpがない時に使用した時、mpが足りないって出ると思うのですが、飲むときはmp関係なくできるようにしたいんですよね

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MPは消費するけど,MPRequire:trueを追加すれば行けそう

Comment on lines +16 to +24
# ダメージ
data modify storage api: Argument.Damage set value 10.0f
data modify storage api: Argument.AttackType set value "Physical"
data modify storage api: Argument.ElementType set value "None"
data modify storage api: Argument.DeathMessage append value '[{"translate":"%1$sはガソリン中毒で死んだ","with":[{"selector":"@s"}]}]'
data modify storage api: Argument.DeathMessage append value '[{"translate":"%1$sは自分を車と勘違いした","with":[{"selector":"@s"}]}]'
function api:damage/modifier
function api:damage/
function api:damage/reset

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これプレイヤーの補正かかるけどソレは意図してる?

Comment on lines +8 to +9
data modify storage api: Argument.UUID set value [I;1,3,372,0]
data modify storage api: Argument.UUID set from storage asset:context this.UUID

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Argument.UUIDはどっちかだけにしよう
個人的にはregisterにUUIDを設定しておいて、this.UUIDから拾ってくる方が確実だと思う (removeも同様)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants