Fix a few clang tidy/clazy warnings and a few other current issues

Closes #273, #272, #243, and #260

See merge request !228

commit c8fc1491 missed adding code to optionsmodel.cpp implenting
optionType and OptionsModel::allValues(), also ensure we use optionList
for virtual desktops on wayland. x11PropertyHash had outdates entries
update them aswell. fixes crashes when trying to Configure Special
Window Settings on both x11 and wayland.

m_settings was never initialized and can contain arbitrary random data
and cause issues down the line.

m_settings was never being initialized and could potentially cause
issues further down the line having arbitrary random data.

commit fa105ea7 added an second connect to the signal
virtualDesktopsUpdated while it should have simply modified the
currently existing one, this fixes a nullptr crash when trying to access
window rules.

backport of 1b215e6c, EffectQuickView is used by Aurorae.
As long as Aurorae decorations are rendered by kwin, it's not a problem.
However, kde-gtk-config kded module can also render them.
This creates a problem. If effects object is not created,
accessing any of its getters or setters will result in a segmentation fault.

This change rewrites the share context check so the effects object is
not accessed anymore.

One could argue that kde-gtk-config has to provide a dummy EffectsHandler
but it doesn't use effects and libkwineffects is not a dependency of
KDecoration2. So, providing a dummy EffectsHandler makes no sense.

In hindsight, we need to revisit the usage of EffectQuickView in Aurorae
as accessing the EffectsHandler in EffectQuickView is a totally valid
usecase and integration of QOpenGLContext.

Co-authored-by: Vlad Zahorodnii

on wayland we use rgba colors, so set QImage::rgba8888 instead of
::argb32. fixes weird color distortions. also with the right colors set
we dont have to flip and invert the image.

seems a typo has been left in there.
actually check maxsize in maxsize.

fixes popups sometimes being in the wrong stacking order and also the
situation when the dock has an popup and setshowingdesktop gets called
which iterates over the stackingorder and calls update_layer and then
belong_to_layer, now without this it means its entering
win->layer_for_dock() because this popup has windowtype of net::dock but
is an xdgxshell popup without control and hits the assert in there.

add missing includes.

slide effects might delete themself while iterating, and iterating in a
for ranges loop that gets modified is undefined behaviour.

if internal window is not a deleted we call delete_window_from_space
that deletes "this" which means assigning m_internalWindow to nullptr
afterwards is a write to pointer after free, also move delete this, to
not double delete it.

ensures we dont deep copy on each call.

QFileInfo::exists(file) is documented to be faster then QFileInfo(file).exists().

seems focus stealing prevention level was never being set, ensure its being
set.

ensures it doesnt deep copy the container on each call.

dont call .first or .last on temporarys, use constFirst or constLast.
ensures they dont unnecesserly detach/deep copies. found with clang
tidy/clazy.

ensure Qt containers does not detach by using qAsConst, found with clang
tidy/clazy.

add missing references and const, found by clang tidy/clazy.

Store space and input platform in base

See merge request !227

As with space store the input platform in base.

Move the functionality from the output class to a free function template. This
way we can add additional parameters.

This way we don't need to access it via the global getter.

This allows to afterwards template the functions.

This way we don't need to access it via the global getter.

Move the D-Bus interface parts out into a separate object. This way we can
template the manager class.

Prepare templating over the platform by defining selected functions in the
header.

Instead of using Qt's ownership model.

Instead of accessing it via the global input getter.

The includes are not needed as we use templates. We need to fix the includes in
other parts though afterwards.

Getting the platform via a reference but then accessing it as a pointer is
confusing. Better just except the caller to provide a reference to the object.

This way we don't need to access the global getter.

Move the init functions of shortcuts out of the virtual desktop manager class
and access the input platform through the space instead of using the global
getter.

This allows later to make it a class template.

This will allow to call a non-member function on the receiver.

This way we don't have to access the type-unsafe sender function.

The slots and properties are not needed anymore since the D-Bus interface had
been moved to a separate class.

Move it to a free function and add a template parameter on the input platform
to not have to access the global getter.

Instead access it via dependency injection.

This way we don't need to access it via the global getter.