Edit command feature has several issues

Bug #623353 reported by Duane Hinnen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
CLI Companion
Fix Released
Medium
Duane Hinnen

Bug Description

1. spaces being put in weird places.
Highlight the aplay -l command and select Edit Command.
Add a period to the end of the description and select OK.
Now highlight the aplay -l command and click the Apply button.
It now thinks there is a user parameter ' : ' That command does not require user input.

Looks like it is adding spaces to all the fields.

2. After editing the aplay -l command a couple times I closed down clicompanion. I then reopened the program to find I have several aplay -l commands in my 'command dictionary'. Instead of saving over the command it is appending the command to the list creating multiple entries. This is not immediately obvious until you close and reopen the program.

3. After doing the above editing I went to run a command and the program wanted to run a command two or three entries above. The command list that the program uses is not being updated correctly.

EDIT: After some additional research into the problem I think some of the issues I am experiencing are specific to Maverick..
When I run this command in the CLIcompanion Terminal on Maverick:
echo $TERM
I get the result:
dummy

When I run this command in CLIcompanion on Lucid:
echo $TERM
I get the result:
xterm

Looks like there has been a change in libvte. see bug 621606 We need to do the same as is suggested in the comments of that report.:
'We now follow GNOME Terminal's behaviour of hardcoding TERM=xterm'

Related branches

Changed in clicompanion:
assignee: nobody → duanedesign (duanedesign)
summary: - Edit command has several issues
+ Edit command feature has several issues
Changed in clicompanion:
importance: Undecided → Medium
description: updated
Revision history for this message
naesk (naesk) wrote :

Here's a screenshot after editing a command which shows the additional prefixed spaces to the Description field, and the input dialog were it is expecting ":" as a parameter when running the previously edited command.

Revision history for this message
Duane Hinnen (duanedesign) wrote :

@naesk
I put up a new package where I think i have fixed some of these issues. I uploaded it to the PPA as well as the Launchpad page. it is clicompanion_1.0-2_all.deb

If you get a chance could you please test it.

thank you!
duanedesign

Changed in clicompanion:
status: New → Fix Committed
Revision history for this message
David Caro (dcaro) wrote :

I suppose this is merged to trunk (it's very old and I was unable to reproduce it), so i'm closing it.

Thanks,
David

Changed in clicompanion:
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.