software-properties-{qt,kde} attempts to use KUrl but it doesn't exist

[Kristopher Ives]

Release: Kubuntu 18.04
Version: 0.96.24.32.1

When clicking the "Import Key" button in the "Authentication" section of the panel it throws this error:

    Traceback (most recent call last):
      File "/usr/lib/python3/dist-packages/softwareproperties/kde/SoftwarePropertiesKDE.py", line 667, in add_key_clicked
        url = KUrl.fromPath(home)
    NameError: name 'KUrl' is not defined

While this doesn't crash the program it causes nothing to happen when clicking the button. Running the program in a terminal reveals the error being printed to stderr.

After debugging it appears KUrl doesn't exist anywhere and I'm not sure if's a python module that has changed.

[Kristopher Ives (krisives)]

This came from this Ask Ubuntu post:

https://askubuntu.com/questions/1079793/cannot-add-ppa-key-in-18-04-nameerror-name-kurl-is-not-defined/1079823

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Walter Lapchynski (wxl)]

According to the dupe, this affects more recent versions, too. And not just Kubuntu.

[Walter Lapchynski (wxl)]

There are no PyKDE imports anymore, only PyQt. That said, replacing `KUrl.fromPath` with just `QUrl` should resolve that.

But that's not our only issue:
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/softwareproperties/qt/SoftwarePropertiesQt.py", line 713, in add_key_clicked
    filename = KFileDialog.getOpenFileName(url, 'application/pgp-keys', self.userinterface, _("Import key"))
NameError: name 'KFileDialog' is not defined

So we replace `KFileDialog` with `QFileDialog` and move some arguments around:

    filename = QFileDialog.getOpenFileName(self.userinterface, _("Import key"), url, 'application/pgp-keys')

The only other thing missing is that KFileDialog wanted a KUrl while QFileDialog wants a string. So we should cast our QUrl call:

    url = str(QUrl(home))

Maybe there's a prettier/better way to do this, but that's what I got for 10 minutes which is more progress than I can say this bug has had in the past XD If people can test those changes, it would be greatly appreciated.

[Hans P. Möller (hmollercl)]

this should be the diff to be tested:

diff --git a/softwareproperties/qt/SoftwarePropertiesQt.py b/softwareproperties/qt/SoftwarePropertiesQt.py
index 8172e25..b210323 100644
--- a/softwareproperties/qt/SoftwarePropertiesQt.py
+++ b/softwareproperties/qt/SoftwarePropertiesQt.py
@@ -709,8 +709,7 @@ class SoftwarePropertiesQt(SoftwareProperties):
     home = QDir.homePath()
     if "SUDO_USER" in os.environ:
         home = os.path.expanduser("~%s" % os.environ["SUDO_USER"])
- url = KUrl.fromPath(home)
- filename = KFileDialog.getOpenFileName(url, 'application/pgp-keys', self.userinterface, _("Import key"))
+ filename = QFileDialog.getOpenFileName(self.userinterface, _("Import key"), home, 'application/pgp-keys')
     if filename:
       if not self.add_key(filename):
         title = _("Error importing selected file")

[Michal Dziczkowski (mdziczkowski)]

# diff --git a/softwareproperties/qt/SoftwarePropertiesQt.py b/softwareproperties/qt/SoftwarePropertiesQt.p
diff: unrecognized option '--git'
diff: Try 'diff --help' for more information.

[Walter Lapchynski (wxl)]

Michal, that's not how patching works, unfortunately. You can put all that in a file and use the `patch` command. The easy way is to go to the folder containing the file and run `sudo patch < /path/to/diff`.

Or you could simply edit the file (/usr/lib/python3/dist-packages/softwareproperties/qt/SoftwarePropertiesQt.py) according to the instructions given by the diff:

 1. Starting at line 709, look at it and the following 7 lines for a total of eight lines (this is what "-709,8" refers to in the original)
 2. Note, too, that after the change, the same hunk is referenced as "+709,7" meaning that the end result is one less line.
 3. Remove the definition of the "url" variable in line 712.
 4. In the next line (which is now 712) which defines the "filename" variable, change it from using the KFileDialog.getOpenFileName method to the QFileDialog.getOpenFileName method. Note that the arguments and their order are different.
 5. Save it and try again.

[Walter Lapchynski (wxl)]

P.S. added Hirsute and Groovy tags since the latest dupe shows it affects Hirsute and thus should affect Groovy.

[Ubuntu QA Website (ubuntuqa)]

This bug has been reported on the Ubuntu ISO testing tracker.

A list of all reports related to this bug can be found here:
http://iso.qa.ubuntu.com/qatracker/reports/bugs/1795278

[Leó Kolbeinsson (leok)]

@wxl

This is still occuring on Lubuntu Hirsute daily ISO 2021-03-26.

Tested also Ubuntu Mate and Xubuntu same date and no error.

[Leó Kolbeinsson (leok)]

Confirm that this still occuring in Lubuntu Impish - tested daily ISO 2021-08-04
http://cdimage.ubuntu.com/lubuntu/daily-live/20210804/impish-desktop-amd64.iso

[Leó Kolbeinsson (leok)]

Confirm this is still occuring in Lubuntu Kinetic - tested daily 19-05-2022

Machine tested Lenovo YG SL7 in live session.
http://iso.qa.ubuntu.com/qatracker/milestones/433/builds/248375/testcases/1303/results

[KGIII (uninvolved)]

Yup, still occurring in Kinetic.

Ran a quick test on bare metal and in a virtual machine.

[Erich Eickmeyer (eeickmeyer)]

Targeting this correctly. No clue why this was isn't being prioritized.

[Dave Jones (waveform)]

Switching the dependency over to PyQt (or PySide?) sounds like something for the desktop team to look at?

[Leó Kolbeinsson (leok)]

This is still occuring in Lubuntu Lunar.

[Aaron Rainbolt (arraybolt3)]

There is now an MP in software-properties that will fix this bug.

https://code.launchpad.net/~arraybolt3/software-properties/+git/software-properties/+merge/438504