1
recently I could not be able to work with sf.net (send features,bugs or download directly) because sf.net ban Iranian IPs more severe than before.
So please somebody send the below feature request on behalf of me from this link:
https://sourceforge.net/p/xoops/feature-requests/new/title:
sample forms in XoopsForm classes need some enhancements
content:
There is 2 xoops forms in xoops/class/xoopsform folder that needs some enhancements.
1- simpleform.php
It dont have the element class.
I suggest to replace it with this:
/**
* XOOPS simple form
*
* You may not change or alter any portion of this comment or credits
* of supporting developers from this source code or any supporting source code
* which is considered copyrighted (c) material of the original comment or credit authors.
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
*
* @copyright The XOOPS Project http://sourceforge.net/projects/xoops/
* @license GNU GPL 2 (http://www.gnu.org/licenses/old-licenses/gpl-2.0.html)
* @package kernel
* @subpackage form
* @since 2.0.0
* @author Kazumi Ono (AKA onokazu) http://www.myweb.ne.jp/, http://jp.xoops.org/
* @version $Id: simpleform.php 8066 2011-11-06 05:09:33Z beckmi $
*/
defined('XOOPS_ROOT_PATH') or die('Restricted access');
/**
* base class
*/
xoops_load('XoopsForm');
/**
* Form that will output as a simple HTML form with minimum formatting
*/
class XoopsSimpleForm extends XoopsForm
{
/**
* create HTML to output the form with minimal formatting
*
* @return string
*/
function render()
{
$ret = ($this->getTitle() ? "center head ">" . $this->getTitle() . ""
: "");
$ret.= ". $this->getName() . "' id='" . $this->getName() . "' action='" . $this->getAction() . "' method='" . $this->getMethod() . "'" . $this->getExtra() . ">n";
foreach ($this->getElements() as $ele) {
if (!$ele->isHidden()) {
$ret .= "$ele->getClass() . "">" . $ele->getCaption() . "" . $ele->render() . ""
;
} else {
$ret .= $ele->render() . "n";
}
}
$ret .= "n";
return $ret;
}
} I used the above in userlog:
http://svn.code.sf.net/p/xoops/svn/XoopsModules/userlog/trunk/userlog/class/form/simpleform.php2- themeform.php
It should be changed with a full div table with better looking and less hard-codes.
Also an id for each element is very needed like this:
$ret .= '. $ele->getName() . '" valign="top" align="left">';
then module developers can get the element by id.
