From ceb83c70bae495403545157cdfcc43e2bc6572c1 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Sun, 13 Aug 2006 09:11:45 +0000
Subject: [PATCH] Changes to string format for capabilities

---
 admin/roles/manage.html |  2 +-
 lib/accesslib.php       | 33 +++++++++++++++++----------------
 2 files changed, 18 insertions(+), 17 deletions(-)

diff --git a/admin/roles/manage.html b/admin/roles/manage.html
index 3094315da9..c07d65e4ef 100755
--- a/admin/roles/manage.html
+++ b/admin/roles/manage.html
@@ -49,7 +49,7 @@
             ?>
                 
             <tr>
-              <td><?php echo get_capability_string($capability->name); ?></td>
+              <td><span title="<?php echo $capability->name ?>"><?php echo get_capability_string($capability->name); ?></span></td>
                  <td><INPUT TYPE=radio NAME="<?php echo $capability->name; ?>" VALUE="0" <?php if (!isset($localoverride->permission) || $localoverride->permission==0){ echo 'checked="checked"'; }?>></td>
                  <td><INPUT TYPE=radio NAME="<?php echo $capability->name; ?>" VALUE="1" <?php if (isset($localoverride->permission) && $localoverride->permission==1){ echo 'checked="checked"'; }?>></td>
                  <td ><INPUT TYPE=radio NAME="<?php echo $capability->name; ?>" VALUE="-1" <?php if (isset($localoverride->permission) && $localoverride->permission==-1){ echo 'checked="checked"'; }?>></td>
diff --git a/lib/accesslib.php b/lib/accesslib.php
index 151ef3a78c..66cf57bdda 100755
--- a/lib/accesslib.php
+++ b/lib/accesslib.php
@@ -1429,38 +1429,39 @@ function get_role_context_capability($contextid, $capability, $capabilities) {
 
 
 // a big switch statement
-function get_capability_string($capname) {
+function get_capability_string($capabilityname) {
 
-    $names = split('/', $capname);
-    $componentname = split(':', $names[1]);
-    $componentname = $componentname[0];
-    $capability = split(':', $capname);
-    $capability = 'capability_'.$capability[1];
+    // Typical capabilityname is:   mod/choice:readresponses
+
+    $names = split('/', $capabilityname);
+    $stringname = $names[1];                 // choice:readresponses
+    $components = split(':', $stringname);   
+    $componentname = $components[0];               // choice
 
     switch ($names[0]) {
         case 'mod':
-            $string = get_string($capability, $componentname);
+            $string = get_string($stringname, $componentname);
         break;
         
         case 'block':
-            $string = get_string($capability, 'block_'.$componentname);
+            $string = get_string($stringname, 'block_'.$componentname);
         break;
-        
+
         case 'moodle':
-            $string = get_string($capability);
+            $string = get_string($stringname, 'role');
         break;
         
         case 'enrol':
-            $string = get_string($capability, 'enrol_'.$componentname);
+            $string = get_string($stringname, 'enrol_'.$componentname);
+        break;  
         
         default:
-            $string = get_string($capability);
+            $string = get_string($stringname);
         break;  
-            
       
     }
 
-      return $string;
+    return $string;
 }
 
 
@@ -1470,7 +1471,7 @@ function get_component_string($component, $contextlevel) {
     switch ($contextlevel) {
 
         case CONTEXT_SYSTEM:
-            $string = get_string('system');
+            $string = get_string('coresystem');
         break;
 
         case CONTEXT_PERSONAL:
@@ -1510,4 +1511,4 @@ function get_component_string($component, $contextlevel) {
     return $string;
   
 }
-?>
\ No newline at end of file
+?>
-- 
2.39.5