Set Variable in SQL Function with Condition - sql-server

Question

Trying to set a varable based on the current date and pass to query wihtin SQL function. It error at the IF statement. Any ideas?
CREATE FUNCTION CS_AwaredRCPs
(
#currentDate DATE,
#fiscalYear INT
IF DATEPART(m,#currentDate) > 10
SET #fiscalYear = DATEPART(yyyy,#currentDate)
ELSE
SET #fiscalYear = DATEPART(yyyy,#currentDate) - 1
END
)
RETURNS TABLE
AS
RETURN
(
SELECT dbo.tbl_requirementManagement.postaward_specialist_id,
SUM(dbo.tbl_requirementManagement.actual_award_value) AS AwardValue,
COUNT(dbo.tbl_requirementManagement.postaward_specialist_id) AS AwardCount
FROM dbo.tbl_requirementManagement RIGHT OUTER JOIN
dbo.vw_ContractSpecialists ON dbo.tbl_requirementManagement.postaward_specialist_id = dbo.vw_ContractSpecialists.user_certificateSerialNumber
GROUP BY dbo.tbl_requirementManagement.statusID, dbo.tbl_requirementManagement.postaward_specialist_id, dbo.tbl_requirementManagement.fiscal_year
HAVING (dbo.tbl_requirementManagement.statusID = 4) AND
(dbo.tbl_requirementManagement.postaward_specialist_id <> 0) AND
(dbo.tbl_requirementManagement.fiscal_year = #fiscalYear)
)
`

Answer 1

You don't need to use IF at all, you can simply make your predicate:
fiscal_year = DATEPART(YEAR, #currentDate)
+ CASE WHEN DATEPART(MONTH, #CurrentDate) <= 10 THEN -1 ELSE 0 END
Then you don't need the #FiscalYear parameter at all. For what it is worth there are a number of other things you should change in your function
Use table aliases to significantly reduce the amount of text, so rather than using dbo.tbl_requirementManagement over and over again, you could just use rm
The fact that you reference your outer table in the where clause, turns your RIGHT OUTER JOIN into an INNER JOIN so you may as well just use and INNER JOIN. To clarify, you have (dbo.tbl_requirementManagement.statusID = 4), so where you have no match in tbl_requirementManagement, statusID will be NULL, and NULL = 4 returns NULL, which is not true, therefore the row would not be returned.
Your predicates do not reference aggregates so should be in the WHERE clause, not the HAVING
So your final function becomes:
CREATE FUNCTION CS_AwaredRCPs (#currentDate DATE)
RETURNS TABLE
AS
RETURN
( SELECT rm.postaward_specialist_id,
SUM(rm.actual_award_value) AS AwardValue,
COUNT(rm.postaward_specialist_id) AS AwardCount
FROM dbo.tbl_requirementManagement AS rm
INNER JOIN dbo.vw_ContractSpecialists AS cs
ON rm.postaward_specialist_id = cs.user_certificateSerialNumber
WHERE rm.fiscal_year = DATEPART(YEAR,#currentDate)
+ CASE WHEN DATEPART(MONTH, #CurrentDate) <= 10 THEN -1 ELSE 0 END
AND rm.statusID = 4
AND rm.postaward_specialist_id <> 0
GROUP BY rm.statusID, rm.postaward_specialist_id, rm.fiscal_year
);

Answer 2

You cannot do a if-else check inside Function parameter list. Move the If-else part to the function body. Try this.
CREATE FUNCTION Cs_awaredrcps (#currentDate DATE)
RETURNS #ret_table TABLE (
postaward_specialist_id INT,
AwardValue INT,
AwardCount INT )
AS
BEGIN
DECLARE #fiscalYear INT
IF Datepart(Month, #currentDate) > 10
SET #fiscalYear = Datepart(yyyy, #currentDate)
ELSE
SET #fiscalYear = Datepart(yyyy, #currentDate) - 1
INSERT INTO #ret_table
SELECT dbo.tbl_requirementManagement.postaward_specialist_id,
Sum(dbo.tbl_requirementManagement.actual_award_value) AS AwardValue,
Count(dbo.tbl_requirementManagement.postaward_specialist_id) AS AwardCount
FROM dbo.tbl_requirementManagement
RIGHT OUTER JOIN dbo.vw_ContractSpecialists
ON dbo.tbl_requirementManagement.postaward_specialist_id = dbo.vw_ContractSpecialists.user_certificateSerialNumber
GROUP BY dbo.tbl_requirementManagement.statusID,
dbo.tbl_requirementManagement.postaward_specialist_id,
dbo.tbl_requirementManagement.fiscal_year
HAVING ( dbo.tbl_requirementManagement.statusID = 4 )
AND ( dbo.tbl_requirementManagement.postaward_specialist_id <> 0 )
AND ( dbo.tbl_requirementManagement.fiscal_year = #fiscalYear )
RETURN
END