Regression in Trunk: AssetManager.php #1675

There's a regression in trunk. Setting up Piwik locally throws an Exception in https://github.com/piwik/piwik/blob/master/core/AssetManager.php#L371. Cause is the static call to ::isGenerated() which itself isn't declared as static.

Thanks. Do you have any idea which php configuration stopped the exception from being raised on my development box ?

No idea. What setup do you have?

I'm currently using:

• Win7
• Apache 2.2.16
• PHP 5.3.3 + xdebug

Do you have display_errors = On (should be On on a dev. box).

On the other hand - I wonder why removeMergedAsset() gets called in the first place, I'd expect it not to be called when disable_merged_assets = 1 in global.ini.php.

I didn't configure the php strict mode properly. Sorry about that. I will update http://dev.piwik.org/trac/wiki/CodingStandard accordingly.

removeMergedAsset() is always called in development mode (ie. with disable_merged_assets = 1). It ensures merged assets are always regenerated when switching back to production mode (ie. with disable_merged_assets = 0).

In order to see the error, I had to update https://github.com/piwik/piwik/blob/master/config/global.ini.php#L282 to E_ALL | E_STRICT

(In [3072]) fixes #1675

The CI server uses E_ALL | E_STRICT to catch these types of errors.

We should add a note to the appropriate wiki page.