Joomla Templates Club 406,261 posts

[giuliano m Friend giuliano mJoin date: August 2008Posts: 117Downloads:29Uploads:33 Thanks: 91 Thanked: 14 times in 4 posts]

#185579

I updated the template to version 1.0.8 works, but I have this error only in certain categories of items.

Warning: Invalid argument supplied for foreach() in /home/mysite/templates/ja_wall/html/com_k2/ja_wall_blog/item.php on line 278

with firebug

<?php
//Check extrafield value as grid-double or grid-triple
if(count($this->item->extra_fields)):
foreach ($this->item->extra_fields as $key=>$extraField){
if ($extraField->value ==’grid-double’ || $extraField->value ==’grid-triple’) {
unset($this->item->extra_fields[$key]);
}
}

endif;
?>

but I do not understand what is the problem…

[giuliano m Friend giuliano mJoin date: August 2008Posts: 117Downloads:29Uploads:33 Thanks: 91 Thanked: 14 times in 4 posts]

#485641

I solved, I think,:)

in the category of k2 options (Item view options) , the Extra Fields must be selected. (strange first it was not necessary)

---

Attachments:

1. 

3 users say Thank You to giuliano m for this useful post

1. JA Developer
2. Saguaros
3. jsventer

[jsventer Friend jsventerJoin date: April 2010Posts: 19Downloads:0Uploads:0 Thanks: 7 Thanked: 1 times in 1 posts]

#495223

This can’t be the solution. What if we don’t want to show the extra-fields?

[Wall Crasher Developer Wall CrasherJoin date: December 2011Posts: 1113Downloads:0Uploads:15 Thanks: 66 Thanked: 361 times in 300 posts]

#495276

Hi,

We will fix this issue on next release of JA Wall.

For better, you can remove the block code above and add a simple line code as below in the next loop.

<?php foreach ($this->item->extra_fields as $key=>$extraField): ?>
<?php if($extraField->value =='grid-double' || $extraField->value =='grid-triple') continue; ?>
<?php if($extraField->value != ''): ?>

Hope it helps,

Regards

[jsventer Friend jsventerJoin date: April 2010Posts: 19Downloads:0Uploads:0 Thanks: 7 Thanked: 1 times in 1 posts]

#495969

Hi,

The problem is in the foreach statement, and it is probably related to the umpteenth deprecation from the glorious PHP developers.
I think the solution might be the one that stackOverflow suggest:

if(is_array($this->item->extra_fields)):
foreach ($this->item->extra_fields as $key=>$extraField){
if ($extraField->value ==’grid-double’ || $extraField->value ==’grid-triple’) {
unset($this->item->extra_fields[$key]);
}
}

endif;

Regards:

Javier

[Wall Crasher Developer Wall CrasherJoin date: December 2011Posts: 1113Downloads:0Uploads:15 Thanks: 66 Thanked: 361 times in 300 posts]

#496001

Hi jsventer,

Yes, the if condition will make the foreach safe. But we do not need to run the loop twice. That why I move to a single loop and it will gain a bit(very very small) performance.

Regards

[Author]

Posts