If you had tested it first, you would have found most of the errors.

If you cannot tell us how it is not working, how do you expect us to help?

function build_list

That is not wrong in ksh and bash, but it is not the standard syntax for defining a function and will not work in other shells. Standard is:

Code:

build_list()

{
for i in *$1*$2*
do
x=`echo $i | sed 's/\.Sent/\.Done/g'`
done
}

Tha function does not build a list; only the last filename that matched the pattern would be stored in $x.

function show_list
{
echo mv $i DONE/$x
}

It makes your script more undertandable if you use meaningful names; that function doesn't show a list, it moves files.

...and it will fail if any filenames contain spaces or other pathological characters.

..and there is no logic behind that code. Think about what you are doing.

echo "Is this Ok?"
read user_response
if [$user_response="y"|"Y"]

There are five errors in the line above.

then
 mv $i DONE/$x;;
else
if [$user_response="n"|"N"]
then
echo "No changes made ...";;
fi
fi