Removing a job that is not registered causes removal of the last registered job

[kbrezina]

see

gocron/gocron.go

Line 468 in 6771d4b

s.size = s.size - 1

[kbrezina]

A safer implementation could be following:

// Remove specific job j
func (s *Scheduler) Remove(j interface{}) {
	i := 0
	found := false

	for ; i < s.size; i++ {
		if s.jobs[i].jobFunc == getFunctionName(j) {
			found = true
			break
		}
	}

	if !found {
		return
	}

	for j := (i + 1); j < s.size; j++ {
		s.jobs[i] = s.jobs[j]
		i++
	}
	s.size = s.size - 1
}