Data Manager Programming Problems

[calwebsnc]

Hi,
I'm trying to develop a class to write new threads on vbulletin.

This is my code

PHP Code:

<?php 

// ######################### REQUIRE BACK-END ############################
require_once('./global.php');


    class codeigniter_bridge{
        
        function codeigniter_bridge(){
            
            echo "Create Object <br/>";
            
            $this->threaddm =& datamanager_init('Thread', $vbulletin, ERRTYPE_ARRAY, 'threadpost'); 
            
            echo "Object Created<br/>";
        }
        
        
        function postThread(){
            echo "Call Prova<br/>";
            $class_methods = get_class_methods($this->threaddm);

            foreach ($class_methods as $method_name) {
                echo "$method_name\n";
            }
            
        }
    }

    $vb = new codeigniter_bridge();
    $vb->postThread();
?>

and this is what appens:
http://www.cyberludus.com/forum/codeigniter_bridge.php

Where I do wrong?

Thank you!

You need to pass a valid vBulletin registry object to the datamanager; in your script, $vbulletin doesn't have the proper scope in your method.

Pass $vbulletin through the constructor:

PHP Code:

function codeigniter bridge(&$registry) {
    $this->registry =& $registry;
}

// *** .....

 $vb =& new codeigniter_bridge($vbulletin); 


[James Birkett]

Originally Posted by Danny.VBT

You need to pass a valid vBulletin registry object to the datamanager; in your script, $vbulletin doesn't have the proper scope in your method.

Pass $vbulletin through the constructor:

PHP Code:

function codeigniter bridge(&$registry) {
    $this->registry =& $registry;
}

// *** .....

 $vb =& new codeigniter_bridge($vbulletin); 


Wouldn't you consider it bad practice to pass the variables via referencing considering it's deprecated in 5.3?

Originally Posted by James Birkett

Wouldn't you consider it bad practice to pass the variables via referencing considering it's deprecated in 5.3?

Well I saw he had PHP 4 specific syntax (such as the constructor), so I provided my code with the same OO flavor. If you are running < PHP 5, you have to explicitly pass by reference or else you'll get some funky results.

[calwebsnc]

Ok, now it works!
Thank you.

<?php 

// ######################### REQUIRE BACK-END ############################
require_once('./global.php');


	class codeigniter_bridge{
		
		function codeigniter_bridge(&$registry){
			
			echo "Create Registry <br/>";
			
			$this->Registry =& $registry;
			
			echo "Is registry an object? ".is_object($this->Registry)."<br/>";
			
			echo "Create Object <br/>";
			
			$this->threaddm =& datamanager_init('Thread', $this->Registry, ERRTYPE_ARRAY, 'threadpost'); 
			
			echo "Object Created<br/>";
		}
		
		
		function postThread(){
			echo "Call Prova<br/>";
			$class_methods = get_class_methods($this->threaddm);

			foreach ($class_methods as $method_name) {
			    echo "$method_name<br/>\n";
			}
			
		}
	}

	$vb =& new codeigniter_bridge($vbulletin);
	$vb->postThread();
?>