From: Antoine Musso <hashar@users.mediawiki.org>
Date: Thu, 30 Sep 2004 22:48:38 +0000 (+0000)
Subject: moving html forms builders to its own class. Do we want them in wgOut instead ?
X-Git-Tag: 1.5.0alpha1~1721
X-Git-Url: http://git.cyclocoop.org/data/Fool?a=commitdiff_plain;h=46aacac9cbe7d4eece1c85fc3a3b00b1f25efdd9;p=lhc%2Fweb%2Fwiklou.git

moving html forms builders to its own class. Do we want them in wgOut instead ?
---

diff --git a/includes/SpecialSitesettings.php b/includes/SpecialSitesettings.php
index ecc97a9706..0c8f450486 100644
--- a/includes/SpecialSitesettings.php
+++ b/includes/SpecialSitesettings.php
@@ -1,14 +1,14 @@
 <?php
+require_once('HTMLForm.php');
 
-function wfSpecialSiteSettings()
-{
+function wfSpecialSiteSettings() {
 	global $wgRequest;
 
 	$form = new SiteSettingsForm( $wgRequest );
 	$form->execute();
 }
 
-class SiteSettingsForm {
+class SiteSettingsForm extends HTMLForm {
 	var $mPosted, $mRequest, $mReset, $mSaveprefs;
 	
 	function SiteSettingsForm ( &$request ) {
@@ -36,44 +36,6 @@ class SiteSettingsForm {
 		return;
 	}
 
-	/* private */ function fieldset( $name, $content ) {
-		return "<fieldset><legend>".wfMsg($name)."</legend>\n" .
-			$content . "\n</fieldset>\n";
-	}
-
-	/* private */ function checkbox( $varname, $checked=false ) {
-		$checked = isset( $GLOBALS[$varname] ) && $GLOBALS[$varname] ;
-		return "<div><input type='checkbox' value=\"1\" id=\"{$varname}\" name=\"wpOp{$varname}\"" .
-			( $checked ? ' checked="checked"' : '' ) .
-			" /><label for=\"{$varname}\">". wfMsg( "sitesettings-".$varname ) .
-			"</label></div>\n";
-	}
-
-	/* private */ function textbox( $varname, $value='', $size=20 ) {
-		$value = isset( $GLOBALS[$varname] ) ? $GLOBALS[$varname] : '';
-		return "<div><label>". wfMsg( "sitesettings-".$varname ) .
-			"<input type='text' name=\"wpOp{$varname}\" value=\"{$value}\" size=\"{$size}\" /></label></div>\n";
-	}
-	/* private */ function radiobox( $varname, $fields ) {
-		foreach ( $fields as $value => $checked ) {
-			$s .= "<div><label><input type='radio' name=\"wpOp{$varname}\" value=\"{$value}\"" .
-				( $checked ? ' checked="checked"' : '' ) . " />" . wfMsg( 'sitesettings-'.$varname.'-'.$value ) .
-				"</label></div>\n";
-		}
-		return $this->fieldset( 'sitesettings-'.$varname, $s );
-	}
-
-	/* private */ function arraybox( $varname , $size=20 ) {
-		$s = '';
-		if ( isset( $GLOBALS[$varname] ) && is_array( $GLOBALS[$varname] ) ) {
-			foreach ( $GLOBALS[$varname] as $index=>$element ) {
-				$s .= $element."\n";
-			}
-		}
-		return "<div><label>".wfMsg( 'sitesettings-'.$varname ).
-			"<textarea name=\"wpOp{$varname}\" rows=\"5\" cols=\"{$size}\">{$s}</textarea>\n";
-	}
-
 	/* private */ function mainPrefsForm( $err ) {
 		global $wgOut;