User defined PHP function in custom block

[dagg929]

I've searched everywhere and I can't find a definitive answer to my question. I'm having trouble getting a user defined php function to execute inside of a custom block.

$my_output = 'Bad';
if(!function_exists('good'))
{
function good()
{
$my_output = 'Good';
}
}
good();
return $my_output;

This is just a test code I wrote to see if this was indeed my problem and it was. The output is always "Bad". Any suggestions to get this to work would be appreciated.

Also, do these same issues arise if I define the php code as a plugin? I haven't been able to get it to work that way either and I figured it would be better to define the code in the block anyways.

[tbworld]

I've searched everywhere and I can't find a definitive answer to my question. I'm having trouble getting a user defined php function to execute inside of a custom block.

Where are you currently installing your custom code? Typically custom code is installed via the Product/Plugin system.

[kh99]

You need to make sure $my_output is a global variable. You need a global statement inside the good() function, and you also may need one at the beginning of the plugin code, depending on the hook being used. So try something like:

global $my_output;
$my_output = 'Bad';
if(!function_exists('good'))
{
function good()
{
global $my_output;
$my_output = 'Good';
}
}
good();
return $my_output;


ETA: or you could use a reference parameter or return a value from good(), but probably your actual code is different than the example you posted, so maybe that won't work.

[dagg929]

Thank you for the global suggestion I think that would solve part of the problem, but I don't think that was entirely the issue because I've tried returning straight from the function and it never executed.

I did solve my problem though, I just got rid of the function because it seems like too much of a hassle to try to work around. Was easier to just execute the code where needed.

To answer the first question, in a forum block. It's not a large chunk of code, but I needed to do a call to the database and some manipulating. The function was just for output formatting so I just scrapped it.