icfs 0.1.3 → 0.2.0

data/lib/icfs/api.rb

@@ -9,6 +9,8 @@
 # This program is distributed WITHOUT ANY WARRANTY; without even the
 # implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 
+# frozen_string_literal: true
+
 #
 module ICFS
 
@@ -49,12 +51,14 @@ class Api
   # @param users [Users] the User/role/group interface
   # @param cache [Cache] the cache
   # @param store [Store] the store
+  # @param config [Config] the config settings, optional
   #
-  def initialize(stats, users, cache, store)
+  def initialize(stats, users, cache, store, config=nil)
     @users = users
     @cache = cache
     @store = store
     @gstats = stats.map{|st| st.dup.freeze }.freeze
+    @config = config
     reset
   end # def initialize
 
@@ -67,8 +71,8 @@ class Api
   def user=(uname)
     @user = uname.dup.freeze
     urgp = @users.read(uname)
-    raise(Error::NotFound, 'User name not found'.freeze) if !urgp
-    raise(Error::Value, 'Not a user'.freeze) if urgp['type'] != 'user'
+    raise(Error::NotFound, 'User name not found') if !urgp
+    raise(Error::Value, 'Not a user') if urgp['type'] != 'user'
     if urgp['roles']
       @roles = urgp['roles'].map{|rn| rn.freeze }.freeze
     else
@@ -96,6 +100,8 @@ class Api
     @ur.merge @roles
     @ur.freeze
 
+    @config.load(@user) if @config
+
     reset
   end # def user=()
 
@@ -143,6 +149,11 @@ class Api
   attr_reader :urg
 
 
+  ###############################################
+  # Config settings
+  attr_reader :config
+
+
   ###############################################
   # Reset the cached cases and access
   #
@@ -213,7 +224,7 @@ class Api
   ###############################################
   # See if we are tasked
   def tasked?(cid, anum)
-    id = '%s.%d'.freeze % [cid, anum]
+    id = '%s.%d' % [cid, anum]
     unless @tasked.key?(id)
       act = _action_read(cid, anum)
 
@@ -283,12 +294,12 @@ class Api
   def case_read(cid, lnum=0)
     if lnum != 0
       json = @store.case_read(cid, lnum)
-      return Items.parse(json, 'case'.freeze, Items::ItemCase)
+      return Items.parse(json, 'case', Items::ItemCase)
     end
 
     if !@cases.key?(cid)
       json = @cache.case_read(cid)
-      cur = Items.parse(json, 'case'.freeze, Items::ItemCase)
+      cur = Items.parse(json, 'case', Items::ItemCase)
       @cases[cid] = cur
     end
     return @cases[cid]
@@ -308,12 +319,12 @@ class Api
     # get access list
     al = access_list(cid)
     if !al.include?(ICFS::PermRead)
-      raise(Error::Perms, 'missing perms: %s'.freeze % ICFS::PermRead)
+      raise(Error::Perms, 'missing perms: %s' % ICFS::PermRead)
     end
 
     # read
     json = @cache.log_read(cid, lnum)
-    return Items.parse(json, 'log'.freeze, Items::ItemLog)
+    return Items.parse(json, 'log', Items::ItemLog)
   end # def log_read()
 
 
@@ -331,7 +342,7 @@ class Api
     # get access list and current entry
     al = access_list(cid)
     json = @cache.entry_read(cid, enum)
-    ec = Items.parse(json, 'entry'.freeze, Items::ItemEntry)
+    ec = Items.parse(json, 'entry', Items::ItemEntry)
 
     # see if we can read the entry
     need = Set.new
@@ -339,7 +350,7 @@ class Api
     need.merge(ec['perms']) if ec['perms']
     need.subtract(al)
     unless need.empty?
-      raise(Error::Perms, 'missing perms: %s'.freeze %
+      raise(Error::Perms, 'missing perms: %s' %
         need.to_a.sort.join(', ') )
     end
 
@@ -348,7 +359,7 @@ class Api
       return ec
     else
       json = @store.entry_read(cid, enum, lnum)
-      return Items.parse(json, 'entry'.freeze, Items::ItemEntry)
+      return Items.parse(json, 'entry', Items::ItemEntry)
     end
   end # def entry_read()
 
@@ -366,7 +377,7 @@ class Api
   def file_read(cid, enum, lnum, fnum)
     entry_read(cid, enum)
     fi = @store.file_read(cid, enum, lnum, fnum)
-    raise(Error::NotFound, 'file not found'.freeze) if !fi
+    raise(Error::NotFound, 'file not found') if !fi
     return fi
   end # def file_read()
 
@@ -377,10 +388,10 @@ class Api
   # Internal version.
   #
   def _action_read(cid, anum)
-    id = '%s.%d'.freeze % [cid, anum]
+    id = '%s.%d' % [cid, anum]
     unless @actions.key?(id)
       json = @cache.action_read(cid, anum)
-      act = Items.parse(json, 'action'.freeze, Items::ItemAction)
+      act = Items.parse(json, 'action', Items::ItemAction)
       @actions[id] = act
     end
     return @actions[id]
@@ -405,7 +416,7 @@ class Api
 
     # see if we can read the action
     unless _can_read?(cid, anum)
-      raise(Error::Perms, 'missing perms: %s'.freeze % ICFS::PermRead )
+      raise(Error::Perms, 'missing perms: %s' % ICFS::PermRead )
     end
 
     # return the requested version
@@ -413,7 +424,7 @@ class Api
       return ac
     else
       json = @store.action_read( cid, anum, lnum)
-      return Items.parse(json, 'action'.freeze, Items::ItemAction)
+      return Items.parse(json, 'action', Items::ItemAction)
     end
   end # def action_read()
 
@@ -432,19 +443,19 @@ class Api
     # get access list
     al = access_list(cid)
     if !al.include?(ICFS::PermRead)
-      raise(Error::Perms, 'missing perms: %s'.freeze % ICFS::PermRead )
+      raise(Error::Perms, 'missing perms: %s' % ICFS::PermRead )
     end
 
     # read curent index
     json = @cache.index_read(cid, xnum)
-    xc = Items.parse(json, 'index'.freeze, Items::ItemIndex)
+    xc = Items.parse(json, 'index', Items::ItemIndex)
 
     # return the requested version
     if( lnum == 0 || xc['log'] == lnum )
       return xc
     else
       json = @store.index_read(cid, xnum, lnum)
-      return Items.parse(json, 'index'.freeze, Items::ItemIndex)
+      return Items.parse(json, 'index', Items::ItemIndex)
     end
   end # def index_read()
 
@@ -458,11 +469,11 @@ class Api
 
     al = access_list(cid)
     if !al.include?(ICFS::PermRead)
-      raise(Error::Perms, 'missing perms: %s'.freeze % ICFS::PermRead)
+      raise(Error::Perms, 'missing perms: %s' % ICFS::PermRead)
     end
 
     json = @cache.current_read(cid)
-    return Items.parse(json, 'current'.freeze, Items::ItemCurrent)
+    return Items.parse(json, 'current', Items::ItemCurrent)
   end # end def current_read()
 
 
@@ -494,7 +505,7 @@ class Api
   # @param query [Hash] a query
   #
   def case_search(query)
-    Items.validate(query, 'Case Search'.freeze, ValCaseSearch)
+    Items.validate(query, 'Case Search', ValCaseSearch)
     @cache.case_search(query)
   end
 
@@ -516,8 +527,8 @@ class Api
       sort: {
         method: :string,
         allowed: Set[
-          'time_desc'.freeze,
-          'time_asc'.freeze,
+          'time_desc',
+          'time_asc',
         ].freeze,
         whitelist: true,
       }.freeze
@@ -531,7 +542,7 @@ class Api
   # @param query [Hash] a query
   #
   def log_search(query)
-    Items.validate(query, 'Log Search'.freeze, ValLogSearch)
+    Items.validate(query, 'Log Search', ValLogSearch)
     @cache.log_search(query)
   end
 
@@ -556,8 +567,8 @@ class Api
       sort: {
         method: :string,
         allowed: Set[
-          'time_desc'.freeze,
-          'time_asc'.freeze,
+          'time_desc',
+          'time_asc',
         ].freeze,
         whitelist: true,
       }.freeze
@@ -570,14 +581,14 @@ class Api
   # Search for entries
   #
   def entry_search(query)
-    Items.validate(query, 'Entry Search'.freeze, ValEntrySearch)
+    Items.validate(query, 'Entry Search', ValEntrySearch)
 
     # check permissions
     # - have global search permissions / read access to the case
     # - are searching for an action they can read
     unless( _search?(query) || (query[:caseid] &&
         query[:action] && tasked?(query[:caseid], query[:action])))
-      raise(Error::Perms, 'Does not have permission to search'.freeze)
+      raise(Error::Perms, 'Does not have permission to search')
     end
 
     # run the query
@@ -645,8 +656,8 @@ class Api
       sort: {
         method: :string,
         allowed: Set[
-          'time_desc'.freeze,
-          'time_asc'.freeze,
+          'time_desc',
+          'time_asc',
         ].freeze,
         whitelist: true,
       }.freeze
@@ -658,7 +669,7 @@ class Api
   # Search for actions
   #
   def action_search(query)
-    Items.validate(query, 'Action Search'.freeze, ValActionSearch)
+    Items.validate(query, 'Action Search', ValActionSearch)
 
     # permissions check
     # - have global search permissions / read access to the case
@@ -666,7 +677,7 @@ class Api
     unless( _search?(query) || @ur.include?(query[:assigned]) ||
        (query[:assigned] == ICFS::UserCase && query[:caseid] &&
         access_list(query[:caseid]).include?(ICFS::PermAction) ))
-      raise(Error::Perms, 'Does not have permission to search'.freeze)
+      raise(Error::Perms, 'Does not have permission to search')
     end
 
     # run the search
@@ -689,10 +700,10 @@ class Api
       sort: {
         method: :string,
         allowed: Set[
-          'title_desc'.freeze,
-          'title_asc'.freeze,
-          'index_desc'.freeze,
-          'index_asc'.freeze,
+          'title_desc',
+          'title_asc',
+          'index_desc',
+          'index_asc',
         ].freeze,
         whitelist: true,
       }.freeze
@@ -704,12 +715,12 @@ class Api
   # Search for indexes
   #
   def index_search(query)
-    Items.validate(query, 'Index Search'.freeze, ValIndexSearch)
+    Items.validate(query, 'Index Search', ValIndexSearch)
 
     # permissions check
     # - have global search permissions / read access to the case
     unless _search?(query)
-      raise(Error::Perms, 'Do not have permission to search'.freeze)
+      raise(Error::Perms, 'Do not have permission to search')
     end
 
     # run the query
@@ -746,13 +757,13 @@ class Api
   # Analyze stats
   #
   def stats(query)
-    Items.validate(query, 'Stats Search'.freeze, ValStatsSearch)
+    Items.validate(query, 'Stats Search', ValStatsSearch)
 
     # permissions check
     # - have global search permissions / read access to the case
     # - are searching for a user/role/group you have
     unless _search?(query) || (query[:credit] && @urg.include?(query[:credit]))
-      raise(Error::Perms, 'Do not have permissions to search'.freeze)
+      raise(Error::Perms, 'Do not have permissions to search')
     end
 
     @cache.stats(query)
@@ -775,7 +786,7 @@ class Api
   # Get case tags
   #
   def case_tags(query)
-    Items.validate(query, 'Case Tags Search'.freeze, ValCaseTags)
+    Items.validate(query, 'Case Tags Search', ValCaseTags)
     return @cache.case_tags(query)
   end # def case_tags()
 
@@ -796,12 +807,12 @@ class Api
   # Get entry tags
   #
   def entry_tags(query)
-    Items.validate(query, 'Entry Tags Search'.freeze, ValEntryTags)
+    Items.validate(query, 'Entry Tags Search', ValEntryTags)
 
     # permissions
     # - read access to case
     unless access_list(query[:caseid]).include?(ICFS::PermRead)
-      raise(Error::Perms, 'missing perms: %s'.freeze % ICFS::PermRead)
+      raise(Error::Perms, 'missing perms: %s' % ICFS::PermRead)
     end
     return @cache.entry_tags(query)
   end # def entry_tags()
@@ -837,13 +848,13 @@ class Api
   # Get action tags
   #
   def action_tags(query)
-    Items.validate(query, 'Task Tags Search'.freeze, ValActionTags)
+    Items.validate(query, 'Task Tags Search', ValActionTags)
 
     # only allow searches for user/roles you have
     unless @ur.include?(query[:assigned]) ||
        (query[:assigned] == ICFS::UserCase && query[:caseid] &&
           access_list(query[:caseid]).include?(ICFS::PermAction) )
-      raise(Error::Perms, 'May not search for other\'s tasks'.freeze)
+      raise(Error::Perms, 'May not search for other\'s tasks')
     end
 
     # run the search
@@ -867,9 +878,9 @@ class Api
   # Get index tags
   #
   def index_tags(query)
-    Items.validate(query, 'Index Tags'.freeze, ValIndexTags)
+    Items.validate(query, 'Index Tags', ValIndexTags)
     unless access_list(query[:caseid]).include?(ICFS::PermRead)
-      raise(Error::Perms, 'missing perms: %s'.freeze % ICFS::PermRead)
+      raise(Error::Perms, 'missing perms: %s' % ICFS::PermRead)
     end
     return @cache.index_tags(query)
   end
@@ -894,8 +905,8 @@ class Api
     # Sanity checks
 
     # form & values
-    Items.validate(ent, 'entry'.freeze, Items::ItemEntryNew)
-    Items.validate(cse, 'case'.freeze, Items::ItemCaseEdit)
+    Items.validate(ent, 'entry', Items::ItemEntryNew)
+    Items.validate(cse, 'case', Items::ItemCaseEdit)
 
     # access users/roles/groups are valid, unless manually specifying user
     unless unam
@@ -903,7 +914,7 @@ class Api
         acc["grant"].each do |gnt|
           urg = @users.read(gnt)
           if !urg
-            raise(Error::NotFound, 'User/role/group %s not found'.freeze % urg)
+            raise(Error::NotFound, 'User/role/group %s not found' % urg)
           end
         end
       end
@@ -917,26 +928,26 @@ class Api
     if tid
       tmpl = case_read(tid)
       unless tmpl['template']
-        raise(Error::Perms, 'Not a template'.freeze)
+        raise(Error::Perms, 'Not a template')
       end
 
       al = access_list(tid)
       unless al.include?(ICFS::PermManage)
-        raise(Error::Perms, 'May not create cases from this template'.freeze)
+        raise(Error::Perms, 'May not create cases from this template')
       end
     end
 
     # no action/indexes
     if ent['action']
-      raise(Error::Value, 'No Action for a new case entry'.freeze)
+      raise(Error::Value, 'No Action for a new case entry')
     end
     if ent['index']
-      raise(Error::Value, 'No Index for a new case entry'.freeze)
+      raise(Error::Value, 'No Index for a new case entry')
     end
 
     # Allow case creation without a Users system in place
     user = @user ? @user : unam
-    raise(ArgumentError, 'No user specified'.freeze) if user.nil?
+    raise(ArgumentError, 'No user specified') if user.nil?
 
     ####################
     # Prep
@@ -947,7 +958,7 @@ class Api
     cse['caseid'] = cid
     cse['log'] = 1
     cse['tags'] ||= [ ICFS::TagNone ]
-    citem = Items.generate(cse, 'case'.freeze, Items::ItemCase)
+    citem = Items.generate(cse, 'case', Items::ItemCase)
 
     # entry
     ent['icfs'] = 1
@@ -989,7 +1000,7 @@ class Api
     @cache.lock_take(cid)
     begin
       if @cache.case_read(cid)
-        raise(Error::Conflict, 'Case already exists'.freeze)
+        raise(Error::Conflict, 'Case already exists')
       end
 
       now = Time.now.to_i
@@ -997,12 +1008,12 @@ class Api
       # finish items
       ent['time'] ||= now
       ent['files'].each{|fi| fi['log'] ||= 1 } if ent['files']
-      eitem = Items.generate(ent, 'entry'.freeze, Items::ItemEntry)
+      eitem = Items.generate(ent, 'entry', Items::ItemEntry)
       log['time'] = now
       log['entry']['hash'] = ICFS.hash(eitem)
-      litem = Items.generate(log, 'log'.freeze, Items::ItemLog)
+      litem = Items.generate(log, 'log', Items::ItemLog)
       cur['hash'] = ICFS.hash(litem)
-      nitem = Items.generate(cur, 'current'.freeze, Items::ItemCurrent)
+      nitem = Items.generate(cur, 'current', Items::ItemCurrent)
 
       # write to cache
       @cache.entry_write(cid, 1, eitem)
@@ -1041,22 +1052,22 @@ class Api
 
     # form & content
     if idx || cse
-      Items.validate(ent, 'New Entry'.freeze, Items::ItemEntryNew)
+      Items.validate(ent, 'New Entry', Items::ItemEntryNew)
     else
-      Items.validate(ent, 'Editable Entry'.freeze, Items::ItemEntryEdit)
+      Items.validate(ent, 'Editable Entry', Items::ItemEntryEdit)
     end
-    Items.validate(act, 'action'.freeze, Items::ItemActionEdit) if act
-    Items.validate(idx, 'index'.freeze, Items::ItemIndexEdit) if idx
-    Items.validate(cse, 'case'.freeze, Items::ItemCaseEdit) if cse
+    Items.validate(act, 'action', Items::ItemActionEdit) if act
+    Items.validate(idx, 'index', Items::ItemIndexEdit) if idx
+    Items.validate(cse, 'case', Items::ItemCaseEdit) if cse
 
     # edit index OR case, not both
     if idx && cse
-      raise(Error::Value, 'May not edit both case and index at once'.freeze)
+      raise(Error::Value, 'May not edit both case and index at once')
     end
 
     # no changing the action
     if act && ent['action'] && act['action'] && act['action'] != ent['action']
-      raise(Error::Conflict, 'May not change entry\'s action'.freeze)
+      raise(Error::Conflict, 'May not change entry\'s action')
     end
 
     # access users/roles/groups are valid
@@ -1065,7 +1076,7 @@ class Api
         acc['grant'].each do |gnt|
           urg = @users.read(gnt)
           if !urg
-            raise(Error::NotFound, 'User/role/group %s not found'.freeze % gnt)
+            raise(Error::NotFound, 'User/role/group %s not found' % gnt)
           end
         end
       end
@@ -1078,12 +1089,12 @@ class Api
         tsk = act['tasks'][ix]
         ur = @users.read(tsk['assigned'])
         if !ur
-          raise(Error::NotFound, 'User/role %s not found'.freeze %
+          raise(Error::NotFound, 'User/role %s not found' %
              tsk['assigned'])
         end
         type = ur['type']
         if type != 'user' && type != 'role'
-          raise(Error::Values, 'Not a user or role: %s'.freeze %
+          raise(Error::Values, 'Not a user or role: %s' %
              tsk['assigned'])
         end
       end
@@ -1157,13 +1168,13 @@ class Api
 
       # current
       json = @cache.current_read(cid)
-      cur = Items.parse(json, 'current'.freeze, Items::ItemCurrent)
+      cur = Items.parse(json, 'current', Items::ItemCurrent)
 
       # entry
       if ent['entry']
         enum = ent['entry']
         json = @cache.entry_read(cid, enum)
-        ent_pri = Items.parse(json, 'entry'.freeze, Items::ItemEntry)
+        ent_pri = Items.parse(json, 'entry', Items::ItemEntry)
         nxt['entry'] = cur['entry']
       else
         enum = cur['entry'] + 1
@@ -1180,7 +1191,7 @@ class Api
       end
       if anum
         json = @cache.action_read(cid, anum)
-        act_pri = Items.parse(json, 'action'.freeze, Items::ItemAction)
+        act_pri = Items.parse(json, 'action', Items::ItemAction)
         nxt['action'] = cur['action']
       elsif act
         anum = cur['action'] + 1
@@ -1225,7 +1236,7 @@ class Api
 
         # may not change action
         if ent_pri['action'] && (ent['action'] != ent_pri['action'])
-          raise(Error::Conflict, 'May not change entry\'s action'.freeze)
+          raise(Error::Conflict, 'May not change entry\'s action')
         end
 
         # may not remove or add action, index, case tags
@@ -1235,7 +1246,7 @@ class Api
                  ent['tags'].include?(ICFS::TagIndex) ) ||
             (ent_pri['tags'].include?(ICFS::TagCase) !=
                  ent['tags'].include?(ICFS::TagCase) ) )
-          raise(Error::Conflict, 'May not change entry\'s special tags'.freeze)
+          raise(Error::Conflict, 'May not change entry\'s special tags')
         end
       end
 
@@ -1247,7 +1258,7 @@ class Api
 
         # not allowed to delete tasks
         if pri_tsk.size > cur_tsk.size
-          raise(Error::Conflict, 'May not delete tasks'.freeze)
+          raise(Error::Conflict, 'May not delete tasks')
         end
 
         # check each task
@@ -1259,17 +1270,17 @@ class Api
 
           # may not delete a tasking
           if pt && pt['assigned'] != ct['assigned']
-            raise(Error::Conflict, 'May not delete task'.freeze)
+            raise(Error::Conflict, 'May not delete task')
           end
 
           # new taskings require action to be open
           if !pt && !act_open
-            raise(Error::Value, 'New tasks require the action be open'.freeze)
+            raise(Error::Value, 'New tasks require the action be open')
           end
 
           # may not have a task open if action is closed
           if ct['status'] && !act_open
-            raise(Error::Value, 'Open tasks on closed action'.freeze)
+            raise(Error::Value, 'Open tasks on closed action')
           end
 
           # can set any values for our tasks
@@ -1280,7 +1291,7 @@ class Api
 
           # must be flagged if new tasking or re-opening
           if !ct['flag'] && (!pt || (ct['status'] && !pt['status']))
-            raise(Error::Value, 'New or re-opened taskings must flag'.freeze)
+            raise(Error::Value, 'New or re-opened taskings must flag')
           end
 
           # no changing other's taskings, no deflagging, and no
@@ -1289,7 +1300,7 @@ class Api
              (pt['title'] != ct['title']) || (pt['time'] != ct['time']) ||
              (pt['tags'] != ct['tags']) || (pt['flag'] && !ct['flag']) ||
              (pt['status'] && !ct['status'] && !perm_act) )
-            raise(Error::Value, 'May not change other\'s tasks'.freeze)
+            raise(Error::Value, 'May not change other\'s tasks')
           end
         end
 
@@ -1306,7 +1317,7 @@ class Api
       if cse
         # no changing template
         unless cse['template'] == cse_pri['template']
-          raise(Error::Conflict, 'May not change template status'.freeze)
+          raise(Error::Conflict, 'May not change template status')
         end
 
         # manage required
@@ -1321,7 +1332,7 @@ class Api
       # permissions
       perms_miss = perms - al
       unless perms_miss.empty?
-        raise(Error::Perms, 'Missing perms: %s'.freeze %
+        raise(Error::Perms, 'Missing perms: %s' %
           perms_miss.to_a.sort.join(', ') )
       end
 
@@ -1342,7 +1353,7 @@ class Api
         end
       end
       ent['files'].each{|fi| fi['log'] ||= lnum } if ent['files']
-      eitem = Items.generate(ent, 'entry'.freeze, Items::ItemEntry)
+      eitem = Items.generate(ent, 'entry', Items::ItemEntry)
       log['entry'] = {
         'num' => enum,
         'hash' => ICFS.hash(eitem)
@@ -1352,7 +1363,7 @@ class Api
       if act
         act['action'] = anum
         act['log'] = lnum
-        aitem = Items.generate(act, 'action'.freeze, Items::ItemAction)
+        aitem = Items.generate(act, 'action', Items::ItemAction)
         log['action'] = {
           'num' => anum,
           'hash' => ICFS.hash(aitem)
@@ -1363,7 +1374,7 @@ class Api
       if idx
         idx['index'] = xnum
         idx['log'] = lnum
-        xitem = Items.generate(idx, 'index'.freeze, Items::ItemIndex)
+        xitem = Items.generate(idx, 'index', Items::ItemIndex)
         log['index'] = {
           'num' => xnum,
           'hash' => ICFS.hash(xitem)
@@ -1373,7 +1384,7 @@ class Api
       # case
       if cse
         cse['log'] = lnum
-        citem = Items.generate(cse, 'case'.freeze, Items::ItemCase)
+        citem = Items.generate(cse, 'case', Items::ItemCase)
         log['case_hash'] = ICFS.hash(citem)
       end
 
@@ -1381,11 +1392,11 @@ class Api
       log['log'] = lnum
       log['prev'] = cur['hash']
       log['time'] = now
-      litem = Items.generate(log, 'log'.freeze, Items::ItemLog)
+      litem = Items.generate(log, 'log', Items::ItemLog)
       nxt['hash'] = ICFS.hash(litem)
 
       # next
-      nitem = Items.generate(nxt, 'current'.freeze, Items::ItemCurrent)
+      nitem = Items.generate(nxt, 'current', Items::ItemCurrent)
 
 
       ####################

data/lib/icfs/cache.rb

@@ -9,6 +9,8 @@
 # This program is distributed WITHOUT ANY WARRANTY; without even the
 # implied warranty of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 
+# frozen_string_literal: true
+
 #
 module ICFS