"Send To" function replaces spaces in file name with 0s and consequently cannot find the file when attempting to email it

[Geoffrey van Wyk]

Evince 3.4 on Ubuntu 12.04

When I try to email a PDF with spaces in the file name, Evince tells me that
the file does not exist and so could not be attached.

[Impact]

 * For certain named PDFs Evince won't you email them. It's easy to workaround, but why should users have to when they have an email option in evince.
 * The diff is the upstream fix that is in all future versions of Ubuntu.

[Test Case]

1) Open the PDF file with spaces in the file name, e.g. "GNOME Rules.pdf", with
Evince.

2) In Evince, click the File menu. On the file menu click Send To... item. The
Send To dialog appears.

3) On the Send To dialog, in the Destination section, Email will be selected in
the "Send as:" dropdown list. In the "Send to:" textbox, enter a valid email
address.

4) On the Send To dialog, click the Send button.

Actual results:
The File Attach error message will appear, stating that the PDF file you were
trying to send does not exist and so could not be attached.

In the File Attach dialog you will notice that spaces in the file name have
been replaced by 0s. This is the reason why the program cannot find the file.

Expected results:
A new email message composing window from the default email client should
appear, with the email address entered in to Send To dialog appearing in the To
field, and with the PDF document attached.

[Regression Potential]

 * We could somehow make the Send to functionality worse.

Upstream bug report:
I also filed the bug on GNOME's bugzilla. The bug report number is 681144.

[Geoffrey van Wyk (geoffrey-van-wyk)]

The evince developer has committed the fix to the code repository.

[Sebastien Bacher (seb128)]

the new version is in quantal, closing the bug

[Sebastien Bacher (seb128)]

The commit which fixed it is:
http://git.gnome.org/browse/evince/commit/?id=00acd7274797fe3576be92f21faa32fd8692c95d

Not sure if it's worth a stable update...

[Bryan Quigley (bryanquigley)]

Attaching a debdiff.. I don't understand why the Uploader line got changed in it...

(I followed https://wiki.debian.org/BuildingTutorial, with http://raphaelhertzog.com/2012/08/08/how-to-use-quilt-to-manage-patches-in-debian-packages/)

[Sebastien Bacher (seb128)]

thanks, you should probably subscribe ubuntu-sponsors as well there

[Sebastien Bacher (seb128)]

@Bryan, thanks! The uploader is computed from most recent uploaders, using changelog, that's why it changed

[Sebastien Bacher (seb128)]

uploaded (with a line wrap added in the changelog text)

[Brian Murray (brian-murray)]

Hello Geoffrey, or anyone else affected,

Accepted evince into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/evince/3.4.0-0ubuntu1.8 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

[Bryan Quigley (bryanquigley)]

Verification confirmed by customer

[Chris J Arges (arges)]

The verification of the Stable Release Update for evince has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

[Launchpad Janitor (janitor)]

This bug was fixed in the package evince - 3.4.0-0ubuntu1.8

---------------
evince (3.4.0-0ubuntu1.8) precise; urgency=low

  * debian/patches/git_fix_sent_to_when_filename_contains_spaces.patch:
    - Unscape the uri before building the command line and use
       quotes for the command argument (lp: #1032680)
 -- Bryan Quigley <email address hidden> Wed, 15 Oct 2014 15:20:36 +0000