Bug #1226

wrong pivknode lovk sequence

Details

Submitted	2004-04-16 19:11 UTC
From	priit at e-positive dot ee
Assigned	datenpunk
Status	Closed
Package	DB_NestedSet
PHP Version	4.3.4
OS	win
Roadmaps	(Not assigned)

Comments

[2004-04-16 19:11 UTC] priit at e-positive dot ee

Description:
------------
All create/update functions begin with something like this:

if (!($thisnode = $this->pickNode($id, true))) {
$epr = array('createLeftNode()', $id);
return $this->_raiseError(NESE_ERROR_NOT_FOUND, PEAR_ERROR_TRIGGER, E_USER_ERROR, $epr);
}

if (PEAR::isError($lock = $this->_setLock())) {
return $lock;
}

It's theoretically possible that picknode is successful, but that node get's erased by concurrent user just before lock is set.

[2004-07-23 22:50 UTC] dk at webcluster dot at

This bug has been fixed in CVS.

In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pear.php.net.

In case this was a pear.php.net website problem, the change will show
up on the website in short time.

Thank you for the report, and for helping us make PEAR better.

This was a very good suggestion. Never really thought of that.
Sorry for the delay :)