Roxen.git/ server/ etc/ modules/ Variable.pmod/ module.pmod

2008-12-19

2008-12-19 12:17:39 by Henrik Grubbström (Grubba) <grubba@grubba.org>

• 86ae1e74ae6f697b1e6756d2899281ec3bebd60c (27 lines) (+19/-8) ^{[ Show | Annotate ]}
  Branch: 4.5

Improved sorting of ModuleChoice and ProviderChoice choices.

Rev: server/etc/modules/Variable.pmod/module.pmod:1.104

1: - // $Id: module.pmod,v 1.103 2008/12/18 15:05:05 grubba Exp $ + // $Id: module.pmod,v 1.104 2008/12/19 12:17:39 grubba Exp $      #include <module.h>   #include <roxen.h>
1453:    return res;    }    +  //! Fetch the unsorted list of choices. +  protected array low_get_choice_list() +  { +  return indices(conf->otomod); +  } +     array get_choice_list()    { -  array res = indices(conf->otomod); -  // FIXME: Sort on priority as well? +  array res = low_get_choice_list(); +  // Sort order: +  // priority: descending, +  // title: ascending, +  // module_id: ascending. +  sort(res->module_local_id(), res);    sort(map(res, _title), res); -  +  res = reverse(res); +  sort(res->query("_priority"), res); +  res = reverse(res);    return res;    }   
1517:    constant type = "ProviderChoice";    protected string provides;    -  array get_choice_list() +  protected array low_get_choice_list()    { -  array res = conf->get_providers(provides); -  // FIXME: Sort on priority as well? -  sort(map(res, _title), res); -  return res; +  return conf->get_providers(provides);    }       //! @param default_id