From 7c990f68d68f691533f8bf745f67a9c8be8277c0 Mon Sep 17 00:00:00 2001
From: moodler <moodler>
Date: Sat, 14 Aug 2004 11:54:32 +0000
Subject: [PATCH] Fixed window choices on resources so that they use a radio
 button now.

bug 1741
---
 mod/resource/lib.php                      | 55 ++++++++++++-----------
 mod/resource/type/file/file.html          | 17 ++++---
 mod/resource/type/file/resource.class.php |  4 ++
 mod/resource/type/html/html.html          | 18 +++-----
 mod/resource/type/html/resource.class.php |  4 ++
 mod/resource/type/text/resource.class.php |  4 ++
 mod/resource/type/text/text.html          | 16 +++----
 7 files changed, 59 insertions(+), 59 deletions(-)

diff --git a/mod/resource/lib.php b/mod/resource/lib.php
index 99b9006e95..feb0ff9b2a 100644
--- a/mod/resource/lib.php
+++ b/mod/resource/lib.php
@@ -136,23 +136,23 @@ function add_instance($resource) {
     $resource->timemodified = time();
 
     if (isset($resource->windowpopup)) {
-        $optionlist = array();
-        foreach ($RESOURCE_WINDOW_OPTIONS as $option) {
-            if (isset($resource->$option)) {
-                $optionlist[] = $option."=".$resource->$option;
+        if ($resource->windowpopup) {
+            $optionlist = array();
+            foreach ($RESOURCE_WINDOW_OPTIONS as $option) {
+                if (isset($resource->$option)) {
+                    $optionlist[] = $option."=".$resource->$option;
+                }
             }
-        }
-        $resource->popup = implode(',', $optionlist);
-        $resource->options = "";
-
-    } else if (isset($resource->windowpage)) {
-
-        if (isset($resource->framepage)) {
-            $resource->options = "frame";
-        } else {
+            $resource->popup = implode(',', $optionlist);
             $resource->options = "";
+        } else {
+            if (isset($resource->framepage)) {
+                $resource->options = "frame";
+            } else {
+                $resource->options = "";
+            }
+            $resource->popup = "";
         }
-        $resource->popup = "";
     }
 
     return insert_record("resource", $resource);
@@ -170,22 +170,23 @@ function update_instance($resource) {
     $resource->timemodified = time();
 
     if (isset($resource->windowpopup)) {
-        $optionlist = array();
-        foreach ($RESOURCE_WINDOW_OPTIONS as $option) {
-            if (isset($resource->$option)) {
-                $optionlist[] = $option."=".$resource->$option;
+        if ($resource->windowpopup) {
+            $optionlist = array();
+            foreach ($RESOURCE_WINDOW_OPTIONS as $option) {
+                if (isset($resource->$option)) {
+                    $optionlist[] = $option."=".$resource->$option;
+                }
             }
-        }
-        $resource->popup = implode(',', $optionlist);
-        $resource->options = "";
-
-    } else if (isset($resource->windowpage)) {
-        if (isset($resource->framepage)) {
-            $resource->options = "frame";
-        } else {
+            $resource->popup = implode(',', $optionlist);
             $resource->options = "";
+        } else {
+            if (isset($resource->framepage)) {
+                $resource->options = "frame";
+            } else {
+                $resource->options = "";
+            }
+            $resource->popup = "";
         }
-        $resource->popup = "";
     }
 
     return update_record("resource", $resource);
diff --git a/mod/resource/type/file/file.html b/mod/resource/type/file/file.html
index 9a2da093a3..33565068cc 100644
--- a/mod/resource/type/file/file.html
+++ b/mod/resource/type/file/file.html
@@ -28,7 +28,9 @@
             var frameitem = [<?php echo $frameoption; ?>];
             var allitems = [<?php echo $alloptions; ?>];
         </script>
-        <input type="checkbox" name="windowpage" value=1 <?php echo ($windowtype == "page") ? "checked" : "" ?> onClick="lockoptions('form', 'windowpage', frameitem); document.form.windowpopup.checked=false; return lockoptions('form', 'windowpopup', popupitems);">
+        <input type="radio" name="windowpopup" value=0 <?php echo ($windowtype != "popup") ? "checked" : "" ?> 
+        onClick="lockoptions('form', 'windowpopup[0]', frameitem); 
+        return lockoptions('form', 'windowpopup[1]', popupitems);">
         <?php print_string("pagedisplay", "resource") ?>
         <ul>
             <input type="hidden" name="hframepage" value=0>
@@ -44,8 +46,9 @@
     </td>
     
     <td>
-        <input name="setnewwindow" type=hidden value=1>
-        <input name="windowpopup" type=checkbox value=1 onclick="lockoptions('form','windowpopup', popupitems); document.form.windowpage.checked=false; return lockoptions('form', 'windowpage', frameitem);">
+        <input name="windowpopup" type=radio value=1 <?php echo ($windowtype == "popup") ? "checked" : "" ?>
+        onclick="lockoptions('form', 'windowpopup[0]', frameitem); 
+        return lockoptions('form', 'windowpopup[1]', popupitems);">
         <?php p($strnewwindowopen) ?>
         <ul>
         <?php
@@ -70,15 +73,11 @@
         <?php
             if ($windowtype == "page") {
                 echo "<script>";
-                echo "lockoptions('form','windowpopup', popupitems);";
-                echo "document.form.windowpage.checked=true;";
-                echo "document.form.windowpopup.checked=false;";
+                echo "lockoptions('form','windowpopup[1]', popupitems);";
                 echo "</script>";
             } else {
                 echo "<script>";
-                echo "lockoptions('form','windowpage', frameitem);";
-                echo "document.form.windowpopup.checked=true;";
-                echo "document.form.windowpage.checked=false;";
+                echo "lockoptions('form','windowpopup[0]', frameitem);";
                 echo "</script>";
             }
         ?>
diff --git a/mod/resource/type/file/resource.class.php b/mod/resource/type/file/resource.class.php
index 57ee8ab711..a4bdac5578 100644
--- a/mod/resource/type/file/resource.class.php
+++ b/mod/resource/type/file/resource.class.php
@@ -479,6 +479,10 @@ function setup($form) {
     if ($form->instance) {     // Re-editing
         if (!$form->popup) {
             $windowtype = "page";   // No popup text => in page
+            foreach ($RESOURCE_WINDOW_OPTIONS as $optionname) {
+                $defaultvalue = "resource_popup$optionname";
+                $window->$optionname = $CFG->$defaultvalue;
+            }
         } else {
             $windowtype = "popup";
             $rawoptions = explode(',', $form->popup);
diff --git a/mod/resource/type/html/html.html b/mod/resource/type/html/html.html
index ce32d708ea..fb06350310 100644
--- a/mod/resource/type/html/html.html
+++ b/mod/resource/type/html/html.html
@@ -8,7 +8,7 @@
         </font>
     </td>
     <td>
-        <?php print_textarea($usehtmleditor, 20, 60, 680, 400, "alltext", $form->alltext); ?>
+        <?php print_textarea($usehtmleditor, 30, 60, 680, 500, "alltext", $form->alltext); ?>
     </td>
 </tr> 
 
@@ -24,7 +24,8 @@
             var popupitems = [<?php echo $popupoptions; ?>];
             var allitems = [<?php echo $alloptions; ?>];
         </script>
-        <input type="checkbox" name="windowpage" value=1 <?php echo ($windowtype == "page") ? "checked" : "" ?> onClick="document.form.windowpopup.checked=false; return lockoptions('form', 'windowpopup', popupitems);">
+        <input type="radio" name="windowpopup" value=0 <?php echo ($windowtype != "popup") ? "checked" : "" ?> 
+        onClick="return lockoptions('form', 'windowpopup[1]', popupitems);">
         <?php print_string("pagedisplay", "resource") ?>
     </td>
 </tr>
@@ -35,8 +36,8 @@
     </td>
     
     <td>
-        <input name="setnewwindow" type=hidden value=1>
-        <input name="windowpopup" type=checkbox value=1 onclick="lockoptions('form','windowpopup', popupitems); document.form.windowpage.checked=false;">
+        <input name="windowpopup" type=radio value=1 <?php echo ($windowtype == "popup") ? "checked" : "" ?>
+        onclick=" return lockoptions('form', 'windowpopup[1]', popupitems);">
         <?php p($strnewwindowopen) ?>
         <ul>
         <?php
@@ -61,14 +62,7 @@
         <?php
             if ($windowtype == "page") {
                 echo "<script>";
-                echo "lockoptions('form','windowpopup', popupitems);";
-                echo "document.form.windowpage.checked=true;";
-                echo "document.form.windowpopup.checked=false;";
-                echo "</script>";
-            } else {
-                echo "<script>";
-                echo "document.form.windowpopup.checked=true;";
-                echo "document.form.windowpage.checked=false;";
+                echo "lockoptions('form','windowpopup[1]', popupitems);";
                 echo "</script>";
             }
         ?>
diff --git a/mod/resource/type/html/resource.class.php b/mod/resource/type/html/resource.class.php
index e94518b2d3..8289cfa793 100644
--- a/mod/resource/type/html/resource.class.php
+++ b/mod/resource/type/html/resource.class.php
@@ -162,6 +162,10 @@ function setup($form) {
     if ($form->instance) {     // Re-editing
         if (!$form->popup) {
             $windowtype = "page";   // No popup text => in page
+            foreach ($RESOURCE_WINDOW_OPTIONS as $optionname) {
+                $defaultvalue = "resource_popup$optionname";
+                $window->$optionname = $CFG->$defaultvalue;
+            }
         } else {
             $windowtype = "popup";
             $rawoptions = explode(',', $form->popup);
diff --git a/mod/resource/type/text/resource.class.php b/mod/resource/type/text/resource.class.php
index a3a7066dd8..60160a40c1 100644
--- a/mod/resource/type/text/resource.class.php
+++ b/mod/resource/type/text/resource.class.php
@@ -163,6 +163,10 @@ function setup($form) {
     if ($form->instance) {     // Re-editing
         if (!$form->popup) {
             $windowtype = "page";   // No popup text => in page
+            foreach ($RESOURCE_WINDOW_OPTIONS as $optionname) {
+                $defaultvalue = "resource_popup$optionname";
+                $window->$optionname = $CFG->$defaultvalue;
+            }
         } else {
             $windowtype = "popup";
             $rawoptions = explode(',', $form->popup);
diff --git a/mod/resource/type/text/text.html b/mod/resource/type/text/text.html
index a639aa50cc..9384fbd084 100644
--- a/mod/resource/type/text/text.html
+++ b/mod/resource/type/text/text.html
@@ -34,7 +34,8 @@
             var popupitems = [<?php echo $popupoptions; ?>];
             var allitems = [<?php echo $alloptions; ?>];
         </script>
-        <input type="checkbox" name="windowpage" value=1 <?php echo ($windowtype == "page") ? "checked" : "" ?> onClick="document.form.windowpopup.checked=false; return lockoptions('form', 'windowpopup', popupitems);">
+        <input type="radio" name="windowpopup" value=0 <?php echo ($windowtype != "popup") ? "checked" : "" ?> 
+        onClick="return lockoptions('form', 'windowpopup[1]', popupitems);">
         <?php print_string("pagedisplay", "resource") ?>
     </td>
 </tr>
@@ -45,8 +46,8 @@
     </td>
     
     <td>
-        <input name="setnewwindow" type=hidden value=1>
-        <input name="windowpopup" type=checkbox value=1 onclick="lockoptions('form','windowpopup', popupitems); document.form.windowpage.checked=false;">
+        <input name="windowpopup" type=radio value=1 <?php echo ($windowtype == "popup") ? "checked" : "" ?>
+        onclick=" return lockoptions('form', 'windowpopup[1]', popupitems);">
         <?php p($strnewwindowopen) ?>
         <ul>
         <?php
@@ -71,14 +72,7 @@
         <?php
             if ($windowtype == "page") {
                 echo "<script>";
-                echo "lockoptions('form','windowpopup', popupitems);";
-                echo "document.form.windowpage.checked=true;";
-                echo "document.form.windowpopup.checked=false;";
-                echo "</script>";
-            } else {
-                echo "<script>";
-                echo "document.form.windowpopup.checked=true;";
-                echo "document.form.windowpage.checked=false;";
+                echo "lockoptions('form','windowpopup[1]', popupitems);";
                 echo "</script>";
             }
         ?>
-- 
2.39.5