MalitsPlus / ShizuruNotes

An unofficial Android tool application for "Princess Connect Re:Dive"

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

ホマレ's unknown skill target

HerDataSam opened this issue · comments

I found that ホマレ's target of IfForChildAction (UB Action 1 and 3, Skill 1 Action 2, and Skill 2 Action 1) is Unknown.

This issue comes from the wrong exception handling of ifType where you have assigned as IfType.unknown in line 125 in the IfForChildAction.
Her skill's target is not affected by the action_detail_1 but target_type as you handled.

Please consider the change below on both line 18 and line 43.

-    if(ifType != null) {
+    if(ifType != IfType.unknown) {

I already changed this to my forked project and it works well.

HerDataSam@6930386#diff-1ac07a801dee23cade0865922d03c2d6889be3c9bac6c5e9b440ae19ba323f55

Ah thanks, I haven't noticed this until you told me.
Besides, if you already have a solution you can definitely open a PR rather than an issue instead :)

Oh, that way is better. I have not thought about that because your project and my project are a lot different.
I will use Pull Requests next time. At this time, please fix the issue accordingly. :)