inkscape command line quoting

Bug #692799 reported by Winston Ewert
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenShot Video Editor
Fix Released
Medium
Unassigned

Bug Description

When I request to modify a title in Inkscape this fails if the title contains the character: '

An error message printing to stdout indicates a quoting error.

Version 1.3.0~Alpha1

OS. Ubuntu 10.04

Tags: patch
moimael (moimael)
Changed in openshot:
status: New → Confirmed
importance: Undecided → Low
Revision history for this message
moimael (moimael) wrote :

Here is the patch. It fixes the problem, remove the deprecated os.system functions and replace them with subprocess.Popen and subprocess.call.

Changed in openshot:
assignee: nobody → moimael (moimael)
milestone: none → 1.3.1
Revision history for this message
Jonathan Thomas (jonoomph) wrote :

Applied the patch, and made a few other changes. There is also an "Advanced Edit" menu on the main form which I applied the same basic code to. And... the error messages were not being correctly translated, so I fixed that as well. subprocess.Popen() is asynchronous, so I added another call to make OpenShot wait for Inkscape to finish, so we can update the thumbnail and preview window correctly, etc...

Changed in openshot:
assignee: moimael (moimael) → nobody
importance: Low → Medium
status: Confirmed → Fix Committed
moimael (moimael)
Changed in openshot:
status: Fix Committed → Fix Released
tags: added: patch
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.