flo/todo.txt-cli
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix erroneously removing pipe chars from input

Browse Source 
cleaninput() unintentionally deletes the '|' character from input.  This
commit fixes it and adds test for adding tasks with a bunch of symbols.
This commit is contained in:
Nick Wilson
2011-01-31 21:07:35 -08:00
parent f1d339a537
commit 8e4364f5e1
2 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
tests/t1000-addlist.sh
View File

@@ -58,19 +58,23 @@ TODO: 3 added.
TODO: 2 of 3 tasks shown TODO: 2 of 3 tasks shown
EOF EOF
test_todo_session 'add with &' <<EOF test_todo_session 'add with symbols' <<EOF
>>> todo.sh add "dig the garden & water the flowers" >>> todo.sh add "~@#$%^&*()-_=+[{]}|;:',<.>/?"
4 dig the garden & water the flowers 4 ~@#$%^&*()-_=+[{]}|;:',<.>/?
TODO: 4 added. TODO: 4 added.
>>> todo.sh add '\`!\\"'
5 \`!\\"
TODO: 5 added.
>>> todo.sh list >>> todo.sh list
4 dig the garden & water the flowers
1 notice the daisies 1 notice the daisies
2 smell the roses 2 smell the roses
3 smell the uppercase Roses 3 smell the uppercase Roses
5 \`!\\"
4 ~@#$%^&*()-_=+[{]}|;:',<.>/?
-- --
TODO: 4 of 4 tasks shown TODO: 5 of 5 tasks shown
EOF EOF
test_done test_done

2
todo.sh
View File

@@ -260,7 +260,7 @@ cleaninput()
{ {
# Cleanup the input # Cleanup the input
# Replace newlines with spaces Always # Replace newlines with spaces Always
input=`echo $input | tr -d '\r|\n'` input=`echo $input | tr -d '\r\n'`
action_regexp="^\(append\|app\|prepend\|prep\|replace\)$" action_regexp="^\(append\|app\|prepend\|prep\|replace\)$"