From 0de331182d265067e30527b333c2ec1b6331786a Mon Sep 17 00:00:00 2001
From: Julian Prud'homme <j.prudhomme@elixir-interactive.com>
Date: Tue, 17 Apr 2007 15:43:30 +0200
Subject: [PATCH] [svn r12044] Code refactoring : use formvalidator instead of
 old form and fix the "bug" : http://www.dokeos.com/forum/viewtopic.php?t=8778

---
 main/newscorm/lp_edit.php | 150 ++++++++++++++++++--------------------
 1 file changed, 72 insertions(+), 78 deletions(-)

diff --git a/main/newscorm/lp_edit.php b/main/newscorm/lp_edit.php
index 268dc24039..6dc23741b8 100644
--- a/main/newscorm/lp_edit.php
+++ b/main/newscorm/lp_edit.php
@@ -4,93 +4,87 @@
  * @package dokeos.learnpath
  * @author Yannick Warnier <ywarnier@beeznest.org>
  */
-/**
- * Script
- */
+
+require_once (api_get_path(LIBRARY_PATH).'formvalidator/FormValidator.class.php');
 
 $show_description_field = false; //for now
+
 $nameTools = get_lang("Doc");
+
 event_access_tool(TOOL_LEARNPATH);
+
 if (! $is_allowed_in_course) api_not_allowed();
-/**
- * Display
- */
+
 $interbreadcrumb[]= array ("url"=>"lp_controller.php?action=list", "name"=> get_lang("_learning_path"));
 $interbreadcrumb[]= array ("url"=>$_SERVER['PHP_SELF']."?action=admin_view&lp_id=$learnpath_id", "name" => $_SESSION['oLP']->get_name());
+
 Display::display_header(null,'Path');
-api_display_tool_title($nameTools);
-if($my_version=='1.8'){
-	Display::display_introduction_section(TOOL_LEARNPATH);
+
+//Page subtitle
+echo '<h4>'.get_lang('_edit_learnpath').'</h4>';
+
+$defaults=array();
+$form = new FormValidator('form1', 'post', 'lp_controller.php');
+
+
+//Title
+$form -> addElement('text', 'lp_name', get_lang('_title'));
+
+
+//Ecoding
+$encoding_select = &$form->addElement('select', 'lp_encoding', get_lang('Charset'));
+$encodings = array('UTF-8','ISO-8859-1','ISO-8859-15','cp1251','cp1252','KOI8-R','BIG5','GB2312','Shift_JIS','EUC-JP');
+foreach($encodings as $encoding){
+	if($encoding == $_SESSION['oLP']->encoding){
+  		$s_selected_encoding = $encoding;
+  	}
+  	$encoding_select->addOption($encoding,$encoding);
+}
+
+
+//Origin
+$origin_select = &$form->addElement('select', 'lp_maker', get_lang('Origin'));
+$lp_orig = $_SESSION['oLP']->get_maker();
+include('content_makers.inc.php');
+foreach($content_origins as $origin){
+	if($lp_orig == $origin){
+		$s_selected_origin = $origin;
+	}
+	$origin_select->addOption($origin,$origin);
+}
+
+
+//Content proximity
+$content_proximity_select = &$form->addElement('select', 'lp_proximity', get_lang('ContentProximity'));
+$lp_prox = $_SESSION['oLP']->get_proximity();
+if($lp_prox != 'local'){
+	$s_selected_proximity = 'remote';
 }else{
-	api_introductionsection(TOOL_LEARNPATH);
+	$s_selected_proximity = 'local';
 }
-?>
-<form name="form1" method="post" action="lp_controller.php">
-  <h4>
-	<?php echo get_lang('_edit_learnpath'); ?>
-  </h4>
-  <table width="400" border="0" cellspacing="2" cellpadding="0">
-	<tr>
-	  <td align="right"><?php echo get_lang('_title');?></td>
-	  <td><input name="lp_name" type="text" value="<?php echo $_SESSION['oLP']->get_name();?>" size="50" /></td>
-	</tr>
-	<?php if($show_description_field){ ?>
-	<tr>
-	  <td align="right" valign="top"><?php echo get_lang('_description');?></td>
-	  <td><textarea name='lp_description' cols='45'><?php echo $_SESSION['oLP']->get_description();?></textarea></td>
-	</tr>
-	<?php } ?>
-	<tr>
-	  <td align="right"><?php echo get_lang('Charset');?></td>
-	  <td><select name="lp_encoding">
-	  <?php
-	  $encodings = array('UTF-8','ISO-8859-1','ISO-8859-15','cp1251','cp1252','KOI8-R','BIG5','GB2312','Shift_JIS','EUC-JP');
-	  foreach($encodings as $encoding){
-	  	if($encoding == $_SESSION['oLP']->encoding){
-	  		echo "	<option value='$encoding' selected='selected'>$encoding</option>\n";
-	  	}else{
-	  		echo "	<option value='$encoding'>$encoding</option>\n";
-	  	}
-	  }
-	  ?></select></td>
-	</tr>
-	<tr>
-	  <td align="right"><?php echo get_lang('Origin');?></td>
-	  <td><select name="lp_maker">
-	  <?php
-	  $lp_orig = $_SESSION['oLP']->get_maker();
-	  include('content_makers.inc.php');
-	  foreach($content_origins as $origin){
-	  		$selected = '';
-	  		if($lp_orig == $origin){
-	  			$selected = ' selected = "selected"';
-	  		}
-	  		echo '    <option value="'.$origin.'"'.$selected.'>'.$origin.'</option>' ;
-	  }
-	  ?></select></td>
-	</tr>
-	<tr>
-	  <td align="right"><?php echo get_lang('Location');?></td>
-	  <td><select name="lp_proximity">
-	  <?php
-	  	$lp_prox = $_SESSION['oLP']->get_proximity();
-	  	if($lp_prox != 'local'){
-			echo '    <option value="local">'.get_lang('Local')."</option>" .
-			'    <option value="remote" selected="selected">'.get_lang('Remote')."</option>";
-	  	}else{
-			echo '    <option value="local" selected="selected">'.get_lang('Local')."</option>" .
-			'    <option value="remote">'.get_lang('Remote')."</option>";
-	  	}
-	  ?></select></td>
-	</tr>
-	<tr>
-      <td align="right">&nbsp;</td>
-		<input type="hidden" name="action" value="update_lp" />
-		<input type="hidden" name="lp_id" value="<?php echo $_SESSION['oLP']->get_id(); ?>" />
-		<td><input type="submit" name="Submit" value="<?php echo get_lang('Ok'); ?>" /></td>
-	</tr>
-  </table>
-</form>
-<?php
+$content_proximity_select->addOption(get_lang('Local'), 'local');
+$content_proximity_select->addOption(get_lang('Remote'), 'remote');
+
+
+//default values
+$content_proximity_select -> setSelected($s_selected_proximity);
+$origin_select -> setSelected($s_selected_origin);
+$encoding_select -> setSelected($s_selected_encoding);
+$defaults["lp_name"]=$_SESSION['oLP']->get_name();
+
+
+//Submit button
+$form->addElement('submit', 'Submit', get_lang('Ok'));
+
+
+//Hidden fields
+$form->addElement('hidden', 'action', 'update_lp');
+$form->addElement('hidden', 'lp_id', $_SESSION['oLP']->get_id());
+
+
+$form->setDefaults($defaults);
+$form -> display();
+
 Display::display_footer();
+
 ?>
\ No newline at end of file