isn't this a dupe of #264?

not a blocker for 0.4 but should be done in the next release

For consistency, I think:

* ArchiveProcessing_Day.compute
* ArchiveProcessing_Period.compute

Should be renamed to:

* ArchiveProcessing.Day.compute
* ArchiveProcessing.Period.compute

re: comment:5 - this would break third party plugins like GeoIP, UserLanguage, and EntryPage

re: hook documentation

Assuming it's possible with phpdoc, would it be easier if all standard hooks were defined in one place, e.g.,

class Piwik_Event
{
    /**
     * Authenticate API request
     *
     * @param $token_auth
     * @see Piwik_View::factory()
     * @link https://github.com/piwik/piwik/blob/master/core/API/Request.php#L107
     */
    const API_REQUEST_AUTHENTICATE = 'API.Request.authenticate';

    // ...
}

vipsoft, the problem with pre-defining hooks is that you then have to write them twice in the source code. It is better if you just can add the hook in one line. A simple grep script with pre-defined comment format would do the trick to automatically generate the documentation, wouldn't it - or should we use phpdoc?

The flip side is that misspelling a hook generates an error; whereas a typo in a string would go undetected.

For the automatic hook documentation, maybe we can use Zend_Reflection docbook retrieval features: http://framework.zend.com/manual/en/zend.reflection.reference.html

Vipsoft, you are right, your solution would work well, excuse my previous comment.

It would need to work also for hooks triggered in the Plugins code. Plugins can not add their hook to the core hook class, as technically they are loosely coupled. We would also need a way for plugins to define their own hooks, using the same class interface.

This solution would also make it easier to spot names that don't follow the naming standard.

It also makes #264 a lot easier of course, as we just need to parse the core hook class, and all the plugins hook classes.

Hook names make sense in Piwik atm, appart from ArchiveProcessing Anthon pointed out, but already used in plugins. Closing