pgn 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 70f3a7855b4d29274f4393cdc84bac75792dad94
-  data.tar.gz: 191c65ca1e5f3d7067629d2c167bdcfd495d8885
+  metadata.gz: d868495f784cc03dedf3465023bffe390d7228f5
+  data.tar.gz: d12ab910e19cac3c20cb2ed965fd460eadc1b657
 SHA512:
-  metadata.gz: 475172b8b7b9b84031b0de1fc4010b5b63250ae586d1bc018e2d5e5bd59ffbc530f369989bea9406169d3f94be93ff67d0be88d1dcb535a28398538cb4e0d96e
-  data.tar.gz: 78cc6d760518aaad4b0fa1a0d758199a037c29dd17877583eefaa6242f831c4f55c91d41750a8dfea2454cb73b5c264770edabb6c2775fe1a65a95523b534f28
+  metadata.gz: 790ce8940ab3aa672edb725073847d6c2154335afb9403d71acfdfa835a68658b6d2e2dced3d80f99291cafc361721482ae2f0982bd1221023bd9727b256a710
+  data.tar.gz: dac60e56ab33829619c706f2e4aab434b1fa4ef61833dd8ff1b1ed6e6547ab95557b72cb371d3462d9dc9cc78995ea73b91da1cb393dfb6717e6add315fcf7e1

data/lib/pgn/move_calculator.rb

@@ -105,26 +105,31 @@ module PGN
     # @return [Array<String>] which castling moves are no longer available
     #
     def castling_restrictions
-      restrict = case self.move.piece
-      when "K" then "KQ"
-      when "k" then "kq"
-      when "R"
-        {"a1" => "Q", "h1" => "K"}[self.origin]
-      when "r"
-        {"a8" => "q", "h8" => "k"}[self.origin]
+      restrict = []
+
+      # when a king or rook is moved
+      case self.move.piece
+      when 'K'
+        restrict += ['K', 'Q']
+      when 'k'
+        restrict += ['k', 'q']
+      when 'R'
+        restrict << {'a1' => 'Q', 'h1' => 'K'}[self.origin]
+      when 'r'
+        restrict << {'a8' => 'q', 'h8' => 'k'}[self.origin]
       end
 
-      restrict = "KQ" if ['K', 'Q'].include? move.castle
-      restrict = "kq" if ['k', 'q'].include? move.castle
-      
-      restrict += 'Q' if self.move.destination == 'a1' && !restrict.include?('Q')
-      restrict += 'q' if self.move.destination == 'a8' && !restrict.include?('q')
-      restrict += 'K' if self.move.destination == 'h1' && !restrict.include?('K')
-      restrict += 'k' if self.move.destination == 'h8' && !restrict.include?('k')
+      # when castling occurs
+      restrict += ['K', 'Q'] if ['K', 'Q'].include? move.castle
+      restrict += ['k', 'q'] if ['k', 'q'].include? move.castle
 
-      restrict ||= ''
+      # when a rook is taken
+      restrict << 'Q' if self.move.destination == 'a1'
+      restrict << 'q' if self.move.destination == 'a8'
+      restrict << 'K' if self.move.destination == 'h1'
+      restrict << 'k' if self.move.destination == 'h8'
 
-      restrict.split('')
+      restrict.compact.uniq
     end
 
     # @return [Boolean] whether to increment the halfmove clock

data/lib/pgn/version.rb

@@ -1,3 +1,3 @@
 module PGN
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

data/spec/fen_spec.rb

@@ -21,6 +21,12 @@ describe PGN::FEN do
       next_pos.to_fen.castling.should match(/q/)
     end
 
+    it "should remove one castling option when a rook is taken" do
+      pos = PGN::FEN.new("rn1qkbnr/pbpppppp/1p6/8/6P1/2N4P/PPPPPP2/R1BQKBNR b KQkq - 2 3").to_position
+      next_pos = pos.move("Bxh1")
+      next_pos.to_fen.castling.should_not match(/K/)
+    end
+
     it "should change to a hyphen once no side can castle" do
       pos = PGN::FEN.new("r1bq1rk1/pp1nbppp/3ppn2/8/2PP1N2/P1N5/1P2BPPP/R1BQK2R w KQ - 2 9").to_position
       pos.to_fen.castling.should_not == "-"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pgn
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Stacey Touset
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-16 00:00:00.000000000 Z
+date: 2016-03-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: whittle