Merge remote-tracking branch 'joomla/staging' into 3.4-dev
[imp][33293] Apply Authorised Groups changes through the backend to the logged in user in the frontend without re-login (Fix #3972)
[imp][33978] Bad views on mobile devices (Fix #3933)
[bug][32989] with escape in tooltipText(), entire result should be escaped (Fix #3224)
JFormField classes can't test standalone due to DB dependency
Merge pull request #3547 from pjwiseman/33685_featured_articles_order

[#33685] Featured article order now orders by a tertiary date
using $this->getState('article.id') instead of $id
Merge pull request #4062 from infograf768/multilang_editarticle

[#34037] multilanguage: cannot update an associated article from the frontend
JFormField test classes need database connection
Update install.xml
Update install.xml
Adding cases to emailcloak
Merge pull request #4068 from zero-24/patch-3

Add new files to the install.xml
[fix] Address issues with current PHP 5.6 RC. Fixes #3961

Add PHP 5.6 to Travis config

Add PHP 5.6 to skipped tests with unserialize issues
Merge pull request #4069 from infograf768/mailcloaknewcases

Regression: Adding cases to emailcloak
( ! ) Parse error: syntax error, unexpected 'endif' (T_ENDIF) in
ROOT/administrator/modules/mod_feed/tmpl/default.php on line 111
Merge pull request #4049 from osignell/staging

Tracker #34010 fix
Revert "Updated PR - Fixed wrong call trigger after render component #2095 by brunobatista"

This reverts commit 718da9070f5925699eb683cc55d99c51e38b333a.
JAccess::checkGroup() doesn't work without an $asset

# Executive summary

JAccess::checkGroup() does not work when the optional third argument $asset is not defined. This is in stark contrast with the docblock which states that it "Defaults to the global asset node."

# Technical explanation

The root caused is a botched copy-pasted if-block spanning lines 144-147. It was copied from lines 101-104 except the last line `$asset = $rootId` was never copied. As a result, whenever $asset is empty we look into the `#__assets` table, get the root asset ID... and never use it. This causes null to always be returned, even if the given group does have the requested global privilege enabled.

The solution is to merely change line 144 from `$rootId = $assets->getRootId();` to `$asset = $assets->getRootId();` as it was intended.

# Testing

Unless you are using the affected method in your own custom code you will not be able to reproduce this issue. Apparently everyone was avoiding to use checkGroup because it's buggy but nobody figured out why.
Merge pull request #4073 from nikosdion/patch-1

JAccess::checkGroup() doesn't work without an $asset