Bug 1217961

Summary: Open plugin location button doesn't work
Product: [Fedora] Fedora Reporter: Tom Georgoulias <tom.georgoulias>
Component: curaAssignee: Miro Hrončok <mhroncok>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 21CC: mhroncok
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: cura-14.12.1-4.fc21 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-06-09 15:00:08 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Tom Georgoulias 2015-05-03 12:13:12 UTC
Description of problem: The "Open plugin location" doesn't work


Version-Release number of selected component (if applicable):
cura-14.12.1-3.fc21.noarch

How reproducible:
Every time

Steps to Reproduce:
1. Start cura
2. Click on "Open plugin location" button at the bottom of the Plugins tab panel
3.

Actual results:
Nothing happens, the button looks like it's clicked but no window opens up showing the plugin location


Expected results:
A window should open up with the plugin location, allowing user to add and remove plugins.

Additional info:

Comment 1 Miro Hrončok 2015-05-03 13:21:27 UTC
I can reproduce this.

Comment 2 Tom Georgoulias 2015-05-29 01:20:18 UTC
This problem is still present in Fedora 22.

Comment 3 Miro Hrončok 2015-05-29 08:24:30 UTC
Ok, this is done by this (very bad) code:

    if sys.platform.startswith('linux'):
        #TODO: On linux we cannot seem to select a certain file, only open the specified path.
        if os.path.isfile('/usr/bin/nautilus'):
            subprocess.Popen(['/usr/bin/nautilus', os.path.split(filename)[0]])
        elif os.path.isfile('/usr/bin/dolphin'):
            subprocess.Popen(['/usr/bin/dolphin', os.path.split(filename)[0]])

The entire "logic" can be seen here https://github.com/daid/Cura/blob/SteamEngine/Cura/util/explorer.py

Replacing this with /usr/bin/xdg-open seems to work, so I'll send upstream Pull request and prepare a patch for Fedora cura and cura-lulzbot packages.

Comment 4 Miro Hrončok 2015-05-29 08:43:35 UTC
https://github.com/daid/Cura/pull/1261

Comment 5 Fedora Update System 2015-05-29 09:21:14 UTC
cura-14.12.1-4.fc22 has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/cura-14.12.1-4.fc22

Comment 6 Fedora Update System 2015-05-29 09:22:19 UTC
cura-14.12.1-4.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/cura-14.12.1-4.fc21

Comment 7 Fedora Update System 2015-05-29 09:30:43 UTC
cura-lulzbot-14.12-5.fc22 has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/cura-lulzbot-14.12-5.fc22

Comment 8 Fedora Update System 2015-05-29 09:31:18 UTC
cura-lulzbot-14.12-5.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/cura-lulzbot-14.12-5.fc21

Comment 9 Fedora Update System 2015-05-30 15:54:09 UTC
Package cura-14.12.1-4.fc21:
* should fix your issue,
* was pushed to the Fedora 21 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing cura-14.12.1-4.fc21'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2015-9199/cura-14.12.1-4.fc21
then log in and leave karma (feedback).

Comment 10 Fedora Update System 2015-06-09 14:59:24 UTC
cura-14.12.1-4.fc22 has been pushed to the Fedora 22 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 11 Fedora Update System 2015-06-09 14:59:35 UTC
cura-lulzbot-14.12-5.fc22 has been pushed to the Fedora 22 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Fedora Update System 2015-06-09 15:00:08 UTC
cura-14.12.1-4.fc21 has been pushed to the Fedora 21 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 13 Fedora Update System 2015-06-09 15:04:34 UTC
cura-lulzbot-14.12-5.fc21 has been pushed to the Fedora 21 stable repository.  If problems still persist, please make note of it in this bug report.