search

mickyscreggs / SWEN222_GroupPage_ArtGame

0 stars 1 forks source link

does Art need a removeWall method? #12

Closed return-string closed 8 years ago

return-string commented 8 years ago

Are we just removing Walls directly from an Art object's set of walls?

mickyscreggs commented 8 years ago

It's for when a player takes a painting that's on multiple walls. It fixes a bug where a player would take a one of these paintings, but the painting would still be on the other walls as well as in the player's inventory.

Check the facebook convo and commit 6c84656 for details

kaishuoy commented 8 years ago

i dont think it does because you dont need to remove walls from a art's set of walls. you need to do it the other way around(remove art from a wall) hence each art has a set of walls it appears on to deal with 1 art, 2 walls scenario

return-string commented 8 years ago

OK, sweet, just wondering if anything was depending on that! Thanks guys!