PutShipCommand boundary check bug

[erjillsison]

isBodyWithinBounds() method in PutShipCommand does not work properly.

Line 123 and 124 should have > inequality instead of >=.
Also, because of the || on line 124, it ignores orientation which should not be the case.
e.g Put a length 2 ship horizontally on e4 on a size 5 map. Will result in invalid because it's not within bounds vertically.

[iwle]

Good catch! I will rectify the inequality comparison bug.

For the second bug, until the put by orientation ability is included in the PutShipCommand, I will leave the default to horizontal boundary checking.

[iwle]

Fixed bug. See #112.