From ca319818fc7be7a011895c17cc0d7a77f7d2e41a Mon Sep 17 00:00:00 2001
From: defacer <defacer>
Date: Tue, 20 Apr 2004 07:43:26 +0000
Subject: [PATCH] Don't display blocks that have no content at all. I 'm not
 sure if this could cause any confusion, but it solves the problem with the
 admin block being displayed to guests.

The thought about a "display only to X and Y kinds of users" feature came
to mind, but maybe that would be more confusing than useful.
---
 blocks/moodleblock.class.php | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/blocks/moodleblock.class.php b/blocks/moodleblock.class.php
index 1e36b53385..b515b8d8fb 100644
--- a/blocks/moodleblock.class.php
+++ b/blocks/moodleblock.class.php
@@ -49,15 +49,20 @@ class MoodleBlock {
         return $this->get_content();
     }
     function print_block() {
-        // Wrap it up, in case we have buttons too
+        // Wrap the title in a floating DIV, in case we have edit controls to display
+        // These controls will always be wrapped on a right-floating DIV
         $title = '<div style="float: left;">'.$this->title.'</div>';
         if($this->edit_controls !== NULL) {
             $title .= $this->edit_controls;
         }
+
         $this->get_content();
 
         switch($this->content_type) {
             case BLOCK_TYPE_TEXT:
+                if(empty($this->content->text)) {
+                    break;
+                }
                 if ($this->edit_controls !== NULL || !$this->hide_header()) {
                     print_side_block($title, $this->content->text, NULL, NULL, $this->content->footer);
                 } else {
@@ -65,6 +70,9 @@ class MoodleBlock {
                 }
             break;
             case BLOCK_TYPE_LIST:
+                if(empty($this->content->items)) {
+                    break;
+                }
                 if ($this->edit_controls !== NULL || !$this->hide_header()) {
                     print_side_block($title, '', $this->content->items, $this->content->icons, $this->content->footer);
                 } else {
-- 
2.39.5