Issue 1553

[c3ho]

Hi, this is the PR for issue-1553.

A few things I did:

• removed the button to materialize creatures when viewing a creature. Only right clicking on an empty hex or the godlet printer will display the materialize button
• I commented a ton of lines out, was a bit afraid to remove them, if requested I can since they'll be displayed in the commit
• added two new parameters to showCreature, although I'm pretty sure the first one is not needed, now that we don't need to keep track of the creature being viewed and the creature we want to summon(which needed to be tracked when right clicking on a creature on the field). The second one lets showCreature know to disable the materialize button if we are right clicking on a unit from the field

[vercel]

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

[c3ho]

@DreadKnight give this a try and see if the changes are to your liking. I can clean up some of the functions afterwards!

[DreadKnight]

@c3ho I've tested this and works fine, hope you'll clean up the stuff a little bit; I've found a few other bugs in the process while testing this, new issues opened xD

[DreadKnight]

@c3ho This is full of problems now, you need to test it more. The materialize button shows up a lot!

[c3ho]

@DreadKnight I’ll double check, might’ve pushed the wrong branch. Thanks!

[c3ho]

@DreadKnight
Tested the commit thoroughly this time =D. materialize button should appear a lot less now.
Materialize Button will not appear if:

• the active creature is not a Dark Priest
• the Dark Priest has already used materialize for the turn
• right clicking on a hex with a summoned creature

Materialize Button will appear if:

1. the active creature is a Dark Priest and has not summoned a creature for the turn
2. Condition 1 is true and if player right clicked on a hex without a creature

[DreadKnight]

@c3ho Found a bug: after materializing an unit, right clicking on an empty space still shows the button.

[c3ho]

@DreadKnight Oh I see.

[c3ho]

@DreadKnight were there other instances where you saw the materialize button appear when it shouldn't?

[DreadKnight]

@c3ho Did more testing and found another one, by clicking on avatars from the unit queue located top.

[c3ho]

@DreadKnight thanks, I think I just need to fix the clicking on portraits now that's left. Thank you for putting up with me haha.

[DreadKnight]

@c3ho No worries, it's always awesome when people like you take the time to contribute to the project!

[c3ho]

@DreadKnight When clicking on the portrait of the creatures, should the materialize bar appear if the player clicks on the portrait of their Dark Priest, or should it behave just like right clicking on a hex with a creature on it?

[DreadKnight]

@c3ho The logic is that when active unit is Dark Priest, the materialization button should guide the player into finding a suitable creature to materialize, hence why the status/error strings for that button. When not Dark Priest's turn, materialization button should be hidden 100%.

[c3ho]

@DreadKnight fixed the portrait issue as well. Materialize button should only appear if and only it is their Dark Priest's turn AND the materialize ability has not been used for the turn yet.

I think there's is going to be at least 1 maybe 2 bug(s) that need to be filed.

1. If a player views any creature from the Godlet Printer, but doesn't materialize, and clicks on the portrait of their own Dark Priest, the materialize button text will display with the plasma cost of the last viewed creature even though the Dark Priest is displayed.

2. Sometimes after clicking on the portrait and clicking to another creature after the materialize ability has been used, the message "This unit is under heavy development" appears.

[DreadKnight]

@c3ho Indeed, also:
3 . After materializing a unit and checking out the dash again, an error message should be displayed under the cards of available (non dead and non queued) creatures, as your PR makes "Materialization has already been used this round" string not shown anymore

[c3ho]

@DreadKnight sorry, mistakenly rebased with a commit from the master branch. Had to drop the commit and re-push!

Let me know how it goes. For some reason I got a bug when trying to summon the creature Stomper.

[DreadKnight]

@c3ho No worries, we dealt with some extra pipeline issues, had a temporary workaround committed for now. Regarding Stomper and a few others, they're not yet playable. See this regarding their "schedule" https://github.com/FreezingMoon/AncientBeast/projects/3
Stomper is extra problematic atm just by selecting the card itself #1563

If you're not poking at the remaining issues, I can merge and open up new issues in the tracker.

[c3ho]

Alright this one should be able to be merged. If the Godlet Printer has been used in the same turn, The Godlet Printer will show the 'Materialization has already been used this round' when a player re-opens dash, but if a player clicked on another creature in the grid afterwards the materialization bar disappeared. This last commit was to fix that issue.

[DreadKnight]

@c3ho I don't get why your package-lock.json file is committed xD
Materialization has already been used this round shouldn't be showing for queued or dead units btw, but I'll make that a separate issue.

[c3ho]

@DreadKnight I committed the file because the deployment thing kept bothering me about it. Sorry!

[DreadKnight]

@c3ho No worries. Well done with the issue!
I've opened #1566 regarding that error button and will see about the rest as well soon.