From fdb0322d72689556bc3ad57e5e54e5ac03429fe8 Mon Sep 17 00:00:00 2001 From: Davo Smith Date: Tue, 7 Feb 2012 20:56:04 +0000 Subject: [PATCH] MDL-31541 Repositories - make sure all repository instances have unique 'sortorder' values when retrieved --- repository/lib.php | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/repository/lib.php b/repository/lib.php index 8652aa9b9c9..eae607275dc 100644 --- a/repository/lib.php +++ b/repository/lib.php @@ -818,6 +818,9 @@ abstract class repository { } else { $accepted_types = '*'; } + // Sortorder should be unique, which is not true if we use $record->sortorder + // and there are multiple instances of any repository type + $sortorder = 1; foreach ($records as $record) { if (!file_exists($CFG->dirroot . '/repository/'. $record->repositorytype.'/lib.php')) { continue; @@ -826,7 +829,7 @@ abstract class repository { $options['visible'] = $record->visible; $options['type'] = $record->repositorytype; $options['typeid'] = $record->typeid; - $options['sortorder'] = $record->sortorder; + $options['sortorder'] = $sortorder++; // tell instance what file types will be accepted by file picker $classname = 'repository_' . $record->repositorytype; -- 2.43.0