replaces \r by carriage return

Bug #63168 reported by finalbeta
4
Affects Status Importance Assigned to Milestone
alacarte (Ubuntu)
Fix Released
Low
Travis Watkins

Bug Description

When adding a menu item, using \r in any of the fields causes it to be converted to a carriage return.

The same problem also happens when you use \n.

Example : "wine explorer /desktop=FlashFXP,1250x920 "C:\Program Files\Radmin\radmin.exe""
This command would have the \r from \radmin.exe converted to a carriage return. Obviously the command no longer works after editing it with alacarte.

Tested in Edgy

Revision history for this message
Mateusz Drożdżyński (matid) wrote :

I confirm this problem on Edgy.

description: updated
Changed in alacarte:
importance: Undecided → Low
status: Unconfirmed → Confirmed
Revision history for this message
Mateusz Drożdżyński (matid) wrote :

I attach a screenshot of a menu item having \r in the command field.

Revision history for this message
Travis Watkins (amaranth) wrote :

As far as I can tell alacarte never broke the actual .desktop file, it just rendering it's contents incorrectly. Either way, this should be fixed in CVS and will get into 0.10.1

Changed in alacarte:
assignee: nobody → amaranth
status: Confirmed → Fix Committed
Revision history for this message
Sebastien Bacher (seb128) wrote :

This upload fixes the issue:

" alacarte (0.10.1-0ubuntu1) edgy; urgency=low
 .
   * New upstream version:
     - lots of small fixes
     - fix "Browse dialog doesn't escape spaces in command path"
       (Ubuntu: #54748)
     - fix "replaces \r by carriage return" (Ubuntu: #63168)
"

Changed in alacarte:
status: Fix Committed → Fix Released
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.