Is OK to put chaining Query Builder methods inside Controller?

It's breaking the MVC pattern. You should put your queries-functions inside the model and pass the rest with function parameters or using set functions.

e.g. setYear() and setDate()
and use getYearMonthData()

or getYearMonthData($year, $month)