```ruby
module MetasploitDataModels::ActiveRecordModels::Session
  def self.included(base)
    base.class_eval {
      belongs_to :host, :class_name => "Mdm::Host"

      has_one :workspace, :through => :host, :class_name => "Mdm::Workspace"

      has_many :events, :class_name => "Mdm::SessionEvent", :order => "created_at", :dependent => :delete_all
      has_many :routes, :class_name => "Mdm::Route", :dependent => :delete_all

      scope :alive, where("closed_at IS NULL")
      scope :dead, where("closed_at IS NOT NULL")
      scope :upgradeable, where("closed_at IS NULL AND stype = 'shell' and platform ILIKE '%win%'")

      serialize :datastore, ::MetasploitDataModels::Base64Serializer.new

      before_destroy :stop
      
      def upgradeable?
         return true if (self.stype == 'shell' and self.platform =~ /win/)
         else return false
       end
      

      private

      def stop
				c = Pro::Client.get rescue nil 
        c.session_stop(self.local_id) rescue nil # ignore exceptions (XXX - ideally, stopped an already-stopped session wouldn't throw XMLRPCException)
      end

    }
  end
end
```

Fix to the upgradeable? method to be all around better
user
```ruby
module MetasploitDataModels::ActiveRecordModels::Session
  def self.included(base)
    base.class_eval {
      belongs_to :host, :class_name => "Mdm::Host"

      has_one :workspace, :through => :host, :class_name => "Mdm::Workspace"

      has_many :events, :class_name => "Mdm::SessionEvent", :order => "created_at", :dependent => :delete_all
      has_many :routes, :class_name => "Mdm::Route", :dependent => :delete_all

      scope :alive, where("closed_at IS NULL")
      scope :dead, where("closed_at IS NOT NULL")
      scope :upgradeable, where("closed_at IS NULL AND stype = 'shell' and platform ILIKE '%win%'")

      serialize :datastore, ::MetasploitDataModels::Base64Serializer.new

      before_destroy :stop
      
      def upgradeable?
        (self.platform =~ /win/ and self.stype == 'shell')
      end
      

      private

      def stop
				c = Pro::Client.get rescue nil 
        c.session_stop(self.local_id) rescue nil # ignore exceptions (XXX - ideally, stopped an already-stopped session wouldn't throw XMLRPCException)
      end

    }
  end
end
```
assistant