PHP Warning: in_array() expects parameter 2 to be array

@linux4me2

I had one of our Support Reps check into this. He was not able to reproduce the issue with the information given. To investigate this further, we’re going to need to get a bit more detailed. If you would, please call/email our Support Team directly so they can dig into this more with you.

Thank you!

@lindzconnell

For some reason, I never got the notice that you replied to this, but when I updated to version 3.4.0 of dsIDXpress and my changes were overwritten, the error returned, though with the new version it is on line 107 rather than 109.

I’ll give Support a call.

@linux4me2

Sounds good! Our team will be happy to help.

I spoke to Tony in Support, and he was unable to reproduce the PHP warning I was getting. We decided I would add some code to the conditional I was using to see if I could get some more info. I ended up using the following on line 107 of /wp-content/plugins/dsidxpress/widget-idx-guided-search.php to output some data to the error log when the warning occurred:

if (is_array($values['idx-q-Cities'])) {
  $selected = in_array(strtolower(trim($city)), array_map('strtolower', $values['idx-q-Cities']))?' selected="selected"':'';
} else {
  // Begin debugging code here.
  error_log('idx-q-Cities is not an array with the following variables:' . "\n" . print_r($_REQUEST, true) . "\nIP:" . $_SERVER['REMOTE_ADDR'] . "\n" . $_SERVER['REQUEST_URI'] . "\n" . $_SERVER['HTTP_USER_AGENT']);
  // End debugging code. 
  $selected = '';
}

I waited a while, and the Bing bot obliged by triggering the debugging code. It turns out I can reliably reproduce the warning by using a URL like:

https://mysite.com/idx/city/cityname/

I tried with two cities that are in our MLS area, and both resulted in 547 entries in the error log. One city didn’t return any results, so the 547 isn’t related to the number of results.

I called Support back and explained what I’d found. Ian told me they would look into it further. I’ll post back when I hear from them.

Hello @linux4me2,

Had some time to look over this, and here’s the issue with the /idx/city/location pages (line ~52):

$specialSlugs = array(
	'city' 		=> 'idx-q-Cities',
	'community' => 'idx-q-Communities',
	'tract' 	=> 'idx-q-TractIdentifiers',
	'zip' 		=> 'idx-q-ZipCodes'
);

$urlParts = explode('/', $_SERVER['REQUEST_URI']);
$count = 0;
foreach($urlParts as $p){
	if(array_key_exists($p, $specialSlugs) && isset($urlParts[$count + 1])){
		$values[$specialSlugs[$p]] = $urlParts[$count + 1];
	}
	$count++;
}

This piece of code handles the pretty URL pages for /idx/city/, /idx/community/, etc. However, what’s happening is on these pages it’s then setting the $values[] array to a string value.

The solution is to replace this line (~63):
$values[$specialSlugs[$p]] = $urlParts[$count + 1];

With this:
array_push($values[$specialSlugs[$p]], $urlParts[$count + 1]);

This will simply push the value to the end of the array, which should be empty.

The number of warnings (not errors) that you get in your logs has to do with how many values you have in the IDX Guided Search widget fields. So 540+ warnings means that you have that many locations in your City field.

Thanks,
Tony.

• This reply was modified 5 years, 9 months ago by dstonyh. Reason: Added line numbers
• This reply was modified 5 years, 9 months ago by dstonyh. Reason: Added explanation of warnings
• This reply was modified 5 years, 9 months ago by dstonyh.

Hi @dstonyh,

That fix works. I appreciate the explanation of the 540+ warnings. There are indeed 547 cities in the cities list of the Guided Search Widget on the site.

I hope this fix can be added to the next release of the plugin. Although they are warnings and not errors, they were racking up some impressive error log file sizes and chewing through the disk space this site is allowed on the server.

Thanks for all your help!