Question

I am using the Rollup function and I am almost at the perfect point of what I need, just need to remove one little thing.

declare @FromDate datetime
declare @ToDate datetime
declare @StoreNumber varchar(10)
declare @AllStores varchar(1)
set @FromDate = '1/1/12'
set @ToDate = '1/1/13'
set @StoreNumber = '01'
set @AllStores = 'y'

If @AllStores = 'Y'
Begin
select 
CASE WHEN (GROUPING(s.StoreID) = 1) THEN 'All Stores' ELSE ISNULL(s.StoreID, 'UNKNOWN') END as 'Store #',
CASE WHEN (GROUPING(b.currencydesc) = 1) THEN 'Store #' + s.StoreID + ' TOTAL' ELSE ISNULL(b.currencydesc, 'UNKNOWN') END as 'Tender Type', SUM(amount) as Total
from RPTrs s 
join rpPay p on s.storeid = p.StoreID and s.ReceiptNO = p.ReceiptNo
join Currencies b on b.POSCurrency = LEFT(p.paytype,1)
where trsdate between @FromDate and @ToDate
group by s.storeid, b.currencydesc with rollup
End
Else
Begin
select 
s.StoreID as 'Store #',
CASE WHEN (GROUPING(b.currencydesc) = 1) THEN 'Store #' + s.StoreID + ' TOTAL' ELSE ISNULL(b.currencydesc, 'UNKNOWN') END as 'Tender Type', SUM(amount) as Total
from RPTrs s 
join rpPay p on s.StoreID = p.StoreID and s.ReceiptNO = p.ReceiptNo
join Currencies b on b.POSCurrency = LEFT(p.paytype,1)
where trsdate between @FromDate and @ToDate
and s.StoreID = @StoreNumber
group by grouping sets((s.StoreID),(s.StoreID, b.CurrencyDesc)) 
End

With this output.

Store # Tender Type Total
01  Amex    250.00
01  C.Card  3138.00
01  Cash    53553.17
01  Gift Card   1595.35
01  MasterCard  813.10
01  Off Line C.Card 247.53
01  Str Cr  -544.45
01  Visa/MC 437.35
01  Store #01 TOTAL 59490.05
02  Cash    238.15
02  Store #02 TOTAL 238.15
All Stores  NULL    59728.20

How can I make the NULL in All Stores total disappear?

Was it helpful?

Solution

One way to do this is to wrap the query in a SELECT and then use COALESCE on the columns you want to replace the null values:

SELECT [store #], 
       COALESCE([tender type], '') [Tender Type], 
       total 
FROM   
(
    SELECT 
        CASE 
            WHEN ( Grouping(s.storeid) = 1 ) 
            THEN 'All Stores' 
            ELSE Isnull(s.storeid, 'UNKNOWN') 
        END         AS [Store #], 
        CASE 
            WHEN ( Grouping(b.currencydesc) = 1 ) 
            THEN 'Store #' + s.storeid + ' TOTAL' 
            ELSE Isnull(b.currencydesc, 'UNKNOWN') 
        END         AS [Tender Type], 
        Sum(amount) AS Total 
    FROM   rptrs s 
    JOIN rppay p 
        ON s.storeid = p.storeid 
        AND s.receiptno = p.receiptno 
    JOIN currencies b 
        ON b.poscurrency = LEFT(p.paytype, 1) 
    WHERE  trsdate BETWEEN @FromDate AND @ToDate 
    GROUP  BY s.storeid, b.currencydesc WITH rollup
) src 

Or you can wrap the column in a COALESCE() without the subquery:

SELECT 
    CASE 
        WHEN ( Grouping(s.storeid) = 1 ) 
        THEN 'All Stores' 
        ELSE Isnull(s.storeid, 'UNKNOWN') 
    END               AS [Store #], 
    COALESCE(CASE 
                WHEN ( Grouping(b.currencydesc) = 1 ) 
                THEN 'Store #' + s.storeid + ' TOTAL' 
                ELSE Isnull(b.currencydesc, 'UNKNOWN') 
            END, '') AS [Tender Type], 
    Sum(amount)       AS Total 
FROM   rptrs s 
JOIN rppay p 
    ON s.storeid = p.storeid 
    AND s.receiptno = p.receiptno 
JOIN currencies b 
    ON b.poscurrency = LEFT(p.paytype, 1) 
WHERE  trsdate BETWEEN @FromDate AND @ToDate 
GROUP  BY s.storeid, b.currencydesc WITH rollup 

OTHER TIPS

It seems your issue is in the following line:

CASE WHEN (GROUPING(b.currencydesc) = 1) THEN 'Store #' + s.StoreID + ' TOTAL' ELSE ISNULL(b.currencydesc, 'UNKNOWN') END as [Tender Type], 

Consider this:

SELECT '1' + NULL 

This will return NULL. Use COALESCE instead:

SELECT COALESCE('1' + NULL,'') 

This will return ''. I suppose StoreId is NULL in your above scenario. Here is the full code:

CASE WHEN (GROUPING(b.currencydesc) = 1) THEN COALESCE('Store #' + s.StoreID + ' TOTAL','') ELSE ISNULL(b.currencydesc, 'UNKNOWN') END as [Tender Type], 

Good luck.

Licensed under: CC-BY-SA with attribution
Not affiliated with StackOverflow
scroll top