Tickets :: [#5423] api.php / _kronolith_put require statement and global definition swapped

Summary	api.php / _kronolith_put require statement and global definition swapped
Queue	Kronolith
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	1. Low
Owners
Requester	d.bussink (at) student (dot) utwente (dot) nl
Created	06/01/2007 (6643 days ago)
Due
Updated	06/01/2007 (6643 days ago)
Assigned
Resolved	06/01/2007 (6643 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

06/01/2007 08:59:45 PM	Chuck Hagenbuch	Comment #3 State ⇒ Resolved	Reply to this comment
Fixed in CVS, thanks.

06/01/2007 07:19:15 PM	d (dot) bussink (at) student (dot) utwente (dot) nl	Comment #2	Reply to this comment
In lib/api.php the lines 350 and 351 should be swapped. Oops, I see these line numbers are not exactly accurate (I've fiddled with them myself). The real fix is that line 353 is moved up so it is the first statement of the function (but I guess that should be deducable from my initial bug report).

06/01/2007 07:16:13 PM	d (dot) bussink (at) student (dot) utwente (dot) nl	Comment #1 State ⇒ Unconfirmed Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ api.php / _kronolith_put require statement and global definition swapped Queue ⇒ Kronolith	Reply to this comment
In lib/api.php the lines 350 and 351 should be swapped. 348 function _kronolith_put($path, $content, $content_type) 349 { 350 global $kronolith; 351 require_once dirname(__FILE__) . '/base.php'; 352 $ids = array(); This way, the $kronolith object doesn't contain the correct object, because the global is created in the base.php object. This prevents the _kronolith_put from working correctly.