From 6979c65b9e77dc2b8f17769b38bdebace679a474 Mon Sep 17 00:00:00 2001
From: "Michael J. Rubinsky" <mrubinsk@horde.org>
Date: Thu, 15 Jul 2010 12:17:25 -0400
Subject: [PATCH] Don't allow fixed block types to be changed.

Bug: 9141
---
 framework/Block/lib/Horde/Block/Collection.php | 7 ++++++-
 horde/templates/portal/layout.inc              | 2 +-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/framework/Block/lib/Horde/Block/Collection.php b/framework/Block/lib/Horde/Block/Collection.php
index 145d8b70b..4568f570e 100644
--- a/framework/Block/lib/Horde/Block/Collection.php
+++ b/framework/Block/lib/Horde/Block/Collection.php
@@ -217,16 +217,21 @@ class Horde_Block_Collection
      *
      * @param string $cur_app    The block from this application gets selected.
      * @param string $cur_block  The block with this name gets selected.
+     * @param boolean $onchange  Include the onchange action
+     * @param boolean $readonly  Indicates if this block type is changeable.
      *
      * @return string  The select tag with all available blocks.
      */
     public function getBlocksWidget($cur_app = null, $cur_block = null,
-                                    $onchange = false)
+                                    $onchange = false, $readonly = false)
     {
         $widget = '<select name="app"';
         if ($onchange) {
             $widget .= ' onchange="document.blockform.action.value=\'save-resume\';document.blockform.submit()"';
         }
+        if ($readonly) {
+            $widget .= ' disabled="disabled"';
+        }
         $widget .= ">\n";
 
         foreach ($this->getBlocksList() as $id => $name) {
diff --git a/horde/templates/portal/layout.inc b/horde/templates/portal/layout.inc
index 3587badd4..1b63037b4 100644
--- a/horde/templates/portal/layout.inc
+++ b/horde/templates/portal/layout.inc
@@ -73,7 +73,7 @@
           </tr>
           <tr>
             <td colspan="2">
-              <?php echo $blocks->getBlocksWidget($blockInfo['app'], $blockInfo['block'], true) ?>
+              <?php echo $blocks->getBlocksWidget($blockInfo['app'], $blockInfo['block'], ($layout->isRemovable($row, $col) ? false : true), ($layout->isRemovable($row, $col) ? false : true)) ?>
             </td>
           </tr>
 <?php
-- 
2.11.0