Use newly-upstreamed rename dialog from KIO
AbandonedPublic

Authored by ngraham on Dec 15 2018, 5:05 AM.

Details

Summary

With the rename dialog upstreamed in D17595, we can use it from there.

This will require the KF5 dep to be bumped to 5.67, which should be feasible given that we're at the very beginning of a new Applications cycle.

Depends on D17595

Test Plan
  1. Dolphin Settings > uncheck "rename inline"
  2. Rename one or more files. Observe that it still works

Tests still pass.

Diff Detail

Repository
R318 Dolphin
Branch
use-upstream-rename-dialog-from-kio (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 6044
Build 6062: arc lint + arc unit
ngraham created this revision.Dec 15 2018, 5:05 AM
Restricted Application added a project: Dolphin. · View Herald TranscriptDec 15 2018, 5:05 AM
Restricted Application added a subscriber: kfm-devel. · View Herald Transcript
ngraham requested review of this revision.Dec 15 2018, 5:05 AM

looks good

src/panels/folders/folderspanel.cpp
107

Could also do KFileItemList{item}

src/views/dolphinview.cpp
639

Why not keep the new connect syntax?

ngraham updated this revision to Diff 72640.Jan 2 2020, 7:05 PM
  • Rebase
  • Fix build failure
  • Update minimum frameworks version
ngraham edited the summary of this revision. (Show Details)Jan 2 2020, 7:16 PM
ngraham updated this revision to Diff 72642.Jan 2 2020, 7:20 PM
ngraham marked an inline comment as done.

Create the KFileItemList inline

ngraham marked an inline comment as done.Jan 2 2020, 7:20 PM
elvisangelaccio accepted this revision.Jan 6 2020, 12:45 PM
This revision is now accepted and ready to land.Jan 6 2020, 12:45 PM
meven added a subscriber: meven.Jan 6 2020, 1:10 PM
meven added inline comments.
CMakeLists.txt
10–11

The new depending KIO version is 5.67 as D17595 missed 5.66

meven requested changes to this revision.Jan 6 2020, 1:13 PM

Please update version mentioned in commit message.
I intend to land D17595 soonish.

This revision now requires changes to proceed.Jan 6 2020, 1:13 PM
ngraham edited the summary of this revision. (Show Details)Jan 6 2020, 3:25 PM
ngraham updated this revision to Diff 72898.Jan 6 2020, 3:26 PM

Fix required frameworks version

ngraham marked an inline comment as done.Jan 6 2020, 3:26 PM
meven accepted this revision as: meven.Jan 6 2020, 3:27 PM

Please wait for D17595 to land

This revision was not accepted when it landed; it landed in state Needs Review.Jan 6 2020, 9:25 PM
This revision was automatically updated to reflect the committed changes.

The Frameworks version in question does not exist, this has broken the build of Dolphin.
See https://build.kde.org/view/Failing/job/Applications/job/dolphin/job/kf5-qt5%20SUSEQt5.12/lastFailedBuild

As Dolphin forms part of the dependencies of other projects and is therefore included in the Dependency Builds, i'm going to have to require you to revert this until David performs the version bump of Frameworks.
The Dependency Builds are currently required to all pass as their has been an ABI breakage in the underlying platform on FreeBSD (requiring them to run, otherwise we have no FreeBSD CI)

ngraham reopened this revision.Jan 6 2020, 9:36 PM

Sorry, I thought the version bump happened a few days ago, but that was for 5.66, not 5.67 (I always forget that frameworks version bumps happen at the end of the release cycle, not the beginning).

Reverted in 41105103b063c2e538bf0071e54fd429a841238b. Will commit again once 5.67 exists.

ngraham abandoned this revision.Jan 11 2020, 8:24 PM

Just pushed this again because KIO 5.67 was tagged today, so landing it won't break the CI this time. Phab won't let me close this, so I guess I have to abandon it for some reason. :/