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

Aftershock atomic smartphone not playing music #37420

Closed
Entity-y opened this issue Jan 26, 2020 · 6 comments · Fixed by #37584
Closed

Aftershock atomic smartphone not playing music #37420

Entity-y opened this issue Jan 26, 2020 · 6 comments · Fixed by #37584
Labels
<Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact Mods: Aftershock Anything to do with the Aftershock mod (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@Entity-y
Copy link
Contributor

Describe the bug

The atomic smartphone doesn't play music.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Use Aftershock mod.
  2. Get an atomic smartphone.
  3. Activate listen to music.

Expected behavior

When activated, music should be playing through headphones like with the mp3 or smartphone.

Versions and configuration

  • OS: Windows 10
  • Game Version: 0.D-11598-g075afa3
  • Graphics version: Tiles
  • Mods loaded: Aftershock
@Unrepentant-Atheist
Copy link

Confirmed.

@ZhilkinSerg ZhilkinSerg added (S2 - Confirmed) Bug that's been confirmed to exist <Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact Mods: Aftershock Anything to do with the Aftershock mod labels Jan 27, 2020
@Maleclypse
Copy link
Member

My understanding is this has to do with hard coded mp3 use. If this is a json fixable issue I’m happy to work on it myself.

@Entity-y
Copy link
Contributor Author

Maybe it would make sense to also add the mp3 function to the Wraitheon executive's smartphone as well, since it's custom variation to the atomic smartphone and lacks that feature.

@eilaattwood
Copy link
Contributor

My understanding is this has to do with hard coded mp3 use. If this is a json fixable issue I’m happy to work on it myself.

Exactly. The list of devices capable of playing music is hardcoded into mp3 function. Adding atomic smartphone there is simple, but not an elegant solution that prevents of adding new player devices into the game without C++ code editing.

@eilaattwood
Copy link
Contributor

@Maleclypse
I can make mentioned changes to the code and create PR to quickfix this for now, but as I mentioned above it will still work only for limited list of devices. Should I add just an atomic smartphone, or something else from the mod?

@Night-Pryanik
Copy link
Contributor

A quickfix is still a fix, it's better than nothing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Items / Item Actions / Item Qualities Items and how they work and interact Mods: Aftershock Anything to do with the Aftershock mod (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants