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

Abolished Fiery Touch throws error when used facing left #2442

Open
andretchen0 opened this issue Jul 19, 2023 · 1 comment
Open

Abolished Fiery Touch throws error when used facing left #2442

andretchen0 opened this issue Jul 19, 2023 · 1 comment
Labels
abilities The issue is related to an unit ability or more coding This issue requires some programming
Milestone

Comments

@andretchen0
Copy link
Contributor

Description

When using the Abolished creature's "Fiery Touch" move while the creature is facing left, an error is thrown:

target is undefined

I believe this is a long-standing error. I checked through several months of commits and all exhibit the same error.

There is no target in the path here:

const target = arrayUtils.last(path).creature;

That's as far up the call stack as I've looked.

To reproduce

Here's a game log. Paste it into the browser console.

AB.restoreGame("Ancient Beast 0.5.0 GameLog | Generated on 2023-07-19 ||eyJhY3Rpb25zIjpbeyJhY3Rpb24iOiJhYmlsaXR5IiwidGFyZ2V0Ijp7InR5cGUiOiJoZXgiLCJ4Ijo0LCJ5IjozfSwiaWQiOjMsImFyZ3MiOnsiMSI6eyJjcmVhdHVyZSI6IlA3IiwiY29zdCI6MTB9LCIyIjp7InF1ZXJ5T3B0aW9ucyI6eyJhcmdzIjp7ImNyZWF0dXJlIjoiUDciLCJjb3N0IjoxMH0sImhleGVzIjpbeyJ4IjoyLCJ5IjowfSx7IngiOjEsInkiOjB9LHsieCI6MCwieSI6MH0seyJ4IjoyLCJ5IjoxfSx7IngiOjEsInkiOjF9LHsieCI6MCwieSI6MX0seyJ4IjozLCJ5IjoxfSx7IngiOjIsInkiOjF9LHsieCI6MSwieSI6MX0seyJ4IjoyLCJ5IjoyfSx7IngiOjEsInkiOjJ9LHsieCI6MCwieSI6Mn0seyJ4IjozLCJ5IjoyfSx7IngiOjIsInkiOjJ9LHsieCI6MSwieSI6Mn0seyJ4IjoyLCJ5IjozfSx7IngiOjEsInkiOjN9LHsieCI6MCwieSI6M30seyJ4IjozLCJ5IjozfSx7IngiOjIsInkiOjN9LHsieCI6MSwieSI6M30seyJ4Ijo0LCJ5IjozfSx7IngiOjMsInkiOjN9LHsieCI6MiwieSI6M30seyJ4IjozLCJ5Ijo0fSx7IngiOjIsInkiOjR9LHsieCI6MSwieSI6NH0seyJ4Ijo0LCJ5Ijo0fSx7IngiOjMsInkiOjR9LHsieCI6MiwieSI6NH0seyJ4IjoyLCJ5Ijo1fSx7IngiOjEsInkiOjV9LHsieCI6MCwieSI6NX0seyJ4IjozLCJ5Ijo1fSx7IngiOjIsInkiOjV9LHsieCI6MSwieSI6NX0seyJ4Ijo0LCJ5Ijo1fSx7IngiOjMsInkiOjV9LHsieCI6MiwieSI6NX0seyJ4IjoyLCJ5Ijo2fSx7IngiOjEsInkiOjZ9LHsieCI6MCwieSI6Nn0seyJ4IjozLCJ5Ijo2fSx7IngiOjIsInkiOjZ9LHsieCI6MSwieSI6Nn0seyJ4IjoyLCJ5Ijo3fSx7IngiOjEsInkiOjd9LHsieCI6MCwieSI6N30seyJ4IjozLCJ5Ijo3fSx7IngiOjIsInkiOjd9LHsieCI6MSwieSI6N30seyJ4IjoyLCJ5Ijo4fSx7IngiOjEsInkiOjh9LHsieCI6MCwieSI6OH1dLCJoZXhlc0Rhc2hlZCI6W10sInNocnVua2VuSGV4ZXMiOltdLCJoZXhlc0RlYWRab25lIjpbXSwic2l6ZSI6MywiaWQiOjAsImZsaXBwZWQiOmZhbHNlLCJoaWRlTm9uVGFyZ2V0IjpmYWxzZSwib3duQ3JlYXR1cmVIZXhTaGFkZSI6ZmFsc2UsInRhcmdldGluZyI6dHJ1ZSwiZmlsbEhleE9uSG92ZXIiOnRydWV9fX19LHsiYWN0aW9uIjoic2tpcCJ9LHsiYWN0aW9uIjoiYWJpbGl0eSIsInRhcmdldCI6eyJ0eXBlIjoiaGV4IiwieCI6MTMsInkiOjZ9LCJpZCI6MywiYXJncyI6eyIxIjp7ImNyZWF0dXJlIjoiUDciLCJjb3N0IjoxMH0sIjIiOnsicXVlcnlPcHRpb25zIjp7ImFyZ3MiOnsiY3JlYXR1cmUiOiJQNyIsImNvc3QiOjEwfSwiaGV4ZXMiOlt7IngiOjE0LCJ5IjowfSx7IngiOjEzLCJ5IjowfSx7IngiOjEyLCJ5IjowfSx7IngiOjE0LCJ5IjoxfSx7IngiOjEzLCJ5IjoxfSx7IngiOjEyLCJ5IjoxfSx7IngiOjE1LCJ5IjoxfSx7IngiOjE0LCJ5IjoxfSx7IngiOjEzLCJ5IjoxfSx7IngiOjEzLCJ5IjoyfSx7IngiOjEyLCJ5IjoyfSx7IngiOjExLCJ5IjoyfSx7IngiOjE0LCJ5IjoyfSx7IngiOjEzLCJ5IjoyfSx7IngiOjEyLCJ5IjoyfSx7IngiOjEzLCJ5IjozfSx7IngiOjEyLCJ5IjozfSx7IngiOjExLCJ5IjozfSx7IngiOjE0LCJ5IjozfSx7IngiOjEzLCJ5IjozfSx7IngiOjEyLCJ5IjozfSx7IngiOjE1LCJ5IjozfSx7IngiOjE0LCJ5IjozfSx7IngiOjEzLCJ5IjozfSx7IngiOjEyLCJ5Ijo0fSx7IngiOjExLCJ5Ijo0fSx7IngiOjEwLCJ5Ijo0fSx7IngiOjEzLCJ5Ijo0fSx7IngiOjEyLCJ5Ijo0fSx7IngiOjExLCJ5Ijo0fSx7IngiOjEzLCJ5Ijo1fSx7IngiOjEyLCJ5Ijo1fSx7IngiOjExLCJ5Ijo1fSx7IngiOjE0LCJ5Ijo1fSx7IngiOjEzLCJ5Ijo1fSx7IngiOjEyLCJ5Ijo1fSx7IngiOjE1LCJ5Ijo1fSx7IngiOjE0LCJ5Ijo1fSx7IngiOjEzLCJ5Ijo1fSx7IngiOjEzLCJ5Ijo2fSx7IngiOjEyLCJ5Ijo2fSx7IngiOjExLCJ5Ijo2fSx7IngiOjE0LCJ5Ijo2fSx7IngiOjEzLCJ5Ijo2fSx7IngiOjEyLCJ5Ijo2fSx7IngiOjE0LCJ5Ijo3fSx7IngiOjEzLCJ5Ijo3fSx7IngiOjEyLCJ5Ijo3fSx7IngiOjE1LCJ5Ijo3fSx7IngiOjE0LCJ5Ijo3fSx7IngiOjEzLCJ5Ijo3fSx7IngiOjE0LCJ5Ijo4fSx7IngiOjEzLCJ5Ijo4fSx7IngiOjEyLCJ5Ijo4fV0sImhleGVzRGFzaGVkIjpbXSwic2hydW5rZW5IZXhlcyI6W10sImhleGVzRGVhZFpvbmUiOltdLCJzaXplIjozLCJpZCI6MCwiZmxpcHBlZCI6dHJ1ZSwiaGlkZU5vblRhcmdldCI6ZmFsc2UsIm93bkNyZWF0dXJlSGV4U2hhZGUiOmZhbHNlLCJ0YXJnZXRpbmciOnRydWUsImZpbGxIZXhPbkhvdmVyIjp0cnVlfX19fSx7ImFjdGlvbiI6InNraXAifSx7ImFjdGlvbiI6InNraXAifSx7ImFjdGlvbiI6InNraXAifSx7ImFjdGlvbiI6ImFiaWxpdHkiLCJ0YXJnZXQiOnsidHlwZSI6ImhleCIsIngiOjMsInkiOjZ9LCJpZCI6MywiYXJncyI6eyIxIjp7ImNyZWF0dXJlIjoiUDMiLCJjb3N0Ijo1fSwiMiI6eyJxdWVyeU9wdGlvbnMiOnsiYXJncyI6eyJjcmVhdHVyZSI6IlAzIiwiY29zdCI6NX0sImhleGVzIjpbeyJ4IjoxLCJ5IjowfSx7IngiOjAsInkiOjB9LHsieCI6MiwieSI6MH0seyJ4IjoxLCJ5IjowfSx7IngiOjEsInkiOjF9LHsieCI6MCwieSI6MX0seyJ4IjoyLCJ5IjoxfSx7IngiOjEsInkiOjF9LHsieCI6MywieSI6MX0seyJ4IjoyLCJ5IjoxfSx7IngiOjEsInkiOjJ9LHsieCI6MCwieSI6Mn0seyJ4IjoyLCJ5IjoyfSx7IngiOjEsInkiOjJ9LHsieCI6MywieSI6Mn0seyJ4IjoyLCJ5IjoyfSx7IngiOjEsInkiOjN9LHsieCI6MCwieSI6M30seyJ4IjoyLCJ5Ijo0fSx7IngiOjEsInkiOjR9LHsieCI6MywieSI6NH0seyJ4IjoyLCJ5Ijo0fSx7IngiOjQsInkiOjR9LHsieCI6MywieSI6NH0seyJ4IjoxLCJ5Ijo1fSx7IngiOjAsInkiOjV9LHsieCI6MiwieSI6NX0seyJ4IjoxLCJ5Ijo1fSx7IngiOjMsInkiOjV9LHsieCI6MiwieSI6NX0seyJ4Ijo0LCJ5Ijo1fSx7IngiOjMsInkiOjV9LHsieCI6MSwieSI6Nn0seyJ4IjowLCJ5Ijo2fSx7IngiOjIsInkiOjZ9LHsieCI6MSwieSI6Nn0seyJ4IjozLCJ5Ijo2fSx7IngiOjIsInkiOjZ9LHsieCI6MSwieSI6N30seyJ4IjowLCJ5Ijo3fSx7IngiOjIsInkiOjd9LHsieCI6MSwieSI6N30seyJ4IjozLCJ5Ijo3fSx7IngiOjIsInkiOjd9LHsieCI6MSwieSI6OH0seyJ4IjowLCJ5Ijo4fSx7IngiOjIsInkiOjh9LHsieCI6MSwieSI6OH1dLCJoZXhlc0Rhc2hlZCI6W10sInNocnVua2VuSGV4ZXMiOltdLCJoZXhlc0RlYWRab25lIjpbXSwic2l6ZSI6MiwiaWQiOjAsImZsaXBwZWQiOmZhbHNlLCJoaWRlTm9uVGFyZ2V0IjpmYWxzZSwib3duQ3JlYXR1cmVIZXhTaGFkZSI6ZmFsc2UsInRhcmdldGluZyI6dHJ1ZSwiZmlsbEhleE9uSG92ZXIiOnRydWV9fX19LHsiYWN0aW9uIjoic2tpcCJ9LHsiYWN0aW9uIjoiYWJpbGl0eSIsInRhcmdldCI6eyJ0eXBlIjoiaGV4IiwieCI6MTIsInkiOjN9LCJpZCI6MywiYXJncyI6eyIxIjp7ImNyZWF0dXJlIjoiUDMiLCJjb3N0Ijo1fSwiMiI6eyJxdWVyeU9wdGlvbnMiOnsiYXJncyI6eyJjcmVhdHVyZSI6IlAzIiwiY29zdCI6NX0sImhleGVzIjpbeyJ4IjoxMywieSI6MH0seyJ4IjoxMiwieSI6MH0seyJ4IjoxNCwieSI6MH0seyJ4IjoxMywieSI6MH0seyJ4IjoxMywieSI6MX0seyJ4IjoxMiwieSI6MX0seyJ4IjoxNCwieSI6MX0seyJ4IjoxMywieSI6MX0seyJ4IjoxNSwieSI6MX0seyJ4IjoxNCwieSI6MX0seyJ4IjoxMiwieSI6Mn0seyJ4IjoxMSwieSI6Mn0seyJ4IjoxMywieSI6Mn0seyJ4IjoxMiwieSI6Mn0seyJ4IjoxNCwieSI6Mn0seyJ4IjoxMywieSI6Mn0seyJ4IjoxMiwieSI6M30seyJ4IjoxMSwieSI6M30seyJ4IjoxMywieSI6M30seyJ4IjoxMiwieSI6M30seyJ4IjoxNCwieSI6M30seyJ4IjoxMywieSI6M30seyJ4IjoxNSwieSI6M30seyJ4IjoxNCwieSI6M30seyJ4IjoxMSwieSI6NH0seyJ4IjoxMCwieSI6NH0seyJ4IjoxMiwieSI6NH0seyJ4IjoxMSwieSI6NH0seyJ4IjoxMywieSI6NH0seyJ4IjoxMiwieSI6NH0seyJ4IjoxMiwieSI6NX0seyJ4IjoxMSwieSI6NX0seyJ4IjoxMywieSI6NX0seyJ4IjoxMiwieSI6NX0seyJ4IjoxNCwieSI6NX0seyJ4IjoxMywieSI6NX0seyJ4IjoxNSwieSI6NX0seyJ4IjoxNCwieSI6NX0seyJ4IjoxMywieSI6N30seyJ4IjoxMiwieSI6N30seyJ4IjoxNCwieSI6N30seyJ4IjoxMywieSI6N30seyJ4IjoxNSwieSI6N30seyJ4IjoxNCwieSI6N30seyJ4IjoxMywieSI6OH0seyJ4IjoxMiwieSI6OH0seyJ4IjoxNCwieSI6OH0seyJ4IjoxMywieSI6OH1dLCJoZXhlc0Rhc2hlZCI6W10sInNocnVua2VuSGV4ZXMiOltdLCJoZXhlc0RlYWRab25lIjpbXSwic2l6ZSI6MiwiaWQiOjAsImZsaXBwZWQiOnRydWUsImhpZGVOb25UYXJnZXQiOmZhbHNlLCJvd25DcmVhdHVyZUhleFNoYWRlIjpmYWxzZSwidGFyZ2V0aW5nIjp0cnVlLCJmaWxsSGV4T25Ib3ZlciI6dHJ1ZX19fX0seyJhY3Rpb24iOiJza2lwIn0seyJhY3Rpb24iOiJza2lwIn0seyJhY3Rpb24iOiJza2lwIn0seyJhY3Rpb24iOiJtb3ZlIiwidGFyZ2V0Ijp7IngiOjgsInkiOjZ9fSx7ImFjdGlvbiI6InNraXAifSx7ImFjdGlvbiI6Im1vdmUiLCJ0YXJnZXQiOnsieCI6OCwieSI6M319LHsiYWN0aW9uIjoic2tpcCJ9LHsiYWN0aW9uIjoic2tpcCJ9LHsiYWN0aW9uIjoic2tpcCJ9LHsiYWN0aW9uIjoiYWJpbGl0eSIsInRhcmdldCI6eyJ0eXBlIjoiYXJyYXkiLCJhcnJheSI6W3sieCI6NSwieSI6M30seyJ4Ijo2LCJ5IjozfSx7IngiOjcsInkiOjN9LHsieCI6OCwieSI6M31dfSwiaWQiOjEsImFyZ3MiOnsiMSI6eyJkaXJlY3Rpb24iOjEsImhleCI6eyJ4Ijo3LCJ5IjozfSwiY2hvaWNlSW5kZXgiOjB9LCIyIjp7InF1ZXJ5T3B0aW9ucyI6eyJ0ZWFtIjoxLCJyZXF1aXJlQ3JlYXR1cmUiOnRydWUsImlkIjoyLCJhcmdzIjp7ImRpcmVjdGlvbiI6MSwiaGV4Ijp7IngiOjcsInkiOjN9LCJjaG9pY2VJbmRleCI6MH0sImZsaXBwZWQiOmZhbHNlLCJjaG9pY2VzIjpbW3sieCI6NSwieSI6M30seyJ4Ijo2LCJ5IjozfSx7IngiOjcsInkiOjN9LHsieCI6OCwieSI6M31dXSwiaGV4ZXNEYXNoZWQiOltdLCJoZXhlc0RlYWRab25lIjpbXSwic2hydW5rZW5IZXhlcyI6W3sieCI6OCwieSI6M31dLCJpc0RpcmVjdGlvbnNRdWVyeSI6dHJ1ZSwiaGlkZU5vblRhcmdldCI6dHJ1ZSwiZGFzaGVkSGV4ZXNVbmRlckNyZWF0dXJlIjpmYWxzZSwieCI6NCwieSI6MywiZGlyZWN0aW9ucyI6WzEsMSwxLDEsMSwxXSwiaW5jbHVkZUNyZWF0dXJlIjp0cnVlLCJzdG9wT25DcmVhdHVyZSI6dHJ1ZSwiZGlzdGFuY2UiOjMsIm1pbkRpc3RhbmNlIjowLCJkaXN0YW5jZUZhbGxvZmYiOjMsImRhc2hlZEhleGVzQWZ0ZXJDcmVhdHVyZVN0b3AiOnRydWUsImRhc2hlZEhleGVzRGlzdGFuY2UiOjAsInNvdXJjZUNyZWF0dXJlIjp7fX19fX0seyJhY3Rpb24iOiJza2lwIn0seyJhY3Rpb24iOiJhYmlsaXR5IiwidGFyZ2V0Ijp7InR5cGUiOiJhcnJheSIsImFycmF5IjpbeyJ4Ijo3LCJ5Ijo2fSx7IngiOjgsInkiOjZ9LHsieCI6OSwieSI6Nn0seyJ4IjoxMCwieSI6Nn1dfSwiaWQiOjEsImFyZ3MiOnsiMSI6eyJkaXJlY3Rpb24iOjQsImhleCI6eyJ4Ijo4LCJ5Ijo2fSwiY2hvaWNlSW5kZXgiOjB9LCIyIjp7InF1ZXJ5T3B0aW9ucyI6eyJ0ZWFtIjoxLCJyZXF1aXJlQ3JlYXR1cmUiOnRydWUsImlkIjozLCJhcmdzIjp7ImRpcmVjdGlvbiI6NCwiaGV4Ijp7IngiOjgsInkiOjZ9LCJjaG9pY2VJbmRleCI6MH0sImZsaXBwZWQiOnRydWUsImNob2ljZXMiOltbeyJ4Ijo3LCJ5Ijo2fSx7IngiOjgsInkiOjZ9LHsieCI6OSwieSI6Nn0seyJ4IjoxMCwieSI6Nn1dXSwiaGV4ZXNEYXNoZWQiOltdLCJoZXhlc0RlYWRab25lIjpbXSwic2hydW5rZW5IZXhlcyI6W3sieCI6MTAsInkiOjZ9XSwiaXNEaXJlY3Rpb25zUXVlcnkiOnRydWUsImhpZGVOb25UYXJnZXQiOnRydWUsImRhc2hlZEhleGVzVW5kZXJDcmVhdHVyZSI6ZmFsc2UsIngiOjEzLCJ5Ijo2LCJkaXJlY3Rpb25zIjpbMSwxLDEsMSwxLDFdLCJpbmNsdWRlQ3JlYXR1cmUiOnRydWUsInN0b3BPbkNyZWF0dXJlIjp0cnVlLCJkaXN0YW5jZSI6MywibWluRGlzdGFuY2UiOjAsImRpc3RhbmNlRmFsbG9mZiI6MywiZGFzaGVkSGV4ZXNBZnRlckNyZWF0dXJlU3RvcCI6dHJ1ZSwiZGFzaGVkSGV4ZXNEaXN0YW5jZSI6MCwic291cmNlQ3JlYXR1cmUiOnt9fX19fV0sInZlcnNpb24iOiIwLjUuMCIsImRhdGUiOiIyMDIzLTA3LTE5VDIyOjM2OjQ3Ljg2OFoiLCJjdXN0b20iOnsiY29uZmlnRGF0YSI6eyJwbGF5ZXJNb2RlIjoyLCJjcmVhTGltaXROYnIiOjMsInVuaXREcm9wcyI6MSwiYWJpbGl0eVVwZ3JhZGVzIjozLCJwbGFzbWFfYW1vdW50IjozMCwidHVyblRpbWVQb29sIjotMSwidGltZVBvb2wiOi02MCwiYmFja2dyb3VuZF9pbWFnZSI6IlNoYWRvdyBDYXZlIiwiY29tYmF0TG9jYXRpb24iOiJTaGFkb3cgQ2F2ZSIsImZ1bGxzY3JlZW5Nb2RlIjpmYWxzZX19fQ==")
@DreadKnight DreadKnight added this to the 0.5 - Chimera milestone Jul 20, 2023
@DreadKnight DreadKnight added coding This issue requires some programming abilities The issue is related to an unit ability or more labels Jul 20, 2023
@andretchen0 andretchen0 removed their assignment Aug 4, 2023
@JoelFernandes09
Copy link
Contributor

Looked into this a bit & I think the wider fix is to convert these files to TS & implement the functions since I think a lot of stuff has been refactored. I found out that there's a temp fix in Chimera, Gumble & more that fixes this issue. Should I open a PR with the same fix for this since it will be playable for the time being? Also do we have conversion issues for files that aren't TS yet?

@andretchen0 @DreadKnight

@DreadKnight DreadKnight pinned this issue May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
abilities The issue is related to an unit ability or more coding This issue requires some programming
Projects
None yet
Development

No branches or pull requests

3 participants