refs #14691 - user editing self should not change User.current

Rather than changing the behaviour of #to_label to return persisted
data, the User.current object should not be modified with unsaved data
from the form submission or API update.

User.current is used for authz as well as for display purposes, so...

Fixes #14395 - Label for provision template capitalized in os details page

s

Fixes #10605 - handle logout accessed by GET

Fixes #14369 - Removed default scope from host counter in taxonomies

fixes #14502 - fix documentation of the hosts controller for puppetclass_ids

previously it wrongly stated that the parameter is called
puppet_class_ids

fixes #14511 - bookmark editing ensures the query field is populated

Fixes #14503 - don't join reports with the logs and resources explicitly in API

Using `.includes(:logs => [:message, :source])` in the search scope
for reports caused serious performance issues (ending up with server
taking all of its memory). I've found no reason for the `includes` to...

fixes #14482 - close two-pane when saving compute profile attributes

Fixes #14330 - Provide option in hammer to change display type for
libvirt compute resource

Fixes #14341 - List all images for compute resource

Previously, the listing was paginated, but since the table is loaded
via ajax and handled by datatables, the pagination had no effect,
leading to a list limited to the max per page (20 usually).
Also, improved the lookup to only bring the needed data.