TSQL Hack needed for getting a filter for data
Question
A UI (before the report shows) shows a look up (Combo) that has
- (ID = 0).All Organization Units
- (ID =4).HR
- (ID = 5).DEV
I need to:
- Be able to show data of (4) + (5) if (0) is selected.
- Only (4) OR (5) if either HR or DEV is selected.
Lookup combo code (Selected Feeds the parameter in the below query.)
Select 0 AS ID,'All Org' AS Name from DP_ORG_OrganizationUnit
where DP_ORG_OrganizationUnit.Code IN {AccessData}
Union
SELECT
DP_ORG_OrganizationUnit.ID,
DP_ORG_OrganizationUnit.Name
FROM DP_ORG_OrganizationUnit where DP_ORG_OrganizationUnit.Code IN ('HR','DEV')
Report data row query
SET CONCAT_NULL_YIELDS_NULL OFF
DECLARE @EmpID as int;
DECLARE @OrganizationUnit as int;
DECLARE @StartDate as datetime;
DECLARE @EndDate as datetime;
SET @EmpID = ?;
SET @StartDate = ?;
SET @EndDate = ?;
SET @OrganizationUnit = ?;
SELECT
Employee.Code,
Employee.Name1+' '+Employee.Name2+' '+Employee.Name3+' '+Employee.Name4+' '+Employee.Name5 AS FullName,
Employee.OrganizationUnit,
ContractType.Name,
EmployeeContract.StartDate,
EmployeeContract.EndDate
FROM Employee INNER JOIN (ContractType INNER JOIN EmployeeContract
ON ContractType.ID = EmployeeContract.ContractType)
ON Employee.ID = EmployeeContract.Employee
WHERE (Employee.ID = @EmpID OR @EmpID=0)
AND
(Employee.OrganizationUnit = @OrganizationUnit OR @OrganizationUnit=0)
AND NOT((EndDate < @StartDate or StartDate > @EndDate));
Any way I can achieve it from the looks of it? 0=0 would show all the data from other departments too..
Anybody :-o?
Solution
First off, your lookup combo code could be tightened up a bit:
-- the FROM clause was superfluous
SELECT 0 AS ID,'All Org' AS Name
UNION ALL
-- the two-part identifiers were superfluous (only one table)
SELECT ID, Name
FROM DP_ORG_OrganizationUnit
WHERE Code IN ('HR','DEV')
For the report query, the simplest form would be:
WHERE
((@OrganizationUnit > 0 AND Employee.OrganizationUnit = @OrganizationUnit) OR
(@OrganizationUnit = 0 AND Employee.OrganizationUnit IN (4,5)))
OTHER TIPS
something like this should work
Where (Employee.OrganizationUnit = case when @OrganizationUnit=0 then 4 else @OrganizationUnit end OR case when @OrganizationUnit=0 then 5 else @OrganizationUnit end)
Try this, which should use indexes on your query...
DECALRE @FilterValues (FilterValue int not null primary key)
IF @Param=0
BEGIN
INSERT INTO @FilterValues VALUES (4)
INSERT INTO @FilterValues VALUES (5)
END
ELSE ID @PAram IS NOT NULL
BEGIN
INSERT INTO @FilterValues VALUES (@Param)
END
SELECT
....
FROM YourTable y
INNER JOIN @FilterValues f ON y.Value=f.Value
WHERE .....
KM's version will work, but this query does not need a temp table...
SELECT *
FROM Employee
WHERE (
@OrganizationUnit = 0
OR
(
@OrganizationUnit <> 0
AND
Employee.OrganizationUnit = @OrganizationUnit
)
)
How about
WHERE (Employee.ID = @EmpID OR @EmpID=0)
AND
(Employee.OrganizationUnit BETWEEN ISNULL(NULLIF(@OrganizationUnit,0),0) AND ISNULL(NULLIF(@OrganizationUnit,0),99))
AND NOT((EndDate < @StartDate or StartDate > @EndDate));
Licensed under: CC-BY-SA with attribution
Not affiliated with StackOverflow