Allow the openURLJob to run executables

Otherwise, excecutable files like AppImages won't be allowed to launch
even if they are correctly marked as executable.

BUG: 429603
FIXED-IN: 20.12

Places panel: highlight place only when it is displayed

BUG: 156678
FIXED-IN: 20.12

PlacesItemModelTest: consider also Pictures/Music/Videos folders

See a371e23b2b667bc85c311d04549f3d7348d00af5 in kio.

PlacesItemModelTest: use helper method to increase indexes

Instead of duplicating the logic all over the place.

Select last visited folder when going up/back

This commit fixes a regression introduced by
ad5d3367c7fef4c3c11188a768f21a2ee2b3c025. A lot of connections were
changed there and at least one of them was wrong. The connections can
be separated into GUI connections and internal ones. This commit makes
it so the GUI stuff is connected to m_urlNavigatorConnected and
everything else to m_urlNavigator.

BUG: 429097

GIT_SILENT made messages (after extraction)

Re-allow icons that are not theme icon

Particularly concerned are .desktop files.

See also:
15baa93640bbb49162f26d439e006729ee9c3441

BUG: 429113

Allow having the UrlNavigators below the tab bar

This commit restores the possibility to have the UrlNavigators below
the tab bar. This will happen automatically whenever the UrlNavigator
is removed from the toolbar.

It is also now again possible to have the toolbar on the side. This
option is disabled while the toolbar contains the UrlNavigators.

This commit makes no changes to the new default which is having the
UrlNavigators in the toolbar but makes sure that upgrading users won't
be affected.

PlacesPanel: tooltip, use toLocalFile instead of path

To avoid Windows compatiblity issues.

Increment kpartgui version

The defaults changed in 37327c9b0aae112c5890703cba1f0157043007e0 but
I forgot to also bump the kpartgui version so this commit rectifies
that.

Context Menu: Only add paste action to folders

Either the viewport, or the selected folder is used as destination.
The paste action is only added when enabled.

Fix navigator alignment for right-to-left localizations

Fix qrc icon

Apparently the icon was not null, because the mimetype was known.
But there was no icon associated with it and we got an
icon which is not null, but has a null pixmap.

Set empty value for packagekit transaction flags

Otherwise the value would be Trusted Only, which will
forbid installing local packages

Places panel: show a tooltip after 500 ms

BUG: 426455
FIXED-IN: 20.12

Adress the third round of Angelaccio's review comments

Additionally remove some redundant code concerning UrlNavigator visuals.

Adress most of the second round of Angelaccio's review comments

This commit applies most suggestions which were made on the MR.
Most notably the DolphinUrlNavigator class is split up which leads to
the creation of a DolphinUrlNavigatorsController class.

Additionally some minor coding style and const correctness changes are
included.

The error value of cached integers is changed from -1 to INT_MIN
because situations could come up in which -1 would be a valid value.

Apply Méven Car's suggestions

Fix a crash and extract unrelated changes

The secondary UrlNavigator is now created when and only when:
- split view mode is activated for the active tab
OR
- switching to a tab that has split view already enabled.
This fixes a crash that occurs when the setting to always start in
split view mode is enabled.

An animation for activating split view is also removed from this and
moved into a separate MR. Another unrelated name change left over from
a previous commit (viewContainers() -> activeViewContainers()) is
dropped.

Make UrlNavigators in the toolbar the only option

The UrlNavigators will be automatically added to the toolbar. The Sort By
action is removed from the default toolbar to make space.

Remove all options to have UrlNavigators outside the toolbar and remove
those code paths.

Make it so the new NavigatorsWidgetAction contains two UrlNavigators when
in split view mode. Spacing was also added to align these UrlNavigators
with the ViewContainers when enough space is available.

Force the toolbar to be either at the top or bottom of the window.

Set a sane sizeHint for DolphinUrlNavigator. It would be better to do this
in KUrlNavigator in the future.

This commit also contains a changes which should be moved to a separate
merge requests before this gets merged:
- Add an expansion animation when split view is enabled by the user

Add the UrlNavigator to the toolbar automatically if needed

This commit adds the DolphinUrlNavigatorWidgetAction::addToToolbarAndSave()
method which changes the users toolbar configuration to contain an
Url Navigator. This way the user doesn't need to do anything manually.

Aside from that a bunch of minor fixes like renaming and reordering

Adress the first round of Angelaccio's review comments

- Split the viewContainers(bool includeInActive) into two methods
    without parameters
- Prevent users from accidently hiding all Url Navigators by
    preventing the dangerous action and then displaying a helpful
    message instead
Unrelated to review comments: Remove a useless line of code

trailing whitespaces

Add an option to use an UrlNavigator in the toolbar instead

This commit adds a locationInToolbar KToggleAction to switch between
using a location bar to navigate or using a new custom QWidgetAction
in the toolbar.

A big portion of this MR is refactoring because until now the
UrlNavigator was tightly intertwined with the DolphinViewContainer.
With this MR an UrlNavigator for controlling a View can be freely
connected or disconnected with a single method call.

A DolphinUrlNavigator class is created in the process which contains all
Dolphin-specific UrlNavigator code which did previously reside in the
DolphinViewContainer class. Other application parts that belong to
UrlNavigator-management are also moved here.

PlacesItemListWidget: Use Solid to check if device is a network share

Add missing reference

Iterate over a const copy list of containers

This effectively reverts the mutable iterations approach on
2448f88c5f42d7a2040fcf3bcd3c5f2a2f62cd03, and fix crashes
and ghost items when using the filter bar

BUG: 428374

GIT_SILENT Upgrade release service version to 20.11.80.

Do not expose KCMs in application launchers

We still need to build them, because they
are also loaded on konqueror.

KDirectoryContentsCounter: fix support of symlinks dir

BUG: 428712
FIXED-IN: 20.12

Add support for KUserFeedback

This commit introduces KUserFeedback in dolphin with some basic data
sources and with a settings page to configure the telemetry values.

There are also a couple custom data sources as proof of concept: a bunch
of settings and the count of available network shares as listed by Solid.

The settings page is shown only if the user feedback framework is
enabled, but currently in Plasma we don't have a global kill switch to
disable it.

At the moment we never show an encouragement message. We need to connect
to the `Provider::showEncouragementMessage()` signal, but first we
should agree to a common way to show a non-annoying message to the users.

Details view: display empty text instead of unknown for sizeless dirs

BUG: 420037
BUG: 420040
FIXED-IN: 20.12

Move m_textInfo to fix ctor warning, and document why it's protected.

Use QDeadlineTimer instead of QDateTime.
Use early returns instead of deeply nested if/else.

Cleanup imports

Remove wip debugging

Fix freeSpaceInfo cache not being reset.

Use KDE GitLab url instead of the GitHub mirror

Cleanup unused colors. Use const. Use KColorScheme danger.

The draw code is more like the last state of D21312 now.

Refactor capacity by to use async KIO FreeSpaceJob.

Based on this kio filepicker patch:
https://github.com/KDE/kio/commit/933887dc334f3498505af7a86d25db7faae91019

Only show capacity bar for devices (has a udi).

Only check if place url is mount point once during polish event.

The capacity bars will no longer update when capacity changes. It only
checks the size on init.

Updating during polish event is only a temporarily solution just to
draft code flow for seperating logic from painting. Eventually this
logic update should be in a worker thread detached from blocking the
rendering thread that blocks launching the dolphin window.

Draw 2 Rectangles below mount points to represent a disk capacity bar

Draws overtop text that's goes below the baseline like a 'j'. It's
fairly difficult to add extra space below the text as the places view
is a "details" view, so the text is vertically centered in the
widget's "row" as you'd desire it to in the main file view.

We can make the widget 8px taller, adding 4px above and below so
there's adequate room for the capacitybar, but that wastes 4px for
each mount point.

Ideally we'd use the "compact" view, which displays the text/size in a
column beside the icon. However the compact view was not designed to
take up the entire width of the viewport. It's also designed to
overflow with a horizontal scrollbar.

Improve padding in the Places View

Revert "D-Bus activation systemd service"

This reverts commit eb441b11885fc44dcba1bef693b95819657ed753.

We'll push it again after we branch 20.12. We don't want to push a too
recent version of KF5 on distros.

!27

Compute text width taking into account the zoom level

This makes the icon view less crowded.

SVN_SILENT made messages (.desktop file) - always resolve ours

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

FolderPanel: prevents scanning directory tree recursively

KFileItemListView contents are periodically scanned by KFileItemModelRolesUpdater.
It uses then KDirectoryContentsCounter to scan directories to determine their size possibly recursively.

Introduce a scanDirectories setting to disable directory scanning by KFileItemModelRolesUpdater.

BUG: 426617
FIXED-IN: 20.08.3

GIT_SILENT made messages (after extraction)

Fix PlacesItemModelTest

Since 3dfa0c1c2f87bf1e6bc9 in kio timeline:// is no longer in the places
model.

servicemenuinstaller: use Q_GLOBAL_STATIC

Global statics should be avoided when possible.
This fixes a clazy warning.

Port away from deprecated QWheelEvent::delta()

QWheelEvent::angleDelta() returns a QPoint instead of int, so we need to
use its y() value to get the number of steps.

Use QString::rightRef() as suggested by clazy

make sure we use valid icons

Fix warnings about scaling pixmaps

GIT_SILENT Update Appstream for new release

(cherry picked from commit 92d41a743399f6f38ff55b9cb59657f5c5a5a510)

Port from QStandardPaths::DataLocation to QStandardPaths::AppDataLocation

They are the same on Linux (but DataLocation is deprecated). On Windows this enables the use of the roaming path.

NO_CHANGELOG

Disable session management for daemon

We don't need to autostart the daemon if it was running at logout.

BUG: 417219
FIXED-IN: 20.12

Fix wrong parameter in function call

In KDirectoryContentsCounterWorker::CountResult, the first parameter in
the call of walkDir was wrong. It should be path without conversion,
because the conversion is done in the called function.

BUG: 428282

Use mutable iterators where required

Compile with QT_NO_KEYWORDS

Compile without foreach

DolphinTabWidget: prefer QUrl::toDisplayString() for tooltips

This is better for remote urls.

!35

KItemListSizeHintResolver: remove no longer used member

Don't session-restore invalid paths

When session restore is populating the main window, it's unconditional;
you'll get whatever was there before. This can be a problem if any of
those things are now missing. For example, maybe you were browsing files
on a removable disk, then quit Dolphin, and finally removed the disk. The
next time you launch Dolphin again, it will try to show you the
view from the now-missing removable disk.

To prevent this, we now look at all the URLs in all of the view
containers that were created after session-restore has finished doing
its thing; if any of them are invalid local URLs, we change the URL to
the home folder instead to avoid showing the user a view with an invalid
location in it.

BUG: 427619
FIXED-IN: 20.12

Show home folder if needed after unmounting mounted disk

Right now, when you unmount a device that any active view containers are
displaying, nothing in the view changes. As a result, it's possible to
try to navigate to files or folders in that view, which cannot be done
because the disk that the files or folders are located on has been
unmounted!

With this commit, we detect that case and switch the view containers
to show the home folder after the disk whose contents they are displaying
gets unmounted.

BUG: 158934
FIXED-IN: 20.12

Fix DolphinMainWindow::viewContainers() to take into account split views

The function was previously assuming that each tab view has exactly one
view container, which is not necessarily accurate; some could have a
split view open.

If include is define in .h remove it if it's defined in .cpp too (scripted)

D-Bus activation systemd service

Correctly scopes dolphin in systemd service `app-org.kde.dolphin.service` when dbus-activated (while still working if systemd is not present), as per https://systemd.io/DESKTOP_ENVIRONMENTS/

uses new ECM modules:
- https://invent.kde.org/frameworks/extra-cmake-modules/-/merge_requests/5 to simplify the systemd service templating + install
- https://invent.kde.org/frameworks/extra-cmake-modules/-/merge_requests/17 to generate the dbus service file automatically

needs https://invent.kde.org/frameworks/extra-cmake-modules/-/merge_requests/25

Remove not implmemented method

GIT_SILENT made messages (after extraction)

Port away from QTimeLine::setCurveShape()

As recommended, we now set `easingCurve` directly.

Stop using QVariant < operator

It will be removed in Qt 6 without porting strategy:
https://github.com/qt/qtbase/commit/f43cb31ba00a431c6d0a0b17750483a72ae03bb0

We know that that variants will be either ints (for `count`) or longs (for `size`),
so just convert them to longs (to avoid overflows) and compare those.

Drop obsolete version check

We require Qt 5.14 now.

Port to non-deprecated QLabel::pixmap()

Quoting from:
https://github.com/qt/qtbase/commit/714409b23ce5ec33e56adce5ba1966aad67c3b34

```
The QLabel changes to the pixmap/picture getters provide the following
migration path:

QPixmap *ppix = l->pixmap(); // up to 5.15, warns in 5.15
QPixmap pval = l->pixmap(Qt::ReturnByValue); // new in 5.15, works in 6
QPixmap pixmap = l->pixmap(); // from Qt 6 on
```

Since we can't require 5.15 yet in dolphin, the port is done only when
building with Qt >= 5.15

Port to non-deprecated QString::split() variant

This requires Qt 5.14, which we now depend on.

Port away from QList::toSet()

The QSet range constructor requires Qt 5.14.

In `DolphinView::slotHeaderContextMenuRequested()` the conversion from
QList to QSet was pointless, so we just use a QList now.

Add tooltip to tabbar

Show "Open With" menu items even for empty dirs

We found some legitimate use cases for this after all! :)

This reverts 14f0cd52f61de7539e2d9c751966594708ed0281

BUG: 427573
FIXED-IN: 5.12

When restoring former window state, append any URLs passed as args

Before, when Dolphin was closed but opened with any URLs, it would open
a new window with only those URLs, rather than restoring any window
state (if the user has this feature enabled).

Now, Dolphin will first restore state (if enabled) and then append any
URLs, rather than replacing the existing set of URLs.

The "new instance" case was previously handled automatically and now
needs to be explictly handled by checking for it, to make sure that new
instances give you a clean window rather than a state-restored window.

BUG: 427274
FIXED-IN: 20.12

GIT_SILENT made messages (after extraction)

Update README.md to gitlab :D

Merge branch 'release/20.08' into master

When an invalid protocol is used, reset the navigator location to the previous url

BUG: 413478

Don't trigger rubberband with back/foward mouse buttons

Right now the back/foward mouse buttons work, but also if you hold either
of them and move the mouse the rubberband selection is triggered.

This is the correct fix (instead of commit d0c71a1435bc9d), to match the
original code before the touch events patches,  we need to check the
back/foward buttons after mouseButtonPressed is emitted and before
triggering the rubberbad selection.

Don't warn when closing multiple tabs if restoring window state

When we're using the "restore window state" feature, there's no need to
warn when closing multiple tabs, because they're saved and you can just
re-open the window.

GIT_SILENT Update Appstream for new release

(cherry picked from commit 38f32d52326bc39b9e27319e9f4e89f94a09bb77)

GIT_SILENT Update Appstream for new release

GIT_SILENT Upgrade release service version to 20.08.2.

GIT_SILENT made messages (after extraction)

GIT_SILENT made messages (after extraction)

Fix build failure after last commit

Allow interaction with folder/files with the stylus again

This Patch works for Wayland and X.

BUG: 426582

SVN_SILENT made messages (.desktop file) - always resolve ours

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

Select/Unselect dialogue: Retain a history of entries made there

It's a very useful facilitity, and having a history of entries and being
able to recall earlier ones makes it even more useful for repetitive
operations.

SVN_SILENT made messages (.desktop file) - always resolve ours

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

Make "Show hidden files" action follow HIG

The action is a toggle action, for which the HIG prescribes that the
text should describe the action's checked state, and neither the text
nor the icon should change when the action is checked. Other toggle
actions in Dolphin follow these rules, but this one does not.

This commit makes the action follow those rules.

Check protocol Class before creating a StatJob

StatJob::mostLocalUrl only works with ":local" protocols, adjust the code
accordingly.

Make the code async.

Remove activeContainerLocalPath() method as it isn't needed anymore.

Drive-by change: minimum required version of Qt should be 5.12 because
that's what KF >= 5.73 already requires.

Fix mouse back/forward buttons

We need to emit mouseButtonPressed (now in the onPress method) before the
check needed to not select items on back/foward mouse button press events.

Exclude trash settings from windows