Endless-Sky-Pirate-Campaign / endless-sky

Space exploration, trading, and combat game.
http://endless-sky.github.io/
GNU General Public License v3.0
2 stars 1 forks source link

All map outfitters panel shows all items for sale at 100% of outfitters #21

Closed samrocketman closed 2 years ago

samrocketman commented 2 years ago

Is there an existing issue for this?

Describe the bug

Playing from commit 70f2c2b1700ddc65cb1aa40bbd9333ccc1893f31 on pirate-complete branch.

Finding where outfits are for sale is not currently possible.

Commit introducing bug

I ran git bisect and found this commit introduces the bug: https://github.com/Endless-Sky-Pirate-Campaign/endless-sky/commit/7f33de2caabebc345b3602cd41a27232bde9fd44

Steps to Reproduce

  1. Press M for map
  2. Press O for Outfitters Map panel
  3. Click on different outfits for sale to see which outfitter sells them
  4. Notice 100% of all outfitters are reported as selling every outfit. Even if an outfitter does not sell an item it will still be reported as such.

Expected Behavior

Clicking on different outfitters should show the map having different

Screenshots

Notice every outfitter is orange (only some should be orange).

Screenshot from 2022-01-12 16-08-27

Link to save file

No response

Operating System

Ubuntu 20.04.3 LTS

Game Source

Other (please describe)

Game Version

70f2c2b1700ddc65cb1aa40bbd9333ccc1893f31

Additional Information

Here's is how I build the game to play it https://gist.github.com/samrocketman/6068b09f46c24370c359b2bae438b7a4

Hurleveur commented 2 years ago

just fixed it on my branch! it was a remnant of when the price for outfits would be sent back as 0 when they weren't sold somewhere I optimized the code a bit whilst I was at it

samrocketman commented 2 years ago

Fixed on pirate-complete branch at commit 1cc1a2679d6177452b1edfb71218ab69f2b3d6be