MDL-13255 - hidden activity capability check was wrong, but require_login()

handles it for us when passsed $cm

diff --git a/mod/data/view.php b/mod/data/view.php
index 5b9d1a0ea14ffdc4ee69b8bef0afc128fe5128e3..b12ca265ad08a524582145781f13562c8193c140 100755 (executable)
--- a/mod/data/view.php
+++ b/mod/data/view.php
@@ -85,14 +85,6 @@
     $context = get_context_instance(CONTEXT_MODULE, $cm->id);
     require_capability('mod/data:viewentry', $context);
 
-/// If it's hidden then it's don't show anything.  :)
-    if (empty($cm->visible) and !has_capability('mod/data:managetemplates', $context)) {
-        $navigation = build_navigation('', $cm);
-        print_header_simple(format_string($data->name), "",
-                 $navigation, "", "", true, '', navmenu($course, $cm));
-        notice(get_string("activityiscurrentlyhidden"));
-    }
-
 /// If we have an empty Database then redirect because this page is useless without data
     if (has_capability('mod/data:managetemplates', $context)) {
         if (!record_exists('data_fields','dataid',$data->id)) {      // Brand new database!