Skip to content

Commit

Permalink
ref qax-os#65: new formula function WEEKNUM
Browse files Browse the repository at this point in the history
  • Loading branch information
xuri committed Jun 9, 2022
1 parent a0396ed commit 964af19
Show file tree
Hide file tree
Showing 2 changed files with 101 additions and 0 deletions.
81 changes: 81 additions & 0 deletions calc.go
Expand Up @@ -719,6 +719,7 @@ type formulaFuncs struct {
// VDB
// VLOOKUP
// WEEKDAY
// WEEKNUM
// WEIBULL
// WEIBULL.DIST
// XIRR
Expand Down Expand Up @@ -12803,6 +12804,86 @@ func (fn *formulaFuncs) WEEKDAY(argsList *list.List) formulaArg {
return newErrorFormulaArg(formulaErrorVALUE, formulaErrorVALUE)
}

// weeknum is an implementation of the formula function WEEKNUM.
func (fn *formulaFuncs) weeknum(snTime time.Time, returnType int) formulaArg {
days := snTime.YearDay()
weekMod, weekNum := days%7, math.Ceil(float64(days)/7)
if weekMod == 0 {
weekMod = 7
}
year := snTime.Year()
firstWeekday := int(time.Date(year, time.January, 1, 0, 0, 0, 0, time.UTC).Weekday())
var offset int
switch returnType {
case 1, 17:
offset = 0
case 2, 11, 21:
offset = 1
case 12, 13, 14, 15, 16:
offset = returnType - 10
default:
return newErrorFormulaArg(formulaErrorNUM, formulaErrorNUM)
}
padding := offset + 7 - firstWeekday
if padding > 7 {
padding -= 7
}
if weekMod > padding {
weekNum++
}
if returnType == 21 && (firstWeekday == 0 || firstWeekday > 4) {
if weekNum--; weekNum < 1 {
if weekNum = 52; int(time.Date(year-1, time.January, 1, 0, 0, 0, 0, time.UTC).Weekday()) < 4 {
weekNum++
}
}
}
return newNumberFormulaArg(weekNum)
}

// WEEKNUM function returns an integer representing the week number (from 1 to
// 53) of the year. The syntax of the function is:
//
// WEEKNUM(serial_number,[return_type])
//
func (fn *formulaFuncs) WEEKNUM(argsList *list.List) formulaArg {
if argsList.Len() < 1 {
return newErrorFormulaArg(formulaErrorVALUE, "WEEKNUM requires at least 1 argument")
}
if argsList.Len() > 2 {
return newErrorFormulaArg(formulaErrorVALUE, "WEEKNUM allows at most 2 arguments")
}
sn := argsList.Front().Value.(formulaArg)
num, returnType := sn.ToNumber(), 1
var snTime time.Time
if num.Type != ArgNumber {
dateString := strings.ToLower(sn.Value())
if !isDateOnlyFmt(dateString) {
if _, _, _, _, _, err := strToTime(dateString); err.Type == ArgError {
return err
}
}
y, m, d, _, err := strToDate(dateString)
if err.Type == ArgError {
return err
}
snTime = time.Date(y, time.Month(m), d, 0, 0, 0, 0, time.Now().Location())
} else {
if num.Number < 0 {
return newErrorFormulaArg(formulaErrorNUM, formulaErrorNUM)
}
snTime = timeFromExcelTime(num.Number, false)
}
if argsList.Len() == 2 {
returnTypeArg := argsList.Back().Value.(formulaArg).ToNumber()
if returnTypeArg.Type != ArgNumber {
return returnTypeArg
}
returnType = int(returnTypeArg.Number)
}
return fn.weeknum(snTime, returnType)
}

// Text Functions

// CHAR function returns the character relating to a supplied character set
Expand Down
20 changes: 20 additions & 0 deletions calc_test.go
Expand Up @@ -1560,6 +1560,18 @@ func TestCalcCellValue(t *testing.T) {
"=WEEKDAY(\"12/25/2012\",15)": "5",
"=WEEKDAY(\"12/25/2012\",16)": "4",
"=WEEKDAY(\"12/25/2012\",17)": "3",
// WEEKNUM
"=WEEKNUM(\"01/01/2011\")": "1",
"=WEEKNUM(\"01/03/2011\")": "2",
"=WEEKNUM(\"01/13/2008\")": "3",
"=WEEKNUM(\"01/21/2008\")": "4",
"=WEEKNUM(\"01/30/2008\")": "5",
"=WEEKNUM(\"02/04/2008\")": "6",
"=WEEKNUM(\"01/02/2017\",2)": "2",
"=WEEKNUM(\"01/02/2017\",12)": "1",
"=WEEKNUM(\"12/31/2017\",21)": "52",
"=WEEKNUM(\"01/01/2017\",21)": "52",
"=WEEKNUM(\"01/01/2021\",21)": "53",
// Text Functions
// CHAR
"=CHAR(65)": "A",
Expand Down Expand Up @@ -3484,6 +3496,14 @@ func TestCalcCellValue(t *testing.T) {
"=WEEKDAY(0,0)": "#VALUE!",
"=WEEKDAY(\"January 25, 100\")": "#VALUE!",
"=WEEKDAY(-1,1)": "#NUM!",
// WEEKNUM
"=WEEKNUM()": "WEEKNUM requires at least 1 argument",
"=WEEKNUM(0,1,0)": "WEEKNUM allows at most 2 arguments",
"=WEEKNUM(0,\"\")": "strconv.ParseFloat: parsing \"\": invalid syntax",
"=WEEKNUM(\"\",1)": "#VALUE!",
"=WEEKNUM(\"January 25, 100\")": "#VALUE!",
"=WEEKNUM(0,0)": "#NUM!",
"=WEEKNUM(-1,1)": "#NUM!",
// Text Functions
// CHAR
"=CHAR()": "CHAR requires 1 argument",
Expand Down

0 comments on commit 964af19

Please sign in to comment.