Hack to avoid sending multiple watchlist notifications on the same view (doesn't really fix bug 1879)

Change paths that were referring to view/view.php?id=..&artefact=.. to instead point to view/artefact.php?artefact=..&view=..

This is related to the viewartefactsplit branch. Now the existing code actually uses the new artefact.php script.

A related bug, #1758, has been opened for artefacts sending notifications.

Also changed the activity code for watchlist to look for $data->message rather than $data->subject, which makes the code in there a bit more readable.

Signed-off-by: Andrew McMillan

giant refactor of activity to pave the way for plugins to have their own activity types.  preliminary testing fine. needs more testing & some more changes but I think we're about back to where we were before I started hacking (with nice oo added)

Changed all requires.
Also removed not used json/renderartefact.php

Renamed the 'view' parameter for view/view.php to be 'id'. 'view' will continue to work for a while.

view/blocks.php uses id, and id is used for other objects in the system, so it doesn't make much sense for this one to remain this way.

Now, table names (and other identifiers that could be confused with SQL kewords), should be put into SQL like this:

SELECT * FROM {artefact}

The braces are matched and expanded by DML now to include the prefix and be properly quoted, which means that tables like 'view' and 'group' don't need renaming (and nor should they, there's little reason why we should rename tables to get around such constraints).

This has removed a whole bunch of $prefix = ... and get_config('dbprefix') stuff, which makes things a little simpler yet again.

Renaming communities to groups step 1: everywhere in the code. Filenames and database changes to come.

Renamed a language string to prevent confusion with another string named the same, but that has a different translation

Wrong branch...

This reverts commit 21fedcd0.

access notifications

in a try/catch (eg in case email fails) and if it does fail, internal
messages don't get marked as read) from stable