Bug in lib/whimsy/asf/icla.rb?

[sebb]

Just noticed the following code:

    # list of all ids
    def self.availids
      return [] unless SOURCE
      refresh
      return @@availids if @@availids
      availids = []
      each {|icla| availids << icla.id unless icla.id == 'notinavail'}
      @availids = availids
    end

AFAICT that will never set up @@availids, or have I misunderstood?
Surely the last line of the function should be:

      @@availids = availids