I'm having the following error when I'm viewing my nodes:

Warning: Invalid argument supplied for foreach() in nodeviewcount_get_node_view_modes_count() (line 57 of C:\wamp\www\MYSITE\sites\all\modules\nodeviewcount\includes\nodeviewcount.api.inc).

I solved it by changing lines 56-60 to

  if (is_array($node_view_modes)){
	  foreach ($node_view_modes as $node_view_mode) {
		if ($node_view_mode !== 0) {
		  $node_view_modes_count[] = $node_view_mode;
		}
	  }
  }

Can this be fixed in the next update?

Comments

Jeroen94’s picture

Jeroen94 commented
Issue summary: View changes
Eranga’s picture

Eranga commented

This will fix the issue.

if(isset($node_view_modes)){
	foreach ($node_view_modes as $node_view_mode) {
	    if ($node_view_mode !== 0) {
	      $node_view_modes_count[] = $node_view_mode;
	    }
	}
  }
awolfey’s picture

awolfey commented

The variable_get needs a default value. This should take care of it.

function nodeviewcount_get_node_view_modes_count() {
  $node_view_modes_count = array();
  $node_view_modes = variable_get('nodeviewcount_node_view_modes', array());
  foreach ($node_view_modes as $node_view_mode) {
    if ($node_view_mode !== 0) {
      $node_view_modes_count[] = $node_view_mode;
    }
  }
  return $node_view_modes_count;
}
marco.machado.santos’s picture

marco.machado.santos commented

Thank you awolfey

  • novartema committed 895dbbc on 7.x-2.x 
    Fixed issue #2453377. issue #2417887 by awolfey: add default value to...
Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Fixed

Hello everyone.

Thank you Jeroen94 and awolfey for your solutions.

Best Regards,
Artem.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)