Thread

Using 2.1.4 beta.

The following code generates an error:

{exp:stats channel=“artikel”}{last_entry_date format="{DATE_W3C}”}{/exp:stats}

A PHP Error was encountered
Severity: Warning
Message:  Invalid argument supplied for foreach()
Filename: stats/mod.stats.php
Line Number: 177

Quickly looked at the code and it seems to be related to member counting, not the tag I’m interested (which is printed okay despite the error).

o_cee,

Was the beta a fresh install or an upgrade from EE2.1.3?
Do you have a Freelancer license?

IS that tag nested within another tag?
Can you test it on a completely blank template please?
Is your member module installed?

Pretty sure it was a clean install of 2.1.4 beta.

It is the commercial license.

The tag is not nested, complete code is here:

<?xml version="1.0" encoding="UTF-8"?>
<urlset xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.sitemaps.org/schemas/sitemap/0.9 <a href="http://www.sitemaps.org/schemas/sitemap/0.9/sitemap.xsd">http://www.sitemaps.org/schemas/sitemap/0.9/sitemap.xsd"</a> >
    <url>
        <loc>{homepage}</loc>
        <lastmod>{exp:stats channel="artikel"}{last_entry_date format="{DATE_W3C}"}{/exp:stats}</lastmod>
        <changefreq>hourly</changefreq>
        1.0</priority>
    </url>
</urlset>

Same error with just

{exp:stats channel="artikel"}{last_entry_date format="{DATE_W3C}"}{/exp:stats}

in empty template.

Member module is not enabled, but installing it (enabling it) does not make any difference.

Hi, o_cee. I just tried this with EE 2.1.4 beta on my test-ee site:

http://test-ee.com/index.php/sample/query

I just changed the channel to news, since I’m using the Agile theme.

Hi Sue

I’m not sure how that’s supposed to help me..

/Oscar

Hi Oscar,

I’m afraid that I can’t replicate this error either. Is this being used on a live site with a membership system? Are you using any addons? If so, please let us know what they are.

Cheers

Greg

Hmm..

No membership system. Site not live yet.

Addons: Playa, Channel Images, Channel Videos, Taggable.

I’ll see if I can debug it myself, got any suggestions for what to look for?  Have quite a bit of experience with PHP, but not CI.

Okay, got the reason for it debugged now..

As my error message says, it is line 177 in modules/stats/mod.stats.php. This code block is in a conditional that checks for:

if (count($this->EE->stats->statdata('current_names')) > 0)

The problem is the use of count() and how EE_Stats::statdata returns an empty response:

libraries/Stats.php

/**
 * Get Statdata
 *
 * This method will retrieve items or the entire statdata class property.
 *
 * @param       string          which piece of the array to get (optional)
 * @return      mixed           FALSE on failure, string or array on success
 */

Note that it returns FALSE on failure, which we will get in this case since we don’t have the member module(?). However, count(FALSE) will return TRUE:
http://php.net/manual/en/function.count.php

Which is why the code at line 177 in modules/stats/mod.stats.php is executed at all, when it shouldn’t, thus giving FALSE to the foreach causing the original error I posted.

Hi, o_cee. I turned of the membership module, and still don’t get an error when I post the code you used all by itself in a stand alone template.

I’m using a fairly clean 2.1.4beta install on MediaTemple. Also tested on a local install with 2.1.4 beta. Still no errors. Let me ask my other TSS helpers if they get the same error.

Sue,

What is your “Enable Online User Tracking?” set to?

It’s set to on.

Try turning it off, see if that gives you the error.

Hi o_cee,

Yep, I can replicate this and we’ll get a fix into the next release. For now you can change line 147 (not 177) of system/expressionengine/modules/stats/mod.stats.php to

if ($this->EE->stats->statdata('current_names') AND count($this->EE->stats->statdata('current_names') > 0))

Thanks!

And making sure this is redirected back to the original ticket I created: https://support.ellislab.com/bugs/detail/15599/

Thanks for pointing that out, Oscar! We got it marked as fixed for the next release. I’ll go ahead and close this but please don’t hesitate to post again.