fix(entities): reverts to 1.8 behavior of ElggEntity->subtype reads

The subtype property was never properly implemented to return the string subtype
but rather returned the integer ID of the subtype. Although this was "fixed" before
this commit, it was decided that it was dangerous to start returning strings where
older plugins might be expecting integers. This deprecates reads of the property
altogether (encouraging the use of getSubtype()) and otherwise returns the behavior
of the property in Elgg 1.8.

Fixes #5920
fix(ui): use correct logo in favicon

Fixes #6446
feature(ui): add svg favicon

Adds svg favicon

Refs #5445
feature(ui): adds fallback png favicons

Adds png fallback favicons

Refs #5445
Merge pull request #6466 from hypeJunction/favicon

Fixes favicon color, adds svg favicon
Merge pull request #6332 from mrclay/5920

Fixes #5920: Returns 1.8 behavior of ElggEntity->subtype read