Desperately need help

[millera]

Hello all,

Let me give you a brief synopsis of what happened and why I had to create my own addon. Our local MLS does RETS, however, they put all the agents/office's contact data in a different database separate from all the MLS/property data. So, when having to display a listing page, I had to create my own addon to pull the office name.

So, about two months ago I made my own addon and it works perfectly. If you go to a listing page on www.thebeachsales.com you will see that it shows the Office's name somewhere in the middle of the page. Now, I do use the ShowResults addon where I can put a list of results on another page other than the main search page. I just found out we need to post the office' name there as well. Well, whenever I put in my addon I created in the template, on a regular search it pulls the first office's name and then uses this on all the rest of the listings on that page as well. Additionally, whenever you go to one of the pages where I used the ShowResults plugin, it only displays the tag of my addon "{addon_showofficename_display}" instead of an office name at all.

I wanted to show you my code and see if maybe I'm doing something wrong?

function showofficename_display() {
        global $config, $conn, $jscript;
	require_once($config['basepath'].'/include/misc.inc.php');
	require_once($config['basepath'] . '/addons/showofficename/language/en/lang.inc.php');
	
	$misc = new misc();
	$display = '';
	$listingID = $_GET['listingID'];
	$sql = "SELECT listingsdbelements_field_value FROM " . $config[table_prefix] . "listingsdbelements WHERE listingsdbelements_field_name='LO_CODE' AND listingsdb_id = $listingID";
	$recordSet = $conn->Execute($sql);
	
	if ($recordSet === false) {
		$misc->log_error($sql);
	}
	
	$officeID = $misc->make_db_unsafe ($recordSet->fields['listingsdbelements_field_value']);
	
	$mysqli = new mysqli('localhost','*****','*****','*****');
	$results	= $mysqli->query("SELECT * FROM offices_bcar WHERE officeid='$officeID'");
	if (!$results)
		$display = "Error fetching Listing Office name.";
	else {
		$row 		= $results->fetch_object();
		$display	= ucwords(strtolower($row->officename));
		
		if (empty($display))
			$display = "Couldn't find listing office name.";
	}
	
	$mysqli->close();
	return $display;
}

I am using OR v. 2.8 -- yes, I know I should upgrade, but I'm not ready to move to 3.0 just yet. I would have to be able to spend a lot of time in case of errors/bugs etc.

Your help would be highly appreciated.

UPDATE: Here is what it looks like on just a regular search (it takes the first company name then puts that company name on all listings):


Here is what the search results look like on pages other than the main search page where I use the ShowResults addon:

[the_sandking]

You need to show us the

XXXX_run_action_user_template()

function. Your template tag isn't being parsed, so the problem is "further up" than your showofficename_display() function.

[millera]

Believe it or not I have nothing in that. The reason for this is because I was using another addon as a template for how to create my own. I looked over the documentation and it really doesn't state what this section is for. Should I be using a GET to request the listing ID or something?

By the way, thanks so much for your quick reply.

[the_sandking]

Believe it or not I have nothing in that. The reason for this is because I was using another addon as a template for how to create my own. I looked over the documentation and it really doesn't state what this section is for. Should I be using a GET to request the listing ID or something?

By the way, thanks so much for your quick reply.

Sorry, I meant the XXX_run_template_user_fields() function.

This is what initially parses your custom template tag.

Something like this should exist at least:

function XXX_run_template_user_fields($tag = '') {
        switch ($tag) {
            case 'addon_showofficename_display':
            $data = showofficename_display();
            break;
            default:
            $data = '';
            break;
        } // End switch 
        return $data;
    }

Replace XXX with the name of your add-on.

[millera]

Here's what mine says:

function showofficename_run_template_user_fields($tag = '') {
	switch ($tag) {
		case 'addon_showofficename_display':
			$data = showofficename_display("display"); // TAG: {addon_showofficename_display}
			break;
		default:
			$data = '';
			break;
		}
	return $data;
}

[the_sandking]

Your function

function showofficename_display()

Does not accept input, yet in your run_template_user_fields function, you are passing it "display"

$data = showofficename_display("display");

Your server error logs are probably full of stuff telling you what is wrong, and you have error_reporting turned off if you are not seeing php errors in your browser.

Turn your error reporting up all the way and start examining your server error logs.

[millera]

Hey King,

Even removing "display" from being a variable of the function it doesn't fix both issues. :( I'm just so confused and why it's not parsing it correctly or how it even duplicates itself for each one?

[the_sandking]

What do your server error logs show? You should always be looking at these when you are developing/debugging code.