gillbus 0.16.7 → 0.16.8

data/lib/gillbus/helpers/update_attrs.rb

@@ -1,15 +1,13 @@
 class Gillbus
   module UpdateAttrs
-
-    def initialize(attrs={})
+    def initialize(attrs = {})
       update_attrs attrs if attrs
     end
 
-    def update_attrs(attrs={})
+    def update_attrs(attrs = {})
       attrs.each do |k, v|
         public_send "#{k}=", v
       end
     end
-
   end
 end

data/lib/gillbus/lock_seats.rb

@@ -1,10 +1,8 @@
 class Gillbus
   module LockSeats
-
     Gillbus.register self, :lock_seats
 
     class Request < BaseRequest
-
       def path; '/online2/lockSeats' end
 
       # tripId
@@ -12,11 +10,13 @@ class Gillbus
       attr_accessor :trip_id
 
       # carriageId (не обязательный)
-      # ИД вагона, для которого нужно получить карту мест. По- умолчанию первый попавшийся. Только для железнодорожных рейсов.
+      # ИД вагона, для которого нужно получить карту мест.
+      # По- умолчанию первый попавшийся. Только для железнодорожных рейсов.
       attr_accessor :carriage_id
 
       # backCarriageId (не обязательный)
-      # ИД вагона обратного рейса, для которого нужно получить карту мест. По-умолчанию первый попавшийся. Только для железнодорожных рейсов.
+      # ИД вагона обратного рейса, для которого нужно получить карту мест.
+      # По-умолчанию первый попавшийся. Только для железнодорожных рейсов.
       attr_accessor :back_carriage_id
 
       # segment0carriageId...segmentNcarriageId (не обязательный)
@@ -36,10 +36,10 @@ class Gillbus
       attr_accessor :segments_seat_ids
 
       def params
-        seats = Array(seat_ids).map.with_index {|id, n| [:"seatId#{n}", id] }.to_h
-        back_seats = Array(back_seat_ids).map.with_index {|id, n| [:"backSeatId#{n}", id] }.to_h
+        seats = Array(seat_ids).map.with_index { |id, n| [:"seatId#{n}", id] }.to_h
+        back_seats = Array(back_seat_ids).map.with_index { |id, n| [:"backSeatId#{n}", id] }.to_h
         segments_seats = Hash(segments_seat_ids).map do |segment, seat_ids|
-          Array(seat_ids).map.with_index {|id, n| [:"segment#{segment}seatId#{n}", id] }.to_h
+          Array(seat_ids).map.with_index { |id, n| [:"segment#{segment}seatId#{n}", id] }.to_h
         end.reduce(&:merge).to_h
 
         compact(
@@ -54,7 +54,6 @@ class Gillbus
     end
 
     class Response < BaseResponse
-
       # seat lock, time limit, in seconds
       field :time_limit, :milliseconds
       # [{Symbol => Boolean}] необходимые для ввода типы документов?
@@ -75,8 +74,6 @@ class Gillbus
           result
         end
       end
-
     end
-
   end
 end

data/lib/gillbus/logout.rb

@@ -1,19 +1,13 @@
 class Gillbus
   module Logout
-
     Gillbus.register self, :logout
 
     class Request < BaseRequest
-
       def path; '/online2/logout' end
-
     end
 
     class Response < BaseResponse
-
       field :logout, :bool
-
     end
-
   end
 end

data/lib/gillbus/parse_error.rb

@@ -7,6 +7,8 @@ class Gillbus::ParseError
   attr_accessor :session_id
   attr_accessor :request_time
   def error?; true; end
+
   def error_code; 0; end
-  def error_message; "Malformed response"; end
+
+  def error_message; 'Malformed response'; end
 end

data/lib/gillbus/reserve_tickets.rb

@@ -1,6 +1,5 @@
 class Gillbus
   module ReserveTickets
-
     Gillbus.register self, :reserve_tickets
 
     class Request < Gillbus::TicketsBooking::Request
@@ -8,6 +7,5 @@ class Gillbus
     end
 
     Response = Gillbus::TicketsBooking::Response
-
   end
 end

data/lib/gillbus/return_position.rb

@@ -3,7 +3,6 @@ class Gillbus
     Gillbus.register self, :return_position
 
     class Request < BaseRequest
-
       def path; '/online2/returnPosition' end
 
       # ticketCount

data/lib/gillbus/search_trips.rb

@@ -1,10 +1,8 @@
 class Gillbus
   module SearchTrips
-
     Gillbus.register self, :search_trips
 
     class Request < BaseRequest
-
       def path; '/online2/searchTrips' end
 
       # selectedModes (не обязательный)
@@ -13,11 +11,14 @@ class Gillbus
       attr_accessor :selected_modes
 
       # connectionIds (не обязательный)
-      # Список предполагаемых идентификаторов пересадочных пунктов, разделенных “;”. Используется если выбран вид транспорта 8, 9.
+      # Список предполагаемых идентификаторов пересадочных пунктов, разделенных “;”.
+      # Используется если выбран вид транспорта 8, 9.
       attr_accessor :connection_ids
 
       # tripFullSale (не обязательный)
-      # Признак определяющий количество возвращаемых рейсов. Если true, то возвращаются заказные и регулярные рейсы. Если false, то возвращаются только регулярные рейсы.
+      # Признак определяющий количество возвращаемых рейсов.
+      # Если true, то возвращаются заказные и регулярные рейсы.
+      # Если false, то возвращаются только регулярные рейсы.
       attr_accessor :trip_full_sale
 
       # fullSearch (не обязательный)
@@ -123,13 +124,11 @@ class Gillbus
           PassengerDiscount.wrap(p).params("passenger#{i}")
         end.reduce({}, :merge)
       end
-
     end
 
     class Response < BaseResponse
       field :completed, :bool
       field :trips, [Trip], key: 'TRIP'
     end
-
   end
 end

data/lib/gillbus/session_login.rb

@@ -1,10 +1,8 @@
 class Gillbus
   module SessionLogin
-
     Gillbus.register self, :session_login
 
     class Request < BaseRequest
-
       def initialize(password:, locale: nil)
         super
       end
@@ -19,14 +17,10 @@ class Gillbus
       def params
         compact(locale: translated_locale(locale))
       end
-
     end
 
     class Response < BaseResponse
-
       field :logged, :bool
-
     end
-
   end
 end

data/lib/gillbus/structs/commission.rb

@@ -1,7 +1,6 @@
-# FIXME it's possible I got currency for some fields wrong.
+# FIXME: it's possible I got currency for some fields wrong.
 class Gillbus
   class Commission
-
     extend Fields
     include UpdateAttrs
 
@@ -14,7 +13,9 @@ class Gillbus
     # Collecting description
     field :description
 
-    # type of commission according to tariff (1 - inside tariff, 2 - outside of tariff, 3 - inside of tariff type II) NB: only in ALL_COMMISSIONS
+    # type of commission according to tariff
+    # (1 - inside tariff, 2 - outside of tariff, 3 - inside of tariff type II)
+    # NB: only in ALL_COMMISSIONS
     field :in_out_type
 
     # Value of collecting in sale currency
@@ -59,6 +60,5 @@ class Gillbus
         Monetize.parse(val, doc['CURRENCY'])
       end
     end
-
   end
 end

data/lib/gillbus/structs/item.rb

@@ -1,6 +1,5 @@
 class Gillbus
   class Item
-
     extend Fields
     include UpdateAttrs
 
@@ -9,6 +8,5 @@ class Gillbus
     field :is_direct, :bool
     # Признак рейса с пересадкой
     field :is_transfer, :bool
-
   end
 end

data/lib/gillbus/structs/passenger_discount.rb

@@ -2,7 +2,6 @@ class Gillbus
   # not a request, neither a response part
   # just a params record for SearchTrips
   class PassengerDiscount < BaseRequest
-
     def self.wrap(passenger_or_params)
       return passenger_or_params if passenger_or_params.is_a? self
       new(passenger_or_params)
@@ -31,14 +30,14 @@ class Gillbus
     # с предоставлением места, если такой имеется.
     attr_accessor :with_seat
 
-    def params(prefix="")
+    def params(prefix = '')
       compact(
         birthday: date(birthday),
         studentTicket: student_ticket,
         studentYear: student_year,
         ISIC: isic,
         withSeat: bool(with_seat),
-      ).map {|k, v| [:"#{prefix}#{k}", v] }.to_h
+      ).map { |k, v| [:"#{prefix}#{k}", v] }.to_h
     end
   end
 end

data/lib/gillbus/structs/point.rb

@@ -1,6 +1,5 @@
 class Gillbus
   class Point
-
     extend Fields
     include UpdateAttrs
 

data/lib/gillbus/structs/return_cause.rb

@@ -1,6 +1,5 @@
 class Gillbus
   class ReturnCause
-
     extend Fields
     include UpdateAttrs
 
@@ -18,6 +17,5 @@ class Gillbus
         Monetize.parse(val, doc[:_currency])
       end
     end
-
   end
 end

data/lib/gillbus/structs/seat.rb

@@ -1,6 +1,5 @@
 class Gillbus
   class Seat
-
     extend Fields
     include UpdateAttrs
 
@@ -33,6 +32,5 @@ class Gillbus
     def isle?
       type == 0 || type.nil?
     end
-
   end
 end

data/lib/gillbus/structs/segment.rb

@@ -1,6 +1,5 @@
 class Gillbus
   class Segment
-
     extend Fields
     include UpdateAttrs
 
@@ -137,11 +136,11 @@ class Gillbus
     # => "643"
     field :currency_code
 
-    field :tariffs, [:tariff], key: "TARIFF"
+    field :tariffs, [:tariff], key: 'TARIFF'
 
-    field :points, [Point], key: "POINT"
+    field :points, [Point], key: 'POINT'
 
-    field :segments, [Segment], key: "SEGMENT"
+    field :segments, [Segment], key: 'SEGMENT'
 
     field :options, TripOptions, key: 'OPTIONS'
 
@@ -164,11 +163,11 @@ class Gillbus
     # to hold unserialized data
     attr_accessor :data
 
-    def self.parse(doc, _ = nil, _ = nil)
+    # rubocop:disable Lint/UnusedMethodArgument
+    def self.parse(doc, instance: nil, parent: nil, options: {})
       instance = super
       instance.data = doc
       instance
     end
-
   end
 end

data/lib/gillbus/structs/tariff.rb

@@ -1,6 +1,5 @@
 class Gillbus
   class Tariff
-
     extend Fields
     include UpdateAttrs
 
@@ -58,7 +57,7 @@ class Gillbus
 
       def string_with_possible_attributes(val)
         [val].flatten.map do |value|
-          if value.kind_of? Hash
+          if value.is_a? Hash
             value['__content__']
           else
             value

data/lib/gillbus/structs/tariff/return_cause.rb

@@ -1,7 +1,6 @@
 class Gillbus
   class Tariff
     class ReturnCause
-
       extend Fields
       include UpdateAttrs
 
@@ -9,9 +8,10 @@ class Gillbus
 
       field :cause, :string
 
-      def self.parse(doc, _ = nil, _ = nil)
+      # rubocop:disable Lint/UnusedMethodArgument
+      def self.parse(doc, instance: nil, parent: nil, options: {})
         instance = super
-        if doc.kind_of? Hash
+        if doc.is_a? Hash
           instance.cause = doc['__content__']
           instance.lossless = doc['lossless'] == 'true'
         else

data/lib/gillbus/structs/ticket.rb

@@ -1,6 +1,5 @@
 class Gillbus
   class Ticket
-
     extend Fields
     include UpdateAttrs
 
@@ -286,7 +285,6 @@ class Gillbus
       def return_cause(val)
         ReturnCause.parse(val.merge(_currency: doc['SALE_CUR_CODE']))
       end
-
     end
   end
 end

data/lib/gillbus/structs/timetable_trip.rb

@@ -42,11 +42,11 @@ class Gillbus
     # => "1"
     field :trip_mode, :int
 
-    field :segments, [TimetableSegment], key: "SEGMENT"
+    field :segments, [TimetableSegment], key: 'SEGMENT'
 
-    field :days_of_week, :days_of_week, key: "DAY_OR_DATE"
+    field :days_of_week, :days_of_week, key: 'DAY_OR_DATE'
 
-    field :dates, :dates, key: "DAY_OR_DATE"
+    field :dates, :dates, key: 'DAY_OR_DATE'
 
     parser do
       DAYS_INTO_WEEK = {
@@ -57,7 +57,7 @@ class Gillbus
         5 => :thursday,
         6 => :friday,
         7 => :saturday,
-      }
+      }.freeze
 
       def days_of_week(value)
         return unless instance.frequency == 3

data/lib/gillbus/structs/trip.rb

@@ -1,6 +1,5 @@
 class Gillbus
   class Trip
-
     extend Fields
     include UpdateAttrs
 
@@ -151,11 +150,11 @@ class Gillbus
     # => "643"
     field :currency_code
 
-    field :tariffs, [:tariff], key: "TARIFF"
+    field :tariffs, [:tariff], key: 'TARIFF'
 
-    field :points, [Point], key: "POINT"
+    field :points, [Point], key: 'POINT'
 
-    field :segments, [Segment], key: "SEGMENT"
+    field :segments, [Segment], key: 'SEGMENT'
 
     field :options, TripOptions, key: 'OPTIONS'
 
@@ -178,11 +177,11 @@ class Gillbus
     # to hold unserialized data
     attr_accessor :data
 
-    def self.parse(doc, _ = nil, _ = nil)
+    # rubocop:disable Lint/UnusedMethodArgument
+    def self.parse(doc, instance: nil, parent: nil, options: {})
       instance = super
       instance.data = doc
       instance
     end
-
   end
 end