diff --git a/controllers/templates.go b/controllers/templates.go index 6648603..4659e26 100644 --- a/controllers/templates.go +++ b/controllers/templates.go @@ -13,7 +13,6 @@ var Templates, _ = template.Must(template.ParseFiles( "templates/executeFailure.html", "templates/executeSuccess.html", "templates/footer.html", - "templates/head.html", "templates/index.html", "templates/officeHourTable.html", "templates/requestNotFound.html")). diff --git a/main.go b/main.go index 6499bcc..83e3065 100644 --- a/main.go +++ b/main.go @@ -40,7 +40,7 @@ func main() { roomRepo := repositories.NewRoomRepo(db) courseRepo := repositories.NewCourseRepo(db) tutorRepo := repositories.NewTutorRepo(db) - officeHourRepo := repositories.NewOfficeHourRepo(db, roomRepo, tutorRepo, courseRepo, conf) + officeHourRepo := repositories.NewOfficeHourRepo(db, roomRepo, tutorRepo, courseRepo) requestRepo := repositories.NewRequestRepo(db, officeHourRepo, conf) h := controllers.NewBaseHandler(roomRepo, officeHourRepo, courseRepo, tutorRepo, requestRepo, conf) diff --git a/repositories/officeHour.go b/repositories/officeHour.go index 5dff4a0..ff04b82 100644 --- a/repositories/officeHour.go +++ b/repositories/officeHour.go @@ -4,8 +4,6 @@ package repositories import ( "database/sql" "fmt" - "log" - "sprechstundentool/config" "sprechstundentool/models" ) @@ -14,16 +12,14 @@ type OfficeHourRepo struct { roomRepo *RoomRepo tutorRepo *TutorRepo courseRepo *CourseRepo - config config.Config } -func NewOfficeHourRepo(db *sql.DB, roomRepo *RoomRepo, tutorRepo *TutorRepo, courseRepo *CourseRepo, conf config.Config) *OfficeHourRepo { +func NewOfficeHourRepo(db *sql.DB, roomRepo *RoomRepo, tutorRepo *TutorRepo, courseRepo *CourseRepo) *OfficeHourRepo { return &OfficeHourRepo{ db: db, roomRepo: roomRepo, tutorRepo: tutorRepo, courseRepo: courseRepo, - config: conf, } } @@ -172,18 +168,14 @@ func (r *OfficeHourRepo) NumberByTimeSpanAndRoom(date models.Date, duration int, return 0, err } var count int - // iterate over all points in the new officehour to get the maximum parallel officehours - for minute := 0; minute < duration; minute += r.config.Date.MinuteGranularity { - var minuteCount int = 0 - for _, officeHour := range officeHours { - // increase count if officehour starts before this point in time and ends later - if models.DateLess(officeHour.Date, models.GetEndDate(date, minute, false)) && models.DateLess(models.GetEndDate(date, minute, false), models.GetEndDate(officeHour.Date, officeHour.Duration, false)) { - minuteCount += 1 - } + for _, officeHour := range officeHours { + if models.DateLess(models.GetEndDate(officeHour.Date, officeHour.Duration, false), date) || models.GetEndDate(officeHour.Date, officeHour.Duration, false) == date { + continue } - if minuteCount > count { - count = minuteCount + if models.DateLess(models.GetEndDate(date, duration, false), officeHour.Date) || models.GetEndDate(date, duration, false) == officeHour.Date { + continue } + count += 1 } return count, nil } diff --git a/templates/addFailure.html b/templates/addFailure.html index f160553..e032c42 100644 --- a/templates/addFailure.html +++ b/templates/addFailure.html @@ -1,8 +1,6 @@ - - +
{{range .Errors}}{{.}}
{{end}}
+
+ {{if ne .Config.Tutor.MailSuffix ""}}Du musst hier eine Email-Adresse angeben, die auf „{{.Config.Tutor.MailSuffix}}“ endet.