Yes you're right:

ALTER Procedure [dbo].[BS_Voucher_CreateVoucher]

@vouchertypeID int = 1,

@denomination int = 100,

@quantity int = 10,

@amountvalue int = 100,

@userid varchar(50) = 1,

@validateDays int = 40

AS

--BEGIN

SET NOCOUNT ON;

DECLARE

@vouchertype varchar(20),

@transactionID varchar(20),

@count int =1,

@VoucherNumber int, -- was @max-2

@vouchertypenew varchar(20),

@retval varchar(20),

@voucherstatus int

SELECT @vouchertype = VoucherType FROM GV_VoucherType WHERE VoucherTypeID = @vouchertypeID

IF @vouchertype IS NULL

RETURN -1

SELECT @voucherstatus = VoucherStatusId FROM GV_VoucherStatus WHERE VoucherStatus = 'New'

SELECT @transactionID = ISNULL(vouchervalue,0) FROM gv_vouchervalue WHERE voucherabbreviation = 'TRNID'

SET @vouchertypenew = RTRIM(LTRIM(@vouchertype));

SET @retval = LEFT(@vouchertypenew,1);

WHILE CHARINDEX(' ',@vouchertypenew,1)>0

BEGIN

SET @vouchertypenew = LTRIM(RIGHT(@vouchertype,LEN(@vouchertype) - CHARINDEX(' ',@vouchertype,1)));

SET @retval += LEFT(@vouchertypenew,1);

END

SELECT @VoucherNumber = ISNULL(vouchervalue,0) FROM gv_vouchervalue WHERE voucherabbreviation = 'V'+@retval

WHILE @count <= @quantity

BEGIN

--If EXISTS (SELECT VoucherType FROM GV_VoucherType WHERE VoucherType = @vouchertype)

--BEGIN

INSERT INTO GV_Voucher

VALUES

(

@vouchertypeID,

'V'+@retval + Replace( Str( @VoucherNumber + @count, 7), ' ', '0'),

@denomination,

DATEADD(MM,6,GETDATE()),

GETDATE(),

GETDATE(),

@voucherstatus,

'TRN' + Replace( Str( @transactionID + 1, 7), ' ', '0'),

@quantity,

CASE @vouchertypenew

WHEN 'Discount' THEN @amountvalue

ELSE 0

END,

@userid,

@userid,

@validateDays,

0

)

SET @count = @count + 1

--END

END

UPDATE gv_vouchervalue

SET vouchervalue = @VoucherNumber + @count

WHERE voucherabbreviation = 'V'+@retval

UPDATE gv_vouchervalue

SET VoucherValue = @transactionID + 1

WHERE VoucherAbbreviation = 'TRNID'

--END

RETURN 0

Can you post some values of VoucherType FROM GV_VoucherType?