From 6e7207b14aa50a0bd03c997b3cfb5b00b7bd9ca7 Mon Sep 17 00:00:00 2001
From: Florent Bruneau <florent.bruneau@polytechnique.org>
Date: Sun, 1 Jun 2008 15:35:43 +0200
Subject: [PATCH]  * Fix some issues with blog edition.  * avoid 'echo <html>'

Signed-off-by: Florent Bruneau <florent.bruneau@polytechnique.org>
---
 widget.blog.owner.php | 45 ++++++++++++++++++++++++++++++++++++++-------
 widget.post.perms.php | 32 +++++++++++++++++++++++---------
 2 files changed, 61 insertions(+), 16 deletions(-)
diff --git a/widget.blog.owner.php b/widget.blog.owner.php
index 272716c..66a234c 100644
--- a/widget.blog.owner.php
+++ b/widget.blog.owner.php
@@ -3,27 +3,58 @@
 class xorgBlogOwnerWidget {
   public static function behavior_adminBlogPreferencesForm(&$core) {
     if ($core->auth->isSuperAdmin()) {
+      if (isset($_GET['id'])) {
+        $settings = new dcSettings($core, $_GET['id']);
+      } else {
+        $settings =& $core->blog->settings;
+      }
       $types = array('user' => array('text' => 'Blog d\'utilisateur',
                                      'selected' => false),
                      'group-member' => array('text' => 'Blog de groupe, Ã©dition par les membres',
                                       'selected' => false),
                      'group-admin' => array('text' => 'Blog de groupe, Ã©dition par les administrateurs',
                                             'selected' => false));
-      $type = $core->blog->settings->get('xorg_blog_type');
+      $type = $settings->get('xorg_blog_type');
       if (!$type) {
         $type = 'user';
       }
       $types[$type]['selected'] = true;
-      echo '<fieldset><legend>Authentification X.org</legend><div class="two-cols"><div class="col">';
-      echo '<p><label>Type de blog&nbsp;:'
-         . '<select name="xorg_blog_type">';
+      ?>
+      <fieldset>
+        <legend>Authentification X.org</legend>
+        <div class="two-cols">
+          <div class="col">
+            <p>
+              <label>
+                Type de blog&nbsp;:
+                <select name="xorg_blog_type">
+      <?php
       foreach ($types as $key => $fields) {
         echo '<option value="' . $key . '"' . ($fields['selected'] ? ' selected="selected"' : '') . '>'
            . $fields['text'] . '</option>';
       }
-      echo '</select></label></p></div>';
-      echo '<div class="col"><p><label>PropriÃ©taire du blog (*)&nbsp;:<input type="text" name="xorg_blog_owner" value="' . $core->blog->settings->get('xorg_blog_owner') . '" /></label></p>';
-      echo '<p><label>(*) Dans le cas d\'un blog de groupe, le propriÃ©taire est le diminutif X.net du groupe<br />(*) Dans le cas d\'un blog d\'utilisateur, le propriÃ©taire est le forlife de l\'utilisateur</label></p></div></div></fieldset>';
+      ?>
+                </select>
+              </label>
+            </p>
+          </div>
+          <div class="col">
+            <p>
+              <label>
+                PropriÃ©taire du blog (*)&nbsp;:
+                <input type="text" name="xorg_blog_owner" value="<?php echo $settings->get('xorg_blog_owner'); ?> " />
+              </label>
+            </p>
+            <p>
+              <label>
+                (*) Dans le cas d'un blog de groupe, le propriÃ©taire est le diminutif X.net du groupe<br />
+                (*) Dans le cas d'un blog d'utilisateur, le propriÃ©taire est le forlife de l'utilisateur
+              </label>
+            </p>
+          </div>
+        </div>
+      </fieldset>
+      <?php
     }
   }
 
diff --git a/widget.post.perms.php b/widget.post.perms.php
index 415d371..ba0dc59 100644
--- a/widget.post.perms.php
+++ b/widget.post.perms.php
@@ -25,14 +25,21 @@ class xorgPostPermsWidget {
       }
     }
     $choices[$pos]['selected'] = true;
-    echo '<p><label>VisibilitÃ© du billet&nbsp;:'
-         . '  <select name="post_xorg_perms">';
+    ?>
+    <p>
+      <label>
+        VisibilitÃ© du billet&nbsp;:
+        <select name="post_xorg_perms">
+    <?php
     foreach ($choices as $val => $fields) {
       echo '<option value="' . $val . '"' . ($fields['selected'] ? ' selected="selected"' : '') . '>'
          . $fields['text'] . '</option>';
     }
-    echo '  </select>'
-       . '</label></p>';
+    ?>
+        </select>
+      </label>
+    </p>
+    <?php
   }
 
   private static function setPermsMeta(&$cur) {
@@ -68,15 +75,22 @@ class xorgPostPermsWidget {
       $pos = 'public';
     }
     $levels[$pos]['selected'] = true;
-    echo '<p><label>VisibilitÃ© nouveaux billets par dÃ©faut&nbsp;:'
-       . '  <select name="post_xorg_perms">';
+    ?>
+    <p>
+      <label>
+        VisibilitÃ© nouveaux billets par dÃ©faut&nbsp;:
+        <select name="post_xorg_perms">
+    <?php
     foreach ($levels as $key => $fields) {
       echo '<option value="' . $key . '"' . ($fields['selected'] ? ' selected="selected"' : '') . '>'
          . $fields['text'] . '</option>';
     }
-    echo '</select>'
-       . '(1) Ne concerne que les blogs de groupes X. Equivaut Ã  "Visible par les X" sur les autres blogs"'
-       . '</label></p>';
+    ?>
+        </select>
+        (1) Ne concerne que les blogs de groupes X. Equivaut Ã  "Visible par les X" sur les autres blogs"
+      </label>
+    </p>
+    <?php
   }
 
   public static function behavior_adminBeforeUserUpdate(&$cur, $user_id) {
-- 
2.1.4

