Bug #2713

LiveUser_Admin_Perm_Container_MDB_Medium throws error when optional fields used

Details

Submitted	2004-11-07 18:05 UTC
From	bmihelac at mihelac dot org
Assigned	dufuz
Status	Closed
Package	LiveUser
PHP Version	Irrelevant
Roadmaps	(Not assigned)

Comments

[2004-11-07 18:05 UTC] bmihelac at mihelac dot org

Description:
------------
Using addGroup() in LiveUser_Admin_Perm_Container_MDB_Medium container throw's an error: Fatal error: function quote() does not exists.

Problem calls are in the MDB_Medium file, lines 103 and 113:

$val[] = $this->dbc->quote($value, $this->groupTableCols['optional'][$alias]['type']);

$val[] = $this->dbc->quote($value, $this->groupTableCols['custom'][$alias]['type']);

if this two lines are changed to:
$val[] = $value;

it works well. I am not sure if quote is required for MDB inserts, but I think everything would work good without them.

cheers,
Bojan

[2004-11-07 19:10 UTC] bmihelac at mihelac dot org

...after some research, I notice that my previous fix would not work, because without quoting values is_active=TRUE would be saved in database as '1' (and 'Y' should be).

So the new fix could be:
$val[] = $this->dbc->getValue($this->groupTableCols['optional'][$alias]['type'], $value);

Bojan