luciorota
Module Developer
Posted on: 2009/1/26 13:52
(maybe a [code] tag bug?)
wishcraft wrote:
Hi all how are you, this is a core issue not an issue with the application, the formselecteditor.php does not have a argument for the extra but also has a fault.. the remmed out lines are in error
To Fix this in the core here is how you do it, see it is using an none standardised form of Java which ensures you can have no use in the western world or anywhere else..
From line 51 of XOOPS_ROOT_PATH/class/xoopsform/formselecteditor.php you will have to alter it like so..function render()
{
xoops_load('XoopsEditorHandler');
$editor_handler = XoopsEditorHandler::getInstance();
$editor_handler->allowed_editors = $this->allowed_editors;
$option_select = new XoopsFormSelect("", $this->name, $this->value);
##[ Incorrect Javascript ]#################################################
// $extra = 'onchange="if(this.options[this.selectedIndex].value.length > 0 ){
// window.document.forms.'.$this->form->getName().'.submit();
// }"';
###########################################################################
// Here is the universal version of this it will work on any system as the location.href is the best method for backward compatibility..
$extra = 'onchange='location.href="'.XOOPS_URL.$_SERVER['PHP_SELF']."?";
foreach ($_GET as $key => $value)
if ($key!=$this->name)
$extra .= $key."=".$value."&";
$extra .= $this->name."="+this.options[this.selectedIndex].value'";
/// End of admendment.
$option_select->setExtra($extra);
$option_select->addOptionArray($editor_handler->getList($this->nohtml));
$this->addElement($option_select);
return parent::render();
}
function render()
{
xoops_load('XoopsEditorHandler');
$editor_handler = XoopsEditorHandler::getInstance();
$editor_handler->allowed_editors = $this->allowed_editors;
$option_select = new XoopsFormSelect("", $this->name, $this->value);
##[ Incorrect Javascript ]#################################################
// $extra = 'onchange="if(this.options[this.selectedIndex].value.length > 0 ){
// window.document.forms.'.$this->form->getName().'.submit();
// }"';
###########################################################################
// Here is the universal version of this it will work on any system as the location.href is the best method for backward compatibility..
$extra = 'onchange='location.href="'.$_SERVER['PHP_SELF'].'?';
foreach ($_GET as $key => $value)
if ($key!=$this->name)
$extra .= $key.'='.$value.'&';
$extra .= $this->name.'="+this.options[this.selectedIndex].value+"';
$extra .= '"'';
/// End of admendment.
$option_select->setExtra($extra);
$option_select->addOptionArray($editor_handler->getList($this->nohtml));
$this->addElement($option_select);
return parent::render();
}
by Mamba on 2008/9/23 20:53:43
Sounds very exciting!!! I'll have to install it!!!
If your module works with XOOPS 2.3, please consider changing your logo to the blue one, as described here.
dslchart wrote:
nice idea, but i become an
Quote:Errore 500 :(
if i visit your site.
kerkyra wrote:
hey mate,nice idea for a module!! i tried to install it in 2.3 php 5 and it gives me errors in MySQL syntax! tried to find out whats going on but no luck...:(
