From d847bb32fddfa2af9d34b74d1fb05001cba48286 Mon Sep 17 00:00:00 2001
From: skodak <skodak>
Date: Sun, 17 Sep 2006 17:32:45 +0000
Subject: [PATCH] fixed non-working edit button on course page for teachers,
 added explanation comments

---
 lib/pagelib.php | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/lib/pagelib.php b/lib/pagelib.php
index 49950ade3c..e116c8fb09 100644
--- a/lib/pagelib.php
+++ b/lib/pagelib.php
@@ -343,19 +343,20 @@ class page_course extends page_base {
 
     // USER-RELATED THINGS
 
-    // When is a user said to have "editing rights" in this page? This would have something
-    // to do with roles, in the future.
+    // Can user edit the course page or "sticky page"?
+    // This is also about editting of blocks BUT mainly activities in course page layout, see
+    // update_course_icon() - it must use the same capability
     function user_allowed_editing() {
         if (has_capability('moodle/site:manageblocks', get_context_instance(CONTEXT_SYSTEM, SITEID)) && defined('ADMIN_STICKYBLOCKS')) {
             return true;
         }
-        return has_capability('moodle/site:manageblocks', get_context_instance(CONTEXT_COURSE, $this->id));
+        return has_capability('moodle/course:manageactivities', get_context_instance(CONTEXT_COURSE, $this->id));
     }
 
-    // Is the user actually editing this page right now? This would have something
-    // to do with roles, in the future.
+    // Is the user actually editing this course page or "sticky page" right now?
     function user_is_editing() {
         if (has_capability('moodle/site:manageblocks', get_context_instance(CONTEXT_SYSTEM, SITEID)) && defined('ADMIN_STICKYBLOCKS')) {
+            //always in edit mode on sticky page
             return true;
         }
         return isediting($this->id);
-- 
2.39.5