openra 1.0.1 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 53319a87b7e4de611a2dff2eb7ff000d31214070
-  data.tar.gz: fdb078643277f89a02e6822dbefac5c37703af61
+  metadata.gz: ab7a16c3f01e989e67ae2aa310703ffff750855b
+  data.tar.gz: 1b0a85ea770c2d1a9f3e2914f900da66949eb50d
 SHA512:
-  metadata.gz: d60e345f5f50e8f2fcba27042e85fc9e6eb56f3076216d42450b57f202c5d0245295bfb33101f36a17682b8b2b00aa4cc8fd1674a84e2f2ed89d8531b734c97d
-  data.tar.gz: 67bef8e53bc78c6a6ee8c020fd87856177f1889e53cb825d6deec16806d7876585654d2e20110e8bdeb8a43add000f872fc686ce610df0ada0d72190278c14cd
+  metadata.gz: 4b5e36e2e31a514e081dc0f2d990f201d7db44de8d0c34eb6f47e13eff28d7a3753b0762580fe55d99483f1acd3904a4e5c68f84e7f939f07d91b56523530715
+  data.tar.gz: 9bab1534a3739e98b9b0182f98ca147890ef17646ba8b5523be0c9d219275f1d727a72ca461f1c43a4e09a52c9f45824824c8ac2c44c6c1a3938ecdc726345b6

data/CHANGELOG.md

@@ -2,10 +2,19 @@
 
 ## Fixed
 
+* [replay-data] Make chat messages show name at time of message ([AMHOL](https://github.com/AMHOL))
+* [replay-data] Only include clients present at game start ([AMHOL](https://github.com/AMHOL))
+
+[Compare v1.0.1...HEAD](https://github.com/AMHOL/openra-ruby/compare/v1.0.1...HEAD)
+
+## v1.0.1
+
+## Fixed
+
 * [core] Add bundler as a runtime dependency (thanks to by dragunoff) ([AMHOL](https://github.com/AMHOL))
 * [replay-data] Fixed the game "type" field (by updating it to use players rather than clients array) ([AMHOL](https://github.com/AMHOL))
 
-[Compare v1.0.0...HEAD](https://github.com/AMHOL/openra-ruby/compare/v1.0.0...HEAD)
+[Compare v1.0.0...v1.0.1](https://github.com/AMHOL/openra-ruby/compare/v1.0.0...v1.0.1)
 
 ## v1.0.0
 

data/lib/openra/cli.rb

@@ -1,4 +1,5 @@
 require 'securerandom'
+require 'ostruct'
 require 'yaml'
 require 'json'
 require 'hanami/cli'

data/lib/openra/cli/commands/replay_data.rb

@@ -79,10 +79,20 @@ module Openra
             chat: []
           }
 
+          current_sync_info = OpenStruct.new(
+            clients: []
+          )
+
           replay.orders.each do |order|
-            client = replay.client(order.client_index.to_s)
+            client = current_sync_info.clients.find do |candidate|
+              candidate.index == order.client_index.to_s
+            end
 
             case order.command
+            when 'SyncInfo'
+              current_sync_info = Openra::Struct::SyncInfo.new(
+                Openra::YAML.load(order.target)
+              )
             when 'PlaceBuilding'
               client_hash = replay_data[:clients].find do |candidate|
                 candidate[:index] == order.client_index.to_s

data/lib/openra/replays/replay.rb

@@ -54,22 +54,13 @@ module Openra
       def sync_info
         return @sync_info if @sync_info
 
-        syncs = orders.reverse.each_with_object([]) do |order, arr|
-          next unless order.command == 'SyncInfo'
+        start_game_order = orders.find { |order| order[:command] == 'StartGame' }
+        start_game_index = orders.index(start_game_order)
+        start_game_sync_order = orders[start_game_index.pred]
 
-          arr << Openra::Struct::SyncInfo.new(
-            Openra::YAML.load(order.target)
-          )
-        end
-
-        @sync_info = syncs.inject(syncs.shift) do |next_sync, sync|
-          next_sync.clients.each do |client|
-            next if sync.clients.map(&:index).include?(client.index)
-            sync.clients << client
-          end
-
-          sync
-        end
+        @sync_info = Openra::Struct::SyncInfo.new(
+          Openra::YAML.load(start_game_sync_order.target)
+        )
       end
     end
   end

data/lib/openra/version.rb

@@ -1,3 +1,3 @@
 module Openra
-  VERSION = '1.0.1'.freeze
+  VERSION = '1.1.0'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openra
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.1.0
 platform: ruby
 authors:
 - Andy Holland
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-07-13 00:00:00.000000000 Z
+date: 2018-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler