xoops-trackers Mailing List for XOOPS Web Application System

Bugs item #1229976, was opened at 2005-06-29 22:29
Message generated for change (Settings changed) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1229976&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Modules - Forum
Group: XOOPS 2.0.x
>Status: Closed
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Syntax error in 'html/modules/newbb/post.php'

Initial Comment:
Line #50 of 'html/modules/newbb/post.php' contains a
syntax error.

if(empty($forumdata['allow_html'])) {
  $HTTP_POST_VARS['nohtml'] = 1: # <-- should be ;
}

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2005-06-30 16:23

Message:
Logged In: NO 

This is fixed in 2.0.12a

/Mithrandir

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1229976&group_id=41586

Bugs item #1229976, was opened at 2005-06-29 13:29
Message generated for change (Comment added) made by nobody
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1229976&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Modules - Forum
Group: XOOPS 2.0.x
Status: Open
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Syntax error in 'html/modules/newbb/post.php'

Initial Comment:
Line #50 of 'html/modules/newbb/post.php' contains a
syntax error.

if(empty($forumdata['allow_html'])) {
  $HTTP_POST_VARS['nohtml'] = 1: # <-- should be ;
}

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2005-06-30 07:23

Message:
Logged In: NO 

This is fixed in 2.0.12a

/Mithrandir

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1229976&group_id=41586

Bugs item #1230190, was opened at 2005-06-30 09:50
Message generated for change (Settings changed) made by thevivi
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1230190&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.0.x
Status: Open
Resolution: None
>Priority: 2
Submitted By: {Vivi} (thevivi)
Assigned to: Nobody/Anonymous (nobody)
Summary: Criteria class

Initial Comment:
A little issue in criteria class. Here, the incrimined
function

function render() {
		if ( is_numeric($this->value) ||
strtoupper($this->operator) == 'IN') {
			$value = $this->value;
		} else {
			if ( '' === ($value = trim($this->value)) ) {
				return '';
			}
			if ( (substr($value, 0, 1) != '`') &&
(substr($value, -1) != '`') ) {
				$value = "'$value'";
			}
		}
		$clause = (!empty($this->prefix) ? "{$this->prefix}."
: "") . $this->column;
		if ( !empty($this->function) ) {
			$clause = sprintf($this->function, $clause);
		}
        $clause .= " {$this->operator} $value";
        return $clause;
}


And here the incrimined line

if ( '' === ($value = trim($this->value)) ) {
  return '';
}

This may cause little render fail every time value is
=== ''. But criteria may be : myField <> '' . So with
this code these criteria are impossible.

My suggestion

if ( '' === ($value = trim($this->value)) ) {
  return '';
}

replaced by

$value = trim($this->value);

because checking $this->value === '' will not bring
additional features and it will not be more fast.

Thanks for your attention, Vivi.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1230190&group_id=41586

Bugs item #1230190, was opened at 2005-06-30 09:50
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1230190&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.0.x
Status: Open
Resolution: None
Priority: 5
Submitted By: {Vivi} (thevivi)
Assigned to: Nobody/Anonymous (nobody)
Summary: Criteria class

Initial Comment:
A little issue in criteria class. Here, the incrimined
function

function render() {
		if ( is_numeric($this->value) ||
strtoupper($this->operator) == 'IN') {
			$value = $this->value;
		} else {
			if ( '' === ($value = trim($this->value)) ) {
				return '';
			}
			if ( (substr($value, 0, 1) != '`') &&
(substr($value, -1) != '`') ) {
				$value = "'$value'";
			}
		}
		$clause = (!empty($this->prefix) ? "{$this->prefix}."
: "") . $this->column;
		if ( !empty($this->function) ) {
			$clause = sprintf($this->function, $clause);
		}
        $clause .= " {$this->operator} $value";
        return $clause;
}


And here the incrimined line

if ( '' === ($value = trim($this->value)) ) {
  return '';
}

This may cause little render fail every time value is
=== ''. But criteria may be : myField <> '' . So with
this code these criteria are impossible.

My suggestion

if ( '' === ($value = trim($this->value)) ) {
  return '';
}

replaced by

$value = trim($this->value);

because checking $this->value === '' will not bring
additional features and it will not be more fast.

Thanks for your attention, Vivi.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1230190&group_id=41586

Bugs item #1229976, was opened at 2005-06-29 13:29
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1229976&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Modules - Forum
Group: XOOPS 2.0.x
Status: Open
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Syntax error in 'html/modules/newbb/post.php'

Initial Comment:
Line #50 of 'html/modules/newbb/post.php' contains a
syntax error.

if(empty($forumdata['allow_html'])) {
  $HTTP_POST_VARS['nohtml'] = 1: # <-- should be ;
}

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1229976&group_id=41586

Bugs item #1228600, was opened at 2005-06-28 00:12
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1228600&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.0.10
Status: Open
Resolution: None
Priority: 5
Submitted By: wdsl (wdsl)
Assigned to: Nobody/Anonymous (nobody)
Summary: Constants of Confignames to short

Initial Comment:
I tried to write a module for Xoops 2.0.11:

I write in xoops_version.php this:
=====================
$modversion['config'][1]['name'] = 'stf_modulid';
$modversion['config'][1]['title'] =
'_MI__STF_MOVIES__CONFIG_STF_ID_TITLE';
$modversion['config'][1]['description'] =
'_MI__STF_MOVIES__CONFIG_STF_ID_DESC';
$modversion['config'][1]['formtype'] = 'textbox';
$modversion['config'][1]['valuetype'] = 'int';
$modversion['config'][1]['default'] = '1';

$modversion['config'][2]['name'] =
'stf_num_movies_per_side';
$modversion['config'][2]['title'] =
'_MI__STF_MOVIES__CONFIG_MOVIES_PER_SIDE';
$modversion['config'][2]['description'] =
'_MI__STF_MOVIES__CONFIG_MOVIES_PER_SIDE_DESC';
$modversion['config'][2]['formtype'] = 'textbox';
$modversion['config'][2]['valuetype'] = 'int';
$modversion['config'][2]['default'] = 30;

But later i get this errors when i would like to change
the preferences:
 Warning [PHP]: constant() [function.constant]:
Couldn't find constant _MI__STF_MOVIES__CONFIG_STF_ID
in file modules/system/admin/preferences/main.php line 260
Warning [PHP]: constant() [function.constant]: Couldn't
find constant _MI__STF_MOVIES__CONFIG_MOVIES in file
modules/system/admin/preferences/main.php line 260

I know my Names are long but i think bigger SQL filds
should solve such problems.

=========================================
Solution of this in /install/sql/mysql.structure.sql
=========================================

#
# Table structure for table `config`
#

CREATE TABLE config (
  conf_id smallint(5) unsigned NOT NULL auto_increment,
  conf_modid smallint(5) unsigned NOT NULL default '0',
  conf_catid smallint(5) unsigned NOT NULL default '0',
  conf_name varchar(25) NOT NULL default '',
  conf_title varchar(30) NOT NULL default '',
  conf_value text NOT NULL,
  conf_desc varchar(30) NOT NULL default '',
  conf_formtype varchar(15) NOT NULL default '',
  conf_valuetype varchar(10) NOT NULL default '',
  conf_order smallint(5) unsigned NOT NULL default '0',
  PRIMARY KEY  (conf_id),
  KEY conf_mod_cat_id (conf_modid,conf_catid)
) TYPE=MyISAM;

=========================================
changing: conf_title varchar(30) NOT NULL default '',
in: conf_title varchar(60) NOT NULL default '',


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1228600&group_id=41586

Bugs item #1161813, was opened at 2005-03-12 05:02
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1161813&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.2.x
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Jan Pedersen (mithyt2)
Summary: edituser.php allows change to a preexisting e-mail address

Initial Comment:
When you try to register using an e-mail address that
someone has registered already, the system tells you
(correctly) that the e-mail address is already in use.

However, when you change the e-mail address of a
pre-existing account to an address that's already in
use, it doesn't tell you that the new e-mail address is
already in use.

Adding the following (based on the code in
register.php) to the validation code in edituser.php
solves the problem:

        $count = 0;
        if ( $email ) {
                $sql = "SELECT COUNT(*) FROM
".$xoopsDB->prefix('users')." where
email='".$myts->makeTboxData4Save($email)."'";
                $result = $xoopsDB->query($sql);
                list($count) = $xoopsDB->fetchRow($result);
                if ( $count > 0 ) {
                        $errors[] = _US_EMAILTAKEN;
                }
        }



----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-26 17:10

Message:
Logged In: YES 
user_id=841117

In XOOPS 2.2, changing an email address requires a response
to an email sent to the new address.

We will be improving on this procedure in future XOOPS versions

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-03-25 14:20

Message:
Logged In: YES 
user_id=841117

OK, I'll take care of this one personally. 

----------------------------------------------------------------------

Comment By: Nobody/Anonymous (nobody)
Date: 2005-03-21 00:39

Message:
Logged In: NO 

Update: That should be something like:

$count = 0;
if ( $email ) {
$sql = "SELECT COUNT(*) FROM
".$xoopsDB->prefix('users')." where
email='".$myts->makeTboxData4Save($email)."'" and uid <> $uid;
$result = $xoopsDB->query($sql);
list($count) = $xoopsDB->fetchRow($result);
if ( $count > 0 ) {
$errors[] = _US_EMAILTAKEN;
}
}

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1161813&group_id=41586

Bugs item #1227758, was opened at 2005-06-26 16:06
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227758&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
>Group: XOOPS 2.2.x
>Status: Closed
>Resolution: Accepted
Priority: 1
Submitted By: Valcilon Silva (valcilon)
>Assigned to: Jan Pedersen (mithyt2)
Summary: xoops 2.0.11 - index.html missing

Initial Comment:
index.html missing in these folders:
/docs
/docs/images
/html/class/mail
/html/class/mail/phpmailer
/html/class/smarty
/html/class/smarty/config
/html/class/smarty/core
/html/class/smarty/plugins
/html/extras
/html/extras/theme_x2t
/html/extras/theme_x2t/doc
/html/extras/theme_x2t/extras
/html/install/class
/html/install/img
/html/install/language/
/html/install/language/english
/html/install/sql
/html/install/sql/upgrade
/html/kernel
/html/english/mail_template
/html/modules/mydownloads/language/english/mail_template
/html/modules/mylinks/language/english/mail_template
/html/modules/newbb/language/english/mail_template
/html/modules/news/language/english/mail_template
/html/modules/system/images
/html/modules/xoopsheadline/include
/html/modules/xoopsheadline/sql
/html/modules/xoopspoll/language/english/mail_template
/html/themes/default
/html/themes/phpkaox/

Valcilon Silva AKA Valker

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-26 17:06

Message:
Logged In: YES 
user_id=841117

Fixed in CVS and waiting for review before merging to the trunk.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227758&group_id=41586

Bugs item #1227758, was opened at 2005-06-26 11:06
Message generated for change (Settings changed) made by valcilon
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227758&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.0.x
Status: Open
Resolution: None
>Priority: 1
Submitted By: Valcilon Silva (valcilon)
Assigned to: Nobody/Anonymous (nobody)
Summary: xoops 2.0.11 - index.html missing

Initial Comment:
index.html missing in these folders:
/docs
/docs/images
/html/class/mail
/html/class/mail/phpmailer
/html/class/smarty
/html/class/smarty/config
/html/class/smarty/core
/html/class/smarty/plugins
/html/extras
/html/extras/theme_x2t
/html/extras/theme_x2t/doc
/html/extras/theme_x2t/extras
/html/install/class
/html/install/img
/html/install/language/
/html/install/language/english
/html/install/sql
/html/install/sql/upgrade
/html/kernel
/html/english/mail_template
/html/modules/mydownloads/language/english/mail_template
/html/modules/mylinks/language/english/mail_template
/html/modules/newbb/language/english/mail_template
/html/modules/news/language/english/mail_template
/html/modules/system/images
/html/modules/xoopsheadline/include
/html/modules/xoopsheadline/sql
/html/modules/xoopspoll/language/english/mail_template
/html/themes/default
/html/themes/phpkaox/

Valcilon Silva AKA Valker

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227758&group_id=41586

Bugs item #1227758, was opened at 2005-06-26 11:06
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227758&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.0.x
Status: Open
Resolution: None
Priority: 5
Submitted By: Valcilon Silva (valcilon)
Assigned to: Nobody/Anonymous (nobody)
Summary: xoops 2.0.11 - index.html missing

Initial Comment:
index.html missing in these folders:
/docs
/docs/images
/html/class/mail
/html/class/mail/phpmailer
/html/class/smarty
/html/class/smarty/config
/html/class/smarty/core
/html/class/smarty/plugins
/html/extras
/html/extras/theme_x2t
/html/extras/theme_x2t/doc
/html/extras/theme_x2t/extras
/html/install/class
/html/install/img
/html/install/language/
/html/install/language/english
/html/install/sql
/html/install/sql/upgrade
/html/kernel
/html/english/mail_template
/html/modules/mydownloads/language/english/mail_template
/html/modules/mylinks/language/english/mail_template
/html/modules/newbb/language/english/mail_template
/html/modules/news/language/english/mail_template
/html/modules/system/images
/html/modules/xoopsheadline/include
/html/modules/xoopsheadline/sql
/html/modules/xoopspoll/language/english/mail_template
/html/themes/default
/html/themes/phpkaox/

Valcilon Silva AKA Valker

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227758&group_id=41586

Feature Requests item #1227716, was opened at 2005-06-26 12:17
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430843&aid=1227716&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Fritz Schmidt (xer0c)
Assigned to: Nobody/Anonymous (nobody)
Summary: Search/Filter function for imagemanager

Initial Comment:
I'd really like to see a search or filter function for
the imagemanger.
It is extremely unconfortable to click through 20pages
of images to just find the one you need. 

It should not be too difficult to implement, because
you even give the pictures a name during upload. 

Thx

XeRoc
-- linux-gamers.net

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430843&aid=1227716&group_id=41586

Patches item #1227678, was opened at 2005-06-26 02:32
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430842&aid=1227678&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: XOOPS 2.0.x
Group: Patches
Status: Open
Resolution: None
Priority: 5
Submitted By: caskater (caskater)
Assigned to: Nobody/Anonymous (nobody)
Summary: Adding registration keys module permanently to system core

Initial Comment:
I would like the registration keys add-on module to be
incorporated as a standard feature in the system
module. Many people have found it incredibly useful. It
is proven to have no issues as many users already have
it installed. It offers users a more customizeable
experience with Xoops that can be of great use for
businesses, organizations, etc, that want to have more
control over how specific individuals register for
their site. Registration Keys offers the following
features to its users:
- Allow or Require XOOPS registrants to enter a special
alpha-numeric case-sensative registration key.
 - Create multiple registration keys
 - Register users to specific group(s).
 - Keys can be enabled and disabled
 - Give non-webmaster groups access to create keys and
specify what groups they have control over

Adding this as a permanent feature to the system core
module would provide just one more reason why web
designers and webmasters should choose Xoops over other
CMS' available.

Thanks for your time and consideration.

Jean-Philippe Steinmetz
cas...@gm...

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430842&aid=1227678&group_id=41586

Patches item #1015560, was opened at 2004-08-24 23:01
Message generated for change (Settings changed) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430842&aid=1015560&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: XOOPS 2.0.x
>Group: XOOPS 2.2
>Status: Pending
Resolution: None
Priority: 5
Submitted By: phppp (phppp)
Assigned to: Nobody/Anonymous (nobody)
Summary: better rendering [ code ][ /code ]

Initial Comment:
in class/module.textsanitizer.php, use:
--------------------------------------

	function codeConv($text, $xcode = 1, $image 
= 1){
		if($xcode != 0){
			$patterns = "/\[code](.*)
\[\/code\]/esU";
			if ($image != 0) {
				// image allowed
				$replacements 
= "'<pre 
class=\xoopsCode\>'.MyTextSanitizer::codeSanitizer
('$1').'</pre>'"; // hack for [code] rendering by <pre> -- 
phppp
				//$text =& $this-
>xoopsCodeDecode($text);
			} else {
				// image not 
allowed
				$replacements 
= "'<pre 
class=\xoopsCode\>'.MyTextSanitizer::codeSanitizer
('$1', 0).'</pre>'"; // hack for [code] rendering by 
<pre> -- phppp
				//$text =& $this-
>xoopsCodeDecode($text, 0);
			}
			$text =  preg_replace
($patterns, $replacements, $text);
		}
		return $text;
	}


then add following to xoops.css or 
themes/yourtheme/style.css, styleNN.css, styleMAC.css 
-----------------------------------------------
pre.xoopscode {
	background: #fffff8;
	border: 1px solid #6C87B0;
	color: #385a72;
	width: 500px;	
	height: 200px;
	margin: 1em;
	overflow: auto;
	padding: 3px 6px 6px 6px;
}


----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 18:37

Message:
Logged In: YES 
user_id=841117

Can you explain to me what this does that isn't done by
adding the CSS style to div.xoopsCode {} in the theme's CSS?

I mean, what's the difference between putting the class on
the <pre> tag as opposed to the <div> tag?

----------------------------------------------------------------------

Comment By: nick89 (nick89)
Date: 2004-10-02 00:24

Message:
Logged In: YES 
user_id=1131183

yes this is much like the latest vbulletin style and it
works very well.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430842&aid=1015560&group_id=41586

Patches item #1208446, was opened at 2005-05-25 15:27
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430842&aid=1208446&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: XOOPS 2.1.x
Group: XOOPS 2.2
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Bruno (bruno2000)
Assigned to: Jan Pedersen (mithyt2)
Summary: a xoops foreign key combo

Initial Comment:
Here is 2 classes to display a combo mapping to a
class. One extend the seconds with a combo implementation.

Here is a example :
$f_qualification_id = new XoopsFormComboForeignClass
(_FO_QUALIFICATION_ID, 'qualification_id',
50,150,$'qualification_id,'ox_qualification');

There 2 conditions to uses it :
- a 'ox_qualification' class exist with it's handler
respecting the xoopsObjetc modèle
- a toString() methode must be implemented in the
xoopsObject (here the ox_qualification class.


----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 18:23

Message:
Logged In: YES 
user_id=841117

I have added a XoopsFormSelectList element that will use
xoops_getmodulehandler to get the handler to use.

This handler must have a getList() method to retrieve the list.
See XoopsPersistableObjectHandler child classes constructors
for how to set the identifier used for getList()

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-05-27 17:42

Message:
Logged In: YES 
user_id=841117

Ahh - so it is a way to get a list of [generic item] in a
<select> ?

Interesting.

I will take a look at it at earliest convenience. It won't
be for the 2.0.x series, though, but perhaps for 2.1/2.2
Thanks.

----------------------------------------------------------------------

Comment By: Bruno (bruno2000)
Date: 2005-05-27 17:31

Message:
Logged In: YES 
user_id=1093822

It is a generique combo mapping directly to a database
table. For exemple actualy in xoops the is several
XoopsFormSelectxxxx. XoopsFormComboForeignClass can render
the same service 

For example pour class "group" : 
1/ you implement in the class the toSting function like that

class XoopsGroup extends XoopsObject
{
function toString()
    {
    	return $this->getVar('name');
}

2/ you use directly a combo with all group 

include XOOPS_ROOT_PATH.'kernel/group.php';
$f_group = new XoopsFormComboForeignClass (_FO_GROUPE, '
groupid', 50,150,$groupid,'group');
$form->addElement($f_group);

In my modules exemple after modify the kernel\group.php you
can add this code in 84 of form\form_fks_use_classif.php


----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-05-27 16:57

Message:
Logged In: YES 
user_id=841117

yes, but what is it for?

----------------------------------------------------------------------

Comment By: Bruno (bruno2000)
Date: 2005-05-27 16:47

Message:
Logged In: YES 
user_id=1093822

I made a sample module witch use this foreign key combo.

In this module, there is 2 classes :
FKS_classification : the foreign table, basicly a
classification.
FKS_use_classif : A class witch use the classification as
foreign key.

The foreign key form is used in the form named
FKSample\form\form_fks_use_classif.php in the
add_fields_fks_use_classif method.
To use it, install the module and type the index

NB : There is no main menu but only a block to install




----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-05-26 12:44

Message:
Logged In: YES 
user_id=841117

Can you elaborate on its use?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430842&aid=1208446&group_id=41586

Bugs item #977360, was opened at 2004-06-22 12:58
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=977360&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.2.x
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: zoullou (zoullou)
Assigned to: Jan Pedersen (mithyt2)
Summary: Wrong icon in comment bloc

Initial Comment:
When an user post a comment without check message icon,
xoopsdir/images/subject/icon1.gif are diplayed then
it's must be xoopsdir/images/icons/no_posticon.gif who
are diplayed.

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 18:05

Message:
Logged In: YES 
user_id=841117

Fixed in CVS and waiting for review before merging to the trunk.

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-03-25 13:05

Message:
Logged In: YES 
user_id=841117

I'll see what I can find out about why this is.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=977360&group_id=41586

Bugs item #1043481, was opened at 2004-10-09 09:44
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1043481&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.2.x
>Status: Pending
Resolution: None
Priority: 5
Submitted By: zoullou (zoullou)
Assigned to: Jan Pedersen (mithyt2)
Summary: XOBJ_DTYPE_EMAIL and XOBJ_DTYPE_URL

Initial Comment:
When i get var with function &getVar($key, $format =
's') and XOBJ_DTYPE_EMAIL or XOBJ_DTYPE_URL this method
return a var without use MYTS function
&makeClickable(&$text).

----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 17:57

Message:
Logged In: YES 
user_id=841117

I am not convinced that it is always desirable for these
types of variables to be made clickable (although I do see
your point)

Will think about this some more, but don't think I will do
much about it. Unless you have good arguments, of course.

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-03-25 13:32

Message:
Logged In: YES 
user_id=841117

OK, I'll take care of this one personally. 

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1043481&group_id=41586

Bugs item #1198680, was opened at 2005-05-10 02:49
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1198680&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: CVS Nightly
Group: None
>Status: Closed
Resolution: None
Priority: 2
Submitted By: Graham Davies (gravies)
Assigned to: Jan Pedersen (mithyt2)
Summary: Headline Modules with allow_url_fopen OFF

Initial Comment:
Headline module does not work with allow_url_fopen OFF...

Fix published using snoopy.php:

http://xoops-tips.com/news-article.storyid-26.htm
http://www.xoops.org/modules/newbb/viewtopic.php?topic_id=22451&forum=8&post_id=98068#forumpost98068

----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 17:53

Message:
Logged In: YES 
user_id=841117

The headlines module will not be in XOOPS 2.2, instead we
will compile packages of core files and a selection of modules.

The headline module, we will put in those packages and
recommend that people use, is GIJOE's xhld module that does
not use fopen

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1198680&group_id=41586

Bugs item #1005686, was opened at 2004-08-09 01:15
Message generated for change (Settings changed) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1005686&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Pending
Resolution: None
Priority: 5
Submitted By: phppp (phppp)
Assigned to: Nobody/Anonymous (nobody)
Summary: module.textsanitizer [ code] [ /code]

Initial Comment:
currently the codePreConv(), codeConv() functions do 
not consider recursive [ code] [ /code]

----------------------------------------------------------------------

Comment By: Brian Wahoff (ackbarr)
Date: 2004-10-26 00:03

Message:
Logged In: YES 
user_id=678143

maybe I'm missing something, but when would recursive [code]
blocks be necessary? and what would the desired behavior be?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1005686&group_id=41586

Bugs item #1180338, was opened at 2005-04-10 22:55
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1180338&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  install/upgrade
Group: XOOPS 2.2.x
>Status: Pending
Resolution: None
Priority: 5
Submitted By: Leonard den Ottolander (leonardjo)
Assigned to: Jan Pedersen (mithyt2)
Summary: General configuration shows db password

Initial Comment:
Setting up Xoops for the first time it shows the db
password in the general configuration form. The more
usual approach is to hide the password and let the user
retype it to avoid typos.


----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 17:31

Message:
Logged In: YES 
user_id=841117

We are talking about a one-time typing of a database server
username and password and not a user's own password.

When someone is installing XOOPS, I should think they did it
somewhere, where prying eyes did not look over the shoulder
- and again, it is a one-time process that takes seconds and
then it is over and done with.

I don't find it necessary to change this.

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-04-12 17:39

Message:
Logged In: YES 
user_id=841117

OK, I'll take care of this one personally. 

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1180338&group_id=41586

Bugs item #1196330, was opened at 2005-05-06 04:28
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1196330&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.0.10
>Status: Pending
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Search engines

Initial Comment:
I was just goofing around checking the search engines 
for my site and when I clicked a link to a registerd user 
only section it took me right there I thout maybe 
because I was loged in  another browser so I loged out 
and it still took me to the restricted area.


Just wanted to let you know


Thanks

----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 17:29

Message:
Logged In: YES 
user_id=841117

This is not something I can reproduce and it should not be
connected to where the link came from - XOOPS should check
the permissions regardless.

Can you give me an example of this? (mithrandir [at] xoops
dottie org if you don't want it out in public)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1196330&group_id=41586

Bugs item #979818, was opened at 2004-06-25 17:42
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=979818&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.2.x
>Status: Pending
Resolution: None
Priority: 5
Submitted By: Andrea Rimicci (arimicci)
Assigned to: Jan Pedersen (mithyt2)
Summary: User registration with wrong email

Initial Comment:
Actually, when an user register first time and setup an 
incorrect email address, he has no way to correct this, 
because he can't login, can't retry registration and also 
can't get activation email.

I suggest inactivate users can at least change their 
email address and asking for activation email being 
resent or, more simple, don't mark inactivated usernames 
as 'taken' when someone with same username tries to 
register.

User arimicci is an example. He (I...) forgot to 
add "users." on the right of the @...


----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 17:26

Message:
Logged In: YES 
user_id=841117

After some consideration, I think this is a bad idea.

Moving the check of doublets to when the user is activated
instead of when the user registers can lead to unexpected
results - or at least quite a number of places where this
check will have to be added.

I hold that the user, who mistyped the email address should
have means for contacting the site administrator and the
problem being dealt with this way. I know it is a manual way
of doing things, but I think this is an example where it
should be that way.

----------------------------------------------------------------------

Comment By: Herko Coomans (herkocoomans)
Date: 2005-04-14 16:04

Message:
Logged In: YES 
user_id=441189

Inactive users changing their e-mail addresses is a bad idea.
Not marking inactive usernames as 'taken' is the solution
IMHO. Should also be easy to fix?

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=979818&group_id=41586

Bugs item #983805, was opened at 2004-07-02 03:05
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=983805&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.2.x
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Bender (the_bender)
Assigned to: Jan Pedersen (mithyt2)
Summary: Members: Search results links not Opera compatible

Initial Comment:
When you do a member search and get more results than 
could be displayed on one page you will get links for more 
pages at the bottom. ( like 1 2 3 ... next)

You can click those links in IE 6, Firefox 0.9 fine and get 
to the selected page. However Opera (7.5x) will not work 
with the current solution.  

A click on one of those links results in a very very short 
loading and repeated display of the first results page.

(example link:  http://www.xoops.
org/modules/xoopsmembers/index.php#2)


----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 17:22

Message:
Logged In: YES 
user_id=841117

Changed approach to use XoopsPagenav in the Profile module
in XOOPS 2.2

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-03-25 13:13

Message:
Logged In: YES 
user_id=841117

Will change to use the XoopsPagenav class

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=983805&group_id=41586

Bugs item #1219708, was opened at 2005-06-13 15:48
Message generated for change (Comment added) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1219708&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Core -  Core
Group: XOOPS 2.2.x
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: Andre Pereira (nckdr)
Assigned to: Jan Pedersen (mithyt2)
Summary: Disable an user

Initial Comment:
Should have a way to disable an user. For example, I'm
using Xoops as Intranet, and I need to disable the
users who are fired. Now a days I'm moving these users
to "Anonymous" group. But that users are still showing
in the lists of users of Xoops, like in the private
messages. And I coult not delete these uses without
losing their descriptions and contributions.

Other way to solve this problem is creating a flag to
indicate if the user is enabled and then filter in all
lists of users, which are enabled.

----------------------------------------------------------------------

>Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 10:44

Message:
Logged In: YES 
user_id=841117

Strictly speaking, this is a feature request and not a bug
report.

Anyway, it is added in XOOPS 2.2

----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-13 16:28

Message:
Logged In: YES 
user_id=841117

The flag already exists, but it is designed for
not-yet-active users and therefore it is possible for the
user to request resending of activation email (new feature
in XOOPS 2.2) and re-activate his account.

Will see what I can do about it for XOOPS 2.2, but may be
that it is not until 2.4 that this is fully implemented in a
satisfactory manner.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1219708&group_id=41586

Bugs item #1227330, was opened at 2005-06-25 02:17
Message generated for change (Settings changed) made by mithyt2
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227330&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: CVS Nightly
Group: XOOPS 2.2.x
>Status: Closed
>Resolution: Fixed
Priority: 5
Submitted By: French Referentiel Xoops-France (xoopsmods)
Assigned to: Jan Pedersen (mithyt2)
Summary: blocs can be displayed more than 1 time

Initial Comment:
Hi, for the CVS Nightly, blocs can be displayed more 
than 1 time !

For example, if i choose "search bloc", and selected :
- top page
AND
- All front side pages
, we can see 2 same bloc !

I know that it's not a correct selection, but the system 
could verify and if i choosed "all", the others selected 
lines must be ignored !

Same trouble on a module with "all module pages" and 
any pages of this module.




----------------------------------------------------------------------

Comment By: Jan Pedersen (mithyt2)
Date: 2005-06-25 10:25

Message:
Logged In: YES 
user_id=841117

Fixed in CVS and waiting for review before merging to the trunk.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227330&group_id=41586

Bugs item #1227330, was opened at 2005-06-25 02:17
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227330&group_id=41586

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: CVS Nightly
Group: XOOPS 2.2.x
Status: Open
Resolution: None
Priority: 5
Submitted By: French Referentiel Xoops-France (xoopsmods)
Assigned to: Jan Pedersen (mithyt2)
Summary: blocs can be displayed more than 1 time

Initial Comment:
Hi, for the CVS Nightly, blocs can be displayed more 
than 1 time !

For example, if i choose "search bloc", and selected :
- top page
AND
- All front side pages
, we can see 2 same bloc !

I know that it's not a correct selection, but the system 
could verify and if i choosed "all", the others selected 
lines must be ignored !

Same trouble on a module with "all module pages" and 
any pages of this module.




----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=430840&aid=1227330&group_id=41586