[Okta Webinar] Learn how to a build a cloud-first strategyRegister Now

x
?
Solved

Compare Expiration dates

Posted on 2014-02-05
6
Medium Priority
?
186 Views
Last Modified: 2014-02-25
I have a stored  procedure that assign a dollar amount based on an expiration date.  Initially a user would pay a $1000 for each brand family.  But if the user would like to add styles they can do so without being charged.  Therefore  I have the case statement below.

 SELECT l.vchTransNum, l.vchManufacturer, b.vchBrand AS 'Brand Family', 
      CASE
       
      --check online tables to see if brands have been submitted for this particular manu
      WHEN EXISTS (SELECT vchBrand, vchManufacturer 
      FROM tblOnlineCLBrands o  WHERE o.vchBrand = b.vchBrand and o.vchManufacturer =l.vchManufacturer) 
      THEN 0.00
      
      
       --checks in-house table to see if brands have been submitted for this manu and if this is renewal year
      WHEN EXISTS (SELECT vchBrand, vchManufacturer   
      FROM tblCLBrands bf
      INNER JOIN tblCLPermit p on bf.vchRecordID = p.vchRecordID 
      --WHERE bf.vchBrand = b.vchBrand and bf.vchManufacturer =l.vchManufacturer and Year(DATEADD (Year , 3, dtpermitDate))<= YEAR(GetDate()) )
      WHERE bf.vchBrand = b.vchBrand and bf.vchManufacturer =l.vchManufacturer and dtExpirationDate > GETDATE())
      THEN 0.00
     
      
      ELSE 1000
      END AS 'Cost of Brand'
            
      FROM tblOnlineCLTempLab l
      
      INNER JOIN (Select distinct vchBrand, vchTransNum  from tblOnlineCLTempBrands) b on l.vchTransNum  = b.vchTransNum 
      
      WHERE  l.vchTransNum = @TransNum
      GROUP BY l.vchTransNum , l.vchManufacturer, b.vchbrand

Open in new window


The second case line is giving me a hard time.  If users want to renew before the expiration date, it still assigns it zero dollars instead of $1000. How can I program it to allow the user to pay before their date expires?
0
Comment
Question by:dlavar
  • 3
  • 2
6 Comments
 
LVL 13

Expert Comment

by:Koen Van Wielink
ID: 39837818
There's nothing obviously wrong with the query as far as I can tell, so is it possible that the first case statement always evaluates to True?
0
 
LVL 25

Expert Comment

by:chaau
ID: 39837826
Could it be that dtExpirationDate  exists in both tblCLBrands and tblCLPermit tables, or in tblOnlineCLTempLab table itself?
0
 

Author Comment

by:dlavar
ID: 39838858
When I commented out the first case statement, it resulted in a $1000. The Expiration date is only in the Permit table.  I'm thinking that I may have to set a grace period of when the can renew their permit.
0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 
LVL 13

Expert Comment

by:Koen Van Wielink
ID: 39838916
In that case you'll have to check why the first case statement evaluates to True when you don't want it to. The problem is most likely not with the expiration date.
0
 

Accepted Solution

by:
dlavar earned 0 total points
ID: 39874646
I had to set a grace period of 60 days in the stored procedure

 SELECT l.vchTransNum, l.vchManufacturer, b.vchBrand AS 'Brand Family', 
      
      CASE
      -- check online tables to see if brands have been submitted for this particular manu
	  WHEN EXISTS (SELECT vchBrand, vchManufacturer 
	  FROM tblOnlineCLBrands o  WHERE o.vchBrand = b.vchBrand and o.vchManufacturer =l.vchManufacturer) 
	  THEN 0.00
      
      --checks in-house table to see if brands have been submitted for this manu and if this is renewal year
      --If today is 60 days before the expiration (before renewal time), charge 0. 
      WHEN EXISTS (SELECT vchBrand, vchManufacturer 
      FROM tblCLBrands bf
      INNER JOIN tblCLPermit p on bf.vchRecordID = p.vchRecordID 
      WHERE bf.vchBrand = b.vchBrand and bf.vchManufacturer =l.vchManufacturer and  GETDATE() < DATEADD(Day , -60, dtExpirationDate) )
      THEN 0.00
               
      
      ELSE 1000
      END AS 'Cost of Brand'
            
      FROM tblOnlineCLTempLab l
      INNER JOIN (Select distinct vchBrand, vchTransNum  from tblOnlineCLTempBrands) b on l.vchTransNum  = b.vchTransNum 
      WHERE  l.vchTransNum = @TransNum
      GROUP BY l.vchTransNum , l.vchManufacturer, b.vchbrand

Open in new window

0
 

Author Closing Comment

by:dlavar
ID: 39885097
My solution was the one that worked and was most effective.
0

Featured Post

Concerto Cloud for Software Providers & ISVs

Can Concerto Cloud Services help you focus on evolving your application offerings, while delivering the best cloud experience to your customers? From DevOps to revenue models and customer support, the answer is yes!

Learn how Concerto can help you.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Windocks is an independent port of Docker's open source to Windows.   This article introduces the use of SQL Server in containers, with integrated support of SQL Server database cloning.
This shares a stored procedure to retrieve permissions for a given user on the current database or across all databases on a server.
Viewers will learn how to use the INSERT statement to insert data into their tables. It will also introduce the NULL statement, to show them what happens when no value is giving for any given column.
Viewers will learn how to use the SELECT statement in SQL to return specific rows and columns, with various degrees of sorting and limits in place.
Suggested Courses

868 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question